[PATCH 02/86] i2c/i801: linux/pci_ids.h -> uapi/linux/pci_ids.h

[PATCH 02/86] i2c/i801: linux/pci_ids.h -> uapi/linux/pci_ids.h

[Michael S. Tsirkin]

Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h,
update i2c documentation accordingly.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
 Documentation/i2c/busses/i2c-i801 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/i2c/busses/i2c-i801 b/Documentation/i2c/busses/i2c-i801
index 82f48f7..8036644 100644
--- a/Documentation/i2c/busses/i2c-i801
+++ b/Documentation/i2c/busses/i2c-i801
@@ -141,7 +141,7 @@ host bridge PCI device. Get yours with "lspci -n -v -s 00:00.0":
 
 Here the host bridge ID is 2570 (82865G/PE/P), the subvendor ID is 1043
 (Asus) and the subdevice ID is 80f2 (P4P800-X). You can find the symbolic
-names for the bridge ID and the subvendor ID in include/linux/pci_ids.h,
+names for the bridge ID and the subvendor ID in include/uapi/linux/pci_ids.h,
 and then add a case for your subdevice ID at the right place in
 drivers/pci/quirks.c. Then please give it very good testing, to make sure
 that the unhidden SMBus doesn't conflict with e.g. ACPI.
-- 
MST

[PATCH 39/86] i2c/i801: use uapi/linux/pci_ids.h directly

[Michael S. Tsirkin]

Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h,
use the new header directly so we can drop
the wrapper in include/linux/pci_ids.h.

Signed-off-by: Michael S. Tsirkin <mst-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
---
 drivers/i2c/busses/i2c-i801.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
index 8fafb25..948a1a2 100644
--- a/drivers/i2c/busses/i2c-i801.c
+++ b/drivers/i2c/busses/i2c-i801.c
@@ -164,7 +164,7 @@
 #define STATUS_FLAGS		(SMBHSTSTS_BYTE_DONE | SMBHSTSTS_INTR | \
 				 STATUS_ERROR_FLAGS)
 
-/* Older devices have their ID defined in <linux/pci_ids.h> */
+/* Older devices have their ID defined in <uapi/linux/pci_ids.h> */
 #define PCI_DEVICE_ID_INTEL_BAYTRAIL_SMBUS		0x0f12
 #define PCI_DEVICE_ID_INTEL_BRASWELL_SMBUS		0x2292
 #define PCI_DEVICE_ID_INTEL_COUGARPOINT_SMBUS		0x1c22
-- 
MST

Re: [PATCH 02/86] i2c/i801: linux/pci_ids.h -> uapi/linux/pci_ids.h

[Jean Delvare]

On Sun, 29 Mar 2015 15:37:30 +0200, Michael S. Tsirkin wrote:
> Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h,
> update i2c documentation accordingly.
> 
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> ---
>  Documentation/i2c/busses/i2c-i801 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/i2c/busses/i2c-i801 b/Documentation/i2c/busses/i2c-i801
> index 82f48f7..8036644 100644
> --- a/Documentation/i2c/busses/i2c-i801
> +++ b/Documentation/i2c/busses/i2c-i801
> @@ -141,7 +141,7 @@ host bridge PCI device. Get yours with "lspci -n -v -s 00:00.0":
>  
>  Here the host bridge ID is 2570 (82865G/PE/P), the subvendor ID is 1043
>  (Asus) and the subdevice ID is 80f2 (P4P800-X). You can find the symbolic
> -names for the bridge ID and the subvendor ID in include/linux/pci_ids.h,
> +names for the bridge ID and the subvendor ID in include/uapi/linux/pci_ids.h,
>  and then add a case for your subdevice ID at the right place in
>  drivers/pci/quirks.c. Then please give it very good testing, to make sure
>  that the unhidden SMBus doesn't conflict with e.g. ACPI.

No objection from me.

Reviewed-by: Jean Delvare <jdelvare@suse.de>

-- 
Jean Delvare
SUSE L3 Support

Re: [PATCH 39/86] i2c/i801: use uapi/linux/pci_ids.h directly

[Jean Delvare]

On Sun, 29 Mar 2015 15:40:42 +0200, Michael S. Tsirkin wrote:
> Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h,
> use the new header directly so we can drop
> the wrapper in include/linux/pci_ids.h.
> 
> Signed-off-by: Michael S. Tsirkin <mst-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> ---
>  drivers/i2c/busses/i2c-i801.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
> index 8fafb25..948a1a2 100644
> --- a/drivers/i2c/busses/i2c-i801.c
> +++ b/drivers/i2c/busses/i2c-i801.c
> @@ -164,7 +164,7 @@
>  #define STATUS_FLAGS		(SMBHSTSTS_BYTE_DONE | SMBHSTSTS_INTR | \
>  				 STATUS_ERROR_FLAGS)
>  
> -/* Older devices have their ID defined in <linux/pci_ids.h> */
> +/* Older devices have their ID defined in <uapi/linux/pci_ids.h> */
>  #define PCI_DEVICE_ID_INTEL_BAYTRAIL_SMBUS		0x0f12
>  #define PCI_DEVICE_ID_INTEL_BRASWELL_SMBUS		0x2292
>  #define PCI_DEVICE_ID_INTEL_COUGARPOINT_SMBUS		0x1c22

No objection from me.

Reviewed-by: Jean Delvare <jdelvare-l3A5Bk7waGM@public.gmane.org>


-- 
Jean Delvare
SUSE L3 Support

Re: [PATCH 02/86] i2c/i801: linux/pci_ids.h -> uapi/linux/pci_ids.h

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 388 bytes --]

On Sun, Mar 29, 2015 at 03:37:30PM +0200, Michael S. Tsirkin wrote:
> Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h,
> update i2c documentation accordingly.
> 
> Signed-off-by: Michael S. Tsirkin <mst-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

Acked-by: Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>

I don't know if I should take it. If so, let me know.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH 39/86] i2c/i801: use uapi/linux/pci_ids.h directly

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 436 bytes --]

On Sun, Mar 29, 2015 at 03:40:42PM +0200, Michael S. Tsirkin wrote:
> Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h,
> use the new header directly so we can drop
> the wrapper in include/linux/pci_ids.h.
> 
> Signed-off-by: Michael S. Tsirkin <mst-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

Acked-by: Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>

I don't know if I should take it. If so, let me know.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH 02/86] i2c/i801: linux/pci_ids.h -> uapi/linux/pci_ids.h

[Jean Delvare]

Hi Wolfram,

On Fri, 3 Apr 2015 21:09:17 +0200, Wolfram Sang wrote:
> On Sun, Mar 29, 2015 at 03:37:30PM +0200, Michael S. Tsirkin wrote:
> > Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h,
> > update i2c documentation accordingly.
> > 
> > Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> 
> Acked-by: Wolfram Sang <wsa@the-dreams.de>
> 
> I don't know if I should take it. If so, let me know.

Don't, the series was nacked meanwhile.

Thanks,
-- 
Jean Delvare
SUSE L3 Support