2023-06-14 11:57:12

by Rasmus Villemoes

[permalink] [raw]
Subject: [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible

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 <[email protected]>
---
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



2023-06-14 18:04:25

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible

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 <[email protected]>
> ---
> This allows what imx8mp.dtsi already does, namely
>
> compatible = "fsl,imx8mp-gpt", "fsl,imx6dl-gpt";
>

https://lore.kernel.org/all/[email protected]/

Best regards,
Krzysztof


2023-06-14 19:36:44

by Rasmus Villemoes

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible

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 <[email protected]>
>> ---
>> This allows what imx8mp.dtsi already does, namely
>>
>> compatible = "fsl,imx8mp-gpt", "fsl,imx6dl-gpt";
>>
>
> https://lore.kernel.org/all/[email protected]/

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


2023-06-15 17:44:15

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: timer: fsl,imxgpt: Add imx8mp-gpt compatible

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 <[email protected]>
> >> ---
> >> This allows what imx8mp.dtsi already does, namely
> >>
> >> compatible = "fsl,imx8mp-gpt", "fsl,imx6dl-gpt";
> >>
> >
> > https://lore.kernel.org/all/[email protected]/
>
> 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