[RFC V1] devicetree: Dialog Semiconductor consolidate existing vendor prefixes to standardise on 'dlg'
Mark Rutland
mark.rutland at arm.com
Mon Jul 14 09:21:01 PDT 2014
On Mon, Jul 14, 2014 at 05:07:20PM +0100, Opensource [Steve Twiss] wrote:
> From: Opensource [Steve Twiss] <stwiss.opensource at 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 at 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
>
>
More information about the linux-arm-kernel
mailing list