* [PATCH 2/6] ARM: dts: imx51-zii-scu3-esb: Disable VPU
2018-09-11 20:10 [PATCH 1/6] ARM: dts: imx51: Add label for VPU node Fabio Estevam
@ 2018-09-11 20:10 ` Fabio Estevam
2018-09-25 15:07 ` Shawn Guo
2018-09-11 20:10 ` [PATCH 3/6] ARM: dts: imx51-zii-scu2-mezz: " Fabio Estevam
` (4 subsequent siblings)
5 siblings, 1 reply; 17+ messages in thread
From: Fabio Estevam @ 2018-09-11 20:10 UTC (permalink / raw)
To: linux-arm-kernel
imx51-zii-scu3-esb does not have any video encoding/decoding needs,
so disable the VPU.
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
arch/arm/boot/dts/imx51-zii-scu3-esb.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/imx51-zii-scu3-esb.dts b/arch/arm/boot/dts/imx51-zii-scu3-esb.dts
index e6ebac8..14b2077 100644
--- a/arch/arm/boot/dts/imx51-zii-scu3-esb.dts
+++ b/arch/arm/boot/dts/imx51-zii-scu3-esb.dts
@@ -350,6 +350,10 @@
vcc-supply = <&vusb2_reg>;
};
+&vpu {
+ status = "disabled";
+};
+
&wdog1 {
status = "disabled";
};
--
2.7.4
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 2/6] ARM: dts: imx51-zii-scu3-esb: Disable VPU
2018-09-11 20:10 ` [PATCH 2/6] ARM: dts: imx51-zii-scu3-esb: Disable VPU Fabio Estevam
@ 2018-09-25 15:07 ` Shawn Guo
2018-09-25 15:13 ` Andrey Smirnov
0 siblings, 1 reply; 17+ messages in thread
From: Shawn Guo @ 2018-09-25 15:07 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 11, 2018 at 05:10:43PM -0300, Fabio Estevam wrote:
> imx51-zii-scu3-esb does not have any video encoding/decoding needs,
> so disable the VPU.
@Andrey, do you agree?
Shawn
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
> arch/arm/boot/dts/imx51-zii-scu3-esb.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx51-zii-scu3-esb.dts b/arch/arm/boot/dts/imx51-zii-scu3-esb.dts
> index e6ebac8..14b2077 100644
> --- a/arch/arm/boot/dts/imx51-zii-scu3-esb.dts
> +++ b/arch/arm/boot/dts/imx51-zii-scu3-esb.dts
> @@ -350,6 +350,10 @@
> vcc-supply = <&vusb2_reg>;
> };
>
> +&vpu {
> + status = "disabled";
> +};
> +
> &wdog1 {
> status = "disabled";
> };
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 17+ messages in thread* [PATCH 2/6] ARM: dts: imx51-zii-scu3-esb: Disable VPU
2018-09-25 15:07 ` Shawn Guo
@ 2018-09-25 15:13 ` Andrey Smirnov
0 siblings, 0 replies; 17+ messages in thread
From: Andrey Smirnov @ 2018-09-25 15:13 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 25, 2018 at 8:07 AM Shawn Guo <shawnguo@kernel.org> wrote:
>
> On Tue, Sep 11, 2018 at 05:10:43PM -0300, Fabio Estevam wrote:
> > imx51-zii-scu3-esb does not have any video encoding/decoding needs,
> > so disable the VPU.
>
> @Andrey, do you agree?
>
Oh, sorry, the though of providing an Acked-by didn't occur to me for
some reason, my apologies. Yeah, those boards definitely don't use
VPU.
Acked-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Thanks,
Andrey Smirnov
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 3/6] ARM: dts: imx51-zii-scu2-mezz: Disable VPU
2018-09-11 20:10 [PATCH 1/6] ARM: dts: imx51: Add label for VPU node Fabio Estevam
2018-09-11 20:10 ` [PATCH 2/6] ARM: dts: imx51-zii-scu3-esb: Disable VPU Fabio Estevam
@ 2018-09-11 20:10 ` Fabio Estevam
2018-09-25 15:10 ` Shawn Guo
2018-09-11 20:10 ` [PATCH 4/6] ARM: dts: imx51-zii-scu2-mezz: Disable the internal watchdog Fabio Estevam
` (3 subsequent siblings)
5 siblings, 1 reply; 17+ messages in thread
From: Fabio Estevam @ 2018-09-11 20:10 UTC (permalink / raw)
To: linux-arm-kernel
imx51-zii-scu2-mezz does not have any video encoding/decoding needs,
so disable the VPU.
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
arch/arm/boot/dts/imx51-zii-scu2-mezz.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts b/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
index 26cf085..fc4a32c 100644
--- a/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
+++ b/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
@@ -342,6 +342,10 @@
vcc-supply = <&vusb2_reg>;
};
+&vpu {
+ status = "disabled";
+};
+
&iomuxc {
pinctrl_ecspi1: ecspi1grp {
fsl,pins = <
--
2.7.4
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 3/6] ARM: dts: imx51-zii-scu2-mezz: Disable VPU
2018-09-11 20:10 ` [PATCH 3/6] ARM: dts: imx51-zii-scu2-mezz: " Fabio Estevam
@ 2018-09-25 15:10 ` Shawn Guo
2018-09-25 15:13 ` Andrey Smirnov
0 siblings, 1 reply; 17+ messages in thread
From: Shawn Guo @ 2018-09-25 15:10 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 11, 2018 at 05:10:44PM -0300, Fabio Estevam wrote:
> imx51-zii-scu2-mezz does not have any video encoding/decoding needs,
> so disable the VPU.
@Andrey, do you agree?
Shawn
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
> arch/arm/boot/dts/imx51-zii-scu2-mezz.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts b/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
> index 26cf085..fc4a32c 100644
> --- a/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
> +++ b/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
> @@ -342,6 +342,10 @@
> vcc-supply = <&vusb2_reg>;
> };
>
> +&vpu {
> + status = "disabled";
> +};
> +
> &iomuxc {
> pinctrl_ecspi1: ecspi1grp {
> fsl,pins = <
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 4/6] ARM: dts: imx51-zii-scu2-mezz: Disable the internal watchdog
2018-09-11 20:10 [PATCH 1/6] ARM: dts: imx51: Add label for VPU node Fabio Estevam
2018-09-11 20:10 ` [PATCH 2/6] ARM: dts: imx51-zii-scu3-esb: Disable VPU Fabio Estevam
2018-09-11 20:10 ` [PATCH 3/6] ARM: dts: imx51-zii-scu2-mezz: " Fabio Estevam
@ 2018-09-11 20:10 ` Fabio Estevam
2018-09-25 15:12 ` Shawn Guo
2018-09-11 20:10 ` [PATCH 5/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sdio' property Fabio Estevam
` (2 subsequent siblings)
5 siblings, 1 reply; 17+ messages in thread
From: Fabio Estevam @ 2018-09-11 20:10 UTC (permalink / raw)
To: linux-arm-kernel
imx51-zii-scu2-mezz has an external watchdog in the environment
microcontroller, so disable the internal one.
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
arch/arm/boot/dts/imx51-zii-scu2-mezz.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts b/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
index fc4a32c..243d1c8 100644
--- a/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
+++ b/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
@@ -346,6 +346,10 @@
status = "disabled";
};
+&wdog1 {
+ status = "disabled";
+};
+
&iomuxc {
pinctrl_ecspi1: ecspi1grp {
fsl,pins = <
--
2.7.4
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 4/6] ARM: dts: imx51-zii-scu2-mezz: Disable the internal watchdog
2018-09-11 20:10 ` [PATCH 4/6] ARM: dts: imx51-zii-scu2-mezz: Disable the internal watchdog Fabio Estevam
@ 2018-09-25 15:12 ` Shawn Guo
2018-09-25 15:14 ` Andrey Smirnov
0 siblings, 1 reply; 17+ messages in thread
From: Shawn Guo @ 2018-09-25 15:12 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 11, 2018 at 05:10:45PM -0300, Fabio Estevam wrote:
> imx51-zii-scu2-mezz has an external watchdog in the environment
> microcontroller, so disable the internal one.
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
@Andrey, are you okay with this one?
Shawn
> ---
> arch/arm/boot/dts/imx51-zii-scu2-mezz.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts b/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
> index fc4a32c..243d1c8 100644
> --- a/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
> +++ b/arch/arm/boot/dts/imx51-zii-scu2-mezz.dts
> @@ -346,6 +346,10 @@
> status = "disabled";
> };
>
> +&wdog1 {
> + status = "disabled";
> +};
> +
> &iomuxc {
> pinctrl_ecspi1: ecspi1grp {
> fsl,pins = <
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 17+ messages in thread* [PATCH 4/6] ARM: dts: imx51-zii-scu2-mezz: Disable the internal watchdog
2018-09-25 15:12 ` Shawn Guo
@ 2018-09-25 15:14 ` Andrey Smirnov
0 siblings, 0 replies; 17+ messages in thread
From: Andrey Smirnov @ 2018-09-25 15:14 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 25, 2018 at 8:12 AM Shawn Guo <shawnguo@kernel.org> wrote:
>
> On Tue, Sep 11, 2018 at 05:10:45PM -0300, Fabio Estevam wrote:
> > imx51-zii-scu2-mezz has an external watchdog in the environment
> > microcontroller, so disable the internal one.
> >
> > Signed-off-by: Fabio Estevam <festevam@gmail.com>
>
> @Andrey, are you okay with this one?
>
Yep, that's one of the boards that have RAVE SP MCU that implements
watchdog functionality there.
Acked-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Thanks,
Andrey Smirnov
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 5/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sdio' property
2018-09-11 20:10 [PATCH 1/6] ARM: dts: imx51: Add label for VPU node Fabio Estevam
` (2 preceding siblings ...)
2018-09-11 20:10 ` [PATCH 4/6] ARM: dts: imx51-zii-scu2-mezz: Disable the internal watchdog Fabio Estevam
@ 2018-09-11 20:10 ` Fabio Estevam
2018-09-25 15:18 ` Shawn Guo
2018-09-11 20:10 ` [PATCH 6/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sd' property Fabio Estevam
2018-09-26 2:19 ` [PATCH 1/6] ARM: dts: imx51: Add label for VPU node Shawn Guo
5 siblings, 1 reply; 17+ messages in thread
From: Fabio Estevam @ 2018-09-11 20:10 UTC (permalink / raw)
To: linux-arm-kernel
No SDIO devices are connected to these ports, so pass the 'no-sdio'
property.
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
arch/arm/boot/dts/vf610-zii-cfu1.dts | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/boot/dts/vf610-zii-cfu1.dts b/arch/arm/boot/dts/vf610-zii-cfu1.dts
index de3ed03..aba4cec 100644
--- a/arch/arm/boot/dts/vf610-zii-cfu1.dts
+++ b/arch/arm/boot/dts/vf610-zii-cfu1.dts
@@ -122,6 +122,7 @@
non-removable;
no-1-8-v;
keep-power-in-suspend;
+ no-sdio;
status = "okay";
};
@@ -129,6 +130,7 @@
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_esdhc1>;
bus-width = <4>;
+ no-sdio;
status = "okay";
};
--
2.7.4
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 5/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sdio' property
2018-09-11 20:10 ` [PATCH 5/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sdio' property Fabio Estevam
@ 2018-09-25 15:18 ` Shawn Guo
2018-09-25 15:40 ` Andrey Smirnov
0 siblings, 1 reply; 17+ messages in thread
From: Shawn Guo @ 2018-09-25 15:18 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 11, 2018 at 05:10:46PM -0300, Fabio Estevam wrote:
> No SDIO devices are connected to these ports, so pass the 'no-sdio'
> property.
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
@Andrey, are you okay?
Shawn
> ---
> arch/arm/boot/dts/vf610-zii-cfu1.dts | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/boot/dts/vf610-zii-cfu1.dts b/arch/arm/boot/dts/vf610-zii-cfu1.dts
> index de3ed03..aba4cec 100644
> --- a/arch/arm/boot/dts/vf610-zii-cfu1.dts
> +++ b/arch/arm/boot/dts/vf610-zii-cfu1.dts
> @@ -122,6 +122,7 @@
> non-removable;
> no-1-8-v;
> keep-power-in-suspend;
> + no-sdio;
> status = "okay";
> };
>
> @@ -129,6 +130,7 @@
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_esdhc1>;
> bus-width = <4>;
> + no-sdio;
> status = "okay";
> };
>
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 5/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sdio' property
2018-09-25 15:18 ` Shawn Guo
@ 2018-09-25 15:40 ` Andrey Smirnov
0 siblings, 0 replies; 17+ messages in thread
From: Andrey Smirnov @ 2018-09-25 15:40 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 25, 2018 at 8:18 AM Shawn Guo <shawnguo@kernel.org> wrote:
>
> On Tue, Sep 11, 2018 at 05:10:46PM -0300, Fabio Estevam wrote:
> > No SDIO devices are connected to these ports, so pass the 'no-sdio'
> > property.
> >
> > Signed-off-by: Fabio Estevam <festevam@gmail.com>
>
> @Andrey, are you okay?
>
Acked-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Thanks,
Andrey Smirnov
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 6/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sd' property
2018-09-11 20:10 [PATCH 1/6] ARM: dts: imx51: Add label for VPU node Fabio Estevam
` (3 preceding siblings ...)
2018-09-11 20:10 ` [PATCH 5/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sdio' property Fabio Estevam
@ 2018-09-11 20:10 ` Fabio Estevam
2018-09-25 15:19 ` Shawn Guo
2018-09-26 2:19 ` [PATCH 1/6] ARM: dts: imx51: Add label for VPU node Shawn Guo
5 siblings, 1 reply; 17+ messages in thread
From: Fabio Estevam @ 2018-09-11 20:10 UTC (permalink / raw)
To: linux-arm-kernel
Pass the 'no-sd' for esdhc0 controller as it is wired to eMMC.
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
arch/arm/boot/dts/vf610-zii-cfu1.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/vf610-zii-cfu1.dts b/arch/arm/boot/dts/vf610-zii-cfu1.dts
index aba4cec..b76c3d0 100644
--- a/arch/arm/boot/dts/vf610-zii-cfu1.dts
+++ b/arch/arm/boot/dts/vf610-zii-cfu1.dts
@@ -123,6 +123,7 @@
no-1-8-v;
keep-power-in-suspend;
no-sdio;
+ no-sd;
status = "okay";
};
--
2.7.4
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 6/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sd' property
2018-09-11 20:10 ` [PATCH 6/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sd' property Fabio Estevam
@ 2018-09-25 15:19 ` Shawn Guo
2018-09-25 15:41 ` Andrey Smirnov
0 siblings, 1 reply; 17+ messages in thread
From: Shawn Guo @ 2018-09-25 15:19 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 11, 2018 at 05:10:47PM -0300, Fabio Estevam wrote:
> Pass the 'no-sd' for esdhc0 controller as it is wired to eMMC.
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
@Andrey?
Shawn
> ---
> arch/arm/boot/dts/vf610-zii-cfu1.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/vf610-zii-cfu1.dts b/arch/arm/boot/dts/vf610-zii-cfu1.dts
> index aba4cec..b76c3d0 100644
> --- a/arch/arm/boot/dts/vf610-zii-cfu1.dts
> +++ b/arch/arm/boot/dts/vf610-zii-cfu1.dts
> @@ -123,6 +123,7 @@
> no-1-8-v;
> keep-power-in-suspend;
> no-sdio;
> + no-sd;
> status = "okay";
> };
>
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 6/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sd' property
2018-09-25 15:19 ` Shawn Guo
@ 2018-09-25 15:41 ` Andrey Smirnov
0 siblings, 0 replies; 17+ messages in thread
From: Andrey Smirnov @ 2018-09-25 15:41 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 25, 2018 at 8:19 AM Shawn Guo <shawnguo@kernel.org> wrote:
>
> On Tue, Sep 11, 2018 at 05:10:47PM -0300, Fabio Estevam wrote:
> > Pass the 'no-sd' for esdhc0 controller as it is wired to eMMC.
> >
> > Signed-off-by: Fabio Estevam <festevam@gmail.com>
>
> @Andrey?
Acked-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Thanks,
Andrey Smirnov
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 1/6] ARM: dts: imx51: Add label for VPU node
2018-09-11 20:10 [PATCH 1/6] ARM: dts: imx51: Add label for VPU node Fabio Estevam
` (4 preceding siblings ...)
2018-09-11 20:10 ` [PATCH 6/6] ARM: dts: vf610-zii-cfu1: Pass the 'no-sd' property Fabio Estevam
@ 2018-09-26 2:19 ` Shawn Guo
5 siblings, 0 replies; 17+ messages in thread
From: Shawn Guo @ 2018-09-26 2:19 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 11, 2018 at 05:10:42PM -0300, Fabio Estevam wrote:
> Add label for VPU node so that board dts files can disable VPU
> if needed.
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
Applied all, thanks.
^ permalink raw reply [flat|nested] 17+ messages in thread