* [PATCH v3 0/4] Correct vendor-prefix and document device isl29028 @ 2014-10-07 11:45 Darshana Padmadas 2014-10-07 11:45 ` [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with " Darshana Padmadas ` (3 more replies) 0 siblings, 4 replies; 17+ messages in thread From: Darshana Padmadas @ 2014-10-07 11:45 UTC (permalink / raw) To: linux-arm-kernel 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 ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028 2014-10-07 11:45 [PATCH v3 0/4] Correct vendor-prefix and document device isl29028 Darshana Padmadas @ 2014-10-07 11:45 ` Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:02 ` Mark Rutland 2014-10-07 11:45 ` [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning Darshana Padmadas ` (2 subsequent siblings) 3 siblings, 2 replies; 17+ 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] 17+ messages in thread
* [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028 2014-10-07 11:45 ` [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with " Darshana Padmadas @ 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:02 ` Mark Rutland 1 sibling, 0 replies; 17+ 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] 17+ messages in thread
* [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with device isl29028 2014-10-07 11:45 ` [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with " Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann @ 2014-10-07 12:02 ` Mark Rutland 2014-10-07 12:17 ` Mark Rutland 1 sibling, 1 reply; 17+ 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] 17+ 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 0 siblings, 0 replies; 17+ 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] 17+ messages in thread
* [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning 2014-10-07 11:45 [PATCH v3 0/4] Correct vendor-prefix and document device isl29028 Darshana Padmadas 2014-10-07 11:45 ` [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with " Darshana Padmadas @ 2014-10-07 11:45 ` Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:03 ` Mark Rutland 2014-10-07 11:45 ` [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028 Darshana Padmadas 2014-10-07 11:45 ` [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property Darshana Padmadas 3 siblings, 2 replies; 17+ 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] 17+ messages in thread
* [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning 2014-10-07 11:45 ` [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning Darshana Padmadas @ 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:03 ` Mark Rutland 1 sibling, 0 replies; 17+ 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] 17+ messages in thread
* [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning 2014-10-07 11:45 ` [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann @ 2014-10-07 12:03 ` Mark Rutland 1 sibling, 0 replies; 17+ 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] 17+ messages in thread
* [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028 2014-10-07 11:45 [PATCH v3 0/4] Correct vendor-prefix and document device isl29028 Darshana Padmadas 2014-10-07 11:45 ` [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with " Darshana Padmadas 2014-10-07 11:45 ` [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning Darshana Padmadas @ 2014-10-07 11:45 ` Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:08 ` Mark Rutland 2014-10-07 11:45 ` [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property Darshana Padmadas 3 siblings, 2 replies; 17+ 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] 17+ messages in thread
* [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028 2014-10-07 11:45 ` [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028 Darshana Padmadas @ 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:08 ` Mark Rutland 1 sibling, 0 replies; 17+ 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] 17+ messages in thread
* [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028 2014-10-07 11:45 ` [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028 Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann @ 2014-10-07 12:08 ` Mark Rutland 1 sibling, 0 replies; 17+ 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] 17+ 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 [PATCH v3 0/4] Correct vendor-prefix and document device isl29028 Darshana Padmadas ` (2 preceding siblings ...) 2014-10-07 11:45 ` [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028 Darshana Padmadas @ 2014-10-07 11:45 ` Darshana Padmadas 2014-10-07 11:49 ` Arnd Bergmann 3 siblings, 1 reply; 17+ 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] 17+ 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 ` [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property Darshana Padmadas @ 2014-10-07 11:49 ` Arnd Bergmann 2014-10-07 12:08 ` Mark Rutland 0 siblings, 1 reply; 17+ 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] 17+ 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 2014-10-07 12:31 ` Arnd Bergmann 0 siblings, 1 reply; 17+ 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] 17+ 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 2014-10-07 12:35 ` Mark Rutland 0 siblings, 1 reply; 17+ 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] 17+ 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 [not found] ` <CAEZObdLz-77b41tE=TLb0iFyhOsAyW+UV7YfAyFbMYRBJvRwTg@mail.gmail.com> 0 siblings, 1 reply; 17+ 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] 17+ messages in thread
[parent not found: <CAEZObdLz-77b41tE=TLb0iFyhOsAyW+UV7YfAyFbMYRBJvRwTg@mail.gmail.com>]
* [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property [not found] ` <CAEZObdLz-77b41tE=TLb0iFyhOsAyW+UV7YfAyFbMYRBJvRwTg@mail.gmail.com> @ 2014-10-07 13:54 ` Arnd Bergmann 0 siblings, 0 replies; 17+ 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] 17+ messages in thread
end of thread, other threads:[~2014-10-07 13:54 UTC | newest] Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-10-07 11:45 [PATCH v3 0/4] Correct vendor-prefix and document device isl29028 Darshana Padmadas 2014-10-07 11:45 ` [PATCH v3 1/4] Documentation: devicetree: bindings: Document correct and deprecated vendor-prefix with " Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:02 ` Mark Rutland 2014-10-07 12:17 ` Mark Rutland 2014-10-07 11:45 ` [PATCH v3 2/4] Documentation: devicetree: bindings: Document deprecated device vendor name to fix related warning Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:03 ` Mark Rutland 2014-10-07 11:45 ` [PATCH v3 3/4] Staging: iio: light: Added correct vendor-prefix for device isl29028 Darshana Padmadas 2014-10-07 11:46 ` Arnd Bergmann 2014-10-07 12:08 ` Mark Rutland 2014-10-07 11:45 ` [PATCH v3 4/4] arch: arm: boot: dts: Added correct vendor-prefix with device name to compatible property Darshana Padmadas 2014-10-07 11:49 ` Arnd Bergmann 2014-10-07 12:08 ` Mark Rutland 2014-10-07 12:31 ` Arnd Bergmann 2014-10-07 12:35 ` Mark Rutland [not found] ` <CAEZObdLz-77b41tE=TLb0iFyhOsAyW+UV7YfAyFbMYRBJvRwTg@mail.gmail.com> 2014-10-07 13:54 ` Arnd Bergmann
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).