* [PATCH] pata_falcon: Add missing __iomem annotations
@ 2021-11-24 4:25 Finn Thain
2021-11-24 4:52 ` Michael Schmitz
2021-11-24 7:51 ` Geert Uytterhoeven
0 siblings, 2 replies; 9+ messages in thread
From: Finn Thain @ 2021-11-24 4:25 UTC (permalink / raw)
To: Damien Le Moal
Cc: Michael Schmitz, Bartlomiej Zolnierkiewicz, Jens Axboe,
linux-ide, linux-kernel
The zero day bot reported some sparse complaints in pata_falcon.c. E.g.
drivers/ata/pata_falcon.c:58:41: warning: cast removes address space '__iomem' of expression
drivers/ata/pata_falcon.c:58:41: warning: incorrect type in argument 1 (different address spaces)
drivers/ata/pata_falcon.c:58:41: expected unsigned short volatile [noderef] [usertype] __iomem *port
drivers/ata/pata_falcon.c:58:41: got unsigned short [usertype] *
The same thing shows up in 8 places, all told. Avoid this by use of
__iomem type casts.
Cc: Michael Schmitz <schmitzmic@gmail.com>
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: Jens Axboe <axboe@kernel.dk>
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Finn Thain <fthain@linux-m68k.org>
---
drivers/ata/pata_falcon.c | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/ata/pata_falcon.c b/drivers/ata/pata_falcon.c
index 121635aa8c00..e2a88edd9dbf 100644
--- a/drivers/ata/pata_falcon.c
+++ b/drivers/ata/pata_falcon.c
@@ -55,14 +55,14 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
/* Transfer multiple of 2 bytes */
if (rw == READ) {
if (swap)
- raw_insw_swapw((u16 *)data_addr, (u16 *)buf, words);
+ raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
else
- raw_insw((u16 *)data_addr, (u16 *)buf, words);
+ raw_insw((u16 __iomem *)data_addr, (u16 *)buf, words);
} else {
if (swap)
- raw_outsw_swapw((u16 *)data_addr, (u16 *)buf, words);
+ raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
else
- raw_outsw((u16 *)data_addr, (u16 *)buf, words);
+ raw_outsw((u16 __iomem *)data_addr, (u16 *)buf, words);
}
/* Transfer trailing byte, if any. */
@@ -74,16 +74,16 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
if (rw == READ) {
if (swap)
- raw_insw_swapw((u16 *)data_addr, (u16 *)pad, 1);
+ raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)pad, 1);
else
- raw_insw((u16 *)data_addr, (u16 *)pad, 1);
+ raw_insw((u16 __iomem *)data_addr, (u16 *)pad, 1);
*buf = pad[0];
} else {
pad[0] = *buf;
if (swap)
- raw_outsw_swapw((u16 *)data_addr, (u16 *)pad, 1);
+ raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)pad, 1);
else
- raw_outsw((u16 *)data_addr, (u16 *)pad, 1);
+ raw_outsw((u16 __iomem *)data_addr, (u16 *)pad, 1);
}
words++;
}
--
2.26.3
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH] pata_falcon: Add missing __iomem annotations
2021-11-24 4:25 [PATCH] pata_falcon: Add missing __iomem annotations Finn Thain
@ 2021-11-24 4:52 ` Michael Schmitz
2021-11-25 1:09 ` Finn Thain
2021-11-24 7:51 ` Geert Uytterhoeven
1 sibling, 1 reply; 9+ messages in thread
From: Michael Schmitz @ 2021-11-24 4:52 UTC (permalink / raw)
To: Finn Thain, Damien Le Moal
Cc: Bartlomiej Zolnierkiewicz, Jens Axboe, linux-ide, linux-kernel
Hi Finn,
thanks for your patch!
On 24/11/21 17:25, Finn Thain wrote:
> The zero day bot reported some sparse complaints in pata_falcon.c. E.g.
>
> drivers/ata/pata_falcon.c:58:41: warning: cast removes address space '__iomem' of expression
> drivers/ata/pata_falcon.c:58:41: warning: incorrect type in argument 1 (different address spaces)
> drivers/ata/pata_falcon.c:58:41: expected unsigned short volatile [noderef] [usertype] __iomem *port
> drivers/ata/pata_falcon.c:58:41: got unsigned short [usertype] *
>
> The same thing shows up in 8 places, all told. Avoid this by use of
> __iomem type casts.
Seeing as data_addr was explicitly typed as __iomem, your fix is clearly
correct. Bit embarrassing to have missed that (I remember adding __iomem
annotations elsewhere at some stage).
If you think there's any need to test this change, say so.
Reviewed-by: Michael Schmitz <schmitzmic@gmail.com>
>
> Cc: Michael Schmitz <schmitzmic@gmail.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Jens Axboe <axboe@kernel.dk>
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Finn Thain <fthain@linux-m68k.org>
> ---
> drivers/ata/pata_falcon.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/ata/pata_falcon.c b/drivers/ata/pata_falcon.c
> index 121635aa8c00..e2a88edd9dbf 100644
> --- a/drivers/ata/pata_falcon.c
> +++ b/drivers/ata/pata_falcon.c
> @@ -55,14 +55,14 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
> /* Transfer multiple of 2 bytes */
> if (rw == READ) {
> if (swap)
> - raw_insw_swapw((u16 *)data_addr, (u16 *)buf, words);
> + raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> else
> - raw_insw((u16 *)data_addr, (u16 *)buf, words);
> + raw_insw((u16 __iomem *)data_addr, (u16 *)buf, words);
> } else {
> if (swap)
> - raw_outsw_swapw((u16 *)data_addr, (u16 *)buf, words);
> + raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> else
> - raw_outsw((u16 *)data_addr, (u16 *)buf, words);
> + raw_outsw((u16 __iomem *)data_addr, (u16 *)buf, words);
> }
>
> /* Transfer trailing byte, if any. */
> @@ -74,16 +74,16 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
>
> if (rw == READ) {
> if (swap)
> - raw_insw_swapw((u16 *)data_addr, (u16 *)pad, 1);
> + raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)pad, 1);
> else
> - raw_insw((u16 *)data_addr, (u16 *)pad, 1);
> + raw_insw((u16 __iomem *)data_addr, (u16 *)pad, 1);
> *buf = pad[0];
> } else {
> pad[0] = *buf;
> if (swap)
> - raw_outsw_swapw((u16 *)data_addr, (u16 *)pad, 1);
> + raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)pad, 1);
> else
> - raw_outsw((u16 *)data_addr, (u16 *)pad, 1);
> + raw_outsw((u16 __iomem *)data_addr, (u16 *)pad, 1);
> }
> words++;
> }
>
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] pata_falcon: Add missing __iomem annotations
2021-11-24 4:52 ` Michael Schmitz
@ 2021-11-25 1:09 ` Finn Thain
0 siblings, 0 replies; 9+ messages in thread
From: Finn Thain @ 2021-11-25 1:09 UTC (permalink / raw)
To: Michael Schmitz
Cc: Damien Le Moal, Bartlomiej Zolnierkiewicz, Jens Axboe, linux-ide,
linux-kernel
On Wed, 24 Nov 2021, Michael Schmitz wrote:
> On 24/11/21 17:25, Finn Thain wrote:
> > The zero day bot reported some sparse complaints in pata_falcon.c. E.g.
> >
> > drivers/ata/pata_falcon.c:58:41: warning: cast removes address space
> > '__iomem' of expression
> > drivers/ata/pata_falcon.c:58:41: warning: incorrect type in argument 1
> > (different address spaces)
> > drivers/ata/pata_falcon.c:58:41: expected unsigned short volatile
> > [noderef] [usertype] __iomem *port
> > drivers/ata/pata_falcon.c:58:41: got unsigned short [usertype] *
> >
> > The same thing shows up in 8 places, all told. Avoid this by use of
> > __iomem type casts.
>
> Seeing as data_addr was explicitly typed as __iomem, your fix is clearly
> correct. Bit embarrassing to have missed that (I remember adding __iomem
> annotations elsewhere at some stage).
>
> If you think there's any need to test this change, say so.
>
There's no change in pata_falcon.o.
> Reviewed-by: Michael Schmitz <schmitzmic@gmail.com>
>
Thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] pata_falcon: Add missing __iomem annotations
2021-11-24 4:25 [PATCH] pata_falcon: Add missing __iomem annotations Finn Thain
2021-11-24 4:52 ` Michael Schmitz
@ 2021-11-24 7:51 ` Geert Uytterhoeven
2021-11-24 20:50 ` Michael Schmitz
2021-11-25 1:06 ` Finn Thain
1 sibling, 2 replies; 9+ messages in thread
From: Geert Uytterhoeven @ 2021-11-24 7:51 UTC (permalink / raw)
To: Finn Thain
Cc: Damien Le Moal, Michael Schmitz, Bartlomiej Zolnierkiewicz,
Jens Axboe, linux-ide, Linux Kernel Mailing List
Hi Finn,
On Wed, Nov 24, 2021 at 8:36 AM Finn Thain <fthain@linux-m68k.org> wrote:
> The zero day bot reported some sparse complaints in pata_falcon.c. E.g.
>
> drivers/ata/pata_falcon.c:58:41: warning: cast removes address space '__iomem' of expression
> drivers/ata/pata_falcon.c:58:41: warning: incorrect type in argument 1 (different address spaces)
> drivers/ata/pata_falcon.c:58:41: expected unsigned short volatile [noderef] [usertype] __iomem *port
> drivers/ata/pata_falcon.c:58:41: got unsigned short [usertype] *
>
> The same thing shows up in 8 places, all told. Avoid this by use of
> __iomem type casts.
>
> Cc: Michael Schmitz <schmitzmic@gmail.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Jens Axboe <axboe@kernel.dk>
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Finn Thain <fthain@linux-m68k.org>
Thanks for your patch!
> --- a/drivers/ata/pata_falcon.c
> +++ b/drivers/ata/pata_falcon.c
> @@ -55,14 +55,14 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
> /* Transfer multiple of 2 bytes */
> if (rw == READ) {
> if (swap)
> - raw_insw_swapw((u16 *)data_addr, (u16 *)buf, words);
> + raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> else
> - raw_insw((u16 *)data_addr, (u16 *)buf, words);
> + raw_insw((u16 __iomem *)data_addr, (u16 *)buf, words);
> } else {
> if (swap)
> - raw_outsw_swapw((u16 *)data_addr, (u16 *)buf, words);
> + raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> else
> - raw_outsw((u16 *)data_addr, (u16 *)buf, words);
> + raw_outsw((u16 __iomem *)data_addr, (u16 *)buf, words);
Can't you just drop the casts? data_addr is an __iomem void *.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] pata_falcon: Add missing __iomem annotations
2021-11-24 7:51 ` Geert Uytterhoeven
@ 2021-11-24 20:50 ` Michael Schmitz
2021-11-25 7:33 ` Geert Uytterhoeven
2021-11-25 1:06 ` Finn Thain
1 sibling, 1 reply; 9+ messages in thread
From: Michael Schmitz @ 2021-11-24 20:50 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Finn Thain, Damien Le Moal, Bartlomiej Zolnierkiewicz,
Jens Axboe, linux-ide, Linux Kernel Mailing List
Hi Geert,
On Wed, Nov 24, 2021 at 8:51 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > --- a/drivers/ata/pata_falcon.c
> > +++ b/drivers/ata/pata_falcon.c
> > @@ -55,14 +55,14 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
> > /* Transfer multiple of 2 bytes */
> > if (rw == READ) {
> > if (swap)
> > - raw_insw_swapw((u16 *)data_addr, (u16 *)buf, words);
> > + raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > else
> > - raw_insw((u16 *)data_addr, (u16 *)buf, words);
> > + raw_insw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > } else {
> > if (swap)
> > - raw_outsw_swapw((u16 *)data_addr, (u16 *)buf, words);
> > + raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > else
> > - raw_outsw((u16 *)data_addr, (u16 *)buf, words);
> > + raw_outsw((u16 __iomem *)data_addr, (u16 *)buf, words);
>
> Can't you just drop the casts? data_addr is an __iomem void *.
It's not u16 though, and the raw_ IO functions require that. But we
could cast data_addr as __iomem u16 * (compile tested).
Cheers,
Michael
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] pata_falcon: Add missing __iomem annotations
2021-11-24 20:50 ` Michael Schmitz
@ 2021-11-25 7:33 ` Geert Uytterhoeven
0 siblings, 0 replies; 9+ messages in thread
From: Geert Uytterhoeven @ 2021-11-25 7:33 UTC (permalink / raw)
To: Michael Schmitz
Cc: Finn Thain, Damien Le Moal, Bartlomiej Zolnierkiewicz,
Jens Axboe, linux-ide, Linux Kernel Mailing List
Hi Michael,
On Wed, Nov 24, 2021 at 9:50 PM Michael Schmitz <schmitzmic@gmail.com> wrote:
> On Wed, Nov 24, 2021 at 8:51 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > > --- a/drivers/ata/pata_falcon.c
> > > +++ b/drivers/ata/pata_falcon.c
> > > @@ -55,14 +55,14 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
> > > /* Transfer multiple of 2 bytes */
> > > if (rw == READ) {
> > > if (swap)
> > > - raw_insw_swapw((u16 *)data_addr, (u16 *)buf, words);
> > > + raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > > else
> > > - raw_insw((u16 *)data_addr, (u16 *)buf, words);
> > > + raw_insw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > > } else {
> > > if (swap)
> > > - raw_outsw_swapw((u16 *)data_addr, (u16 *)buf, words);
> > > + raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > > else
> > > - raw_outsw((u16 *)data_addr, (u16 *)buf, words);
> > > + raw_outsw((u16 __iomem *)data_addr, (u16 *)buf, words);
> >
> > Can't you just drop the casts? data_addr is an __iomem void *.
>
> It's not u16 though, and the raw_ IO functions require that. But we
> could cast data_addr as __iomem u16 * (compile tested).
The raw_ IO functions do not require that: you can pass a void *
to a function that expects a different pointer type.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] pata_falcon: Add missing __iomem annotations
2021-11-24 7:51 ` Geert Uytterhoeven
2021-11-24 20:50 ` Michael Schmitz
@ 2021-11-25 1:06 ` Finn Thain
2021-11-25 7:39 ` Geert Uytterhoeven
1 sibling, 1 reply; 9+ messages in thread
From: Finn Thain @ 2021-11-25 1:06 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Damien Le Moal, Michael Schmitz, Bartlomiej Zolnierkiewicz,
Jens Axboe, linux-ide, Linux Kernel Mailing List
On Wed, 24 Nov 2021, Geert Uytterhoeven wrote:
> Thanks for your patch!
>
> > --- a/drivers/ata/pata_falcon.c
> > +++ b/drivers/ata/pata_falcon.c
> > @@ -55,14 +55,14 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
> > /* Transfer multiple of 2 bytes */
> > if (rw == READ) {
> > if (swap)
> > - raw_insw_swapw((u16 *)data_addr, (u16 *)buf, words);
> > + raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > else
> > - raw_insw((u16 *)data_addr, (u16 *)buf, words);
> > + raw_insw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > } else {
> > if (swap)
> > - raw_outsw_swapw((u16 *)data_addr, (u16 *)buf, words);
> > + raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > else
> > - raw_outsw((u16 *)data_addr, (u16 *)buf, words);
> > + raw_outsw((u16 __iomem *)data_addr, (u16 *)buf, words);
>
> Can't you just drop the casts? data_addr is an __iomem void *.
>
Yes, that works here (i.e. removing the data_addr casts and not the buf
casts). But is it prudent?
Given the implementation of raw_in/out is subject to change, it seems like
the original casts were defensive programming.
Here's an example of a recent regression that was fixed by casting a macro
argument to a specific width:
https://lore.kernel.org/linuxppc-dev/79ae1f49-f6b1-e9ad-977d-0cc7e553c7b9@csgroup.eu/
https://lore.kernel.org/linuxppc-dev/08bbe7240b384016e0b2912ecf3bf5e2d25ef2c6.1636501628.git.fthain@linux-m68k.org/
BTW, that bug eventually got fixed using a different patch.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/arch/powerpc/kernel/signal.h?id=5499802b2284331788a440585869590f1bd63f7f
Note that __get_user_sigset() still lacks width casts here, so it remains
non-portable.
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] pata_falcon: Add missing __iomem annotations
2021-11-25 1:06 ` Finn Thain
@ 2021-11-25 7:39 ` Geert Uytterhoeven
2021-11-25 22:36 ` Finn Thain
0 siblings, 1 reply; 9+ messages in thread
From: Geert Uytterhoeven @ 2021-11-25 7:39 UTC (permalink / raw)
To: Finn Thain
Cc: Damien Le Moal, Michael Schmitz, Bartlomiej Zolnierkiewicz,
Jens Axboe, linux-ide, Linux Kernel Mailing List
Hi Finn,
On Thu, Nov 25, 2021 at 2:06 AM Finn Thain <fthain@linux-m68k.org> wrote:
> On Wed, 24 Nov 2021, Geert Uytterhoeven wrote:
> > > --- a/drivers/ata/pata_falcon.c
> > > +++ b/drivers/ata/pata_falcon.c
> > > @@ -55,14 +55,14 @@ static unsigned int pata_falcon_data_xfer(struct ata_queued_cmd *qc,
> > > /* Transfer multiple of 2 bytes */
> > > if (rw == READ) {
> > > if (swap)
> > > - raw_insw_swapw((u16 *)data_addr, (u16 *)buf, words);
> > > + raw_insw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > > else
> > > - raw_insw((u16 *)data_addr, (u16 *)buf, words);
> > > + raw_insw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > > } else {
> > > if (swap)
> > > - raw_outsw_swapw((u16 *)data_addr, (u16 *)buf, words);
> > > + raw_outsw_swapw((u16 __iomem *)data_addr, (u16 *)buf, words);
> > > else
> > > - raw_outsw((u16 *)data_addr, (u16 *)buf, words);
> > > + raw_outsw((u16 __iomem *)data_addr, (u16 *)buf, words);
> >
> > Can't you just drop the casts? data_addr is an __iomem void *.
>
> Yes, that works here (i.e. removing the data_addr casts and not the buf
> casts). But is it prudent?
>
> Given the implementation of raw_in/out is subject to change, it seems like
> the original casts were defensive programming.
>
> Here's an example of a recent regression that was fixed by casting a macro
> argument to a specific width:
>
> https://lore.kernel.org/linuxppc-dev/79ae1f49-f6b1-e9ad-977d-0cc7e553c7b9@csgroup.eu/
> https://lore.kernel.org/linuxppc-dev/08bbe7240b384016e0b2912ecf3bf5e2d25ef2c6.1636501628.git.fthain@linux-m68k.org/
Yeah, you do have to be careful with macros that derive a size from
the type of the passed data. The *{in,out}sw() functions do not suffer
from that: they are defined to operate on a 16-bit I/O register.
It is very unlikely these semantics will ever change.
Here I'm more worried about the other danger: keeping casts will
silence any warning that may be introduced in a future change to the
driver code.
BTW, insw() and readsw() in asm-generic take void *.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] pata_falcon: Add missing __iomem annotations
2021-11-25 7:39 ` Geert Uytterhoeven
@ 2021-11-25 22:36 ` Finn Thain
0 siblings, 0 replies; 9+ messages in thread
From: Finn Thain @ 2021-11-25 22:36 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Damien Le Moal, Michael Schmitz, Bartlomiej Zolnierkiewicz,
Jens Axboe, linux-ide, Linux Kernel Mailing List
On Thu, 25 Nov 2021, Geert Uytterhoeven wrote:
>
> Yeah, you do have to be careful with macros that derive a size from the
> type of the passed data. The *{in,out}sw() functions do not suffer from
> that: they are defined to operate on a 16-bit I/O register. It is very
> unlikely these semantics will ever change.
>
> Here I'm more worried about the other danger: keeping casts will silence
> any warning that may be introduced in a future change to the driver
> code.
>
Fair enough. I'll send v2.
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-11-25 22:38 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-24 4:25 [PATCH] pata_falcon: Add missing __iomem annotations Finn Thain
2021-11-24 4:52 ` Michael Schmitz
2021-11-25 1:09 ` Finn Thain
2021-11-24 7:51 ` Geert Uytterhoeven
2021-11-24 20:50 ` Michael Schmitz
2021-11-25 7:33 ` Geert Uytterhoeven
2021-11-25 1:06 ` Finn Thain
2021-11-25 7:39 ` Geert Uytterhoeven
2021-11-25 22:36 ` Finn Thain
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).