2014-07-31 16:10:18

by Andreas Färber

[permalink] [raw]
Subject: [PATCH v4 1/4] ARM: dts: Fix MMC pinctrl for exynos5250-snow

The pinctrl properties should be on the device directly and not on the
slot sub-node.

Reported-by: Doug Anderson <[email protected]>
Cc: Jaehoon Chung <[email protected]>
Signed-off-by: Andreas Färber <[email protected]>
---
v3 -> v4: Unchanged

v3: New (Doug Anderson)
Redundant with Jaehoon Chung's general slot@0 deprecation,
in case that hits the tree earlier.

arch/arm/boot/dts/exynos5250-snow.dts | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
index f2b8c4116541..eb437f6afec1 100644
--- a/arch/arm/boot/dts/exynos5250-snow.dts
+++ b/arch/arm/boot/dts/exynos5250-snow.dts
@@ -240,10 +240,8 @@
*/
mmc@12230000 {
status = "okay";
- slot@0 {
- pinctrl-names = "default";
- pinctrl-0 = <&sd3_clk &sd3_cmd &sd3_bus4>;
- };
+ pinctrl-names = "default";
+ pinctrl-0 = <&sd3_clk &sd3_cmd &sd3_bus4>;
};

i2c@12CD0000 {
--
1.9.3


2014-07-31 18:41:48

by Kukjin Kim

[permalink] [raw]
Subject: Re: [PATCH v4 1/4] ARM: dts: Fix MMC pinctrl for exynos5250-snow

On 08/01/14 01:08, Andreas Färber wrote:
> The pinctrl properties should be on the device directly and not on the
> slot sub-node.
>
> Reported-by: Doug Anderson<[email protected]>
> Cc: Jaehoon Chung<[email protected]>
> Signed-off-by: Andreas Färber<[email protected]>
> ---
> v3 -> v4: Unchanged
>
> v3: New (Doug Anderson)
> Redundant with Jaehoon Chung's general slot@0 deprecation,
> in case that hits the tree earlier.
>
> arch/arm/boot/dts/exynos5250-snow.dts | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
> index f2b8c4116541..eb437f6afec1 100644
> --- a/arch/arm/boot/dts/exynos5250-snow.dts
> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
> @@ -240,10 +240,8 @@
> */
> mmc@12230000 {
> status = "okay";
> - slot@0 {
> - pinctrl-names = "default";
> - pinctrl-0 =<&sd3_clk&sd3_cmd&sd3_bus4>;
> - };
> + pinctrl-names = "default";
> + pinctrl-0 =<&sd3_clk&sd3_cmd&sd3_bus4>;
> };
>
> i2c@12CD0000 {

Once I get Doug's reviewed tag on exynos5250-snow dt changes, I will
take this series :) Though Doug suggested or reported...

Thanks,
Kukjin

2014-07-31 19:20:24

by Tomasz Figa

[permalink] [raw]
Subject: Re: [PATCH v4 1/4] ARM: dts: Fix MMC pinctrl for exynos5250-snow

On 31.07.2014 18:08, Andreas Färber wrote:
> The pinctrl properties should be on the device directly and not on the
> slot sub-node.
>
> Reported-by: Doug Anderson <[email protected]>
> Cc: Jaehoon Chung <[email protected]>
> Signed-off-by: Andreas Färber <[email protected]>
> ---
> v3 -> v4: Unchanged
>
> v3: New (Doug Anderson)
> Redundant with Jaehoon Chung's general slot@0 deprecation,
> in case that hits the tree earlier.
>
> arch/arm/boot/dts/exynos5250-snow.dts | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>

Reviewed-by: Tomasz Figa <[email protected]>

Best regards,
Tomasz