| Message ID | 20230306172249.74003-7-stefan.wahren@i2se.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | ARM: dts: imx6ull: Add chargebyte Tarragon support | expand |
On 06/03/2023 18:22, Stefan Wahren wrote: > From: Stefan Wahren <stefan.wahren@chargebyte.com> > > This adds the compatibles for the chargebyte Tarragon boards. > > Signed-off-by: Stefan Wahren <stefan.wahren@chargebyte.com> > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > --- > Documentation/devicetree/bindings/arm/fsl.yaml | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml > index dece3e9ba7fd..2b430e20a7a6 100644 > --- a/Documentation/devicetree/bindings/arm/fsl.yaml > +++ b/Documentation/devicetree/bindings/arm/fsl.yaml > @@ -781,6 +781,15 @@ properties: > - const: tq,imx6ull-tqma6ull2l # MCIMX6Y2, LGA SoM variant > - const: fsl,imx6ull > > + - description: chargebyte Tarragon Boards > + items: > + - enum: > + - chargebyte,imx6ull-tarragon-master > + - chargebyte,imx6ull-tarragon-micro > + - chargebyte,imx6ull-tarragon-slave > + - chargebyte,imx6ull-tarragon-slavext > + - const: fsl,imx6ull I don't think you put it in correct place. Didn't you just stuff it at the end? Best regards, Krzysztof
Hi Krzysztof, Am 07.03.23 um 09:24 schrieb Krzysztof Kozlowski: > On 06/03/2023 18:22, Stefan Wahren wrote: >> From: Stefan Wahren <stefan.wahren@chargebyte.com> >> >> This adds the compatibles for the chargebyte Tarragon boards. >> >> Signed-off-by: Stefan Wahren <stefan.wahren@chargebyte.com> >> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> >> --- >> Documentation/devicetree/bindings/arm/fsl.yaml | 9 +++++++++ >> 1 file changed, 9 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml >> index dece3e9ba7fd..2b430e20a7a6 100644 >> --- a/Documentation/devicetree/bindings/arm/fsl.yaml >> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml >> @@ -781,6 +781,15 @@ properties: >> - const: tq,imx6ull-tqma6ull2l # MCIMX6Y2, LGA SoM variant >> - const: fsl,imx6ull >> >> + - description: chargebyte Tarragon Boards >> + items: >> + - enum: >> + - chargebyte,imx6ull-tarragon-master >> + - chargebyte,imx6ull-tarragon-micro >> + - chargebyte,imx6ull-tarragon-slave >> + - chargebyte,imx6ull-tarragon-slavext >> + - const: fsl,imx6ull > I don't think you put it in correct place. Didn't you just stuff it at > the end? Yes, i just put it at the end. But to be honest, i'm not sure about the order: armadeus,imx6ull-opos6uldev .. dh,imx6ull-dhcom-drc02 .. phytec,imx6ull-pbacd10-emmc .. toradex,colibri-imx6ull-aster .. kontron,bl-imx6ull .. tq,imx6ull-tqma6ull2-mba6ulx Does the order depend on the compatibles and vendor is relevant? So your expectation was between armadeus and dh? > > Best regards, > Krzysztof >
On 10/03/2023 11:08, Stefan Wahren wrote: > Hi Krzysztof, > > Am 07.03.23 um 09:24 schrieb Krzysztof Kozlowski: >> On 06/03/2023 18:22, Stefan Wahren wrote: >>> From: Stefan Wahren <stefan.wahren@chargebyte.com> >>> >>> This adds the compatibles for the chargebyte Tarragon boards. >>> >>> Signed-off-by: Stefan Wahren <stefan.wahren@chargebyte.com> >>> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> >>> --- >>> Documentation/devicetree/bindings/arm/fsl.yaml | 9 +++++++++ >>> 1 file changed, 9 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml >>> index dece3e9ba7fd..2b430e20a7a6 100644 >>> --- a/Documentation/devicetree/bindings/arm/fsl.yaml >>> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml >>> @@ -781,6 +781,15 @@ properties: >>> - const: tq,imx6ull-tqma6ull2l # MCIMX6Y2, LGA SoM variant >>> - const: fsl,imx6ull >>> >>> + - description: chargebyte Tarragon Boards >>> + items: >>> + - enum: >>> + - chargebyte,imx6ull-tarragon-master >>> + - chargebyte,imx6ull-tarragon-micro >>> + - chargebyte,imx6ull-tarragon-slave >>> + - chargebyte,imx6ull-tarragon-slavext >>> + - const: fsl,imx6ull >> I don't think you put it in correct place. Didn't you just stuff it at >> the end? > > Yes, i just put it at the end. But to be honest, i'm not sure about the > order: > > armadeus,imx6ull-opos6uldev > .. > dh,imx6ull-dhcom-drc02 > .. > phytec,imx6ull-pbacd10-emmc > .. > toradex,colibri-imx6ull-aster > .. > kontron,bl-imx6ull > .. > tq,imx6ull-tqma6ull2-mba6ulx > > Does the order depend on the compatibles and vendor is relevant? > > So your expectation was between armadeus and dh? This I don't know :). I think order was by description (name of board) but maybe by compatibles. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml index dece3e9ba7fd..2b430e20a7a6 100644 --- a/Documentation/devicetree/bindings/arm/fsl.yaml +++ b/Documentation/devicetree/bindings/arm/fsl.yaml @@ -781,6 +781,15 @@ properties: - const: tq,imx6ull-tqma6ull2l # MCIMX6Y2, LGA SoM variant - const: fsl,imx6ull + - description: chargebyte Tarragon Boards + items: + - enum: + - chargebyte,imx6ull-tarragon-master + - chargebyte,imx6ull-tarragon-micro + - chargebyte,imx6ull-tarragon-slave + - chargebyte,imx6ull-tarragon-slavext + - const: fsl,imx6ull + - description: i.MX6ULZ based Boards items: - enum: