On Thu, Dec 5, 2013 at 11:43 PM, Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > On Thu, Dec 05, 2013 at 05:06:33PM +0800, ZHAO Gang wrote: >> On Tue, Dec 3, 2013 at 7:26 AM, Jingoo Han <jg1....@samsung.com> wrote: >> > Don't use DEFINE_PCI_DEVICE_TABLE macro, because this macro >> > is not preferred. >> > >> > Signed-off-by: Jingoo Han <jg1....@samsung.com> >> > >> >> I think you misunderstood the checkpatch.pl warning, it tells you what >> to do, not what not to do. >> >> WARNING: Use DEFINE_PCI_DEVICE_TABLE for struct pci_device_id >> >> This means use DEFINE_PCI_DEVICE_TABLE to replace struct >> pci_device_id, not reverse. > > No, checkpatch is wrong, and is being fixed, this patch is correct.
Oh, I didn't notice the recent checkpatch update, sorry for the noise. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
