* [PATCH] qla1820 iomem annotations
@ 2004-10-20 23:58 viro
2004-10-21 9:00 ` Christoph Hellwig
0 siblings, 1 reply; 6+ messages in thread
From: viro @ 2004-10-20 23:58 UTC (permalink / raw)
To: Linus Torvalds; +Cc: linux-scsi
Signed-off-by: Al Viro <viro@parcelfarce.linux.theplanet.co.uk>
----
diff -urN RC9-bk4-aacraid/drivers/scsi/qla1280.c RC9-bk4-qla1820/drivers/scsi/qla1280.c
--- RC9-bk4-aacraid/drivers/scsi/qla1280.c Wed Oct 20 10:56:03 2004
+++ RC9-bk4-qla1820/drivers/scsi/qla1280.c Wed Oct 20 13:46:21 2004
@@ -540,7 +540,7 @@
struct list_head *);
static uint16_t qla1280_get_nvram_word(struct scsi_qla_host *, uint32_t);
static uint16_t qla1280_nvram_request(struct scsi_qla_host *, uint32_t);
-static uint16_t qla1280_debounce_register(volatile uint16_t *);
+static uint16_t qla1280_debounce_register(volatile uint16_t __iomem *);
static request_t *qla1280_req_pkt(struct scsi_qla_host *);
static int qla1280_check_for_dead_scsi_bus(struct scsi_qla_host *,
unsigned int);
@@ -949,7 +949,7 @@
static void qla1280_mailbox_timeout(unsigned long __data)
{
struct scsi_qla_host *ha = (struct scsi_qla_host *)__data;
- struct device_reg *reg;
+ struct device_reg __iomem *reg;
reg = ha->iobase;
ha->mailbox_out[0] = RD_REG_WORD(®->mailbox0);
@@ -1360,7 +1360,7 @@
qla1280_intr_handler(int irq, void *dev_id, struct pt_regs *regs)
{
struct scsi_qla_host *ha;
- struct device_reg *reg;
+ struct device_reg __iomem *reg;
u16 data;
int handled = 0;
@@ -1716,7 +1716,7 @@
static inline void
qla1280_enable_intrs(struct scsi_qla_host *ha)
{
- struct device_reg *reg;
+ struct device_reg __iomem *reg;
reg = ha->iobase;
/* enable risc and host interrupts */
@@ -1728,7 +1728,7 @@
static inline void
qla1280_disable_intrs(struct scsi_qla_host *ha)
{
- struct device_reg *reg;
+ struct device_reg __iomem *reg;
reg = ha->iobase;
/* disable risc and host interrupts */
@@ -1750,7 +1750,7 @@
static int __devinit
qla1280_initialize_adapter(struct scsi_qla_host *ha)
{
- struct device_reg *reg;
+ struct device_reg __iomem *reg;
int status;
int bus;
#if LINUX_VERSION_CODE > 0x020500
@@ -1921,7 +1921,7 @@
qla1280_chip_diag(struct scsi_qla_host *ha)
{
uint16_t mb[MAILBOX_REGISTER_COUNT];
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
int status = 0;
int cnt;
uint16_t data;
@@ -2513,7 +2513,7 @@
static int
qla1280_nvram_config(struct scsi_qla_host *ha)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
struct nvram *nv = &ha->nvram;
int bus, target, status = 0;
uint16_t mb[MAILBOX_REGISTER_COUNT];
@@ -2699,7 +2699,7 @@
static uint16_t
qla1280_nvram_request(struct scsi_qla_host *ha, uint32_t nv_cmd)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
int cnt;
uint16_t data = 0;
uint16_t reg_data;
@@ -2742,7 +2742,7 @@
static void
qla1280_nv_write(struct scsi_qla_host *ha, uint16_t data)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
WRT_REG_WORD(®->nvram, data | NV_SELECT);
RD_REG_WORD(®->id_l); /* Flush PCI write */
@@ -2773,13 +2773,14 @@
static int
qla1280_mailbox_command(struct scsi_qla_host *ha, uint8_t mr, uint16_t *mb)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
#if 0
LIST_HEAD(done_q);
#endif
int status = 0;
int cnt;
uint16_t *optr, *iptr;
+ uint16_t __iomem *mptr;
uint16_t data;
DECLARE_COMPLETION(wait);
struct timer_list timer;
@@ -2796,15 +2797,15 @@
* available before starting sending the command data
*/
/* Load mailbox registers. */
- optr = (uint16_t *) ®->mailbox0;
+ mptr = (uint16_t __iomem *) ®->mailbox0;
iptr = mb;
for (cnt = 0; cnt < MAILBOX_REGISTER_COUNT; cnt++) {
if (mr & BIT_0) {
- WRT_REG_WORD(optr, (*iptr));
+ WRT_REG_WORD(mptr, (*iptr));
}
mr >>= 1;
- optr++;
+ mptr++;
iptr++;
}
@@ -2880,7 +2881,7 @@
static void
qla1280_poll(struct scsi_qla_host *ha)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
uint16_t data;
LIST_HEAD(done_q);
@@ -3083,7 +3084,7 @@
static void
qla1280_reset_adapter(struct scsi_qla_host *ha)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
ENTER("qla1280_reset_adapter");
@@ -3147,7 +3148,7 @@
static int
qla1280_64bit_start_scsi(struct scsi_qla_host *ha, struct srb * sp)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
struct scsi_cmnd *cmd = sp->cmd;
cmd_a64_entry_t *pkt;
struct scatterlist *sg = NULL;
@@ -3433,7 +3434,7 @@
static int
qla1280_32bit_start_scsi(struct scsi_qla_host *ha, struct srb * sp)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
struct scsi_cmnd *cmd = sp->cmd;
struct cmd_entry *pkt;
struct scatterlist *sg = NULL;
@@ -3693,7 +3694,7 @@
static request_t *
qla1280_req_pkt(struct scsi_qla_host *ha)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
request_t *pkt = NULL;
int cnt;
uint32_t timer;
@@ -3761,7 +3762,7 @@
static void
qla1280_isp_cmd(struct scsi_qla_host *ha)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
ENTER("qla1280_isp_cmd");
@@ -3799,7 +3800,7 @@
static void
qla1280_isr(struct scsi_qla_host *ha, struct list_head *done_q)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
struct response *pkt;
struct srb *sp = NULL;
uint16_t mailbox[MAILBOX_REGISTER_COUNT];
@@ -4258,7 +4259,7 @@
static int
qla1280_abort_isp(struct scsi_qla_host *ha)
{
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
struct srb *sp;
int status = 0;
int cnt;
@@ -4336,7 +4337,7 @@
* register value.
*/
static u16
-qla1280_debounce_register(volatile u16 * addr)
+qla1280_debounce_register(volatile u16 __iomem * addr)
{
volatile u16 ret;
volatile u16 ret2;
@@ -4368,7 +4369,7 @@
qla1280_check_for_dead_scsi_bus(struct scsi_qla_host *ha, unsigned int bus)
{
uint16_t config_reg, scsi_control;
- struct device_reg *reg = ha->iobase;
+ struct device_reg __iomem *reg = ha->iobase;
if (ha->bus_settings[bus].scsi_bus_dead) {
WRT_REG_WORD(®->host_cmd, HC_PAUSE_RISC);
@@ -4787,7 +4788,7 @@
}
host->base = (unsigned long)ha->mmpbase;
- ha->iobase = (struct device_reg *)ha->mmpbase;
+ ha->iobase = (struct device_reg __iomem *)ha->mmpbase;
#else
host->io_port = pci_resource_start(ha->pdev, 0);
if (!request_region(host->io_port, 0xff, "qla1280")) {
diff -urN RC9-bk4-aacraid/drivers/scsi/qla1280.h RC9-bk4-qla1820/drivers/scsi/qla1280.h
--- RC9-bk4-aacraid/drivers/scsi/qla1280.h Mon Oct 18 22:04:50 2004
+++ RC9-bk4-qla1820/drivers/scsi/qla1280.h Wed Oct 20 13:46:24 2004
@@ -1038,9 +1038,9 @@
/* Linux adapter configuration data */
struct Scsi_Host *host; /* pointer to host data */
struct scsi_qla_host *next;
- struct device_reg *iobase; /* Base Memory-mapped I/O address */
+ struct device_reg __iomem *iobase; /* Base Memory-mapped I/O address */
- unsigned char *mmpbase; /* memory mapped address */
+ unsigned char __iomem *mmpbase; /* memory mapped address */
unsigned long host_no;
struct pci_dev *pdev;
uint8_t devnum;
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] qla1820 iomem annotations
2004-10-20 23:58 [PATCH] qla1820 iomem annotations viro
@ 2004-10-21 9:00 ` Christoph Hellwig
2004-10-21 14:24 ` Linus Torvalds
2004-10-21 15:14 ` Al Viro
0 siblings, 2 replies; 6+ messages in thread
From: Christoph Hellwig @ 2004-10-21 9:00 UTC (permalink / raw)
To: viro; +Cc: Linus Torvalds, linux-scsi
This breaks old-style PIO support in this driver which is required for the
Visual Workstations.
On Thu, Oct 21, 2004 at 12:58:37AM +0100, viro@parcelfarce.linux.theplanet.co.uk wrote:
> Signed-off-by: Al Viro <viro@parcelfarce.linux.theplanet.co.uk>
> ----
> diff -urN RC9-bk4-aacraid/drivers/scsi/qla1280.c RC9-bk4-qla1820/drivers/scsi/qla1280.c
> --- RC9-bk4-aacraid/drivers/scsi/qla1280.c Wed Oct 20 10:56:03 2004
> +++ RC9-bk4-qla1820/drivers/scsi/qla1280.c Wed Oct 20 13:46:21 2004
> @@ -540,7 +540,7 @@
> struct list_head *);
> static uint16_t qla1280_get_nvram_word(struct scsi_qla_host *, uint32_t);
> static uint16_t qla1280_nvram_request(struct scsi_qla_host *, uint32_t);
> -static uint16_t qla1280_debounce_register(volatile uint16_t *);
> +static uint16_t qla1280_debounce_register(volatile uint16_t __iomem *);
> static request_t *qla1280_req_pkt(struct scsi_qla_host *);
> static int qla1280_check_for_dead_scsi_bus(struct scsi_qla_host *,
> unsigned int);
> @@ -949,7 +949,7 @@
> static void qla1280_mailbox_timeout(unsigned long __data)
> {
> struct scsi_qla_host *ha = (struct scsi_qla_host *)__data;
> - struct device_reg *reg;
> + struct device_reg __iomem *reg;
> reg = ha->iobase;
>
> ha->mailbox_out[0] = RD_REG_WORD(®->mailbox0);
> @@ -1360,7 +1360,7 @@
> qla1280_intr_handler(int irq, void *dev_id, struct pt_regs *regs)
> {
> struct scsi_qla_host *ha;
> - struct device_reg *reg;
> + struct device_reg __iomem *reg;
> u16 data;
> int handled = 0;
>
> @@ -1716,7 +1716,7 @@
> static inline void
> qla1280_enable_intrs(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg;
> + struct device_reg __iomem *reg;
>
> reg = ha->iobase;
> /* enable risc and host interrupts */
> @@ -1728,7 +1728,7 @@
> static inline void
> qla1280_disable_intrs(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg;
> + struct device_reg __iomem *reg;
>
> reg = ha->iobase;
> /* disable risc and host interrupts */
> @@ -1750,7 +1750,7 @@
> static int __devinit
> qla1280_initialize_adapter(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg;
> + struct device_reg __iomem *reg;
> int status;
> int bus;
> #if LINUX_VERSION_CODE > 0x020500
> @@ -1921,7 +1921,7 @@
> qla1280_chip_diag(struct scsi_qla_host *ha)
> {
> uint16_t mb[MAILBOX_REGISTER_COUNT];
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> int status = 0;
> int cnt;
> uint16_t data;
> @@ -2513,7 +2513,7 @@
> static int
> qla1280_nvram_config(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> struct nvram *nv = &ha->nvram;
> int bus, target, status = 0;
> uint16_t mb[MAILBOX_REGISTER_COUNT];
> @@ -2699,7 +2699,7 @@
> static uint16_t
> qla1280_nvram_request(struct scsi_qla_host *ha, uint32_t nv_cmd)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> int cnt;
> uint16_t data = 0;
> uint16_t reg_data;
> @@ -2742,7 +2742,7 @@
> static void
> qla1280_nv_write(struct scsi_qla_host *ha, uint16_t data)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
>
> WRT_REG_WORD(®->nvram, data | NV_SELECT);
> RD_REG_WORD(®->id_l); /* Flush PCI write */
> @@ -2773,13 +2773,14 @@
> static int
> qla1280_mailbox_command(struct scsi_qla_host *ha, uint8_t mr, uint16_t *mb)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> #if 0
> LIST_HEAD(done_q);
> #endif
> int status = 0;
> int cnt;
> uint16_t *optr, *iptr;
> + uint16_t __iomem *mptr;
> uint16_t data;
> DECLARE_COMPLETION(wait);
> struct timer_list timer;
> @@ -2796,15 +2797,15 @@
> * available before starting sending the command data
> */
> /* Load mailbox registers. */
> - optr = (uint16_t *) ®->mailbox0;
> + mptr = (uint16_t __iomem *) ®->mailbox0;
> iptr = mb;
> for (cnt = 0; cnt < MAILBOX_REGISTER_COUNT; cnt++) {
> if (mr & BIT_0) {
> - WRT_REG_WORD(optr, (*iptr));
> + WRT_REG_WORD(mptr, (*iptr));
> }
>
> mr >>= 1;
> - optr++;
> + mptr++;
> iptr++;
> }
>
> @@ -2880,7 +2881,7 @@
> static void
> qla1280_poll(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> uint16_t data;
> LIST_HEAD(done_q);
>
> @@ -3083,7 +3084,7 @@
> static void
> qla1280_reset_adapter(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
>
> ENTER("qla1280_reset_adapter");
>
> @@ -3147,7 +3148,7 @@
> static int
> qla1280_64bit_start_scsi(struct scsi_qla_host *ha, struct srb * sp)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> struct scsi_cmnd *cmd = sp->cmd;
> cmd_a64_entry_t *pkt;
> struct scatterlist *sg = NULL;
> @@ -3433,7 +3434,7 @@
> static int
> qla1280_32bit_start_scsi(struct scsi_qla_host *ha, struct srb * sp)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> struct scsi_cmnd *cmd = sp->cmd;
> struct cmd_entry *pkt;
> struct scatterlist *sg = NULL;
> @@ -3693,7 +3694,7 @@
> static request_t *
> qla1280_req_pkt(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> request_t *pkt = NULL;
> int cnt;
> uint32_t timer;
> @@ -3761,7 +3762,7 @@
> static void
> qla1280_isp_cmd(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
>
> ENTER("qla1280_isp_cmd");
>
> @@ -3799,7 +3800,7 @@
> static void
> qla1280_isr(struct scsi_qla_host *ha, struct list_head *done_q)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> struct response *pkt;
> struct srb *sp = NULL;
> uint16_t mailbox[MAILBOX_REGISTER_COUNT];
> @@ -4258,7 +4259,7 @@
> static int
> qla1280_abort_isp(struct scsi_qla_host *ha)
> {
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
> struct srb *sp;
> int status = 0;
> int cnt;
> @@ -4336,7 +4337,7 @@
> * register value.
> */
> static u16
> -qla1280_debounce_register(volatile u16 * addr)
> +qla1280_debounce_register(volatile u16 __iomem * addr)
> {
> volatile u16 ret;
> volatile u16 ret2;
> @@ -4368,7 +4369,7 @@
> qla1280_check_for_dead_scsi_bus(struct scsi_qla_host *ha, unsigned int bus)
> {
> uint16_t config_reg, scsi_control;
> - struct device_reg *reg = ha->iobase;
> + struct device_reg __iomem *reg = ha->iobase;
>
> if (ha->bus_settings[bus].scsi_bus_dead) {
> WRT_REG_WORD(®->host_cmd, HC_PAUSE_RISC);
> @@ -4787,7 +4788,7 @@
> }
>
> host->base = (unsigned long)ha->mmpbase;
> - ha->iobase = (struct device_reg *)ha->mmpbase;
> + ha->iobase = (struct device_reg __iomem *)ha->mmpbase;
> #else
> host->io_port = pci_resource_start(ha->pdev, 0);
> if (!request_region(host->io_port, 0xff, "qla1280")) {
> diff -urN RC9-bk4-aacraid/drivers/scsi/qla1280.h RC9-bk4-qla1820/drivers/scsi/qla1280.h
> --- RC9-bk4-aacraid/drivers/scsi/qla1280.h Mon Oct 18 22:04:50 2004
> +++ RC9-bk4-qla1820/drivers/scsi/qla1280.h Wed Oct 20 13:46:24 2004
> @@ -1038,9 +1038,9 @@
> /* Linux adapter configuration data */
> struct Scsi_Host *host; /* pointer to host data */
> struct scsi_qla_host *next;
> - struct device_reg *iobase; /* Base Memory-mapped I/O address */
> + struct device_reg __iomem *iobase; /* Base Memory-mapped I/O address */
>
> - unsigned char *mmpbase; /* memory mapped address */
> + unsigned char __iomem *mmpbase; /* memory mapped address */
> unsigned long host_no;
> struct pci_dev *pdev;
> uint8_t devnum;
> -
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
---end quoted text---
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] qla1820 iomem annotations
2004-10-21 9:00 ` Christoph Hellwig
@ 2004-10-21 14:24 ` Linus Torvalds
2004-10-21 15:14 ` Al Viro
1 sibling, 0 replies; 6+ messages in thread
From: Linus Torvalds @ 2004-10-21 14:24 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: viro, linux-scsi
On Thu, 21 Oct 2004, Christoph Hellwig wrote:
>
> This breaks old-style PIO support in this driver which is required for the
> Visual Workstations.
Really? It doesn't even seem to change any code, it just adds annotations
that gcc won't care about anyway.
Linus
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] qla1820 iomem annotations
2004-10-21 9:00 ` Christoph Hellwig
2004-10-21 14:24 ` Linus Torvalds
@ 2004-10-21 15:14 ` Al Viro
2004-10-21 17:25 ` Christoph Hellwig
1 sibling, 1 reply; 6+ messages in thread
From: Al Viro @ 2004-10-21 15:14 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Linus Torvalds, linux-scsi
On Thu, Oct 21, 2004 at 10:00:32AM +0100, Christoph Hellwig wrote:
> This breaks old-style PIO support in this driver which is required for the
> Visual Workstations.
Where? __iomem is #defined to nothing for gcc. So that leaves only
the chunk below as theoretically possible source of changes, right?
> > @@ -2773,13 +2773,14 @@
> > static int
> > qla1280_mailbox_command(struct scsi_qla_host *ha, uint8_t mr, uint16_t *mb)
> > {
> > - struct device_reg *reg = ha->iobase;
> > + struct device_reg __iomem *reg = ha->iobase;
> > #if 0
> > LIST_HEAD(done_q);
> > #endif
> > int status = 0;
> > int cnt;
> > uint16_t *optr, *iptr;
> > + uint16_t __iomem *mptr;
> > uint16_t data;
> > DECLARE_COMPLETION(wait);
> > struct timer_list timer;
> > @@ -2796,15 +2797,15 @@
> > * available before starting sending the command data
> > */
> > /* Load mailbox registers. */
> > - optr = (uint16_t *) ®->mailbox0;
> > + mptr = (uint16_t __iomem *) ®->mailbox0;
> > iptr = mb;
> > for (cnt = 0; cnt < MAILBOX_REGISTER_COUNT; cnt++) {
> > if (mr & BIT_0) {
> > - WRT_REG_WORD(optr, (*iptr));
> > + WRT_REG_WORD(mptr, (*iptr));
> > }
> >
> > mr >>= 1;
> > - optr++;
> > + mptr++;
> > iptr++;
> > }
Now let's look at the function in question; the next place where optr is
used is
/* Load return mailbox registers. */
optr = mb;
so what we had was
optr = (uint16_t *) ®->mailbox0;
iptr = mb;
<loop using and modifying optr>
<code never using optr>
optr = mb;
Since optr is never used between the end of loop and reassignment, the code
above is equivalent to
{
uint16_t *foo = optr;
optr = (uint16_t *) ®->mailbox0;
iptr = mb;
<loop using and modifying optr>
optr = foo;
<code never using optr>
}
optr = mb;
which is, in turn, equivalent to
{
uint16_t *bar;
bar = (uint16_t *) ®->mailbox0;
iptr = mb;
<loop using and modifying optr> with s/optr/bar/
<code never using optr>
}
optr = mb;
s/bar/mptr/ and lift its declaration to the top of function and that's
what we have now.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] qla1820 iomem annotations
2004-10-21 15:14 ` Al Viro
@ 2004-10-21 17:25 ` Christoph Hellwig
2004-10-21 17:30 ` Al Viro
0 siblings, 1 reply; 6+ messages in thread
From: Christoph Hellwig @ 2004-10-21 17:25 UTC (permalink / raw)
To: Al Viro; +Cc: Linus Torvalds, linux-scsi
On Thu, Oct 21, 2004 at 04:14:06PM +0100, Al Viro wrote:
> On Thu, Oct 21, 2004 at 10:00:32AM +0100, Christoph Hellwig wrote:
> > This breaks old-style PIO support in this driver which is required for the
> > Visual Workstations.
>
> Where? __iomem is #defined to nothing for gcc. So that leaves only
> the chunk below as theoretically possible source of changes, right?
Sorry, this was totlly misworked. What I actually meant is that it gives
the sparse warnings again in the PIO-config, so you trade sparse warnings
for one variant vs ones for another variant.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] qla1820 iomem annotations
2004-10-21 17:25 ` Christoph Hellwig
@ 2004-10-21 17:30 ` Al Viro
0 siblings, 0 replies; 6+ messages in thread
From: Al Viro @ 2004-10-21 17:30 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Linus Torvalds, linux-scsi
On Thu, Oct 21, 2004 at 06:25:47PM +0100, Christoph Hellwig wrote:
> On Thu, Oct 21, 2004 at 04:14:06PM +0100, Al Viro wrote:
> > On Thu, Oct 21, 2004 at 10:00:32AM +0100, Christoph Hellwig wrote:
> > > This breaks old-style PIO support in this driver which is required for the
> > > Visual Workstations.
> >
> > Where? __iomem is #defined to nothing for gcc. So that leaves only
> > the chunk below as theoretically possible source of changes, right?
>
> Sorry, this was totlly misworked. What I actually meant is that it gives
> the sparse warnings again in the PIO-config, so you trade sparse warnings
> for one variant vs ones for another variant.
Huh? We _already_ have these guys as pointers; so PIO variants were not
any happier with non-iomem ones.
Details, please...
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2004-10-21 17:30 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-10-20 23:58 [PATCH] qla1820 iomem annotations viro
2004-10-21 9:00 ` Christoph Hellwig
2004-10-21 14:24 ` Linus Torvalds
2004-10-21 15:14 ` Al Viro
2004-10-21 17:25 ` Christoph Hellwig
2004-10-21 17:30 ` Al Viro
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).