The patch titled potential parse error in ifdef has been added to the -mm tree. Its filename is potential-parse-error-in-ifdef.patch *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this ------------------------------------------------------ Subject: potential parse error in ifdef From: Yoann Padioleau <padator@xxxxxxxxxx> I have made a tool to parse the kernel that does not pre-process the source. That means that my parser tries to parse all the code, including code in the #else branch or code that is not often compiled because the driver is not very used (or not used at all). So, my parser sometimes reports parse error not originally detected by gcc. Here is my (first) patch. Signed-off-by: Yoann Padioleau <padator@xxxxxxxxxx> Acked-by: Matthew Wilcox <matthew@xxxxxx> Cc: Wim Van Sebroeck <wim@xxxxxxxxx> Cc: David Woodhouse <dwmw2@xxxxxxxxxxxxx> Cc: Jeff Garzik <jeff@xxxxxxxxxx> Cc: James Bottomley <James.Bottomley@xxxxxxxxxxxx> Cc: Russell King <rmk@xxxxxxxxxxxxxxxx> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/char/watchdog/ixp2000_wdt.c | 2 +- drivers/mtd/devices/pmc551.c | 2 +- drivers/mtd/nand/autcpu12.c | 2 +- drivers/mtd/nand/ppchameleonevb.c | 2 +- drivers/net/amd8111e.c | 2 +- drivers/net/skfp/smt.c | 2 +- drivers/scsi/aic7xxx/aic79xx_core.c | 2 +- sound/arm/sa11xx-uda1341.c | 2 +- 8 files changed, 8 insertions(+), 8 deletions(-) diff -puN drivers/char/watchdog/ixp2000_wdt.c~potential-parse-error-in-ifdef drivers/char/watchdog/ixp2000_wdt.c --- a/drivers/char/watchdog/ixp2000_wdt.c~potential-parse-error-in-ifdef +++ a/drivers/char/watchdog/ixp2000_wdt.c @@ -205,7 +205,7 @@ static void __exit ixp2000_wdt_exit(void module_init(ixp2000_wdt_init); module_exit(ixp2000_wdt_exit); -MODULE_AUTHOR("Deepak Saxena <dsaxena@xxxxxxxxxxx">); +MODULE_AUTHOR("Deepak Saxena <dsaxena@xxxxxxxxxxx>"); MODULE_DESCRIPTION("IXP2000 Network Processor Watchdog"); module_param(heartbeat, int, 0); diff -puN drivers/mtd/devices/pmc551.c~potential-parse-error-in-ifdef drivers/mtd/devices/pmc551.c --- a/drivers/mtd/devices/pmc551.c~potential-parse-error-in-ifdef +++ a/drivers/mtd/devices/pmc551.c @@ -650,7 +650,7 @@ MODULE_DESCRIPTION(PMC551_VERSION); */ static int msize = 0; #if defined(CONFIG_MTD_PMC551_APERTURE_SIZE) -static int asize = CONFIG_MTD_PMC551_APERTURE_SIZE +static int asize = CONFIG_MTD_PMC551_APERTURE_SIZE; #else static int asize = 0; #endif diff -puN drivers/mtd/nand/autcpu12.c~potential-parse-error-in-ifdef drivers/mtd/nand/autcpu12.c --- a/drivers/mtd/nand/autcpu12.c~potential-parse-error-in-ifdef +++ a/drivers/mtd/nand/autcpu12.c @@ -101,7 +101,7 @@ static void autcpu12_hwcontrol(struct mt struct nand_chip *chip = mtd->priv; if (ctrl & NAND_CTRL_CHANGE) { - void __iomem *addr + void __iomem *addr; unsigned char bits; addr = CS89712_VIRT_BASE + AUTCPU12_SMC_PORT_OFFSET; diff -puN drivers/mtd/nand/ppchameleonevb.c~potential-parse-error-in-ifdef drivers/mtd/nand/ppchameleonevb.c --- a/drivers/mtd/nand/ppchameleonevb.c~potential-parse-error-in-ifdef +++ a/drivers/mtd/nand/ppchameleonevb.c @@ -81,7 +81,7 @@ __setup("ppchameleonevb_fio_pbase=", ppc */ static struct mtd_partition partition_info_hi[] = { { .name = "PPChameleon HI Nand Flash", - offset = 0, + .offset = 0, .size = 128 * 1024 * 1024 } }; diff -puN drivers/net/amd8111e.c~potential-parse-error-in-ifdef drivers/net/amd8111e.c --- a/drivers/net/amd8111e.c~potential-parse-error-in-ifdef +++ a/drivers/net/amd8111e.c @@ -155,7 +155,7 @@ This function will write into PHY regist */ static int amd8111e_write_phy(struct amd8111e_priv* lp,int phy_id, int reg, u32 val) { - unsigned int repeat = REPEAT_CNT + unsigned int repeat = REPEAT_CNT; void __iomem *mmio = lp->mmio; unsigned int reg_val; diff -puN drivers/net/skfp/smt.c~potential-parse-error-in-ifdef drivers/net/skfp/smt.c --- a/drivers/net/skfp/smt.c~potential-parse-error-in-ifdef +++ a/drivers/net/skfp/smt.c @@ -1748,7 +1748,7 @@ char *addr_to_string(struct fddi_addr *a #endif #ifdef AM29K -smt_ifconfig(int argc, char *argv[]) +int smt_ifconfig(int argc, char *argv[]) { if (argc >= 2 && !strcmp(argv[0],"opt_bypass") && !strcmp(argv[1],"yes")) { diff -puN drivers/scsi/aic7xxx/aic79xx_core.c~potential-parse-error-in-ifdef drivers/scsi/aic7xxx/aic79xx_core.c --- a/drivers/scsi/aic7xxx/aic79xx_core.c~potential-parse-error-in-ifdef +++ a/drivers/scsi/aic7xxx/aic79xx_core.c @@ -5180,7 +5180,7 @@ ahd_handle_devreset(struct ahd_softc *ah cur_lun = lun; max_lun = lun; } - for (cur_lun <= max_lun; cur_lun++) { + for (;cur_lun <= max_lun; cur_lun++) { struct ahd_tmode_lstate* lstate; lstate = tstate->enabled_luns[cur_lun]; diff -puN sound/arm/sa11xx-uda1341.c~potential-parse-error-in-ifdef sound/arm/sa11xx-uda1341.c --- a/sound/arm/sa11xx-uda1341.c~potential-parse-error-in-ifdef +++ a/sound/arm/sa11xx-uda1341.c @@ -987,7 +987,7 @@ static int __init sa11xx_uda1341_init(vo if (platform_get_drvdata(device)) return 0; platform_device_unregister(device); - err = -ENODEV + err = -ENODEV; } else err = PTR_ERR(device); platform_driver_unregister(&sa11xx_uda1341_driver); _ Patches currently in -mm which might be from padator@xxxxxxxxxx are atari_pamsnetc-old-declaration-ritchie-style-fix.patch potential-parse-error-in-ifdef.patch - To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html