Stephen Hemminger wrote: > Not sure how you want to fix this: > > CHECK drivers/acpi/pci_irq.c > drivers/ata/ahci.c:323:2: warning: Initializer entry defined twice > drivers/ata/ahci.c:324:3: also defined here > > This happens because macro ATA_BASE_SHT(drv_name) defines .can_queue > then then later in the initializer for ahci_sht, it gets redefined. > > Granted it probably works now, but sparse is correct to complain that > this is bad usage. > Hmm... That use is pretty much intentional to provide generic default initial value and allow specific drivers to override if necessary and I believe it's a valid C usage, no? Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
