Messages in this thread | | | Date | Fri, 6 Dec 2013 08:50:11 +0800 | Subject | Re: [PATCH 28/39] staging: remove DEFINE_PCI_DEVICE_TABLE macro | From | ZHAO Gang <> |
| |
On Thu, Dec 5, 2013 at 11:43 PM, Greg Kroah-Hartman <gregkh@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.han@samsung.com> wrote: >> > Don't use DEFINE_PCI_DEVICE_TABLE macro, because this macro >> > is not preferred. >> > >> > Signed-off-by: Jingoo Han <jg1.han@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.
| |