* [PATCH v3 0/4] Correct vendor-prefix and document device isl29028
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Darshana Padmadas
Patchset documents correct and the deprecated vendor-prefix found by checkpatch
warning and also documents information of device isl29028 for compatibility.
Patchset also includes corrected vendor-prefix and device name in compatible
property for files with checkpatch warning of undocumented string "isil,isl29028".
Darshana Padmadas (4):
Documentation: devicetree: bindings: Document correct and deprecated
vendor-prefix with device isl29028
Documentation: devicetree: bindings: Document deprecated device vendor
name to fix related warning
Staging: iio: light: Added correct vendor-prefix for device isl29028
arch: arm: boot: dts: Added correct vendor-prefix with device name to
compatible property
Changes in v3:
- In PATCH 1/4 corrected alphabetical order of vendor-prefix and device name.
- In PATCH 2/4 inserted deprecated vendor-prefix in list in alphabetical order
and edited vendor/chip string.
- In PATCH 4/4 dropped deprecated string "isil,isl29028" and included correct
vendor-prefix isl with isl29028 to give "isl,isl29028" for device's compatible
property.
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 ++
Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
drivers/staging/iio/light/isl29028.c | 3 ++-
arch/arm/boot/dts/tegra30-cardhu.dtsi | 2 +-
4 files changed, 6 insertions(+), 2 deletions(-)
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread* [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: linux-arm-kernel
This patch documents the device isl29028 with its vendor-prefix. Undocumented deprecated vendor-prefix
found by checkpatch also documented for compatibility reasons.
Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
---
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
index 6af570e..d024613 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -57,6 +57,8 @@ gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire In
infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
isl,isl12057 Intersil ISL12057 I2C RTC Chip
+isil,isl29028 Deprecated name, use isl,isl29028 instead
+isl,isl29028 Intersil ISL29028 Ambient Light and Proximity Sensor
maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
maxim,max6625 9-Bit/12-Bit Temperature Sensors with I?C-Compatible Serial Interface
--
1.9.1
^ permalink raw reply related [flat|nested] 36+ messages in thread
* [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Darshana Padmadas
This patch documents the device isl29028 with its vendor-prefix. Undocumented deprecated vendor-prefix
found by checkpatch also documented for compatibility reasons.
Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
index 6af570e..d024613 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -57,6 +57,8 @@ gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire In
infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
isl,isl12057 Intersil ISL12057 I2C RTC Chip
+isil,isl29028 Deprecated name, use isl,isl29028 instead
+isl,isl29028 Intersil ISL29028 Ambient Light and Proximity Sensor
maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
maxim,max6625 9-Bit/12-Bit Temperature Sensors with I²C-Compatible Serial Interface
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 36+ messages in thread
* [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028
@ 2014-10-07 11:46 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 11:46 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 07 October 2014 17:15:05 Darshana Padmadas wrote:
> This patch documents the device isl29028 with its vendor-prefix. Undocumented deprecated vendor-prefix
> found by checkpatch also documented for compatibility reasons.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028
@ 2014-10-07 11:46 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 11:46 UTC (permalink / raw)
To: Darshana Padmadas
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Tuesday 07 October 2014 17:15:05 Darshana Padmadas wrote:
> This patch documents the device isl29028 with its vendor-prefix. Undocumented deprecated vendor-prefix
> found by checkpatch also documented for compatibility reasons.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>
Acked-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028
@ 2014-10-07 12:02 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:02 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 07, 2014 at 12:45:05PM +0100, Darshana Padmadas wrote:
> This patch documents the device isl29028 with its vendor-prefix. Undocumented deprecated vendor-prefix
> found by checkpatch also documented for compatibility reasons.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
Acked-by: Mark Rutland <mark.rutland@arm.com>
> ---
> Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> index 6af570e..d024613 100644
> --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> @@ -57,6 +57,8 @@ gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire In
> infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
> infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
> isl,isl12057 Intersil ISL12057 I2C RTC Chip
> +isil,isl29028 Deprecated name, use isl,isl29028 instead
> +isl,isl29028 Intersil ISL29028 Ambient Light and Proximity Sensor
> maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
> maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
> maxim,max6625 9-Bit/12-Bit Temperature Sensors with I??C-Compatible Serial Interface
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028
@ 2014-10-07 12:02 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:02 UTC (permalink / raw)
To: Darshana Padmadas
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Pawel Moll,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
On Tue, Oct 07, 2014 at 12:45:05PM +0100, Darshana Padmadas wrote:
> This patch documents the device isl29028 with its vendor-prefix. Undocumented deprecated vendor-prefix
> found by checkpatch also documented for compatibility reasons.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Acked-by: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>
> ---
> Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> index 6af570e..d024613 100644
> --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> @@ -57,6 +57,8 @@ gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire In
> infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
> infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
> isl,isl12057 Intersil ISL12057 I2C RTC Chip
> +isil,isl29028 Deprecated name, use isl,isl29028 instead
> +isl,isl29028 Intersil ISL29028 Ambient Light and Proximity Sensor
> maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
> maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
> maxim,max6625 9-Bit/12-Bit Temperature Sensors with I²C-Compatible Serial Interface
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028
2014-10-07 12:02 ` Mark Rutland
@ 2014-10-07 12:17 ` Mark Rutland
-1 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:17 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 07, 2014 at 01:02:49PM +0100, Mark Rutland wrote:
> On Tue, Oct 07, 2014 at 12:45:05PM +0100, Darshana Padmadas wrote:
> > This patch documents the device isl29028 with its vendor-prefix. Undocumented deprecated vendor-prefix
> > found by checkpatch also documented for compatibility reasons.
> >
> > Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
>
> Acked-by: Mark Rutland <mark.rutland@arm.com>
>
> > ---
> > Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> > index 6af570e..d024613 100644
> > --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> > +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> > @@ -57,6 +57,8 @@ gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire In
> > infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
> > infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
> > isl,isl12057 Intersil ISL12057 I2C RTC Chip
> > +isil,isl29028 Deprecated name, use isl,isl29028 instead
Looking again, could you change the comment to "(deprecated, use isl)",
as I mentioned previously?
Then that will match what we have for powervr, which seems to be the
only other instance of "deprecated" in vendor-prefixes.txt.
Cheers,
Mark.
> > +isl,isl29028 Intersil ISL29028 Ambient Light and Proximity Sensor
> > maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
> > maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
> > maxim,max6625 9-Bit/12-Bit Temperature Sensors with I??C-Compatible Serial Interface
> > --
> > 1.9.1
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe devicetree" in
> > the body of a message to majordomo at vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028
@ 2014-10-07 12:17 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:17 UTC (permalink / raw)
To: Darshana Padmadas
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Pawel Moll,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
On Tue, Oct 07, 2014 at 01:02:49PM +0100, Mark Rutland wrote:
> On Tue, Oct 07, 2014 at 12:45:05PM +0100, Darshana Padmadas wrote:
> > This patch documents the device isl29028 with its vendor-prefix. Undocumented deprecated vendor-prefix
> > found by checkpatch also documented for compatibility reasons.
> >
> > Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>
> Acked-by: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>
>
> > ---
> > Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> > index 6af570e..d024613 100644
> > --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> > +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
> > @@ -57,6 +57,8 @@ gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire In
> > infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
> > infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
> > isl,isl12057 Intersil ISL12057 I2C RTC Chip
> > +isil,isl29028 Deprecated name, use isl,isl29028 instead
Looking again, could you change the comment to "(deprecated, use isl)",
as I mentioned previously?
Then that will match what we have for powervr, which seems to be the
only other instance of "deprecated" in vendor-prefixes.txt.
Cheers,
Mark.
> > +isl,isl29028 Intersil ISL29028 Ambient Light and Proximity Sensor
> > maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
> > maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
> > maxim,max6625 9-Bit/12-Bit Temperature Sensors with I²C-Compatible Serial Interface
> > --
> > 1.9.1
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe devicetree" in
> > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: linux-arm-kernel
This patch documents deprecated vendor name of device isl29028 for compatibility with older kernels.
Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
---
Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index ac7269f..e2503b8 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -68,6 +68,7 @@ img Imagination Technologies Ltd.
intel Intel Corporation
intercontrol Inter Control Group
isee ISEE 2007 S.L.
+isil Intersil (deprecated, use isl)
isl Intersil
karo Ka-Ro electronics GmbH
keymile Keymile GmbH
--
1.9.1
^ permalink raw reply related [flat|nested] 36+ messages in thread
* [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Darshana Padmadas
This patch documents deprecated vendor name of device isl29028 for compatibility with older kernels.
Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index ac7269f..e2503b8 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -68,6 +68,7 @@ img Imagination Technologies Ltd.
intel Intel Corporation
intercontrol Inter Control Group
isee ISEE 2007 S.L.
+isil Intersil (deprecated, use isl)
isl Intersil
karo Ka-Ro electronics GmbH
keymile Keymile GmbH
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 36+ messages in thread
* [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning
@ 2014-10-07 11:46 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 11:46 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 07 October 2014 17:15:06 Darshana Padmadas wrote:
> This patch documents deprecated vendor name of device isl29028 for compatibility with older kernels.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning
@ 2014-10-07 11:46 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 11:46 UTC (permalink / raw)
To: Darshana Padmadas
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Tuesday 07 October 2014 17:15:06 Darshana Padmadas wrote:
> This patch documents deprecated vendor name of device isl29028 for compatibility with older kernels.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>
Acked-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning
@ 2014-10-07 12:03 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:03 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 07, 2014 at 12:45:06PM +0100, Darshana Padmadas wrote:
> This patch documents deprecated vendor name of device isl29028 for compatibility with older kernels.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
Acked-by: Mark Rutland Mark.rutland at arm.com>
> ---
> Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index ac7269f..e2503b8 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -68,6 +68,7 @@ img Imagination Technologies Ltd.
> intel Intel Corporation
> intercontrol Inter Control Group
> isee ISEE 2007 S.L.
> +isil Intersil (deprecated, use isl)
> isl Intersil
> karo Ka-Ro electronics GmbH
> keymile Keymile GmbH
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning
@ 2014-10-07 12:03 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:03 UTC (permalink / raw)
To: Darshana Padmadas
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Pawel Moll,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
On Tue, Oct 07, 2014 at 12:45:06PM +0100, Darshana Padmadas wrote:
> This patch documents deprecated vendor name of device isl29028 for compatibility with older kernels.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Acked-by: Mark Rutland Mark.rutland-5wv7dgnIgG8@public.gmane.org>
> ---
> Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index ac7269f..e2503b8 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -68,6 +68,7 @@ img Imagination Technologies Ltd.
> intel Intel Corporation
> intercontrol Inter Control Group
> isee ISEE 2007 S.L.
> +isil Intersil (deprecated, use isl)
> isl Intersil
> karo Ka-Ro electronics GmbH
> keymile Keymile GmbH
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds the correct vendor-prefix for device isl29028 and
maintains deprecated vendor-prefix found by checkpatch warning
for older kernel releases.
Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
---
drivers/staging/iio/light/isl29028.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/iio/light/isl29028.c b/drivers/staging/iio/light/isl29028.c
index 6014625..60b02a8 100644
--- a/drivers/staging/iio/light/isl29028.c
+++ b/drivers/staging/iio/light/isl29028.c
@@ -537,7 +537,8 @@ static const struct i2c_device_id isl29028_id[] = {
MODULE_DEVICE_TABLE(i2c, isl29028_id);
static const struct of_device_id isl29028_of_match[] = {
- { .compatible = "isil,isl29028", },
+ { .compatible = "isl,isl29028", },
+ { .compatible = "isil,isl29028", }, /* deprecated, don't use */
{ },
};
MODULE_DEVICE_TABLE(of, isl29028_of_match);
--
1.9.1
^ permalink raw reply related [flat|nested] 36+ messages in thread* [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Darshana Padmadas
This patch adds the correct vendor-prefix for device isl29028 and
maintains deprecated vendor-prefix found by checkpatch warning
for older kernel releases.
Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
drivers/staging/iio/light/isl29028.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/iio/light/isl29028.c b/drivers/staging/iio/light/isl29028.c
index 6014625..60b02a8 100644
--- a/drivers/staging/iio/light/isl29028.c
+++ b/drivers/staging/iio/light/isl29028.c
@@ -537,7 +537,8 @@ static const struct i2c_device_id isl29028_id[] = {
MODULE_DEVICE_TABLE(i2c, isl29028_id);
static const struct of_device_id isl29028_of_match[] = {
- { .compatible = "isil,isl29028", },
+ { .compatible = "isl,isl29028", },
+ { .compatible = "isil,isl29028", }, /* deprecated, don't use */
{ },
};
MODULE_DEVICE_TABLE(of, isl29028_of_match);
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 36+ messages in thread* [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028
@ 2014-10-07 11:46 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 11:46 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 07 October 2014 17:15:07 Darshana Padmadas wrote:
> This patch adds the correct vendor-prefix for device isl29028 and
> maintains deprecated vendor-prefix found by checkpatch warning
> for older kernel releases.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028
@ 2014-10-07 11:46 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 11:46 UTC (permalink / raw)
To: Darshana Padmadas
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Tuesday 07 October 2014 17:15:07 Darshana Padmadas wrote:
> This patch adds the correct vendor-prefix for device isl29028 and
> maintains deprecated vendor-prefix found by checkpatch warning
> for older kernel releases.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>
Acked-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028
@ 2014-10-07 12:08 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:08 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 07, 2014 at 12:45:07PM +0100, Darshana Padmadas wrote:
> This patch adds the correct vendor-prefix for device isl29028 and
> maintains deprecated vendor-prefix found by checkpatch warning
> for older kernel releases.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
Acked-by: Mark Rutland <mark.rutland@arm.com>
> ---
> drivers/staging/iio/light/isl29028.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/iio/light/isl29028.c b/drivers/staging/iio/light/isl29028.c
> index 6014625..60b02a8 100644
> --- a/drivers/staging/iio/light/isl29028.c
> +++ b/drivers/staging/iio/light/isl29028.c
> @@ -537,7 +537,8 @@ static const struct i2c_device_id isl29028_id[] = {
> MODULE_DEVICE_TABLE(i2c, isl29028_id);
>
> static const struct of_device_id isl29028_of_match[] = {
> - { .compatible = "isil,isl29028", },
> + { .compatible = "isl,isl29028", },
> + { .compatible = "isil,isl29028", }, /* deprecated, don't use */
> { },
> };
> MODULE_DEVICE_TABLE(of, isl29028_of_match);
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 36+ messages in thread* Re: [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028
@ 2014-10-07 12:08 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:08 UTC (permalink / raw)
To: Darshana Padmadas
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Pawel Moll,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
On Tue, Oct 07, 2014 at 12:45:07PM +0100, Darshana Padmadas wrote:
> This patch adds the correct vendor-prefix for device isl29028 and
> maintains deprecated vendor-prefix found by checkpatch warning
> for older kernel releases.
>
> Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Acked-by: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>
> ---
> drivers/staging/iio/light/isl29028.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/iio/light/isl29028.c b/drivers/staging/iio/light/isl29028.c
> index 6014625..60b02a8 100644
> --- a/drivers/staging/iio/light/isl29028.c
> +++ b/drivers/staging/iio/light/isl29028.c
> @@ -537,7 +537,8 @@ static const struct i2c_device_id isl29028_id[] = {
> MODULE_DEVICE_TABLE(i2c, isl29028_id);
>
> static const struct of_device_id isl29028_of_match[] = {
> - { .compatible = "isil,isl29028", },
> + { .compatible = "isl,isl29028", },
> + { .compatible = "isil,isl29028", }, /* deprecated, don't use */
> { },
> };
> MODULE_DEVICE_TABLE(of, isl29028_of_match);
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds the correct vendor-prefix listed in Documentation/devicetree/bindings/vendor-prefixes.txt
with the device name to compatible property.
Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
---
arch/arm/boot/dts/tegra30-cardhu.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi
index 2063795..dcc6c75 100644
--- a/arch/arm/boot/dts/tegra30-cardhu.dtsi
+++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi
@@ -187,7 +187,7 @@
/* ALS and Proximity sensor */
isl29028 at 44 {
- compatible = "isil,isl29028";
+ compatible = "isl,isl29028";
reg = <0x44>;
interrupt-parent = <&gpio>;
interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
--
1.9.1
^ permalink raw reply related [flat|nested] 36+ messages in thread* [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 11:45 ` Darshana Padmadas
0 siblings, 0 replies; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Darshana Padmadas
This patch adds the correct vendor-prefix listed in Documentation/devicetree/bindings/vendor-prefixes.txt
with the device name to compatible property.
Signed-off-by: Darshana Padmadas <darshanapadmadas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
arch/arm/boot/dts/tegra30-cardhu.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi
index 2063795..dcc6c75 100644
--- a/arch/arm/boot/dts/tegra30-cardhu.dtsi
+++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi
@@ -187,7 +187,7 @@
/* ALS and Proximity sensor */
isl29028@44 {
- compatible = "isil,isl29028";
+ compatible = "isl,isl29028";
reg = <0x44>;
interrupt-parent = <&gpio>;
interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 36+ messages in thread* [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 11:49 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 11:49 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> /* ALS and Proximity sensor */
> isl29028 at 44 {
> - compatible = "isil,isl29028";
> + compatible = "isl,isl29028";
> reg = <0x44>;
> interrupt-parent = <&gpio>;
> interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
>
Does this intentionally list only the new string?
If we want compatibility between new dts files and old kernels, this would
need to be
compatible = "isl,isl29028", compatible = "isil,isl29028";
I don't know if we want to go that far though, given that this is
only a staging driver.
Arnd
^ permalink raw reply [flat|nested] 36+ messages in thread* Re: [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 11:49 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 11:49 UTC (permalink / raw)
To: Darshana Padmadas
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
grant.likely-QSEj5FYQhm4dnm+yROfE0A,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> /* ALS and Proximity sensor */
> isl29028@44 {
> - compatible = "isil,isl29028";
> + compatible = "isl,isl29028";
> reg = <0x44>;
> interrupt-parent = <&gpio>;
> interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
>
Does this intentionally list only the new string?
If we want compatibility between new dts files and old kernels, this would
need to be
compatible = "isl,isl29028", compatible = "isil,isl29028";
I don't know if we want to go that far though, given that this is
only a staging driver.
Arnd
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread* [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
2014-10-07 11:49 ` Arnd Bergmann
@ 2014-10-07 12:08 ` Mark Rutland
-1 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:08 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 07, 2014 at 12:49:02PM +0100, Arnd Bergmann wrote:
> On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> > /* ALS and Proximity sensor */
> > isl29028 at 44 {
> > - compatible = "isil,isl29028";
> > + compatible = "isl,isl29028";
> > reg = <0x44>;
> > interrupt-parent = <&gpio>;
> > interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
> >
>
> Does this intentionally list only the new string?
>
> If we want compatibility between new dts files and old kernels, this would
> need to be
>
> compatible = "isl,isl29028", compatible = "isil,isl29028";
Are we aiming for new dts wto work with old kernels? It's nice if they
do, but I don't think it should be necessary. Old DTBs should definitely
work with a new kernel, though.
Personally I'd rather see one string (preferably the canonical one, but
the deprecated one is ok) than two strings when those strings are for
the exact same device.
Mark.
^ permalink raw reply [flat|nested] 36+ messages in thread* Re: [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 12:08 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:08 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Darshana Padmadas,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Pawel Moll,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
On Tue, Oct 07, 2014 at 12:49:02PM +0100, Arnd Bergmann wrote:
> On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> > /* ALS and Proximity sensor */
> > isl29028@44 {
> > - compatible = "isil,isl29028";
> > + compatible = "isl,isl29028";
> > reg = <0x44>;
> > interrupt-parent = <&gpio>;
> > interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
> >
>
> Does this intentionally list only the new string?
>
> If we want compatibility between new dts files and old kernels, this would
> need to be
>
> compatible = "isl,isl29028", compatible = "isil,isl29028";
Are we aiming for new dts wto work with old kernels? It's nice if they
do, but I don't think it should be necessary. Old DTBs should definitely
work with a new kernel, though.
Personally I'd rather see one string (preferably the canonical one, but
the deprecated one is ok) than two strings when those strings are for
the exact same device.
Mark.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread* [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
2014-10-07 12:08 ` Mark Rutland
@ 2014-10-07 12:31 ` Arnd Bergmann
-1 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 12:31 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 07 October 2014 13:08:12 Mark Rutland wrote:
> On Tue, Oct 07, 2014 at 12:49:02PM +0100, Arnd Bergmann wrote:
> > On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> > > /* ALS and Proximity sensor */
> > > isl29028 at 44 {
> > > - compatible = "isil,isl29028";
> > > + compatible = "isl,isl29028";
> > > reg = <0x44>;
> > > interrupt-parent = <&gpio>;
> > > interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
> > >
> >
> > Does this intentionally list only the new string?
> >
> > If we want compatibility between new dts files and old kernels, this would
> > need to be
> >
> > compatible = "isl,isl29028", compatible = "isil,isl29028";
>
> Are we aiming for new dts wto work with old kernels? It's nice if they
> do, but I don't think it should be necessary. Old DTBs should definitely
> work with a new kernel, though.
>
> Personally I'd rather see one string (preferably the canonical one, but
> the deprecated one is ok) than two strings when those strings are for
> the exact same device.
>
Ok, fair enough. I think we should have the new name in dts files
normally then, in order to avoid spreading the mistake further when
someone copies it.
Arnd
^ permalink raw reply [flat|nested] 36+ messages in thread* Re: [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 12:31 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 12:31 UTC (permalink / raw)
To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Cc: Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Pawel Moll, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
Darshana Padmadas
On Tuesday 07 October 2014 13:08:12 Mark Rutland wrote:
> On Tue, Oct 07, 2014 at 12:49:02PM +0100, Arnd Bergmann wrote:
> > On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> > > /* ALS and Proximity sensor */
> > > isl29028@44 {
> > > - compatible = "isil,isl29028";
> > > + compatible = "isl,isl29028";
> > > reg = <0x44>;
> > > interrupt-parent = <&gpio>;
> > > interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
> > >
> >
> > Does this intentionally list only the new string?
> >
> > If we want compatibility between new dts files and old kernels, this would
> > need to be
> >
> > compatible = "isl,isl29028", compatible = "isil,isl29028";
>
> Are we aiming for new dts wto work with old kernels? It's nice if they
> do, but I don't think it should be necessary. Old DTBs should definitely
> work with a new kernel, though.
>
> Personally I'd rather see one string (preferably the canonical one, but
> the deprecated one is ok) than two strings when those strings are for
> the exact same device.
>
Ok, fair enough. I think we should have the new name in dts files
normally then, in order to avoid spreading the mistake further when
someone copies it.
Arnd
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread* [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
2014-10-07 12:31 ` Arnd Bergmann
@ 2014-10-07 12:35 ` Mark Rutland
-1 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:35 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 07, 2014 at 01:31:23PM +0100, Arnd Bergmann wrote:
> On Tuesday 07 October 2014 13:08:12 Mark Rutland wrote:
> > On Tue, Oct 07, 2014 at 12:49:02PM +0100, Arnd Bergmann wrote:
> > > On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> > > > /* ALS and Proximity sensor */
> > > > isl29028 at 44 {
> > > > - compatible = "isil,isl29028";
> > > > + compatible = "isl,isl29028";
> > > > reg = <0x44>;
> > > > interrupt-parent = <&gpio>;
> > > > interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
> > > >
> > >
> > > Does this intentionally list only the new string?
> > >
> > > If we want compatibility between new dts files and old kernels, this would
> > > need to be
> > >
> > > compatible = "isl,isl29028", compatible = "isil,isl29028";
> >
> > Are we aiming for new dts wto work with old kernels? It's nice if they
> > do, but I don't think it should be necessary. Old DTBs should definitely
> > work with a new kernel, though.
> >
> > Personally I'd rather see one string (preferably the canonical one, but
> > the deprecated one is ok) than two strings when those strings are for
> > the exact same device.
> >
>
> Ok, fair enough. I think we should have the new name in dts files
> normally then, in order to avoid spreading the mistake further when
> someone copies it.
Sounds good to me.
Mark.
^ permalink raw reply [flat|nested] 36+ messages in thread* Re: [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 12:35 ` Mark Rutland
0 siblings, 0 replies; 36+ messages in thread
From: Mark Rutland @ 2014-10-07 12:35 UTC (permalink / raw)
To: Arnd Bergmann
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Pawel Moll,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
Darshana Padmadas
On Tue, Oct 07, 2014 at 01:31:23PM +0100, Arnd Bergmann wrote:
> On Tuesday 07 October 2014 13:08:12 Mark Rutland wrote:
> > On Tue, Oct 07, 2014 at 12:49:02PM +0100, Arnd Bergmann wrote:
> > > On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> > > > /* ALS and Proximity sensor */
> > > > isl29028@44 {
> > > > - compatible = "isil,isl29028";
> > > > + compatible = "isl,isl29028";
> > > > reg = <0x44>;
> > > > interrupt-parent = <&gpio>;
> > > > interrupts = <TEGRA_GPIO(L, 0) IRQ_TYPE_LEVEL_HIGH>;
> > > >
> > >
> > > Does this intentionally list only the new string?
> > >
> > > If we want compatibility between new dts files and old kernels, this would
> > > need to be
> > >
> > > compatible = "isl,isl29028", compatible = "isil,isl29028";
> >
> > Are we aiming for new dts wto work with old kernels? It's nice if they
> > do, but I don't think it should be necessary. Old DTBs should definitely
> > work with a new kernel, though.
> >
> > Personally I'd rather see one string (preferably the canonical one, but
> > the deprecated one is ok) than two strings when those strings are for
> > the exact same device.
> >
>
> Ok, fair enough. I think we should have the new name in dts files
> normally then, in order to avoid spreading the mistake further when
> someone copies it.
Sounds good to me.
Mark.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread* Re: [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
2014-10-07 12:35 ` Mark Rutland
(?)
@ 2014-10-07 13:47 ` Darshana Padmadas
2014-10-07 13:54 ` Arnd Bergmann
-1 siblings, 1 reply; 36+ messages in thread
From: Darshana Padmadas @ 2014-10-07 13:47 UTC (permalink / raw)
To: Mark Rutland
Cc: Arnd Bergmann, linux-arm-kernel@lists.infradead.org,
devicetree@vger.kernel.org, Pawel Moll, robh+dt@kernel.org,
grant.likely@linaro.org
[-- Attachment #1: Type: text/plain, Size: 1896 bytes --]
So we stick to using just the new string, right? Any more changes to this
patch ?
On Tue, Oct 7, 2014 at 6:05 PM, Mark Rutland <mark.rutland@arm.com> wrote:
> On Tue, Oct 07, 2014 at 01:31:23PM +0100, Arnd Bergmann wrote:
> > On Tuesday 07 October 2014 13:08:12 Mark Rutland wrote:
> > > On Tue, Oct 07, 2014 at 12:49:02PM +0100, Arnd Bergmann wrote:
> > > > On Tuesday 07 October 2014 17:15:08 Darshana Padmadas wrote:
> > > > > /* ALS and Proximity sensor */
> > > > > isl29028@44 {
> > > > > - compatible = "isil,isl29028";
> > > > > + compatible = "isl,isl29028";
> > > > > reg = <0x44>;
> > > > > interrupt-parent = <&gpio>;
> > > > > interrupts = <TEGRA_GPIO(L, 0)
> IRQ_TYPE_LEVEL_HIGH>;
> > > > >
> > > >
> > > > Does this intentionally list only the new string?
> > > >
> > > > If we want compatibility between new dts files and old kernels, this
> would
> > > > need to be
> > > >
> > > > compatible = "isl,isl29028", compatible = "isil,isl29028";
> > >
> > > Are we aiming for new dts wto work with old kernels? It's nice if they
> > > do, but I don't think it should be necessary. Old DTBs should
> definitely
> > > work with a new kernel, though.
> > >
> > > Personally I'd rather see one string (preferably the canonical one, but
> > > the deprecated one is ok) than two strings when those strings are for
> > > the exact same device.
> > >
> >
> > Ok, fair enough. I think we should have the new name in dts files
> > normally then, in order to avoid spreading the mistake further when
> > someone copies it.
>
> Sounds good to me.
>
> Mark.
>
--
Regards
Darshana Padmadas <http://darshanapadmadas.wordpress.com>
ज्ञानं परमं ध्येयम्
[-- Attachment #2: Type: text/html, Size: 3014 bytes --]
^ permalink raw reply [flat|nested] 36+ messages in thread* [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 13:54 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 13:54 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 07 October 2014 19:17:34 Darshana Padmadas wrote:
> So we stick to using just the new string, right? Any more changes to this
> patch ?
>
>
No, the patch looks good.
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property
@ 2014-10-07 13:54 ` Arnd Bergmann
0 siblings, 0 replies; 36+ messages in thread
From: Arnd Bergmann @ 2014-10-07 13:54 UTC (permalink / raw)
To: Darshana Padmadas
Cc: Mark Rutland,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Pawel Moll,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org
On Tuesday 07 October 2014 19:17:34 Darshana Padmadas wrote:
> So we stick to using just the new string, right? Any more changes to this
> patch ?
>
>
No, the patch looks good.
Acked-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 36+ messages in thread