* [RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg' @ 2014-07-14 16:07 Opensource [Steve Twiss] 2014-07-14 16:21 ` Mark Rutland 0 siblings, 1 reply; 5+ messages in thread From: Opensource [Steve Twiss] @ 2014-07-14 16:07 UTC (permalink / raw) To: Andrew Lunn, Arnaud Ebalard, DEVICE TREE, Florian Vaussard, Guennadi Liakhovetski, Guenter Roeck, Ian Campbell, Jason Cooper, Jason Gunthorpe, Jonathan Cameron, Kumar Gala, LINUX ARM KERNEL, LINUX KERNEL, Lee Jones, Mark Brown, Mark Rutland, Martin Fuzzey, Pawel Moll, Rob Herring, Russell King, Samuel Ortiz, Sascha Hauer Cc: Philipp Zabel [-- Attachment #1: Type: text/plain, Size: 6965 bytes --] From: Opensource [Steve Twiss] <stwiss.opensource@diasemi.com> This patch series updates the device tree vendor prefix for Dialog Semiconductor Various methods are currently used throughout the kernel: 'diasemi', 'dialog' and 'dlg'. Others have also been suggested. This patch set aims to consolidate the usage of the vendor prefix to use a common standard. The prefix 'dlg' is used. This e-mail is in response to the previous thread here: - https://lkml.org/lkml/2014/6/11/262 Signed-off-by: Opensource [Steve Twiss] <stwiss.opensource@diasemi.com> --- Checks performed with linux-next/next-20140714/scripts/checkpatch.pl trivial-devices.txt total: 0 errors, 0 warnings, 89 lines checked da9210.txt total: 0 errors, 0 warnings, 21 lines checked vendor-prefixes.txt total: 0 errors, 0 warnings, 149 lines checked imx53-smd.dts total: 0 errors, 2 warnings, 279 lines checked imx6qdl-phytec-pfla02.dtsi total: 0 errors, 2 warnings, 357 lines checked da9055-core.c total: 0 errors, 0 warnings, 428 lines checked This patch applies against linux-next and next-20140714 Regards, Steve Twiss, Dialog Semiconductor Ltd. .../devicetree/bindings/i2c/trivial-devices.txt | 2 +- .../devicetree/bindings/regulator/da9210.txt | 4 ++-- .../devicetree/bindings/vendor-prefixes.txt | 1 + arch/arm/boot/dts/imx53-smd.dts | 2 +- arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi | 2 +- drivers/mfd/da9055-core.c | 26 +++++++++++----------- 6 files changed, 19 insertions(+), 18 deletions(-) diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt index 37803eb..a02b8a1 100644 --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt @@ -44,7 +44,7 @@ dallas,ds1775 Tiny Digital Thermometer and Thermostat dallas,ds3232 Extremely Accurate I²C RTC with Integrated Crystal and SRAM dallas,ds4510 CPU Supervisor with Nonvolatile Memory and Programmable I/O dallas,ds75 Digital Thermometer and Thermostat -dialog,da9053 DA9053: flexible system level PMIC with multicore support +dlg,da9053 DA9053: flexible system level PMIC with multicore support epson,rx8025 High-Stability. I2C-Bus INTERFACE REAL TIME CLOCK MODULE epson,rx8581 I2C-BUS INTERFACE REAL TIME CLOCK MODULE fsl,mag3110 MAG3110: Xtrinsic High Accuracy, 3D Magnetometer diff --git a/Documentation/devicetree/bindings/regulator/da9210.txt b/Documentation/devicetree/bindings/regulator/da9210.txt index f120f22..3297c53 100644 --- a/Documentation/devicetree/bindings/regulator/da9210.txt +++ b/Documentation/devicetree/bindings/regulator/da9210.txt @@ -2,7 +2,7 @@ Required properties: -- compatible: must be "diasemi,da9210" +- compatible: must be "dlg,da9210" - reg: the i2c slave address of the regulator. It should be 0x68. Any standard regulator properties can be used to configure the single da9210 @@ -11,7 +11,7 @@ DCDC. Example: da9210@68 { - compatible = "diasemi,da9210"; + compatible = "dlg,da9210"; reg = <0x68>; regulator-min-microvolt = <900000>; diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt index 5d27e5a..35a833e 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.txt +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt @@ -37,6 +37,7 @@ dallas Maxim Integrated Products (formerly Dallas Semiconductor) davicom DAVICOM Semiconductor, Inc. denx Denx Software Engineering digi Digi International Inc. +dlg Dialog Semiconductor dlink D-Link Corporation dmo Data Modul AG ebv EBV Elektronik diff --git a/arch/arm/boot/dts/imx53-smd.dts b/arch/arm/boot/dts/imx53-smd.dts index 5ec1590..1d32557 100644 --- a/arch/arm/boot/dts/imx53-smd.dts +++ b/arch/arm/boot/dts/imx53-smd.dts @@ -265,7 +265,7 @@ }; pmic: dialog@48 { - compatible = "dialog,da9053", "dialog,da9052"; + compatible = "dlg,da9053", "dlg,da9052"; reg = <0x48>; }; }; diff --git a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi index 2694aa8..0e50bb0 100644 --- a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi +++ b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi @@ -83,7 +83,7 @@ }; pmic@58 { - compatible = "dialog,da9063"; + compatible = "dlg,da9063"; reg = <0x58>; interrupt-parent = <&gpio4>; interrupts = <17 0x8>; /* active-low GPIO4_17 */ diff --git a/drivers/mfd/da9055-core.c b/drivers/mfd/da9055-core.c index caf8dcf..b4d920c 100644 --- a/drivers/mfd/da9055-core.c +++ b/drivers/mfd/da9055-core.c @@ -296,73 +296,73 @@ static struct resource da9055_ld05_6_resource = { static const struct mfd_cell da9055_devs[] = { { - .of_compatible = "dialog,da9055-gpio", + .of_compatible = "dlg,da9055-gpio", .name = "da9055-gpio", }, { - .of_compatible = "dialog,da9055-regulator", + .of_compatible = "dlg,da9055-regulator", .name = "da9055-regulator", .id = 1, }, { - .of_compatible = "dialog,da9055-regulator", + .of_compatible = "dlg,da9055-regulator", .name = "da9055-regulator", .id = 2, }, { - .of_compatible = "dialog,da9055-regulator", + .of_compatible = "dlg,da9055-regulator", .name = "da9055-regulator", .id = 3, }, { - .of_compatible = "dialog,da9055-regulator", + .of_compatible = "dlg,da9055-regulator", .name = "da9055-regulator", .id = 4, }, { - .of_compatible = "dialog,da9055-regulator", + .of_compatible = "dlg,da9055-regulator", .name = "da9055-regulator", .id = 5, }, { - .of_compatible = "dialog,da9055-regulator", + .of_compatible = "dlg,da9055-regulator", .name = "da9055-regulator", .id = 6, }, { - .of_compatible = "dialog,da9055-regulator", + .of_compatible = "dlg,da9055-regulator", .name = "da9055-regulator", .id = 7, .resources = &da9055_ld05_6_resource, .num_resources = 1, }, { - .of_compatible = "dialog,da9055-regulator", + .of_compatible = "dlg,da9055-regulator", .name = "da9055-regulator", .resources = &da9055_ld05_6_resource, .num_resources = 1, .id = 8, }, { - .of_compatible = "dialog,da9055-onkey", + .of_compatible = "dlg,da9055-onkey", .name = "da9055-onkey", .resources = &da9055_onkey_resource, .num_resources = 1, }, { - .of_compatible = "dialog,da9055-rtc", + .of_compatible = "dlg,da9055-rtc", .name = "da9055-rtc", .resources = da9055_rtc_resource, .num_resources = ARRAY_SIZE(da9055_rtc_resource), }, { - .of_compatible = "dialog,da9055-hwmon", + .of_compatible = "dlg,da9055-hwmon", .name = "da9055-hwmon", .resources = &da9055_hwmon_resource, .num_resources = 1, }, { - .of_compatible = "dialog,da9055-watchdog", + .of_compatible = "dlg,da9055-watchdog", .name = "da9055-watchdog", }, }; -- end-of-patch for RFC V1 [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg' 2014-07-14 16:07 [RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg' Opensource [Steve Twiss] @ 2014-07-14 16:21 ` Mark Rutland 2014-07-15 10:07 ` Opensource [Steve Twiss] 0 siblings, 1 reply; 5+ messages in thread From: Mark Rutland @ 2014-07-14 16:21 UTC (permalink / raw) To: Opensource [Steve Twiss] Cc: Andrew Lunn, Mark Brown, Thierry Reding, Lee Jones, Russell King, Samuel Ortiz, Jason Gunthorpe, Shawn Guo, Philipp Zabel, Florian Vaussard, DEVICE TREE, Jason Cooper, Pawel Moll, Ian Campbell, Arnaud Ebalard, Rob Herring, LINUX ARM KERNEL, Martin Fuzzey, Support Opensource, LINUX KERNEL, Silvio F, Guenter Roeck On Mon, Jul 14, 2014 at 05:07:20PM +0100, Opensource [Steve Twiss] wrote: > From: Opensource [Steve Twiss] <stwiss.opensource@diasemi.com> > > This patch series updates the device tree vendor prefix for > Dialog Semiconductor > > Various methods are currently used throughout the kernel: 'diasemi', > 'dialog' and 'dlg'. Others have also been suggested. > > This patch set aims to consolidate the usage of the vendor prefix to > use a common standard. The prefix 'dlg' is used. > > This e-mail is in response to the previous thread here: > > - https://lkml.org/lkml/2014/6/11/262 In that response, Rob pointed out the old forms should be marked deprecated, but the patch seems to outright remove them. Are we sure this doesn't break an existing DTB anywhere? Thanks, Mark. > > Signed-off-by: Opensource [Steve Twiss] <stwiss.opensource@diasemi.com> > --- > Checks performed with linux-next/next-20140714/scripts/checkpatch.pl > trivial-devices.txt total: 0 errors, 0 warnings, 89 lines checked > da9210.txt total: 0 errors, 0 warnings, 21 lines checked > vendor-prefixes.txt total: 0 errors, 0 warnings, 149 lines checked > imx53-smd.dts total: 0 errors, 2 warnings, 279 lines checked > imx6qdl-phytec-pfla02.dtsi total: 0 errors, 2 warnings, 357 lines checked > da9055-core.c total: 0 errors, 0 warnings, 428 lines checked > > This patch applies against linux-next and next-20140714 > > Regards, > Steve Twiss, Dialog Semiconductor Ltd. > > .../devicetree/bindings/i2c/trivial-devices.txt | 2 +- > .../devicetree/bindings/regulator/da9210.txt | 4 ++-- > .../devicetree/bindings/vendor-prefixes.txt | 1 + > arch/arm/boot/dts/imx53-smd.dts | 2 +- > arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi | 2 +- > drivers/mfd/da9055-core.c | 26 +++++++++++----------- > 6 files changed, 19 insertions(+), 18 deletions(-) > > diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt > index 37803eb..a02b8a1 100644 > --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt > +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt > @@ -44,7 +44,7 @@ dallas,ds1775 Tiny Digital Thermometer and Thermostat > dallas,ds3232 Extremely Accurate I²C RTC with Integrated Crystal and SRAM > dallas,ds4510 CPU Supervisor with Nonvolatile Memory and Programmable I/O > dallas,ds75 Digital Thermometer and Thermostat > -dialog,da9053 DA9053: flexible system level PMIC with multicore support > +dlg,da9053 DA9053: flexible system level PMIC with multicore support > epson,rx8025 High-Stability. I2C-Bus INTERFACE REAL TIME CLOCK MODULE > epson,rx8581 I2C-BUS INTERFACE REAL TIME CLOCK MODULE > fsl,mag3110 MAG3110: Xtrinsic High Accuracy, 3D Magnetometer > diff --git a/Documentation/devicetree/bindings/regulator/da9210.txt b/Documentation/devicetree/bindings/regulator/da9210.txt > index f120f22..3297c53 100644 > --- a/Documentation/devicetree/bindings/regulator/da9210.txt > +++ b/Documentation/devicetree/bindings/regulator/da9210.txt > @@ -2,7 +2,7 @@ > > Required properties: > > -- compatible: must be "diasemi,da9210" > +- compatible: must be "dlg,da9210" > - reg: the i2c slave address of the regulator. It should be 0x68. > > Any standard regulator properties can be used to configure the single da9210 > @@ -11,7 +11,7 @@ DCDC. > Example: > > da9210@68 { > - compatible = "diasemi,da9210"; > + compatible = "dlg,da9210"; > reg = <0x68>; > > regulator-min-microvolt = <900000>; > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt > index 5d27e5a..35a833e 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt > @@ -37,6 +37,7 @@ dallas Maxim Integrated Products (formerly Dallas Semiconductor) > davicom DAVICOM Semiconductor, Inc. > denx Denx Software Engineering > digi Digi International Inc. > +dlg Dialog Semiconductor > dlink D-Link Corporation > dmo Data Modul AG > ebv EBV Elektronik > diff --git a/arch/arm/boot/dts/imx53-smd.dts b/arch/arm/boot/dts/imx53-smd.dts > index 5ec1590..1d32557 100644 > --- a/arch/arm/boot/dts/imx53-smd.dts > +++ b/arch/arm/boot/dts/imx53-smd.dts > @@ -265,7 +265,7 @@ > }; > > pmic: dialog@48 { > - compatible = "dialog,da9053", "dialog,da9052"; > + compatible = "dlg,da9053", "dlg,da9052"; > reg = <0x48>; > }; > }; > diff --git a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi > index 2694aa8..0e50bb0 100644 > --- a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi > @@ -83,7 +83,7 @@ > }; > > pmic@58 { > - compatible = "dialog,da9063"; > + compatible = "dlg,da9063"; > reg = <0x58>; > interrupt-parent = <&gpio4>; > interrupts = <17 0x8>; /* active-low GPIO4_17 */ > diff --git a/drivers/mfd/da9055-core.c b/drivers/mfd/da9055-core.c > index caf8dcf..b4d920c 100644 > --- a/drivers/mfd/da9055-core.c > +++ b/drivers/mfd/da9055-core.c > @@ -296,73 +296,73 @@ static struct resource da9055_ld05_6_resource = { > > static const struct mfd_cell da9055_devs[] = { > { > - .of_compatible = "dialog,da9055-gpio", > + .of_compatible = "dlg,da9055-gpio", > .name = "da9055-gpio", > }, > { > - .of_compatible = "dialog,da9055-regulator", > + .of_compatible = "dlg,da9055-regulator", > .name = "da9055-regulator", > .id = 1, > }, > { > - .of_compatible = "dialog,da9055-regulator", > + .of_compatible = "dlg,da9055-regulator", > .name = "da9055-regulator", > .id = 2, > }, > { > - .of_compatible = "dialog,da9055-regulator", > + .of_compatible = "dlg,da9055-regulator", > .name = "da9055-regulator", > .id = 3, > }, > { > - .of_compatible = "dialog,da9055-regulator", > + .of_compatible = "dlg,da9055-regulator", > .name = "da9055-regulator", > .id = 4, > }, > { > - .of_compatible = "dialog,da9055-regulator", > + .of_compatible = "dlg,da9055-regulator", > .name = "da9055-regulator", > .id = 5, > }, > { > - .of_compatible = "dialog,da9055-regulator", > + .of_compatible = "dlg,da9055-regulator", > .name = "da9055-regulator", > .id = 6, > }, > { > - .of_compatible = "dialog,da9055-regulator", > + .of_compatible = "dlg,da9055-regulator", > .name = "da9055-regulator", > .id = 7, > .resources = &da9055_ld05_6_resource, > .num_resources = 1, > }, > { > - .of_compatible = "dialog,da9055-regulator", > + .of_compatible = "dlg,da9055-regulator", > .name = "da9055-regulator", > .resources = &da9055_ld05_6_resource, > .num_resources = 1, > .id = 8, > }, > { > - .of_compatible = "dialog,da9055-onkey", > + .of_compatible = "dlg,da9055-onkey", > .name = "da9055-onkey", > .resources = &da9055_onkey_resource, > .num_resources = 1, > }, > { > - .of_compatible = "dialog,da9055-rtc", > + .of_compatible = "dlg,da9055-rtc", > .name = "da9055-rtc", > .resources = da9055_rtc_resource, > .num_resources = ARRAY_SIZE(da9055_rtc_resource), > }, > { > - .of_compatible = "dialog,da9055-hwmon", > + .of_compatible = "dlg,da9055-hwmon", > .name = "da9055-hwmon", > .resources = &da9055_hwmon_resource, > .num_resources = 1, > }, > { > - .of_compatible = "dialog,da9055-watchdog", > + .of_compatible = "dlg,da9055-watchdog", > .name = "da9055-watchdog", > }, > }; > -- > end-of-patch for RFC V1 > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg' 2014-07-14 16:21 ` Mark Rutland @ 2014-07-15 10:07 ` Opensource [Steve Twiss] 2014-07-15 11:09 ` Mark Rutland 0 siblings, 1 reply; 5+ messages in thread From: Opensource [Steve Twiss] @ 2014-07-15 10:07 UTC (permalink / raw) To: Mark Rutland Cc: Andrew Lunn, Mark Brown, Thierry Reding, Lee Jones, Russell King, Samuel Ortiz, Jason Gunthorpe, Shawn Guo, Philipp Zabel, Florian Vaussard, DEVICE TREE, Jason Cooper, Pawel Moll, Ian Campbell, Arnaud Ebalard, Rob Herring, LINUX ARM KERNEL, Martin Fuzzey, Support Opensource, LINUX KERNEL, Silvio F, Guenter Roeck On 14 July 2014 17:21, Mark Rutland wrote: >From: Mark Rutland [mailto:mark.rutland@arm.com] > >On Mon, Jul 14, 2014 at 05:07:20PM +0100, Opensource [Steve Twiss] wrote: >> From: Opensource [Steve Twiss] <stwiss.opensource@diasemi.com> >> >> This patch series updates the device tree vendor prefix for >> Dialog Semiconductor >> >> Various methods are currently used throughout the kernel: 'diasemi', >> 'dialog' and 'dlg'. Others have also been suggested. >> >> This patch set aims to consolidate the usage of the vendor prefix to >> use a common standard. The prefix 'dlg' is used. >> >> This e-mail is in response to the previous thread here: >> >> - https://lkml.org/lkml/2014/6/11/262 > >In that response, Rob pointed out the old forms should be marked >deprecated, but the patch seems to outright remove them. > >Are we sure this doesn't break an existing DTB anywhere? > >Thanks, >Mark. Hi Mark, Rob said that the old forms should be marked as deprecated, but I didn't deprecate any "dialog" or "diasemi" entries in this patch because I have only made two types of change: (1) definitions of "diasemi" or "dialog" that have not been used in the kernel; (2) definitions of "diasemi" or "dialog" that are incompatible with existing driver code. In both of these cases they cannot affect any working DTB. Actually -- I will need to resend a V2 to this patch because I did miss out two files for DA9210 which reference the unused compatible flag "diasemi" and I might as well fix those while I am doing this change. I will resend those changes all my "working" in my next patch RFC V2. Regards, Steve >> >> Signed-off-by: Opensource [Steve Twiss] <stwiss.opensource@diasemi.com> >> --- >> Checks performed with linux-next/next-20140714/scripts/checkpatch.pl >> trivial-devices.txt total: 0 errors, 0 warnings, 89 lines checked >> da9210.txt total: 0 errors, 0 warnings, 21 lines checked >> vendor-prefixes.txt total: 0 errors, 0 warnings, 149 lines checked >> imx53-smd.dts total: 0 errors, 2 warnings, 279 lines checked >> imx6qdl-phytec-pfla02.dtsi total: 0 errors, 2 warnings, 357 lines checked >> da9055-core.c total: 0 errors, 0 warnings, 428 lines checked >> >> This patch applies against linux-next and next-20140714 >> >> Regards, >> Steve Twiss, Dialog Semiconductor Ltd. >> >> .../devicetree/bindings/i2c/trivial-devices.txt | 2 +- >> .../devicetree/bindings/regulator/da9210.txt | 4 ++-- >> .../devicetree/bindings/vendor-prefixes.txt | 1 + >> arch/arm/boot/dts/imx53-smd.dts | 2 +- >> arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi | 2 +- >> drivers/mfd/da9055-core.c | 26 +++++++++++----------- >> 6 files changed, 19 insertions(+), 18 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt >b/Documentation/devicetree/bindings/i2c/trivial-devices.txt >> index 37803eb..a02b8a1 100644 >> --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt >> +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt >> @@ -44,7 +44,7 @@ dallas,ds1775 Tiny Digital Thermometer and >Thermostat >> dallas,ds3232 Extremely Accurate I²C RTC with Integrated Crystal and >SRAM >> dallas,ds4510 CPU Supervisor with Nonvolatile Memory and >Programmable I/O >> dallas,ds75 Digital Thermometer and Thermostat >> -dialog,da9053 DA9053: flexible system level PMIC with >multicore support >> +dlg,da9053 DA9053: flexible system level PMIC with multicore >support >> epson,rx8025 High-Stability. I2C-Bus INTERFACE REAL TIME CLOCK >MODULE >> epson,rx8581 I2C-BUS INTERFACE REAL TIME CLOCK MODULE >> fsl,mag3110 MAG3110: Xtrinsic High Accuracy, 3D Magnetometer >> diff --git a/Documentation/devicetree/bindings/regulator/da9210.txt >b/Documentation/devicetree/bindings/regulator/da9210.txt >> index f120f22..3297c53 100644 >> --- a/Documentation/devicetree/bindings/regulator/da9210.txt >> +++ b/Documentation/devicetree/bindings/regulator/da9210.txt >> @@ -2,7 +2,7 @@ >> >> Required properties: >> >> -- compatible: must be "diasemi,da9210" >> +- compatible: must be "dlg,da9210" >> - reg: the i2c slave address of the regulator. It should be 0x68. >> >> Any standard regulator properties can be used to configure the single da9210 >> @@ -11,7 +11,7 @@ DCDC. >> Example: >> >> da9210@68 { >> - compatible = "diasemi,da9210"; >> + compatible = "dlg,da9210"; >> reg = <0x68>; >> >> regulator-min-microvolt = <900000>; >> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt >b/Documentation/devicetree/bindings/vendor-prefixes.txt >> index 5d27e5a..35a833e 100644 >> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt >> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt >> @@ -37,6 +37,7 @@ dallas Maxim Integrated Products (formerly Dallas >Semiconductor) >> davicom DAVICOM Semiconductor, Inc. >> denx Denx Software Engineering >> digi Digi International Inc. >> +dlg Dialog Semiconductor >> dlink D-Link Corporation >> dmo Data Modul AG >> ebv EBV Elektronik >> diff --git a/arch/arm/boot/dts/imx53-smd.dts b/arch/arm/boot/dts/imx53- >smd.dts >> index 5ec1590..1d32557 100644 >> --- a/arch/arm/boot/dts/imx53-smd.dts >> +++ b/arch/arm/boot/dts/imx53-smd.dts >> @@ -265,7 +265,7 @@ >> }; >> >> pmic: dialog@48 { >> - compatible = "dialog,da9053", "dialog,da9052"; >> + compatible = "dlg,da9053", "dlg,da9052"; >> reg = <0x48>; >> }; >> }; >> diff --git a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi >b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi >> index 2694aa8..0e50bb0 100644 >> --- a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi >> +++ b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi >> @@ -83,7 +83,7 @@ >> }; >> >> pmic@58 { >> - compatible = "dialog,da9063"; >> + compatible = "dlg,da9063"; >> reg = <0x58>; >> interrupt-parent = <&gpio4>; >> interrupts = <17 0x8>; /* active-low GPIO4_17 */ >> diff --git a/drivers/mfd/da9055-core.c b/drivers/mfd/da9055-core.c >> index caf8dcf..b4d920c 100644 >> --- a/drivers/mfd/da9055-core.c >> +++ b/drivers/mfd/da9055-core.c >> @@ -296,73 +296,73 @@ static struct resource da9055_ld05_6_resource = { >> >> static const struct mfd_cell da9055_devs[] = { >> { >> - .of_compatible = "dialog,da9055-gpio", >> + .of_compatible = "dlg,da9055-gpio", >> .name = "da9055-gpio", >> }, >> { >> - .of_compatible = "dialog,da9055-regulator", >> + .of_compatible = "dlg,da9055-regulator", >> .name = "da9055-regulator", >> .id = 1, >> }, >> { >> - .of_compatible = "dialog,da9055-regulator", >> + .of_compatible = "dlg,da9055-regulator", >> .name = "da9055-regulator", >> .id = 2, >> }, >> { >> - .of_compatible = "dialog,da9055-regulator", >> + .of_compatible = "dlg,da9055-regulator", >> .name = "da9055-regulator", >> .id = 3, >> }, >> { >> - .of_compatible = "dialog,da9055-regulator", >> + .of_compatible = "dlg,da9055-regulator", >> .name = "da9055-regulator", >> .id = 4, >> }, >> { >> - .of_compatible = "dialog,da9055-regulator", >> + .of_compatible = "dlg,da9055-regulator", >> .name = "da9055-regulator", >> .id = 5, >> }, >> { >> - .of_compatible = "dialog,da9055-regulator", >> + .of_compatible = "dlg,da9055-regulator", >> .name = "da9055-regulator", >> .id = 6, >> }, >> { >> - .of_compatible = "dialog,da9055-regulator", >> + .of_compatible = "dlg,da9055-regulator", >> .name = "da9055-regulator", >> .id = 7, >> .resources = &da9055_ld05_6_resource, >> .num_resources = 1, >> }, >> { >> - .of_compatible = "dialog,da9055-regulator", >> + .of_compatible = "dlg,da9055-regulator", >> .name = "da9055-regulator", >> .resources = &da9055_ld05_6_resource, >> .num_resources = 1, >> .id = 8, >> }, >> { >> - .of_compatible = "dialog,da9055-onkey", >> + .of_compatible = "dlg,da9055-onkey", >> .name = "da9055-onkey", >> .resources = &da9055_onkey_resource, >> .num_resources = 1, >> }, >> { >> - .of_compatible = "dialog,da9055-rtc", >> + .of_compatible = "dlg,da9055-rtc", >> .name = "da9055-rtc", >> .resources = da9055_rtc_resource, >> .num_resources = ARRAY_SIZE(da9055_rtc_resource), >> }, >> { >> - .of_compatible = "dialog,da9055-hwmon", >> + .of_compatible = "dlg,da9055-hwmon", >> .name = "da9055-hwmon", >> .resources = &da9055_hwmon_resource, >> .num_resources = 1, >> }, >> { >> - .of_compatible = "dialog,da9055-watchdog", >> + .of_compatible = "dlg,da9055-watchdog", >> .name = "da9055-watchdog", >> }, >> }; >> -- >> end-of-patch for RFC V1 >> >> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg' 2014-07-15 10:07 ` Opensource [Steve Twiss] @ 2014-07-15 11:09 ` Mark Rutland 2014-07-15 11:13 ` Mark Brown 0 siblings, 1 reply; 5+ messages in thread From: Mark Rutland @ 2014-07-15 11:09 UTC (permalink / raw) To: Opensource [Steve Twiss] Cc: Andrew Lunn, Mark Brown, Thierry Reding, Lee Jones, Russell King, Samuel Ortiz, Jason Gunthorpe, Shawn Guo, Philipp Zabel, Florian Vaussard, DEVICE TREE, Jason Cooper, Pawel Moll, Ian Campbell, Arnaud Ebalard, Rob Herring, LINUX ARM KERNEL, Martin Fuzzey, Support Opensource, LINUX KERNEL, Silvio F, Guenter Roeck On Tue, Jul 15, 2014 at 11:07:57AM +0100, Opensource [Steve Twiss] wrote: > > > On 14 July 2014 17:21, Mark Rutland wrote: > >From: Mark Rutland [mailto:mark.rutland@arm.com] > > > >On Mon, Jul 14, 2014 at 05:07:20PM +0100, Opensource [Steve Twiss] wrote: > >> From: Opensource [Steve Twiss] <stwiss.opensource@diasemi.com> > >> > >> This patch series updates the device tree vendor prefix for > >> Dialog Semiconductor > >> > >> Various methods are currently used throughout the kernel: 'diasemi', > >> 'dialog' and 'dlg'. Others have also been suggested. > >> > >> This patch set aims to consolidate the usage of the vendor prefix to > >> use a common standard. The prefix 'dlg' is used. > >> > >> This e-mail is in response to the previous thread here: > >> > >> - https://lkml.org/lkml/2014/6/11/262 > > > >In that response, Rob pointed out the old forms should be marked > >deprecated, but the patch seems to outright remove them. > > > >Are we sure this doesn't break an existing DTB anywhere? > > > >Thanks, > >Mark. > > Hi Mark, > > Rob said that the old forms should be marked as deprecated, but I didn't > deprecate any "dialog" or "diasemi" entries in this patch because I have > only made two types of change: > > (1) definitions of "diasemi" or "dialog" that have not been used in the kernel; > (2) definitions of "diasemi" or "dialog" that are incompatible with existing driver code. > > In both of these cases they cannot affect any working DTB. Ok, that sounds fine then. That just wasn't immediately obvious. Could you please note those facts in the commit message when you next post? > Actually -- I will need to resend a V2 to this patch because I did miss out two > files for DA9210 which reference the unused compatible flag "diasemi" and I > might as well fix those while I am doing this change. > > I will resend those changes all my "working" in my next patch RFC V2. That sounds fine to me. Cheers, Mark. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg' 2014-07-15 11:09 ` Mark Rutland @ 2014-07-15 11:13 ` Mark Brown 0 siblings, 0 replies; 5+ messages in thread From: Mark Brown @ 2014-07-15 11:13 UTC (permalink / raw) To: Mark Rutland Cc: Andrew Lunn, Thierry Reding, Lee Jones, Russell King, Samuel Ortiz, Jason Gunthorpe, Shawn Guo, Silvio F, Guenter Roeck, Philipp Zabel, DEVICE TREE, Jason Cooper, Pawel Moll, Ian Campbell, Arnaud Ebalard, Rob Herring, LINUX ARM KERNEL, Martin Fuzzey, Support Opensource, LINUX KERNEL, Opensource [Steve Twiss], Florian [-- Attachment #1.1: Type: text/plain, Size: 971 bytes --] On Tue, Jul 15, 2014 at 12:09:44PM +0100, Mark Rutland wrote: > On Tue, Jul 15, 2014 at 11:07:57AM +0100, Opensource [Steve Twiss] wrote: > > Rob said that the old forms should be marked as deprecated, but I didn't > > deprecate any "dialog" or "diasemi" entries in this patch because I have > > only made two types of change: > > (1) definitions of "diasemi" or "dialog" that have not been used in the kernel; > > (2) definitions of "diasemi" or "dialog" that are incompatible with existing driver code. > > In both of these cases they cannot affect any working DTB. > Ok, that sounds fine then. That just wasn't immediately obvious. > Could you please note those facts in the commit message when you next > post? I thought we were already trying to do the stable ABI thing in wich case at least the first case (where we just happen to not have in tree DTs using the identifiers) should still remain supported? It's not like it really costs us anything to do it. [-- Attachment #1.2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-07-15 11:13 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-07-14 16:07 [RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg' Opensource [Steve Twiss] 2014-07-14 16:21 ` Mark Rutland 2014-07-15 10:07 ` Opensource [Steve Twiss] 2014-07-15 11:09 ` Mark Rutland 2014-07-15 11:13 ` Mark Brown
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).