On Mon, Jan 5, 2015 at 12:48 PM, Andreas Mohr <andi@xxxxxxxx> wrote: > Hi, > >> + if (size == 1) >> + *val = readb(addr); >> + else if (size == 2) >> + *val = readw(addr); >> + else >> + *val = readl(addr); > (pci_generic_config_read() and pci_generic_config_write()) > > switch() > default: WARN_ON(1); > ? If you look at PCI_OP_READ/PCI_OP_WRITE defines, the caller can only have size of 1, 2 or 4. AFAIK, config accesses don't go thru any other caller. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html