Messages in this thread | | | Date | Thu, 6 Apr 2023 20:46:28 +0200 | Subject | Re: [PATCH v6 1/2] dt-bindings: omap: Convert omap.txt to yaml | From | Krzysztof Kozlowski <> |
| |
On 06/04/2023 20:35, Andreas Kemnade wrote: > On Thu, 6 Apr 2023 10:36:36 +0200 > Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > >> On 06/04/2023 10:32, Krzysztof Kozlowski wrote: >>> On 05/04/2023 18:19, Andreas Kemnade wrote: >>>> From: Andrew Davis <afd@ti.com> >>>> >>>> Convert omap.txt to yaml. >>>> >>> >>> >>>> + - description: TI AM43 SoC based platforms >>>> + items: >>>> + - enum: >>>> + - compulab,am437x-cm-t43 >>>> + - ti,am437x-gp-evm >>>> + - ti,am437x-idk-evm >>>> + - ti,am437x-sk-evm >>>> + - pattern: '^ti,am4372[26789]$' >>>> + - const: ti,am43 >>>> + >>>> + - description: TI AM57 SoC based platforms >>>> + items: >>>> + - enum: >>>> + - beagle,am5729-beagleboneai >>>> + - compulab,cl-som-am57x >>>> + - ti,am5718-idk >>>> + - ti,am5728-idk >>>> + - ti,am5748-idk >>>> + - pattern: '^ti,am57[0124][689]$' >>> >>> I don't think my comments were resolved. I asked if it is possible to >>> make a board called "ti,am5718-idk" with "ti,am5749" or with "ti,am5708"? >> >> Hm, I cannot find my concern, so maybe it never left my outbox. Anyway, >> it looks like you allow here many incorrect patterns and combinations. >> > You had concerns about the "dra" stuff for the same valid reasons.
Yes, but also about this. So again: is "ti,am5718-idk" with "ti,am5749" correct?
> >>> >>> What's more, you dropped several variations and compatibles against >>> original binding (all the "dra") and it is not explained in commit msg >>> at all. >> >> All changes against original bindings should be explained. >> > ... which I then just decided not to convert but keep as is in the text file > because there were no comments hints about it more than a month since the first > version was sent so thought that it has to be resolved later.
But you partially remove them - see AM5728 IDK.
Best regards, Krzysztof
| |