[PATCH 1/8] dt-bindings: input: fsl,scu-key: Document wakeup-source

[PATCH 1/8] dt-bindings: input: fsl,scu-key: Document wakeup-source

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

The SCU Key controller can be used as a system wakeup source.

Document the 'wakeup-source' property.

This fixes the following schema warning:

system-controller: keys: 'wakeup-source' does not match any of the regexes: 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#

Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 Documentation/devicetree/bindings/input/fsl,scu-key.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/input/fsl,scu-key.yaml b/Documentation/devicetree/bindings/input/fsl,scu-key.yaml
index e5a3c355ee1f..29921aab9d97 100644
--- a/Documentation/devicetree/bindings/input/fsl,scu-key.yaml
+++ b/Documentation/devicetree/bindings/input/fsl,scu-key.yaml
@@ -24,6 +24,8 @@ properties:
   linux,keycodes:
     maxItems: 1
 
+  wakeup-source: true
+
 required:
   - compatible
   - linux,keycodes
-- 
2.34.1

[PATCH 2/8] dt-bindings: power: fsl,scu-pd: Document imx8dl

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

imx8dxl also contains the SCU PD block.

Add an entry for 'fsl,imx8dl-scu-pd'.

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 Documentation/devicetree/bindings/power/fsl,scu-pd.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
index 407b7cfec783..7a0f1a400868 100644
--- a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
+++ b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
@@ -20,6 +20,7 @@ properties:
   compatible:
     items:
       - enum:
+          - fsl,imx8dl-scu-pd
           - fsl,imx8qm-scu-pd
           - fsl,imx8qxp-scu-pd
       - const: fsl,scu-pd
-- 
2.34.1

Re: [PATCH 2/8] dt-bindings: power: fsl,scu-pd: Document imx8dl

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 52 bytes --]

Acked-by: Conor Dooley <conor.dooley@microchip.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[PATCH 3/8] arm64: dts: imx8dxl: Pass fsl,imx8dl-scu-pd

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

Pass 'fsl,imx8dl-scu-pd' to fix the following schema warning:

system-controller: power-controller:compatible:0: 'fsl,scu-pd' is not one of ['fsl,imx8qm-scu-pd', 'fsl,imx8qxp-scu-pd']
	from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm64/boot/dts/freescale/imx8dxl.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8dxl.dtsi b/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
index 792b7224ca5b..19aa89ea442a 100644
--- a/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
@@ -122,7 +122,7 @@ &lsio_mu1 1 0
 			  &lsio_mu1 3 3>;
 
 		pd: power-controller {
-			compatible = "fsl,scu-pd";
+			compatible = "fsl,imx8dl-scu-pd", "fsl,scu-pd";
 			#power-domain-cells = <1>;
 			wakeup-irq = <160 163 235 236 237 228 229 230 231 238
 				     239 240 166 169>;
-- 
2.34.1

Re: [PATCH 3/8] arm64: dts: imx8dxl: Pass fsl,imx8dl-scu-pd

[Shawn Guo]

On Tue, Sep 26, 2023 at 09:29:52AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Pass 'fsl,imx8dl-scu-pd' to fix the following schema warning:
> 
> system-controller: power-controller:compatible:0: 'fsl,scu-pd' is not one of ['fsl,imx8qm-scu-pd', 'fsl,imx8qxp-scu-pd']
> 	from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Applied all 4 dts changes, thanks!

[PATCH 4/8] arm64: dts: imx8dxl: Remove wakeup-irq

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

wakeup-irq is not documented, and not used anywhere.

Remove it.

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm64/boot/dts/freescale/imx8dxl.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8dxl.dtsi b/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
index 19aa89ea442a..c166ffebe438 100644
--- a/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
@@ -124,8 +124,6 @@ &lsio_mu1 1 0
 		pd: power-controller {
 			compatible = "fsl,imx8dl-scu-pd", "fsl,scu-pd";
 			#power-domain-cells = <1>;
-			wakeup-irq = <160 163 235 236 237 228 229 230 231 238
-				     239 240 166 169>;
 		};
 
 		clk: clock-controller {
-- 
2.34.1

[PATCH 5/8] dt-bindings: thermal: fsl,scu-thermal: Document imx8dl

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

imx8dxl also contains the SCU thermal block.

Add an entry for 'fsl,imx8dxl-sc-thermal'.

Cc: Rafael J. Wysocki <rafael@kernel.org>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Amit Kucheria <amitk@kernel.org>
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 .../devicetree/bindings/thermal/fsl,scu-thermal.yaml          | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml b/Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml
index 3721c8c8ec64..e02d04d4f71e 100644
--- a/Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml
+++ b/Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml
@@ -18,7 +18,9 @@ allOf:
 properties:
   compatible:
     items:
-      - const: fsl,imx8qxp-sc-thermal
+      - enum:
+          - fsl,imx8dxl-sc-thermal
+          - fsl,imx8qxp-sc-thermal
       - const: fsl,imx-sc-thermal
 
   '#thermal-sensor-cells':
-- 
2.34.1

Re: [PATCH 5/8] dt-bindings: thermal: fsl,scu-thermal: Document imx8dl

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 52 bytes --]

Acked-by: Conor Dooley <conor.dooley@microchip.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [PATCH 5/8] dt-bindings: thermal: fsl,scu-thermal: Document imx8dl

[Daniel Lezcano]

On 26/09/2023 14:29, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> imx8dxl also contains the SCU thermal block.
> 
> Add an entry for 'fsl,imx8dxl-sc-thermal'.
> 
> Cc: Rafael J. Wysocki <rafael@kernel.org>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Amit Kucheria <amitk@kernel.org>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---

May I apply this patch or do you want it to be applied with the series 
it belongs?

>   .../devicetree/bindings/thermal/fsl,scu-thermal.yaml          | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml b/Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml
> index 3721c8c8ec64..e02d04d4f71e 100644
> --- a/Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml
> +++ b/Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml
> @@ -18,7 +18,9 @@ allOf:
>   properties:
>     compatible:
>       items:
> -      - const: fsl,imx8qxp-sc-thermal
> +      - enum:
> +          - fsl,imx8dxl-sc-thermal
> +          - fsl,imx8qxp-sc-thermal
>         - const: fsl,imx-sc-thermal
>   
>     '#thermal-sensor-cells':

-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

Re: [PATCH 5/8] dt-bindings: thermal: fsl,scu-thermal: Document imx8dl

[Fabio Estevam]

Hi Daniel,

On 28/09/2023 10:32, Daniel Lezcano wrote:

> May I apply this patch or do you want it to be applied with the series
> it belongs?

Please apply it, thanks.

Re: [PATCH 5/8] dt-bindings: thermal: fsl,scu-thermal: Document imx8dl

[Daniel Lezcano]

On 26/09/2023 14:29, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> imx8dxl also contains the SCU thermal block.
> 
> Add an entry for 'fsl,imx8dxl-sc-thermal'.
> 
> Cc: Rafael J. Wysocki <rafael@kernel.org>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Amit Kucheria <amitk@kernel.org>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---

Applied, thanks

-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

[PATCH 6/8] arm64: dts: imx8dxl: Pass fsl,imx8dxl-sc-thermal

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

Pass 'fsl,imx8dxl-sc-thermal' to fix the following schema warning:

system-controller: thermal-sensor:compatible:0: 'fsl,imx8qxp-sc-thermal' was expected
	from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#
system-controller: thermal-sensor:compatible: ['fsl,imx-sc-thermal'] is too short
	from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#
	
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm64/boot/dts/freescale/imx8dxl.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8dxl.dtsi b/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
index c166ffebe438..5c63976090e6 100644
--- a/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
@@ -171,7 +171,7 @@ watchdog {
 		};
 
 		tsens: thermal-sensor {
-			compatible = "fsl,imx-sc-thermal";
+			compatible = "fsl,imx8dxl-sc-thermal", "fsl,imx-sc-thermal";
 			#thermal-sensor-cells = <1>;
 		};
 	};
-- 
2.34.1

[PATCH 7/8] dt-bindings: watchdog: fsl,scu-wdt: Document imx8dl

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

imx8dxl also contains the SCU watchdog block.

Add an entry for 'fsl,imx8dxl-sc-wdt'.

Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
Cc: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml b/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
index 47701248cd8d..8b7aa922249b 100644
--- a/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
@@ -18,7 +18,9 @@ allOf:
 properties:
   compatible:
     items:
-      - const: fsl,imx8qxp-sc-wdt
+      - enum:
+          - fsl,imx8dxl-sc-wdt
+          - fsl,imx8qxp-sc-wdt
       - const: fsl,imx-sc-wdt
 
 required:
-- 
2.34.1

Re: [PATCH 7/8] dt-bindings: watchdog: fsl,scu-wdt: Document imx8dl

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 52 bytes --]

Acked-by: Conor Dooley <conor.dooley@microchip.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[PATCH 8/8] arm64: dts: imx8dxl: Pass fsl,imx8dxl-sc-wdt

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

Pass 'fsl,imx8dxl-sc-wdt' to fix the following schema warning:

system-controller: watchdog:compatible:0: 'fsl,imx8qxp-sc-wdt' was expected
	from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#
system-controller: watchdog:compatible: ['fsl,imx-sc-wdt'] is too short
	from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm64/boot/dts/freescale/imx8dxl.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8dxl.dtsi b/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
index 5c63976090e6..f580eb6db9a6 100644
--- a/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8dxl.dtsi
@@ -166,7 +166,7 @@ sc_pwrkey: keys {
 		};
 
 		watchdog {
-			compatible = "fsl,imx-sc-wdt";
+			compatible = "fsl,imx8dxl-sc-wdt", "fsl,imx-sc-wdt";
 			timeout-sec = <60>;
 		};
 
-- 
2.34.1

Re: [PATCH 1/8] dt-bindings: input: fsl,scu-key: Document wakeup-source

[Rob Herring]

On Tue, 26 Sep 2023 09:29:50 -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> The SCU Key controller can be used as a system wakeup source.
> 
> Document the 'wakeup-source' property.
> 
> This fixes the following schema warning:
> 
> system-controller: keys: 'wakeup-source' does not match any of the regexes: 'pinctrl-[0-9]+'
> 	from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#
> 
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>  Documentation/devicetree/bindings/input/fsl,scu-key.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 

Acked-by: Rob Herring <robh@kernel.org>

Re: [PATCH 1/8] dt-bindings: input: fsl,scu-key: Document wakeup-source

[Fabio Estevam]

Hi Dmitry,

On Thu, Sep 28, 2023 at 1:48 PM Rob Herring <robh@kernel.org> wrote:
>
>
> On Tue, 26 Sep 2023 09:29:50 -0300, Fabio Estevam wrote:
> > From: Fabio Estevam <festevam@denx.de>
> >
> > The SCU Key controller can be used as a system wakeup source.
> >
> > Document the 'wakeup-source' property.
> >
> > This fixes the following schema warning:
> >
> > system-controller: keys: 'wakeup-source' does not match any of the regexes: 'pinctrl-[0-9]+'
> >       from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#
> >
> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > Signed-off-by: Fabio Estevam <festevam@denx.de>
> > ---
> >  Documentation/devicetree/bindings/input/fsl,scu-key.yaml | 2 ++
> >  1 file changed, 2 insertions(+)
> >
>
> Acked-by: Rob Herring <robh@kernel.org>

Could you please take this one?

Re: [PATCH 1/8] dt-bindings: input: fsl,scu-key: Document wakeup-source

[Dmitry Torokhov]

Hi Fabio,

On Wed, Oct 25, 2023 at 05:07:39PM -0300, Fabio Estevam wrote:
> Hi Dmitry,
> 
> On Thu, Sep 28, 2023 at 1:48 PM Rob Herring <robh@kernel.org> wrote:
> >
> >
> > On Tue, 26 Sep 2023 09:29:50 -0300, Fabio Estevam wrote:
> > > From: Fabio Estevam <festevam@denx.de>
> > >
> > > The SCU Key controller can be used as a system wakeup source.
> > >
> > > Document the 'wakeup-source' property.
> > >
> > > This fixes the following schema warning:
> > >
> > > system-controller: keys: 'wakeup-source' does not match any of the regexes: 'pinctrl-[0-9]+'
> > >       from schema $id: http://devicetree.org/schemas/firmware/fsl,scu.yaml#
> > >
> > > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > > Signed-off-by: Fabio Estevam <festevam@denx.de>
> > > ---
> > >  Documentation/devicetree/bindings/input/fsl,scu-key.yaml | 2 ++
> > >  1 file changed, 2 insertions(+)
> > >
> >
> > Acked-by: Rob Herring <robh@kernel.org>
> 
> Could you please take this one?

Applied, thank you.

-- 
Dmitry