Messages in this thread | | | Date | Thu, 1 Jun 2023 19:20:54 +0100 | From | Conor Dooley <> | Subject | Re: [PATCH v1 2/7] dt-bindings: timer: gpt: Add i.MX6UL support |
| |
On Thu, Jun 01, 2023 at 12:14:46PM +0200, Oleksij Rempel wrote: > Add 'fsl,imx6ul-gpt' compatible to resolve the following dtbs_check > warning: > /arch/arm/boot/dts/imx6ull-jozacp.dtb: timer@20e8000: compatible: > 'oneOf' conditional failed, one must be fixed: > 'fsl,imx6ul-gpt' is not one of ['fsl,imx25-gpt', 'fsl,imx50-gpt', > 'fsl,imx51-gpt', 'fsl,imx53-gpt', 'fsl,imx6q-gpt'] > > Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> > --- > Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml b/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml > index 716c6afcca1f..adf617b8f353 100644 > --- a/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml > +++ b/Documentation/devicetree/bindings/timer/fsl,imxgpt.yaml > @@ -34,6 +34,10 @@ properties: > - fsl,imxrt1050-gpt > - fsl,imxrt1170-gpt > - const: fsl,imx6dl-gpt > + - const: fsl,imx6ul-gpt
Again, why add this as an appears-on its own compatible when it seems to be used only with a fallback to imx6sx? (In-tree at least)
> + - items: > + - const: fsl,imx6ul-gpt > + - const: fsl,imx6sx-gpt > > reg: > maxItems: 1 > -- > 2.39.2 > [unhandled content-type:application/pgp-signature] | |