linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] Enable watchdog support for iMX8QM
@ 2025-04-07 17:03 Thomas Richard
  2025-04-07 17:03 ` [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm Thomas Richard
  2025-04-07 17:03 ` [PATCH 2/2] arm64: dts: imx8qm: add system controller watchdog support Thomas Richard
  0 siblings, 2 replies; 8+ messages in thread
From: Thomas Richard @ 2025-04-07 17:03 UTC (permalink / raw)
  To: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Dong Aisheng, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam
  Cc: Thomas Petazzoni, linux-watchdog, devicetree, linux-kernel, imx,
	linux-arm-kernel, Thomas Richard

This series enables wathdog support for iMX8QM SoC. The first patch defines
a imx8qm compatible, and the second one adds the watchdog node in the
iMX8QM dtsi file.

Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
---
Thomas Richard (2):
      dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm
      arm64: dts: imx8qm: add system controller watchdog support

 Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml | 1 +
 arch/arm64/boot/dts/freescale/imx8qm.dtsi                   | 5 +++++
 2 files changed, 6 insertions(+)
---
base-commit: 0af2f6be1b4281385b618cb86ad946eded089ac8
change-id: 20250203-imx8qm-watchdog-b8363342842e

Best regards,
-- 
Thomas Richard <thomas.richard@bootlin.com>



^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm
  2025-04-07 17:03 [PATCH 0/2] Enable watchdog support for iMX8QM Thomas Richard
@ 2025-04-07 17:03 ` Thomas Richard
  2025-04-07 18:55   ` Frank Li
                     ` (2 more replies)
  2025-04-07 17:03 ` [PATCH 2/2] arm64: dts: imx8qm: add system controller watchdog support Thomas Richard
  1 sibling, 3 replies; 8+ messages in thread
From: Thomas Richard @ 2025-04-07 17:03 UTC (permalink / raw)
  To: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Dong Aisheng, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam
  Cc: Thomas Petazzoni, linux-watchdog, devicetree, linux-kernel, imx,
	linux-arm-kernel, Thomas Richard

Add an entry for 'fsl,imx8qm-sc-wdt' as imx8qm also contains the SCU
watchdog block.

Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
---
 Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml b/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
index 8b7aa922249b..1d9f15ec6657 100644
--- a/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
@@ -20,6 +20,7 @@ properties:
     items:
       - enum:
           - fsl,imx8dxl-sc-wdt
+          - fsl,imx8qm-sc-wdt
           - fsl,imx8qxp-sc-wdt
       - const: fsl,imx-sc-wdt
 

-- 
2.39.5



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 2/2] arm64: dts: imx8qm: add system controller watchdog support
  2025-04-07 17:03 [PATCH 0/2] Enable watchdog support for iMX8QM Thomas Richard
  2025-04-07 17:03 ` [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm Thomas Richard
@ 2025-04-07 17:03 ` Thomas Richard
  2025-04-07 19:00   ` Frank Li
  2025-04-25 12:50   ` Oliver Graute
  1 sibling, 2 replies; 8+ messages in thread
From: Thomas Richard @ 2025-04-07 17:03 UTC (permalink / raw)
  To: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Dong Aisheng, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam
  Cc: Thomas Petazzoni, linux-watchdog, devicetree, linux-kernel, imx,
	linux-arm-kernel, Thomas Richard

Add system controller watchdog support for i.MX8QM.

Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
---
 arch/arm64/boot/dts/freescale/imx8qm.dtsi | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8qm.dtsi b/arch/arm64/boot/dts/freescale/imx8qm.dtsi
index 6fa31bc9ece8..edd1201eb6c0 100644
--- a/arch/arm64/boot/dts/freescale/imx8qm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8qm.dtsi
@@ -352,6 +352,11 @@ fec_mac1: mac@1c6 {
 			};
 		};
 
+		watchdog {
+			compatible = "fsl,imx8qm-sc-wdt", "fsl,imx-sc-wdt";
+			timeout-sec = <60>;
+		};
+
 		tsens: thermal-sensor {
 			compatible = "fsl,imx8qxp-sc-thermal", "fsl,imx-sc-thermal";
 			#thermal-sensor-cells = <1>;

-- 
2.39.5



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm
  2025-04-07 17:03 ` [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm Thomas Richard
@ 2025-04-07 18:55   ` Frank Li
  2025-04-10 21:07   ` Rob Herring (Arm)
  2025-04-25 12:50   ` Oliver Graute
  2 siblings, 0 replies; 8+ messages in thread
From: Frank Li @ 2025-04-07 18:55 UTC (permalink / raw)
  To: Thomas Richard
  Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Dong Aisheng, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, Thomas Petazzoni,
	linux-watchdog, devicetree, linux-kernel, imx, linux-arm-kernel

On Mon, Apr 07, 2025 at 07:03:25PM +0200, Thomas Richard wrote:
> Add an entry for 'fsl,imx8qm-sc-wdt' as imx8qm also contains the SCU
> watchdog block.
>
> Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
> ---

Reviewed-by: Frank Li <Frank.Li@nxp.com>

>  Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml b/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
> index 8b7aa922249b..1d9f15ec6657 100644
> --- a/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
> @@ -20,6 +20,7 @@ properties:
>      items:
>        - enum:
>            - fsl,imx8dxl-sc-wdt
> +          - fsl,imx8qm-sc-wdt
>            - fsl,imx8qxp-sc-wdt
>        - const: fsl,imx-sc-wdt
>
>
> --
> 2.39.5
>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] arm64: dts: imx8qm: add system controller watchdog support
  2025-04-07 17:03 ` [PATCH 2/2] arm64: dts: imx8qm: add system controller watchdog support Thomas Richard
@ 2025-04-07 19:00   ` Frank Li
  2025-04-25 12:50   ` Oliver Graute
  1 sibling, 0 replies; 8+ messages in thread
From: Frank Li @ 2025-04-07 19:00 UTC (permalink / raw)
  To: Thomas Richard
  Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Dong Aisheng, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, Thomas Petazzoni,
	linux-watchdog, devicetree, linux-kernel, imx, linux-arm-kernel

On Mon, Apr 07, 2025 at 07:03:26PM +0200, Thomas Richard wrote:
> Add system controller watchdog support for i.MX8QM.
>
> Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8qm.dtsi | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8qm.dtsi b/arch/arm64/boot/dts/freescale/imx8qm.dtsi
> index 6fa31bc9ece8..edd1201eb6c0 100644
> --- a/arch/arm64/boot/dts/freescale/imx8qm.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8qm.dtsi
> @@ -352,6 +352,11 @@ fec_mac1: mac@1c6 {
>  			};
>  		};
>
> +		watchdog {
> +			compatible = "fsl,imx8qm-sc-wdt", "fsl,imx-sc-wdt";
> +			timeout-sec = <60>;
> +		};
> +

because history reason, node have not ordered. But we can try keep as much
as possible.

So place move watchdog after thermal-sensor

Frank

>  		tsens: thermal-sensor {
>  			compatible = "fsl,imx8qxp-sc-thermal", "fsl,imx-sc-thermal";
>  			#thermal-sensor-cells = <1>;
>
> --
> 2.39.5
>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm
  2025-04-07 17:03 ` [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm Thomas Richard
  2025-04-07 18:55   ` Frank Li
@ 2025-04-10 21:07   ` Rob Herring (Arm)
  2025-04-25 12:50   ` Oliver Graute
  2 siblings, 0 replies; 8+ messages in thread
From: Rob Herring (Arm) @ 2025-04-10 21:07 UTC (permalink / raw)
  To: Thomas Richard
  Cc: Shawn Guo, Krzysztof Kozlowski, Fabio Estevam, linux-watchdog,
	Wim Van Sebroeck, linux-kernel, Guenter Roeck, devicetree,
	Conor Dooley, imx, linux-arm-kernel, Sascha Hauer,
	Pengutronix Kernel Team, Dong Aisheng, Thomas Petazzoni


On Mon, 07 Apr 2025 19:03:25 +0200, Thomas Richard wrote:
> Add an entry for 'fsl,imx8qm-sc-wdt' as imx8qm also contains the SCU
> watchdog block.
> 
> Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
> ---
>  Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

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



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm
  2025-04-07 17:03 ` [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm Thomas Richard
  2025-04-07 18:55   ` Frank Li
  2025-04-10 21:07   ` Rob Herring (Arm)
@ 2025-04-25 12:50   ` Oliver Graute
  2 siblings, 0 replies; 8+ messages in thread
From: Oliver Graute @ 2025-04-25 12:50 UTC (permalink / raw)
  To: Thomas Richard
  Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Dong Aisheng, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, Thomas Petazzoni,
	linux-watchdog, devicetree, linux-kernel, imx, linux-arm-kernel

On 07/04/25, Thomas Richard wrote:
> Add an entry for 'fsl,imx8qm-sc-wdt' as imx8qm also contains the SCU
> watchdog block.
> 
> Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>

Acked-by: Oliver Graute <oliver.graute@kococonnector.com>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] arm64: dts: imx8qm: add system controller watchdog support
  2025-04-07 17:03 ` [PATCH 2/2] arm64: dts: imx8qm: add system controller watchdog support Thomas Richard
  2025-04-07 19:00   ` Frank Li
@ 2025-04-25 12:50   ` Oliver Graute
  1 sibling, 0 replies; 8+ messages in thread
From: Oliver Graute @ 2025-04-25 12:50 UTC (permalink / raw)
  To: Thomas Richard
  Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Dong Aisheng, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, Thomas Petazzoni,
	linux-watchdog, devicetree, linux-kernel, imx, linux-arm-kernel

On 07/04/25, Thomas Richard wrote:
> Add system controller watchdog support for i.MX8QM.
> 
> Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>

Acked-by: Oliver Graute <oliver.graute@kococonnector.com>


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2025-04-25 13:54 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-07 17:03 [PATCH 0/2] Enable watchdog support for iMX8QM Thomas Richard
2025-04-07 17:03 ` [PATCH 1/2] dt-bindings: watchdog: fsl,scu-wdt: Document imx8qm Thomas Richard
2025-04-07 18:55   ` Frank Li
2025-04-10 21:07   ` Rob Herring (Arm)
2025-04-25 12:50   ` Oliver Graute
2025-04-07 17:03 ` [PATCH 2/2] arm64: dts: imx8qm: add system controller watchdog support Thomas Richard
2025-04-07 19:00   ` Frank Li
2025-04-25 12:50   ` Oliver Graute

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).