[RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg'

[mark.rutland@arm.com (Mark Rutland)]

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 at 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 at 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 at 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
> 
>