* [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-08-20 8:42 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:42 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo, Philipp Zabel
Hi,
currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. I figure at this point it is still
possible to change this to use "isil" everywhere without too
much pain, but it might be preferred to keep the already
documented "isl" prefix, even though it doesn't follow the
convention to use the NASDAQ symbol where available.
The current users of the "isil" prefix are the following drivers
and device trees, so we could just as well change those instead:
arch/arm/boot/dts/tegra20-seaboard.dts
arch/arm/boot/dts/tegra20-ventana.dts
arch/arm/boot/dts/tegra30-cardhu.dtsi
arch/powerpc/boot/dts/p1022rdk.dts
drivers/staging/iio/light/isl29018.c
drivers/staging/iio/light/isl29028.c
regards
Philipp
Philipp Zabel (6):
of: Change vendor prefix for Intersil Corporation to isil
Documentation: Add isl1208 and isl12022 to trivial-devices list
ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
isil
rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
isil
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
arch/powerpc/boot/dts/ppa8548.dts | 2 +-
drivers/rtc/rtc-isl12022.c | 3 ++-
drivers/rtc/rtc-isl12057.c | 3 ++-
8 files changed, 12 insertions(+), 8 deletions(-)
--
2.1.0.rc1
--
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] 39+ messages in thread
* [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-08-20 8:42 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:42 UTC (permalink / raw)
To: devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Rob Herring, Paul Mackerras, Philipp Zabel,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Hi,
currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. I figure at this point it is still
possible to change this to use "isil" everywhere without too
much pain, but it might be preferred to keep the already
documented "isl" prefix, even though it doesn't follow the
convention to use the NASDAQ symbol where available.
The current users of the "isil" prefix are the following drivers
and device trees, so we could just as well change those instead:
arch/arm/boot/dts/tegra20-seaboard.dts
arch/arm/boot/dts/tegra20-ventana.dts
arch/arm/boot/dts/tegra30-cardhu.dtsi
arch/powerpc/boot/dts/p1022rdk.dts
drivers/staging/iio/light/isl29018.c
drivers/staging/iio/light/isl29028.c
regards
Philipp
Philipp Zabel (6):
of: Change vendor prefix for Intersil Corporation to isil
Documentation: Add isl1208 and isl12022 to trivial-devices list
ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
isil
rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
isil
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
arch/powerpc/boot/dts/ppa8548.dts | 2 +-
drivers/rtc/rtc-isl12022.c | 3 ++-
drivers/rtc/rtc-isl12057.c | 3 ++-
8 files changed, 12 insertions(+), 8 deletions(-)
--
2.1.0.rc1
^ permalink raw reply [flat|nested] 39+ messages in thread
* [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-08-20 8:42 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:42 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. I figure at this point it is still
possible to change this to use "isil" everywhere without too
much pain, but it might be preferred to keep the already
documented "isl" prefix, even though it doesn't follow the
convention to use the NASDAQ symbol where available.
The current users of the "isil" prefix are the following drivers
and device trees, so we could just as well change those instead:
arch/arm/boot/dts/tegra20-seaboard.dts
arch/arm/boot/dts/tegra20-ventana.dts
arch/arm/boot/dts/tegra30-cardhu.dtsi
arch/powerpc/boot/dts/p1022rdk.dts
drivers/staging/iio/light/isl29018.c
drivers/staging/iio/light/isl29028.c
regards
Philipp
Philipp Zabel (6):
of: Change vendor prefix for Intersil Corporation to isil
Documentation: Add isl1208 and isl12022 to trivial-devices list
ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
isil
rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
isil
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
arch/powerpc/boot/dts/ppa8548.dts | 2 +-
drivers/rtc/rtc-isl12022.c | 3 ++-
drivers/rtc/rtc-isl12057.c | 3 ++-
8 files changed, 12 insertions(+), 8 deletions(-)
--
2.1.0.rc1
^ permalink raw reply [flat|nested] 39+ messages in thread
* [RFC PATCH 1/6] of: Change vendor prefix for Intersil Corporation to isil
2014-08-20 8:42 ` Philipp Zabel
(?)
@ 2014-08-20 8:42 ` Philipp Zabel
-1 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:42 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo, Philipp Zabel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt.
Signed-off-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
---
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 +-
Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
index 6af570e..6504297 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -56,7 +56,7 @@ fsl,sgtl5000 SGTL5000: Ultra Low-Power Audio Codec
gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire Interface
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,isl12057 Intersil ISL12057 I2C RTC Chip
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
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index ac7269f..938e0ce 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -68,7 +68,7 @@ img Imagination Technologies Ltd.
intel Intel Corporation
intercontrol Inter Control Group
isee ISEE 2007 S.L.
-isl Intersil
+isil Intersil Corporation
karo Ka-Ro electronics GmbH
keymile Keymile GmbH
lacie LaCie
--
2.1.0.rc1
--
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] 39+ messages in thread
* [RFC PATCH 1/6] of: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:42 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:42 UTC (permalink / raw)
To: devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Rob Herring, Paul Mackerras, Philipp Zabel,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 +-
Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
index 6af570e..6504297 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -56,7 +56,7 @@ fsl,sgtl5000 SGTL5000: Ultra Low-Power Audio Codec
gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire Interface
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,isl12057 Intersil ISL12057 I2C RTC Chip
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
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index ac7269f..938e0ce 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -68,7 +68,7 @@ img Imagination Technologies Ltd.
intel Intel Corporation
intercontrol Inter Control Group
isee ISEE 2007 S.L.
-isl Intersil
+isil Intersil Corporation
karo Ka-Ro electronics GmbH
keymile Keymile GmbH
lacie LaCie
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 1/6] of: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:42 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:42 UTC (permalink / raw)
To: linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 +-
Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
index 6af570e..6504297 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -56,7 +56,7 @@ fsl,sgtl5000 SGTL5000: Ultra Low-Power Audio Codec
gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire Interface
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,isl12057 Intersil ISL12057 I2C RTC Chip
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
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index ac7269f..938e0ce 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -68,7 +68,7 @@ img Imagination Technologies Ltd.
intel Intel Corporation
intercontrol Inter Control Group
isee ISEE 2007 S.L.
-isl Intersil
+isil Intersil Corporation
karo Ka-Ro electronics GmbH
keymile Keymile GmbH
lacie LaCie
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 2/6] Documentation: Add isl1208 and isl12022 to trivial-devices list
2014-08-20 8:42 ` Philipp Zabel
(?)
@ 2014-08-20 8:43 ` Philipp Zabel
-1 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo, Philipp Zabel
This patch adds the Intersil ISL1208 and ISL12022 I2C RTCs to the
trivial-devices list.
Signed-off-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@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 6504297..ec4c6b2 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -56,6 +56,8 @@ fsl,sgtl5000 SGTL5000: Ultra Low-Power Audio Codec
gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire Interface
infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
+isil,isl1208 Intersil ISL1208 I2C RTC Chip
+isil,isl12022 Intersil ISL12022 I2C RTC Chip
isil,isl12057 Intersil ISL12057 I2C RTC Chip
maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
--
2.1.0.rc1
--
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] 39+ messages in thread
* [RFC PATCH 2/6] Documentation: Add isl1208 and isl12022 to trivial-devices list
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Rob Herring, Paul Mackerras, Philipp Zabel,
Kumar Gala, linuxppc-dev, linux-arm-kernel
This patch adds the Intersil ISL1208 and ISL12022 I2C RTCs to the
trivial-devices list.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
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 6504297..ec4c6b2 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -56,6 +56,8 @@ fsl,sgtl5000 SGTL5000: Ultra Low-Power Audio Codec
gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire Interface
infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
+isil,isl1208 Intersil ISL1208 I2C RTC Chip
+isil,isl12022 Intersil ISL12022 I2C RTC Chip
isil,isl12057 Intersil ISL12057 I2C RTC Chip
maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 2/6] Documentation: Add isl1208 and isl12022 to trivial-devices list
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds the Intersil ISL1208 and ISL12022 I2C RTCs to the
trivial-devices list.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
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 6504297..ec4c6b2 100644
--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
@@ -56,6 +56,8 @@ fsl,sgtl5000 SGTL5000: Ultra Low-Power Audio Codec
gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with Two-Wire Interface
infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz)
+isil,isl1208 Intersil ISL1208 I2C RTC Chip
+isil,isl12022 Intersil ISL12022 I2C RTC Chip
isil,isl12057 Intersil ISL12057 I2C RTC Chip
maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator
maxim,max1237 Low-Power, 4-/12-Channel, 2-Wire Serial, 12-Bit ADCs
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 3/6] ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
2014-08-20 8:42 ` Philipp Zabel
(?)
@ 2014-08-20 8:43 ` Philipp Zabel
-1 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo, Philipp Zabel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt.
Signed-off-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
---
arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/armada-370-netgear-rn102.dts b/arch/arm/boot/dts/armada-370-netgear-rn102.dts
index d6d572e..edc381c 100644
--- a/arch/arm/boot/dts/armada-370-netgear-rn102.dts
+++ b/arch/arm/boot/dts/armada-370-netgear-rn102.dts
@@ -122,7 +122,7 @@
status = "okay";
isl12057: isl12057@68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
diff --git a/arch/arm/boot/dts/armada-370-netgear-rn104.dts b/arch/arm/boot/dts/armada-370-netgear-rn104.dts
index c5fe8b5..7367b4c 100644
--- a/arch/arm/boot/dts/armada-370-netgear-rn104.dts
+++ b/arch/arm/boot/dts/armada-370-netgear-rn104.dts
@@ -117,7 +117,7 @@
status = "okay";
isl12057: isl12057@68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
diff --git a/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts b/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
index 0cf999a..252def8 100644
--- a/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
+++ b/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
@@ -174,7 +174,7 @@
status = "okay";
isl12057: isl12057@68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
--
2.1.0.rc1
--
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] 39+ messages in thread
* [RFC PATCH 3/6] ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Rob Herring, Paul Mackerras, Philipp Zabel,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/armada-370-netgear-rn102.dts b/arch/arm/boot/dts/armada-370-netgear-rn102.dts
index d6d572e..edc381c 100644
--- a/arch/arm/boot/dts/armada-370-netgear-rn102.dts
+++ b/arch/arm/boot/dts/armada-370-netgear-rn102.dts
@@ -122,7 +122,7 @@
status = "okay";
isl12057: isl12057@68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
diff --git a/arch/arm/boot/dts/armada-370-netgear-rn104.dts b/arch/arm/boot/dts/armada-370-netgear-rn104.dts
index c5fe8b5..7367b4c 100644
--- a/arch/arm/boot/dts/armada-370-netgear-rn104.dts
+++ b/arch/arm/boot/dts/armada-370-netgear-rn104.dts
@@ -117,7 +117,7 @@
status = "okay";
isl12057: isl12057@68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
diff --git a/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts b/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
index 0cf999a..252def8 100644
--- a/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
+++ b/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
@@ -174,7 +174,7 @@
status = "okay";
isl12057: isl12057@68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 3/6] ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/armada-370-netgear-rn102.dts b/arch/arm/boot/dts/armada-370-netgear-rn102.dts
index d6d572e..edc381c 100644
--- a/arch/arm/boot/dts/armada-370-netgear-rn102.dts
+++ b/arch/arm/boot/dts/armada-370-netgear-rn102.dts
@@ -122,7 +122,7 @@
status = "okay";
isl12057: isl12057 at 68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
diff --git a/arch/arm/boot/dts/armada-370-netgear-rn104.dts b/arch/arm/boot/dts/armada-370-netgear-rn104.dts
index c5fe8b5..7367b4c 100644
--- a/arch/arm/boot/dts/armada-370-netgear-rn104.dts
+++ b/arch/arm/boot/dts/armada-370-netgear-rn104.dts
@@ -117,7 +117,7 @@
status = "okay";
isl12057: isl12057 at 68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
diff --git a/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts b/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
index 0cf999a..252def8 100644
--- a/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
+++ b/arch/arm/boot/dts/armada-xp-netgear-rn2120.dts
@@ -174,7 +174,7 @@
status = "okay";
isl12057: isl12057 at 68 {
- compatible = "isl,isl12057";
+ compatible = "isil,isl12057";
reg = <0x68>;
};
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 4/6] powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
2014-08-20 8:42 ` Philipp Zabel
(?)
@ 2014-08-20 8:43 ` Philipp Zabel
-1 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo, Philipp Zabel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Signed-off-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
---
arch/powerpc/boot/dts/ppa8548.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/boot/dts/ppa8548.dts b/arch/powerpc/boot/dts/ppa8548.dts
index 27b0699..000262b 100644
--- a/arch/powerpc/boot/dts/ppa8548.dts
+++ b/arch/powerpc/boot/dts/ppa8548.dts
@@ -89,7 +89,7 @@
&soc {
i2c@3000 {
rtc@6f {
- compatible = "intersil,isl1208";
+ compatible = "isil,isl1208";
reg = <0x6f>;
};
};
--
2.1.0.rc1
--
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] 39+ messages in thread
* [RFC PATCH 4/6] powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Rob Herring, Paul Mackerras, Philipp Zabel,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
arch/powerpc/boot/dts/ppa8548.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/boot/dts/ppa8548.dts b/arch/powerpc/boot/dts/ppa8548.dts
index 27b0699..000262b 100644
--- a/arch/powerpc/boot/dts/ppa8548.dts
+++ b/arch/powerpc/boot/dts/ppa8548.dts
@@ -89,7 +89,7 @@
&soc {
i2c@3000 {
rtc@6f {
- compatible = "intersil,isl1208";
+ compatible = "isil,isl1208";
reg = <0x6f>;
};
};
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 4/6] powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
arch/powerpc/boot/dts/ppa8548.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/boot/dts/ppa8548.dts b/arch/powerpc/boot/dts/ppa8548.dts
index 27b0699..000262b 100644
--- a/arch/powerpc/boot/dts/ppa8548.dts
+++ b/arch/powerpc/boot/dts/ppa8548.dts
@@ -89,7 +89,7 @@
&soc {
i2c at 3000 {
rtc at 6f {
- compatible = "intersil,isl1208";
+ compatible = "isil,isl1208";
reg = <0x6f>;
};
};
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 5/6] rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to isil
2014-08-20 8:42 ` Philipp Zabel
(?)
@ 2014-08-20 8:43 ` Philipp Zabel
-1 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo, Philipp Zabel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Patch db04d6284e2a added device tree support using the then
documented isl vendor prefix, so we keep that around for
backwards compatibility.
Signed-off-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
---
drivers/rtc/rtc-isl12022.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/rtc/rtc-isl12022.c b/drivers/rtc/rtc-isl12022.c
index aa55f08..df20f18 100644
--- a/drivers/rtc/rtc-isl12022.c
+++ b/drivers/rtc/rtc-isl12022.c
@@ -275,7 +275,8 @@ static int isl12022_probe(struct i2c_client *client,
#ifdef CONFIG_OF
static struct of_device_id isl12022_dt_match[] = {
- { .compatible = "isl,isl12022" },
+ { .compatible = "isil,isl12022" },
+ { .compatible = "isl,isl12022" }, /* for backwards compatibility */
{ },
};
#endif
--
2.1.0.rc1
--
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] 39+ messages in thread
* [RFC PATCH 5/6] rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Rob Herring, Paul Mackerras, Philipp Zabel,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Patch db04d6284e2a added device tree support using the then
documented isl vendor prefix, so we keep that around for
backwards compatibility.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
drivers/rtc/rtc-isl12022.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/rtc/rtc-isl12022.c b/drivers/rtc/rtc-isl12022.c
index aa55f08..df20f18 100644
--- a/drivers/rtc/rtc-isl12022.c
+++ b/drivers/rtc/rtc-isl12022.c
@@ -275,7 +275,8 @@ static int isl12022_probe(struct i2c_client *client,
#ifdef CONFIG_OF
static struct of_device_id isl12022_dt_match[] = {
- { .compatible = "isl,isl12022" },
+ { .compatible = "isil,isl12022" },
+ { .compatible = "isl,isl12022" }, /* for backwards compatibility */
{ },
};
#endif
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 5/6] rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Patch db04d6284e2a added device tree support using the then
documented isl vendor prefix, so we keep that around for
backwards compatibility.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
drivers/rtc/rtc-isl12022.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/rtc/rtc-isl12022.c b/drivers/rtc/rtc-isl12022.c
index aa55f08..df20f18 100644
--- a/drivers/rtc/rtc-isl12022.c
+++ b/drivers/rtc/rtc-isl12022.c
@@ -275,7 +275,8 @@ static int isl12022_probe(struct i2c_client *client,
#ifdef CONFIG_OF
static struct of_device_id isl12022_dt_match[] = {
- { .compatible = "isl,isl12022" },
+ { .compatible = "isil,isl12022" },
+ { .compatible = "isl,isl12022" }, /* for backwards compatibility */
{ },
};
#endif
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 6/6] rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to isil
2014-08-20 8:42 ` Philipp Zabel
(?)
@ 2014-08-20 8:43 ` Philipp Zabel
-1 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo, Philipp Zabel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Patch 70e123373c05 added this driver with device tree support
using the then documented isl vendor prefix, so we keep that
around for backwards compatibility.
Signed-off-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
---
drivers/rtc/rtc-isl12057.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/rtc/rtc-isl12057.c b/drivers/rtc/rtc-isl12057.c
index 455b601..8276bd6 100644
--- a/drivers/rtc/rtc-isl12057.c
+++ b/drivers/rtc/rtc-isl12057.c
@@ -279,7 +279,8 @@ static int isl12057_probe(struct i2c_client *client,
#ifdef CONFIG_OF
static const struct of_device_id isl12057_dt_match[] = {
- { .compatible = "isl,isl12057" },
+ { .compatible = "isil,isl12057" },
+ { .compatible = "isl,isl12057" }, /* for backwards compatibility */
{ },
};
#endif
--
2.1.0.rc1
--
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] 39+ messages in thread
* [RFC PATCH 6/6] rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Rob Herring, Paul Mackerras, Philipp Zabel,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Patch 70e123373c05 added this driver with device tree support
using the then documented isl vendor prefix, so we keep that
around for backwards compatibility.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
drivers/rtc/rtc-isl12057.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/rtc/rtc-isl12057.c b/drivers/rtc/rtc-isl12057.c
index 455b601..8276bd6 100644
--- a/drivers/rtc/rtc-isl12057.c
+++ b/drivers/rtc/rtc-isl12057.c
@@ -279,7 +279,8 @@ static int isl12057_probe(struct i2c_client *client,
#ifdef CONFIG_OF
static const struct of_device_id isl12057_dt_match[] = {
- { .compatible = "isl,isl12057" },
+ { .compatible = "isil,isl12057" },
+ { .compatible = "isl,isl12057" }, /* for backwards compatibility */
{ },
};
#endif
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* [RFC PATCH 6/6] rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 8:43 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-20 8:43 UTC (permalink / raw)
To: linux-arm-kernel
Currently there is a wild mixture of isl, isil, and intersil
compatibles in the kernel. At this point, changing the vendor
symbol to the most often used variant, which is equal to the
NASDAQ symbol, isil, should not hurt, since the isl1208 driver
doesn't care either way.
Patch 70e123373c05 added this driver with device tree support
using the then documented isl vendor prefix, so we keep that
around for backwards compatibility.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
drivers/rtc/rtc-isl12057.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/rtc/rtc-isl12057.c b/drivers/rtc/rtc-isl12057.c
index 455b601..8276bd6 100644
--- a/drivers/rtc/rtc-isl12057.c
+++ b/drivers/rtc/rtc-isl12057.c
@@ -279,7 +279,8 @@ static int isl12057_probe(struct i2c_client *client,
#ifdef CONFIG_OF
static const struct of_device_id isl12057_dt_match[] = {
- { .compatible = "isl,isl12057" },
+ { .compatible = "isil,isl12057" },
+ { .compatible = "isl,isl12057" }, /* for backwards compatibility */
{ },
};
#endif
--
2.1.0.rc1
^ permalink raw reply related [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
2014-08-20 8:42 ` Philipp Zabel
(?)
@ 2014-08-20 12:16 ` Jason Cooper
-1 siblings, 0 replies; 39+ messages in thread
From: Jason Cooper @ 2014-08-20 12:16 UTC (permalink / raw)
To: Philipp Zabel
Cc: Mark Rutland, devicetree, Alessandro Zummo, Russell King,
Pawel Moll, Ian Campbell, Benjamin Herrenschmidt, Rob Herring,
Paul Mackerras, Michael Ellerman, Kumar Gala, linuxppc-dev,
linux-arm-kernel
Philipp,
On Wed, Aug 20, 2014 at 10:42:58AM +0200, Philipp Zabel wrote:
> Hi,
>
> currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. I figure at this point it is still
> possible to change this to use "isil" everywhere without too
> much pain, but it might be preferred to keep the already
> documented "isl" prefix, even though it doesn't follow the
> convention to use the NASDAQ symbol where available.
>
> The current users of the "isil" prefix are the following drivers
> and device trees, so we could just as well change those instead:
> arch/arm/boot/dts/tegra20-seaboard.dts
> arch/arm/boot/dts/tegra20-ventana.dts
> arch/arm/boot/dts/tegra30-cardhu.dtsi
> arch/powerpc/boot/dts/p1022rdk.dts
> drivers/staging/iio/light/isl29018.c
> drivers/staging/iio/light/isl29028.c
>
> regards
> Philipp
>
> Philipp Zabel (6):
> of: Change vendor prefix for Intersil Corporation to isil
> Documentation: Add isl1208 and isl12022 to trivial-devices list
> ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
> powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
> rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
> isil
> rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
> isil
>
> Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
> Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
> arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
> arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
> arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
> arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> drivers/rtc/rtc-isl12022.c | 3 ++-
> drivers/rtc/rtc-isl12057.c | 3 ++-
> 8 files changed, 12 insertions(+), 8 deletions(-)
This looks good overall. My only nit is that I'd like to see the legacy
name(s) captured in the binding docs.
I'll take the Armada dts changes through mvebu/arm-soc after a few days.
thx,
Jason.
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-08-20 12:16 ` Jason Cooper
0 siblings, 0 replies; 39+ messages in thread
From: Jason Cooper @ 2014-08-20 12:16 UTC (permalink / raw)
To: Philipp Zabel
Cc: Mark Rutland, devicetree, Alessandro Zummo, Russell King,
Pawel Moll, Ian Campbell, Rob Herring, Paul Mackerras,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Philipp,
On Wed, Aug 20, 2014 at 10:42:58AM +0200, Philipp Zabel wrote:
> Hi,
>
> currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. I figure at this point it is still
> possible to change this to use "isil" everywhere without too
> much pain, but it might be preferred to keep the already
> documented "isl" prefix, even though it doesn't follow the
> convention to use the NASDAQ symbol where available.
>
> The current users of the "isil" prefix are the following drivers
> and device trees, so we could just as well change those instead:
> arch/arm/boot/dts/tegra20-seaboard.dts
> arch/arm/boot/dts/tegra20-ventana.dts
> arch/arm/boot/dts/tegra30-cardhu.dtsi
> arch/powerpc/boot/dts/p1022rdk.dts
> drivers/staging/iio/light/isl29018.c
> drivers/staging/iio/light/isl29028.c
>
> regards
> Philipp
>
> Philipp Zabel (6):
> of: Change vendor prefix for Intersil Corporation to isil
> Documentation: Add isl1208 and isl12022 to trivial-devices list
> ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
> powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
> rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
> isil
> rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
> isil
>
> Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
> Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
> arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
> arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
> arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
> arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> drivers/rtc/rtc-isl12022.c | 3 ++-
> drivers/rtc/rtc-isl12057.c | 3 ++-
> 8 files changed, 12 insertions(+), 8 deletions(-)
This looks good overall. My only nit is that I'd like to see the legacy
name(s) captured in the binding docs.
I'll take the Armada dts changes through mvebu/arm-soc after a few days.
thx,
Jason.
^ permalink raw reply [flat|nested] 39+ messages in thread
* [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-08-20 12:16 ` Jason Cooper
0 siblings, 0 replies; 39+ messages in thread
From: Jason Cooper @ 2014-08-20 12:16 UTC (permalink / raw)
To: linux-arm-kernel
Philipp,
On Wed, Aug 20, 2014 at 10:42:58AM +0200, Philipp Zabel wrote:
> Hi,
>
> currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. I figure at this point it is still
> possible to change this to use "isil" everywhere without too
> much pain, but it might be preferred to keep the already
> documented "isl" prefix, even though it doesn't follow the
> convention to use the NASDAQ symbol where available.
>
> The current users of the "isil" prefix are the following drivers
> and device trees, so we could just as well change those instead:
> arch/arm/boot/dts/tegra20-seaboard.dts
> arch/arm/boot/dts/tegra20-ventana.dts
> arch/arm/boot/dts/tegra30-cardhu.dtsi
> arch/powerpc/boot/dts/p1022rdk.dts
> drivers/staging/iio/light/isl29018.c
> drivers/staging/iio/light/isl29028.c
>
> regards
> Philipp
>
> Philipp Zabel (6):
> of: Change vendor prefix for Intersil Corporation to isil
> Documentation: Add isl1208 and isl12022 to trivial-devices list
> ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
> powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
> rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
> isil
> rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
> isil
>
> Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
> Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
> arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
> arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
> arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
> arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> drivers/rtc/rtc-isl12022.c | 3 ++-
> drivers/rtc/rtc-isl12057.c | 3 ++-
> 8 files changed, 12 insertions(+), 8 deletions(-)
This looks good overall. My only nit is that I'd like to see the legacy
name(s) captured in the binding docs.
I'll take the Armada dts changes through mvebu/arm-soc after a few days.
thx,
Jason.
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 6/6] rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to isil
2014-08-20 8:43 ` Philipp Zabel
(?)
@ 2014-08-20 12:40 ` Sergei Shtylyov
-1 siblings, 0 replies; 39+ messages in thread
From: Sergei Shtylyov @ 2014-08-20 12:40 UTC (permalink / raw)
To: Philipp Zabel, devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Kumar Gala, Rob Herring, Paul Mackerras,
linuxppc-dev, linux-arm-kernel
Hello.
On 8/20/2014 12:43 PM, Philipp Zabel wrote:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
> Patch 70e123373c05 added this driver with device tree support
Please also specify that commit's summary line in parens.
> using the then documented isl vendor prefix, so we keep that
> around for backwards compatibility.
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
WBR, Sergei
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 6/6] rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 12:40 ` Sergei Shtylyov
0 siblings, 0 replies; 39+ messages in thread
From: Sergei Shtylyov @ 2014-08-20 12:40 UTC (permalink / raw)
To: Philipp Zabel, devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Kumar Gala, Rob Herring, Paul Mackerras,
linuxppc-dev, linux-arm-kernel
Hello.
On 8/20/2014 12:43 PM, Philipp Zabel wrote:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
> Patch 70e123373c05 added this driver with device tree support
Please also specify that commit's summary line in parens.
> using the then documented isl vendor prefix, so we keep that
> around for backwards compatibility.
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
WBR, Sergei
^ permalink raw reply [flat|nested] 39+ messages in thread
* [RFC PATCH 6/6] rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-20 12:40 ` Sergei Shtylyov
0 siblings, 0 replies; 39+ messages in thread
From: Sergei Shtylyov @ 2014-08-20 12:40 UTC (permalink / raw)
To: linux-arm-kernel
Hello.
On 8/20/2014 12:43 PM, Philipp Zabel wrote:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
> Patch 70e123373c05 added this driver with device tree support
Please also specify that commit's summary line in parens.
> using the then documented isl vendor prefix, so we keep that
> around for backwards compatibility.
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
WBR, Sergei
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 4/6] powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
2014-08-20 8:43 ` Philipp Zabel
(?)
@ 2014-08-22 3:12 ` Rob Herring
-1 siblings, 0 replies; 39+ messages in thread
From: Rob Herring @ 2014-08-22 3:12 UTC (permalink / raw)
To: Philipp Zabel
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
Mark Rutland, Ian Campbell, Kumar Gala, linuxppc-dev,
Benjamin Herrenschmidt, Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo
On Wed, Aug 20, 2014 at 3:43 AM, Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> wrote:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
>
> Signed-off-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
> ---
> arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/boot/dts/ppa8548.dts b/arch/powerpc/boot/dts/ppa8548.dts
> index 27b0699..000262b 100644
> --- a/arch/powerpc/boot/dts/ppa8548.dts
> +++ b/arch/powerpc/boot/dts/ppa8548.dts
> @@ -89,7 +89,7 @@
> &soc {
> i2c@3000 {
> rtc@6f {
> - compatible = "intersil,isl1208";
> + compatible = "isil,isl1208";
This is a mature platform. You cannot change it. You can add the new
string, but the dts and kernel need both strings.
I would just leave it all alone and add the new prefix and mark the
others deprecated.
Rob
> reg = <0x6f>;
> };
> };
> --
> 2.1.0.rc1
>
--
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] 39+ messages in thread
* Re: [RFC PATCH 4/6] powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-22 3:12 ` Rob Herring
0 siblings, 0 replies; 39+ messages in thread
From: Rob Herring @ 2014-08-22 3:12 UTC (permalink / raw)
To: Philipp Zabel
Cc: Mark Rutland, devicetree, Alessandro Zummo, Russell King,
Pawel Moll, Ian Campbell, Rob Herring, Paul Mackerras,
Kumar Gala, linuxppc-dev, linux-arm-kernel
On Wed, Aug 20, 2014 at 3:43 AM, Philipp Zabel <p.zabel@pengutronix.de> wrote:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
> arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/boot/dts/ppa8548.dts b/arch/powerpc/boot/dts/ppa8548.dts
> index 27b0699..000262b 100644
> --- a/arch/powerpc/boot/dts/ppa8548.dts
> +++ b/arch/powerpc/boot/dts/ppa8548.dts
> @@ -89,7 +89,7 @@
> &soc {
> i2c@3000 {
> rtc@6f {
> - compatible = "intersil,isl1208";
> + compatible = "isil,isl1208";
This is a mature platform. You cannot change it. You can add the new
string, but the dts and kernel need both strings.
I would just leave it all alone and add the new prefix and mark the
others deprecated.
Rob
> reg = <0x6f>;
> };
> };
> --
> 2.1.0.rc1
>
^ permalink raw reply [flat|nested] 39+ messages in thread
* [RFC PATCH 4/6] powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-22 3:12 ` Rob Herring
0 siblings, 0 replies; 39+ messages in thread
From: Rob Herring @ 2014-08-22 3:12 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 20, 2014 at 3:43 AM, Philipp Zabel <p.zabel@pengutronix.de> wrote:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
> arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/boot/dts/ppa8548.dts b/arch/powerpc/boot/dts/ppa8548.dts
> index 27b0699..000262b 100644
> --- a/arch/powerpc/boot/dts/ppa8548.dts
> +++ b/arch/powerpc/boot/dts/ppa8548.dts
> @@ -89,7 +89,7 @@
> &soc {
> i2c at 3000 {
> rtc at 6f {
> - compatible = "intersil,isl1208";
> + compatible = "isil,isl1208";
This is a mature platform. You cannot change it. You can add the new
string, but the dts and kernel need both strings.
I would just leave it all alone and add the new prefix and mark the
others deprecated.
Rob
> reg = <0x6f>;
> };
> };
> --
> 2.1.0.rc1
>
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 5/6] rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to isil
2014-08-20 8:43 ` Philipp Zabel
(?)
@ 2014-08-22 8:57 ` Philipp Zabel
-1 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-22 8:57 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo, Wolfram Sang
Am Mittwoch, den 20.08.2014, 10:43 +0200 schrieb Philipp Zabel:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
> Patch db04d6284e2a added device tree support using the then
> documented isl vendor prefix, so we keep that around for
> backwards compatibility.
>
> Signed-off-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
> ---
> drivers/rtc/rtc-isl12022.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/rtc/rtc-isl12022.c b/drivers/rtc/rtc-isl12022.c
> index aa55f08..df20f18 100644
> --- a/drivers/rtc/rtc-isl12022.c
> +++ b/drivers/rtc/rtc-isl12022.c
> @@ -275,7 +275,8 @@ static int isl12022_probe(struct i2c_client *client,
>
> #ifdef CONFIG_OF
> static struct of_device_id isl12022_dt_match[] = {
> - { .compatible = "isl,isl12022" },
> + { .compatible = "isil,isl12022" },
> + { .compatible = "isl,isl12022" }, /* for backwards compatibility */
Actually, is it even necessary to keep this around in here?
If of_driver_match_device fails, i2c_device_match will continue to try
i2c_match_id if driver->id_table is set, which matches against the
result of of_modalias_node (dropping the vendor prefix altogether).
regards
Philipp
--
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] 39+ messages in thread
* Re: [RFC PATCH 5/6] rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-22 8:57 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-22 8:57 UTC (permalink / raw)
To: devicetree
Cc: Mark Rutland, Alessandro Zummo, Russell King, Pawel Moll,
Ian Campbell, Wolfram Sang, Rob Herring, Paul Mackerras,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Am Mittwoch, den 20.08.2014, 10:43 +0200 schrieb Philipp Zabel:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
> Patch db04d6284e2a added device tree support using the then
> documented isl vendor prefix, so we keep that around for
> backwards compatibility.
>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
> drivers/rtc/rtc-isl12022.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/rtc/rtc-isl12022.c b/drivers/rtc/rtc-isl12022.c
> index aa55f08..df20f18 100644
> --- a/drivers/rtc/rtc-isl12022.c
> +++ b/drivers/rtc/rtc-isl12022.c
> @@ -275,7 +275,8 @@ static int isl12022_probe(struct i2c_client *client,
>
> #ifdef CONFIG_OF
> static struct of_device_id isl12022_dt_match[] = {
> - { .compatible = "isl,isl12022" },
> + { .compatible = "isil,isl12022" },
> + { .compatible = "isl,isl12022" }, /* for backwards compatibility */
Actually, is it even necessary to keep this around in here?
If of_driver_match_device fails, i2c_device_match will continue to try
i2c_match_id if driver->id_table is set, which matches against the
result of of_modalias_node (dropping the vendor prefix altogether).
regards
Philipp
^ permalink raw reply [flat|nested] 39+ messages in thread
* [RFC PATCH 5/6] rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to isil
@ 2014-08-22 8:57 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-08-22 8:57 UTC (permalink / raw)
To: linux-arm-kernel
Am Mittwoch, den 20.08.2014, 10:43 +0200 schrieb Philipp Zabel:
> Currently there is a wild mixture of isl, isil, and intersil
> compatibles in the kernel. At this point, changing the vendor
> symbol to the most often used variant, which is equal to the
> NASDAQ symbol, isil, should not hurt, since the isl1208 driver
> doesn't care either way.
> Patch db04d6284e2a added device tree support using the then
> documented isl vendor prefix, so we keep that around for
> backwards compatibility.
>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
> drivers/rtc/rtc-isl12022.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/rtc/rtc-isl12022.c b/drivers/rtc/rtc-isl12022.c
> index aa55f08..df20f18 100644
> --- a/drivers/rtc/rtc-isl12022.c
> +++ b/drivers/rtc/rtc-isl12022.c
> @@ -275,7 +275,8 @@ static int isl12022_probe(struct i2c_client *client,
>
> #ifdef CONFIG_OF
> static struct of_device_id isl12022_dt_match[] = {
> - { .compatible = "isl,isl12022" },
> + { .compatible = "isil,isl12022" },
> + { .compatible = "isl,isl12022" }, /* for backwards compatibility */
Actually, is it even necessary to keep this around in here?
If of_driver_match_device fails, i2c_device_match will continue to try
i2c_match_id if driver->id_table is set, which matches against the
result of of_modalias_node (dropping the vendor prefix altogether).
regards
Philipp
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
2014-08-20 12:16 ` Jason Cooper
(?)
@ 2014-09-07 13:32 ` Jason Cooper
-1 siblings, 0 replies; 39+ messages in thread
From: Jason Cooper @ 2014-09-07 13:32 UTC (permalink / raw)
To: Philipp Zabel
Cc: Mark Rutland, devicetree, Alessandro Zummo, Russell King,
Pawel Moll, Ian Campbell, Benjamin Herrenschmidt, Rob Herring,
Paul Mackerras, Michael Ellerman, Kumar Gala, linuxppc-dev,
linux-arm-kernel
On Wed, Aug 20, 2014 at 08:16:43AM -0400, Jason Cooper wrote:
> Philipp,
>
> On Wed, Aug 20, 2014 at 10:42:58AM +0200, Philipp Zabel wrote:
> > Hi,
> >
> > currently there is a wild mixture of isl, isil, and intersil
> > compatibles in the kernel. I figure at this point it is still
> > possible to change this to use "isil" everywhere without too
> > much pain, but it might be preferred to keep the already
> > documented "isl" prefix, even though it doesn't follow the
> > convention to use the NASDAQ symbol where available.
> >
> > The current users of the "isil" prefix are the following drivers
> > and device trees, so we could just as well change those instead:
> > arch/arm/boot/dts/tegra20-seaboard.dts
> > arch/arm/boot/dts/tegra20-ventana.dts
> > arch/arm/boot/dts/tegra30-cardhu.dtsi
> > arch/powerpc/boot/dts/p1022rdk.dts
> > drivers/staging/iio/light/isl29018.c
> > drivers/staging/iio/light/isl29028.c
> >
> > regards
> > Philipp
> >
> > Philipp Zabel (6):
> > of: Change vendor prefix for Intersil Corporation to isil
> > Documentation: Add isl1208 and isl12022 to trivial-devices list
> > ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
> > powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
> > rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
> > isil
> > rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
> > isil
> >
> > Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
> > Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
> > arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
> > arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
> > arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
> > arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> > drivers/rtc/rtc-isl12022.c | 3 ++-
> > drivers/rtc/rtc-isl12057.c | 3 ++-
> > 8 files changed, 12 insertions(+), 8 deletions(-)
>
> This looks good overall. My only nit is that I'd like to see the legacy
> name(s) captured in the binding docs.
>
> I'll take the Armada dts changes through mvebu/arm-soc after a few days.
Just to dot our i's and cross our t's, would you mind sending out a
non-rfc version of this series?
thx,
Jason.
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-09-07 13:32 ` Jason Cooper
0 siblings, 0 replies; 39+ messages in thread
From: Jason Cooper @ 2014-09-07 13:32 UTC (permalink / raw)
To: Philipp Zabel
Cc: Mark Rutland, devicetree, Alessandro Zummo, Russell King,
Pawel Moll, Ian Campbell, Rob Herring, Paul Mackerras,
Kumar Gala, linuxppc-dev, linux-arm-kernel
On Wed, Aug 20, 2014 at 08:16:43AM -0400, Jason Cooper wrote:
> Philipp,
>
> On Wed, Aug 20, 2014 at 10:42:58AM +0200, Philipp Zabel wrote:
> > Hi,
> >
> > currently there is a wild mixture of isl, isil, and intersil
> > compatibles in the kernel. I figure at this point it is still
> > possible to change this to use "isil" everywhere without too
> > much pain, but it might be preferred to keep the already
> > documented "isl" prefix, even though it doesn't follow the
> > convention to use the NASDAQ symbol where available.
> >
> > The current users of the "isil" prefix are the following drivers
> > and device trees, so we could just as well change those instead:
> > arch/arm/boot/dts/tegra20-seaboard.dts
> > arch/arm/boot/dts/tegra20-ventana.dts
> > arch/arm/boot/dts/tegra30-cardhu.dtsi
> > arch/powerpc/boot/dts/p1022rdk.dts
> > drivers/staging/iio/light/isl29018.c
> > drivers/staging/iio/light/isl29028.c
> >
> > regards
> > Philipp
> >
> > Philipp Zabel (6):
> > of: Change vendor prefix for Intersil Corporation to isil
> > Documentation: Add isl1208 and isl12022 to trivial-devices list
> > ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
> > powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
> > rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
> > isil
> > rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
> > isil
> >
> > Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
> > Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
> > arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
> > arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
> > arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
> > arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> > drivers/rtc/rtc-isl12022.c | 3 ++-
> > drivers/rtc/rtc-isl12057.c | 3 ++-
> > 8 files changed, 12 insertions(+), 8 deletions(-)
>
> This looks good overall. My only nit is that I'd like to see the legacy
> name(s) captured in the binding docs.
>
> I'll take the Armada dts changes through mvebu/arm-soc after a few days.
Just to dot our i's and cross our t's, would you mind sending out a
non-rfc version of this series?
thx,
Jason.
^ permalink raw reply [flat|nested] 39+ messages in thread
* [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-09-07 13:32 ` Jason Cooper
0 siblings, 0 replies; 39+ messages in thread
From: Jason Cooper @ 2014-09-07 13:32 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 20, 2014 at 08:16:43AM -0400, Jason Cooper wrote:
> Philipp,
>
> On Wed, Aug 20, 2014 at 10:42:58AM +0200, Philipp Zabel wrote:
> > Hi,
> >
> > currently there is a wild mixture of isl, isil, and intersil
> > compatibles in the kernel. I figure at this point it is still
> > possible to change this to use "isil" everywhere without too
> > much pain, but it might be preferred to keep the already
> > documented "isl" prefix, even though it doesn't follow the
> > convention to use the NASDAQ symbol where available.
> >
> > The current users of the "isil" prefix are the following drivers
> > and device trees, so we could just as well change those instead:
> > arch/arm/boot/dts/tegra20-seaboard.dts
> > arch/arm/boot/dts/tegra20-ventana.dts
> > arch/arm/boot/dts/tegra30-cardhu.dtsi
> > arch/powerpc/boot/dts/p1022rdk.dts
> > drivers/staging/iio/light/isl29018.c
> > drivers/staging/iio/light/isl29028.c
> >
> > regards
> > Philipp
> >
> > Philipp Zabel (6):
> > of: Change vendor prefix for Intersil Corporation to isil
> > Documentation: Add isl1208 and isl12022 to trivial-devices list
> > ARM: mvebu: Change vendor prefix for Intersil Corporation to isil
> > powerpc/85xx: Change vendor prefix for Intersil Corporation to isil
> > rtc: rtc-isl12022: Change vendor prefix for Intersil Corporation to
> > isil
> > rtc: rtc-isl12057: Change vendor prefix for Intersil Corporation to
> > isil
> >
> > Documentation/devicetree/bindings/i2c/trivial-devices.txt | 4 +++-
> > Documentation/devicetree/bindings/vendor-prefixes.txt | 2 +-
> > arch/arm/boot/dts/armada-370-netgear-rn102.dts | 2 +-
> > arch/arm/boot/dts/armada-370-netgear-rn104.dts | 2 +-
> > arch/arm/boot/dts/armada-xp-netgear-rn2120.dts | 2 +-
> > arch/powerpc/boot/dts/ppa8548.dts | 2 +-
> > drivers/rtc/rtc-isl12022.c | 3 ++-
> > drivers/rtc/rtc-isl12057.c | 3 ++-
> > 8 files changed, 12 insertions(+), 8 deletions(-)
>
> This looks good overall. My only nit is that I'd like to see the legacy
> name(s) captured in the binding docs.
>
> I'll take the Armada dts changes through mvebu/arm-soc after a few days.
Just to dot our i's and cross our t's, would you mind sending out a
non-rfc version of this series?
thx,
Jason.
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
2014-09-07 13:32 ` Jason Cooper
(?)
@ 2014-09-08 9:20 ` Philipp Zabel
-1 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-09-08 9:20 UTC (permalink / raw)
To: Jason Cooper
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll,
Mark Rutland, Ian Campbell, Kumar Gala,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Russell King,
Alessandro Zummo
Hi Jason,
Am Sonntag, den 07.09.2014, 09:32 -0400 schrieb Jason Cooper:
> Just to dot our i's and cross our t's, would you mind sending out a
> non-rfc version of this series?
Thank you for the reminder, I have sent an updated version.
regards
Philipp
--
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] 39+ messages in thread
* Re: [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-09-08 9:20 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-09-08 9:20 UTC (permalink / raw)
To: Jason Cooper
Cc: Mark Rutland, devicetree, Alessandro Zummo, Russell King,
Pawel Moll, Ian Campbell, Rob Herring, Paul Mackerras,
Kumar Gala, linuxppc-dev, linux-arm-kernel
Hi Jason,
Am Sonntag, den 07.09.2014, 09:32 -0400 schrieb Jason Cooper:
> Just to dot our i's and cross our t's, would you mind sending out a
> non-rfc version of this series?
Thank you for the reminder, I have sent an updated version.
regards
Philipp
^ permalink raw reply [flat|nested] 39+ messages in thread
* [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation
@ 2014-09-08 9:20 ` Philipp Zabel
0 siblings, 0 replies; 39+ messages in thread
From: Philipp Zabel @ 2014-09-08 9:20 UTC (permalink / raw)
To: linux-arm-kernel
Hi Jason,
Am Sonntag, den 07.09.2014, 09:32 -0400 schrieb Jason Cooper:
> Just to dot our i's and cross our t's, would you mind sending out a
> non-rfc version of this series?
Thank you for the reminder, I have sent an updated version.
regards
Philipp
^ permalink raw reply [flat|nested] 39+ messages in thread
end of thread, other threads:[~2014-09-08 9:20 UTC | newest]
Thread overview: 39+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-20 8:42 [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation Philipp Zabel
2014-08-20 8:42 ` Philipp Zabel
2014-08-20 8:42 ` Philipp Zabel
[not found] ` <1408524184-31928-1-git-send-email-p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2014-08-20 8:42 ` [RFC PATCH 1/6] of: Change vendor prefix for Intersil Corporation to isil Philipp Zabel
2014-08-20 8:42 ` Philipp Zabel
2014-08-20 8:42 ` Philipp Zabel
2014-08-20 8:43 ` [RFC PATCH 2/6] Documentation: Add isl1208 and isl12022 to trivial-devices list Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
2014-08-20 8:43 ` [RFC PATCH 3/6] ARM: mvebu: Change vendor prefix for Intersil Corporation to isil Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
2014-08-20 8:43 ` [RFC PATCH 4/6] powerpc/85xx: " Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
[not found] ` <1408524184-31928-5-git-send-email-p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2014-08-22 3:12 ` Rob Herring
2014-08-22 3:12 ` Rob Herring
2014-08-22 3:12 ` Rob Herring
2014-08-20 8:43 ` [RFC PATCH 5/6] rtc: rtc-isl12022: " Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
[not found] ` <1408524184-31928-6-git-send-email-p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2014-08-22 8:57 ` Philipp Zabel
2014-08-22 8:57 ` Philipp Zabel
2014-08-22 8:57 ` Philipp Zabel
2014-08-20 8:43 ` [RFC PATCH 6/6] rtc: rtc-isl12057: " Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
2014-08-20 8:43 ` Philipp Zabel
2014-08-20 12:40 ` Sergei Shtylyov
2014-08-20 12:40 ` Sergei Shtylyov
2014-08-20 12:40 ` Sergei Shtylyov
2014-08-20 12:16 ` [RFC PATCH 0/6] Change vendor prefix for Intersil Corporation Jason Cooper
2014-08-20 12:16 ` Jason Cooper
2014-08-20 12:16 ` Jason Cooper
2014-09-07 13:32 ` Jason Cooper
2014-09-07 13:32 ` Jason Cooper
2014-09-07 13:32 ` Jason Cooper
[not found] ` <20140907133218.GA30828-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>
2014-09-08 9:20 ` Philipp Zabel
2014-09-08 9:20 ` Philipp Zabel
2014-09-08 9:20 ` Philipp Zabel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.