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