2016-10-08 20:34:43

by Scott Branden

Subject: [PATCH] ARM: dts: fix naming of pinctrl node

Remove 0x from pinctrl node to match device tree naming convention.

Signed-off-by: Scott Branden <[email protected]>
---
arch/arm/boot/dts/bcm-cygnus.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/bcm-cygnus.dtsi b/arch/arm/boot/dts/bcm-cygnus.dtsi
index fabc9f3..539c58f 100644
--- a/arch/arm/boot/dts/bcm-cygnus.dtsi
+++ b/arch/arm/boot/dts/bcm-cygnus.dtsi
@@ -108,7 +108,7 @@
};
};

- pinctrl: pinctrl@0x0301d0c8 {
+ pinctrl: pinctrl@0301d0c8 {
compatible = "brcm,cygnus-pinmux";
reg = <0x0301d0c8 0x30>,
<0x0301d24c 0x2c>;
--
2.5.0

2016-10-08 21:03:31

by Ray Jui

Subject: Re: [PATCH] ARM: dts: fix naming of pinctrl node

On 10/8/2016 1:34 PM, Scott Branden wrote:
> Remove 0x from pinctrl node to match device tree naming convention.
>
> Signed-off-by: Scott Branden <[email protected]>
> ---
> arch/arm/boot/dts/bcm-cygnus.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/bcm-cygnus.dtsi b/arch/arm/boot/dts/bcm-cygnus.dtsi
> index fabc9f3..539c58f 100644
> --- a/arch/arm/boot/dts/bcm-cygnus.dtsi
> +++ b/arch/arm/boot/dts/bcm-cygnus.dtsi
> @@ -108,7 +108,7 @@
> };
> };
>
> - pinctrl: pinctrl@0x0301d0c8 {
> + pinctrl: pinctrl@0301d0c8 {
> compatible = "brcm,cygnus-pinmux";
> reg = <0x0301d0c8 0x30>,
> <0x0301d24c 0x2c>;
>

Looks good to me!

Reviewed-by: Ray Jui <[email protected]>

2016-10-10 09:07:13

by Florian Fainelli

Subject: Re: [PATCH] ARM: dts: fix naming of pinctrl node

On 10/08/2016 02:03 PM, Ray Jui wrote:
>
>
> On 10/8/2016 1:34 PM, Scott Branden wrote:
>> Remove 0x from pinctrl node to match device tree naming convention.
>>
>> Signed-off-by: Scott Branden <[email protected]>
>> ---
>> arch/arm/boot/dts/bcm-cygnus.dtsi | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/bcm-cygnus.dtsi
>> b/arch/arm/boot/dts/bcm-cygnus.dtsi
>> index fabc9f3..539c58f 100644
>> --- a/arch/arm/boot/dts/bcm-cygnus.dtsi
>> +++ b/arch/arm/boot/dts/bcm-cygnus.dtsi
>> @@ -108,7 +108,7 @@
>> };
>> };
>>
>> - pinctrl: pinctrl@0x0301d0c8 {
>> + pinctrl: pinctrl@0301d0c8 {
>> compatible = "brcm,cygnus-pinmux";
>> reg = <0x0301d0c8 0x30>,
>> <0x0301d24c 0x2c>;
>>
>
> Looks good to me!
>
> Reviewed-by: Ray Jui <[email protected]>

Applied, thanks!
--
Florian