| Message ID | 20241023185841.1183706-1-festevam@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [1/2] dt-bindings: timer: fsl,imxgpt: Fix the fsl,imx7d-gpt fallback | expand |
Hi Fabio, thanks for the patch. Am Mittwoch, 23. Oktober 2024, 20:58:40 CEST schrieb Fabio Estevam: > From: Fabio Estevam <festevam@denx.de> > > imx7s.dtsi correctly describes the GPT node as: > > compatible = "fsl,imx7d-gpt", "fsl,imx6dl-gpt"; > > Document the fallback compatible to be "fsl,imx6dl-gpt" in the bindings. > > This fixes the following dt-schema warnings: > > timer@302f0000: compatible: 'oneOf' conditional failed, one must be fixed: > ['fsl,imx7d-gpt', 'fsl,imx6dl-gpt'] is too long > 'fsl,imx1-gpt' was expected > 'fsl,imx21-gpt' was expected > 'fsl,imx27-gpt' was expected > 'fsl,imx31-gpt' was expected > 'fsl,imx7d-gpt' is not one of ['fsl,imx25-gpt', 'fsl,imx50-gpt', 'fsl,imx51-gpt', 'fsl,imx53-gpt', 'fsl,imx6q-gpt'] > 'fsl,imx6dl-gpt' was expected > 'fsl,imx7d-gpt' is not one of ['fsl,imx6sl-gpt', 'fsl,imx6sx-gpt', 'fsl,imx8mp-gpt', 'fsl,imxrt1050-gpt', 'fsl,imxrt1170-gpt'] > 'fsl,imx6sx-gpt' was expected > > Signed-off-by: Fabio Estevam <festevam@denx.de> Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com> > --- > Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml b/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml > index e2607377cbae..a2fcb1e8e74e 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,imx7d-gpt > - fsl,imx8mp-gpt > - fsl,imxrt1050-gpt > - fsl,imxrt1170-gpt > @@ -38,7 +39,6 @@ properties: > - items: > - enum: > - fsl,imx6ul-gpt > - - fsl,imx7d-gpt > - const: fsl,imx6sx-gpt > > reg: >
On Wed, Oct 23, 2024 at 03:58:40PM -0300, Fabio Estevam wrote: > From: Fabio Estevam <festevam@denx.de> > > imx7s.dtsi correctly describes the GPT node as: > > compatible = "fsl,imx7d-gpt", "fsl,imx6dl-gpt"; > > Document the fallback compatible to be "fsl,imx6dl-gpt" in the bindings. > > This fixes the following dt-schema warnings: > > timer@302f0000: compatible: 'oneOf' conditional failed, one must be fixed: > ['fsl,imx7d-gpt', 'fsl,imx6dl-gpt'] is too long > 'fsl,imx1-gpt' was expected > 'fsl,imx21-gpt' was expected > 'fsl,imx27-gpt' was expected > 'fsl,imx31-gpt' was expected > 'fsl,imx7d-gpt' is not one of ['fsl,imx25-gpt', 'fsl,imx50-gpt', 'fsl,imx51-gpt', 'fsl,imx53-gpt', 'fsl,imx6q-gpt'] > 'fsl,imx6dl-gpt' was expected > 'fsl,imx7d-gpt' is not one of ['fsl,imx6sl-gpt', 'fsl,imx6sx-gpt', 'fsl,imx8mp-gpt', 'fsl,imxrt1050-gpt', 'fsl,imxrt1170-gpt'] > 'fsl,imx6sx-gpt' was expected > > Signed-off-by: Fabio Estevam <festevam@denx.de> Acked-by: Conor Dooley <conor.dooley@microchip.com>
Hi Rob, On Thu, Oct 24, 2024 at 2:05 PM Conor Dooley <conor@kernel.org> wrote: > > On Wed, Oct 23, 2024 at 03:58:40PM -0300, Fabio Estevam wrote: > > From: Fabio Estevam <festevam@denx.de> > > > > imx7s.dtsi correctly describes the GPT node as: > > > > compatible = "fsl,imx7d-gpt", "fsl,imx6dl-gpt"; > > > > Document the fallback compatible to be "fsl,imx6dl-gpt" in the bindings. > > > > This fixes the following dt-schema warnings: > > > > timer@302f0000: compatible: 'oneOf' conditional failed, one must be fixed: > > ['fsl,imx7d-gpt', 'fsl,imx6dl-gpt'] is too long > > 'fsl,imx1-gpt' was expected > > 'fsl,imx21-gpt' was expected > > 'fsl,imx27-gpt' was expected > > 'fsl,imx31-gpt' was expected > > 'fsl,imx7d-gpt' is not one of ['fsl,imx25-gpt', 'fsl,imx50-gpt', 'fsl,imx51-gpt', 'fsl,imx53-gpt', 'fsl,imx6q-gpt'] > > 'fsl,imx6dl-gpt' was expected > > 'fsl,imx7d-gpt' is not one of ['fsl,imx6sl-gpt', 'fsl,imx6sx-gpt', 'fsl,imx8mp-gpt', 'fsl,imxrt1050-gpt', 'fsl,imxrt1170-gpt'] > > 'fsl,imx6sx-gpt' was expected > > > > Signed-off-by: Fabio Estevam <festevam@denx.de> > > Acked-by: Conor Dooley <conor.dooley@microchip.com> Can you apply this series via your tree?
Rob, On Tue, Nov 5, 2024 at 7:48 AM Fabio Estevam <festevam@gmail.com> wrote: > > Acked-by: Conor Dooley <conor.dooley@microchip.com> > > Can you apply this series via your tree? A gentle ping.
On Wed, Nov 13, 2024 at 11:57 AM Fabio Estevam <festevam@gmail.com> wrote: > > Rob, > > On Tue, Nov 5, 2024 at 7:48 AM Fabio Estevam <festevam@gmail.com> wrote: > > > > Acked-by: Conor Dooley <conor.dooley@microchip.com> > > > > Can you apply this series via your tree? > > A gentle ping. Daniel should apply this. That was my assumption and it was out of my queue (which drops the chances of me ever reading this ping). Rob
diff --git a/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml b/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml index e2607377cbae..a2fcb1e8e74e 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,imx7d-gpt - fsl,imx8mp-gpt - fsl,imxrt1050-gpt - fsl,imxrt1170-gpt @@ -38,7 +39,6 @@ properties: - items: - enum: - fsl,imx6ul-gpt - - fsl,imx7d-gpt - const: fsl,imx6sx-gpt reg: