From: Damien Le Moal <dlemoal@kernel.org>
To: Rosen Penev <rosenp@gmail.com>, linux-ide@vger.kernel.org
Cc: Niklas Cassel <cassel@kernel.org>,
Nathan Chancellor <nathan@kernel.org>,
Nick Desaulniers <ndesaulniers@google.com>,
Bill Wendling <morbo@google.com>,
Justin Stitt <justinstitt@google.com>,
open list <linux-kernel@vger.kernel.org>,
"open list:CLANG/LLVM BUILD SUPPORT:Keyword:b(?i:clang|llvm)b"
<llvm@lists.linux.dev>
Subject: Re: [PATCH] ata: pata_pxa: use devm_platform_ioremap_resource
Date: Sat, 18 Jul 2026 12:04:42 +0900 [thread overview]
Message-ID: <65196e27-610e-4b57-8e5e-0af7e8eef06d@kernel.org> (raw)
In-Reply-To: <20260715011343.1270261-1-rosenp@gmail.com>
On 7/15/26 10:13, Rosen Penev wrote:
> Replace the open-coded platform_get_resource() plus devm_ioremap()
> sequences for the CMD and CTL ports with devm_platform_ioremap_resource(),
> which fetches the resource, requests the region and maps it in one call.
> Switch the error checks to IS_ERR()/PTR_ERR() accordingly and drop the
> now-unused cmd_res and ctl_res variables.
>
> The DMA port keeps using devm_ioremap() as its resource is
> IORESOURCE_DMA. The CMD (res 0) and CTL (res 1) MEM resources are
> disjoint in the ATA task-file layout, so the added region reservation
> in devm_ioremap_resource() does not introduce overlap failures.
>
> Built for ARM (drivers/ata/pata_pxa.o) with LLVM=1.
>
> Assisted-by: opencode:hy3-free
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
Applied to for-7.3. Thanks!
--
Damien Le Moal
Western Digital Research
prev parent reply other threads:[~2026-07-18 3:04 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-15 1:13 Rosen Penev
2026-07-18 3:04 ` Damien Le Moal [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=65196e27-610e-4b57-8e5e-0af7e8eef06d@kernel.org \
--to=dlemoal@kernel.org \
--cc=cassel@kernel.org \
--cc=justinstitt@google.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=morbo@google.com \
--cc=nathan@kernel.org \
--cc=ndesaulniers@google.com \
--cc=rosenp@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome