Re: [PATCH] aic7xxx_old: silence GCC warnings
On Mon, 2012-10-29 at 13:45 +0100, Paul Bolle wrote: > On Mon, 2012-10-29 at 12:17 +, James Bottomley wrote: > > mvsas has a maintainer: poke them harder > > According to MAINTAINERS that's you. Is Xiangliang Yu perhaps the actual > maintainer? Building the mvsas driver triggers identical warnings in v3.8-rc5: drivers/scsi/mvsas/mv_sas.c: In function ‘mvs_update_phyinfo’: drivers/scsi/mvsas/mv_sas.c:1156:34: warning: comparison between ‘enum sas_device_type’ and ‘enum sas_dev_type’ [-Wenum-compare] drivers/scsi/mvsas/mv_sas.c:1159:39: warning: comparison between ‘enum sas_device_type’ and ‘enum sas_dev_type’ [-Wenum-compare] And according to MAINTAINERS you're still the maintainer. So I'd guess I'm poking the maintainer now. Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] aic7xxx_old: silence GCC warnings
On Mon, 2012-10-29 at 12:17 +, James Bottomley wrote: > the aic7xxx_old driver is in deep maintenance; I don't think anyone can > actually test changes to it anymore, so we just keep it around unchanged > for the odd really old card that can't be driven by the current driver. > For this reason, we tend only to do tested bug fixes to it because > no-one will notice if any error is introduced. > > mvsas has a maintainer: poke them harder According to MAINTAINERS that's you. Is Xiangliang Yu perhaps the actual maintainer? > and aic94xx is currently > maintainerless, so it will depend on someone finding the time to test. Thanks, Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] aic7xxx_old: silence GCC warnings
On Mon, 2012-10-29 at 11:36 +0100, Paul Bolle wrote: > On Fri, 2012-09-21 at 11:28 +0200, Paul Bolle wrote: > > Building the aic7xxx_old driver triggers these GCC warnings: > > drivers/scsi/aic7xxx_old.c:7901:5: warning: case value '257' not in > > enumerated type 'ahc_chip' [-Wswitch] > > drivers/scsi/aic7xxx_old.c:7898:5: warning: case value '513' not in > > enumerated type 'ahc_chip' [-Wswitch] > > drivers/scsi/aic7xxx_old.c:8517:5: warning: case value '257' not in > > enumerated type 'ahc_chip' [-Wswitch] > > drivers/scsi/aic7xxx_old.c:8510:5: warning: case value '513' not in > > enumerated type 'ahc_chip' [-Wswitch] > > > > Fix these warnings by adopting the idiom used elsewhere in this driver. > > Since AHC_EISA and AHC_VL are only ever set for AHC_AIC7770 this fix > > should not lead to any functional change. > > > > Signed-off-by: Paul Bolle > > --- > > 0) I noticed these warnings while building v3.6-rc6 on current Fedora > > 17, using Fedora's default config. > > Identical warnings can be seen while building v3.7-rc3. What's the > status of my patch? Did anyone found some time to have a look at it? the aic7xxx_old driver is in deep maintenance; I don't think anyone can actually test changes to it anymore, so we just keep it around unchanged for the odd really old card that can't be driven by the current driver. For this reason, we tend only to do tested bug fixes to it because no-one will notice if any error is introduced. mvsas has a maintainer: poke them harder and aic94xx is currently maintainerless, so it will depend on someone finding the time to test. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] aic7xxx_old: silence GCC warnings
On Fri, 2012-09-21 at 11:28 +0200, Paul Bolle wrote: > Building the aic7xxx_old driver triggers these GCC warnings: > drivers/scsi/aic7xxx_old.c:7901:5: warning: case value '257' not in > enumerated type 'ahc_chip' [-Wswitch] > drivers/scsi/aic7xxx_old.c:7898:5: warning: case value '513' not in > enumerated type 'ahc_chip' [-Wswitch] > drivers/scsi/aic7xxx_old.c:8517:5: warning: case value '257' not in > enumerated type 'ahc_chip' [-Wswitch] > drivers/scsi/aic7xxx_old.c:8510:5: warning: case value '513' not in > enumerated type 'ahc_chip' [-Wswitch] > > Fix these warnings by adopting the idiom used elsewhere in this driver. > Since AHC_EISA and AHC_VL are only ever set for AHC_AIC7770 this fix > should not lead to any functional change. > > Signed-off-by: Paul Bolle > --- > 0) I noticed these warnings while building v3.6-rc6 on current Fedora > 17, using Fedora's default config. Identical warnings can be seen while building v3.7-rc3. What's the status of my patch? Did anyone found some time to have a look at it? Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] aic7xxx_old: silence GCC warnings
Building the aic7xxx_old driver triggers these GCC warnings: drivers/scsi/aic7xxx_old.c:7901:5: warning: case value '257' not in enumerated type 'ahc_chip' [-Wswitch] drivers/scsi/aic7xxx_old.c:7898:5: warning: case value '513' not in enumerated type 'ahc_chip' [-Wswitch] drivers/scsi/aic7xxx_old.c:8517:5: warning: case value '257' not in enumerated type 'ahc_chip' [-Wswitch] drivers/scsi/aic7xxx_old.c:8510:5: warning: case value '513' not in enumerated type 'ahc_chip' [-Wswitch] Fix these warnings by adopting the idiom used elsewhere in this driver. Since AHC_EISA and AHC_VL are only ever set for AHC_AIC7770 this fix should not lead to any functional change. Signed-off-by: Paul Bolle --- 0) I noticed these warnings while building v3.6-rc6 on current Fedora 17, using Fedora's default config. 1) Compile tested only. 2) This patch is not checkpatch clean. But this file actually triggers thousands of checkpatch errors and warnings: total: 4779 errors, 7528 warnings, 11149 lines checked So I didn't bother to fix the few errors and warnings this patch triggers by sticking to this file (lack of) coding style. drivers/scsi/aic7xxx_old.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/scsi/aic7xxx_old.c b/drivers/scsi/aic7xxx_old.c index 5b212f0..1a381c8 100644 --- a/drivers/scsi/aic7xxx_old.c +++ b/drivers/scsi/aic7xxx_old.c @@ -7893,12 +7893,12 @@ aic7xxx_register(struct scsi_host_template *template, struct aic7xxx_host *p, printk(KERN_INFO "(scsi%d) <%s> found at ", p->host_no, board_names[p->board_name_index]); - switch(p->chip) + switch(p->chip & ~AHC_CHIPID_MASK) { -case (AHC_AIC7770|AHC_EISA): +case AHC_EISA: printk("EISA slot %d\n", p->pci_device_fn); break; -case (AHC_AIC7770|AHC_VL): +case AHC_VL: printk("VLB slot %d\n", p->pci_device_fn); break; default: @@ -8505,16 +8505,16 @@ aic7xxx_load_seeprom(struct aic7xxx_host *p, unsigned char *sxfrctl1) { printk(KERN_INFO "aic7xxx: Loading serial EEPROM..."); } - switch (p->chip) + switch (p->chip & ~AHC_CHIPID_MASK) { -case (AHC_AIC7770|AHC_EISA): /* None of these adapters have seeproms. */ +case AHC_EISA: /* None of these adapters have seeproms. */ if (aic_inb(p, SCSICONF) & TERM_ENB) p->flags |= AHC_TERM_ENB_A; if ( (p->features & AHC_TWIN) && (aic_inb(p, SCSICONF + 1) & TERM_ENB) ) p->flags |= AHC_TERM_ENB_B; break; -case (AHC_AIC7770|AHC_VL): +case AHC_VL: have_seeprom = read_284x_seeprom(p, (struct seeprom_config *) scarray); break; -- 1.7.11.4 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html