linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible
@ 2023-06-14 11:36 Rasmus Villemoes
  2023-06-14 17:19 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 4+ messages in thread
From: Rasmus Villemoes @ 2023-06-14 11:36 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski
  Cc: Stefan Wahren, Giulio Benetti, Jesse Taube, Rasmus Villemoes,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Sascha Hauer, linux-kernel, devicetree, linux-arm-kernel

The imx8mp has the same GPT as imx6dl. Add fsl,imx8mp-gpt to the set
of values that can be used together with fsl,imx6dl-gpt.

Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
---
This allows what imx8mp.dtsi already does, namely

	compatible = "fsl,imx8mp-gpt", "fsl,imx6dl-gpt";

Currently "make CHECK_DTBS=1 freescale/imx8mp-evk.dtb" results in
dozens of lines, partly because the IP block appears three times in
imx8mp.dtsi, partly because the checker lists all the ways the
compatible string(s) do not match the binding. With this, that command
merely produces three lines of (unrelated) warnings.

 Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml b/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml
index 716c6afcca1f..f5f8b297da13 100644
--- a/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml
+++ b/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml
@@ -31,6 +31,7 @@ properties:
           - enum:
               - fsl,imx6sl-gpt
               - fsl,imx6sx-gpt
+              - fsl,imx8mp-gpt
               - fsl,imxrt1050-gpt
               - fsl,imxrt1170-gpt
           - const: fsl,imx6dl-gpt
-- 
2.37.2


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

* Re: [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible
  2023-06-14 11:36 [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible Rasmus Villemoes
@ 2023-06-14 17:19 ` Krzysztof Kozlowski
  2023-06-14 19:25   ` Rasmus Villemoes
  0 siblings, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2023-06-14 17:19 UTC (permalink / raw)
  To: Rasmus Villemoes, Rob Herring, Krzysztof Kozlowski
  Cc: Stefan Wahren, Giulio Benetti, Jesse Taube,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Sascha Hauer, linux-kernel, devicetree, linux-arm-kernel

On 14/06/2023 13:36, Rasmus Villemoes wrote:
> The imx8mp has the same GPT as imx6dl. Add fsl,imx8mp-gpt to the set
> of values that can be used together with fsl,imx6dl-gpt.
> 
> Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
> ---
> This allows what imx8mp.dtsi already does, namely
> 
> 	compatible = "fsl,imx8mp-gpt", "fsl,imx6dl-gpt";
> 

https://lore.kernel.org/all/20230327173526.851734-2-u.kleine-koenig@pengutronix.de/

Best regards,
Krzysztof


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

* Re: [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible
  2023-06-14 17:19 ` Krzysztof Kozlowski
@ 2023-06-14 19:25   ` Rasmus Villemoes
  2023-06-15 17:31     ` Rob Herring
  0 siblings, 1 reply; 4+ messages in thread
From: Rasmus Villemoes @ 2023-06-14 19:25 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski
  Cc: Stefan Wahren, Giulio Benetti, Jesse Taube,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Sascha Hauer, linux-kernel, devicetree, linux-arm-kernel,
	Uwe Kleine-König, Marek Vasut, Shawn Guo

On 14/06/2023 19.19, Krzysztof Kozlowski wrote:
> On 14/06/2023 13:36, Rasmus Villemoes wrote:
>> The imx8mp has the same GPT as imx6dl. Add fsl,imx8mp-gpt to the set
>> of values that can be used together with fsl,imx6dl-gpt.
>>
>> Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
>> ---
>> This allows what imx8mp.dtsi already does, namely
>>
>> 	compatible = "fsl,imx8mp-gpt", "fsl,imx6dl-gpt";
>>
> 
> https://lore.kernel.org/all/20230327173526.851734-2-u.kleine-koenig@pengutronix.de/

Thanks, I looked in -next and didn't find anything, but didn't think to
check lore.

So could somebody please pick that up? Shawn?

Rasmus


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

* Re: [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible
  2023-06-14 19:25   ` Rasmus Villemoes
@ 2023-06-15 17:31     ` Rob Herring
  0 siblings, 0 replies; 4+ messages in thread
From: Rob Herring @ 2023-06-15 17:31 UTC (permalink / raw)
  To: Rasmus Villemoes
  Cc: Krzysztof Kozlowski, Krzysztof Kozlowski, Stefan Wahren,
	Giulio Benetti, Jesse Taube, Pengutronix Kernel Team,
	Fabio Estevam, NXP Linux Team, Sascha Hauer, linux-kernel,
	devicetree, linux-arm-kernel, Uwe Kleine-König, Marek Vasut,
	Shawn Guo

On Wed, Jun 14, 2023 at 09:25:11PM +0200, Rasmus Villemoes wrote:
> On 14/06/2023 19.19, Krzysztof Kozlowski wrote:
> > On 14/06/2023 13:36, Rasmus Villemoes wrote:
> >> The imx8mp has the same GPT as imx6dl. Add fsl,imx8mp-gpt to the set
> >> of values that can be used together with fsl,imx6dl-gpt.
> >>
> >> Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
> >> ---
> >> This allows what imx8mp.dtsi already does, namely
> >>
> >> 	compatible = "fsl,imx8mp-gpt", "fsl,imx6dl-gpt";
> >>
> > 
> > https://lore.kernel.org/all/20230327173526.851734-2-u.kleine-koenig@pengutronix.de/
> 
> Thanks, I looked in -next and didn't find anything, but didn't think to
> check lore.
> 
> So could somebody please pick that up? Shawn?

Done.

Rob

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

end of thread, other threads:[~2023-06-15 17:32 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-14 11:36 [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible Rasmus Villemoes
2023-06-14 17:19 ` Krzysztof Kozlowski
2023-06-14 19:25   ` Rasmus Villemoes
2023-06-15 17:31     ` Rob Herring

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