[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120715133530.1ffc3f36@endymion.delvare>
Date: Sun, 15 Jul 2012 13:35:30 +0200
From: Jean Delvare <khali@...ux-fr.org>
To: Bjorn Helgaas <bhelgaas@...gle.com>
Cc: Amaury Decrême <amaury.decreme@...il.com>,
linux-i2c@...r.kernel.org, amalysh@....de, nelson@....com,
ben-linux@...ff.org, w.sang@...gutronix.de, rob@...dley.net,
jeffrey.t.kirsher@...el.com, akpm@...ux-foundation.org,
davem@...emloft.net, joe@...ches.com, ralf@...ux-mips.org,
dirk.brandewie@...il.com, jayachandranc@...logicmicro.com,
Xiangzhen.Ye@....com, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-pci@...r.kernel.org,
suzanne.decreme@...il.com
Subject: Re: [PATCH] i2c: sis964: bus driver
Hi Bjorn,
On Fri, 13 Jul 2012 09:36:07 -0600, Bjorn Helgaas wrote:
> On Fri, Jul 13, 2012 at 3:40 AM, Amaury Decrême
> <amaury.decreme@...il.com> wrote:
> > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
> > index ab741b0..0ffc982 100644
> > --- a/include/linux/pci_ids.h
> > +++ b/include/linux/pci_ids.h
> > @@ -699,6 +699,7 @@
> > #define PCI_DEVICE_ID_SI_961 0x0961
> > #define PCI_DEVICE_ID_SI_962 0x0962
> > #define PCI_DEVICE_ID_SI_963 0x0963
> > +#define PCI_DEVICE_ID_SI_964 0x0964
>
> Please read the comment at the top of this file; I don't think this
> addition qualifies as something that should be added.
>
> > #define PCI_DEVICE_ID_SI_965 0x0965
> > #define PCI_DEVICE_ID_SI_966 0x0966
> > #define PCI_DEVICE_ID_SI_968 0x0968
When you reply to a large patch, please limit the quoting to the
relevant portion.
Thanks,
--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists