Re: [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Nishanth,

On 8/27/20 10:14 PM, Lokesh Vutla wrote:
> Hi Nishanth,
> 
> On 28/08/20 6:11 am, Nishanth Menon wrote:
>> On 12:21-20200827, Lokesh Vutla wrote:
>>> Convert TI K3 Board/SoC bindings to DT schema format.
>>>
>>> Signed-off-by: Lokesh Vutla <lokeshvutla@xxxxxx>
>>> ---
>>>  .../devicetree/bindings/arm/ti/k3.txt         | 26 ----------------
>>>  .../devicetree/bindings/arm/ti/k3.yaml        | 31 +++++++++++++++++++
>>>  MAINTAINERS                                   |  2 +-
>>>  3 files changed, 32 insertions(+), 27 deletions(-)
>>>  delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt
>>>  create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml
>>
>>
>> Thanks for doing this, but I have a problem with dbs_check and dtbs
>> W=2 build warnings on existing dts files that this exposes further..
> 
> IMHO, that should not block the conversion to yaml bindings. May I know the
> problem you are seeing?

Agree with Lokesh here. This should neither increase or decrease the existing
warnings present. FWIW, both dt_bindings_check and dtbs_check with the
DT_SCHEMA_FILES set to k3.yaml is clean. And the J7200 dts files are clean even
without any DT_SCHEMA_FILES specified.

It is going to take sometime before the existing dtbs_check warnings are cleaned
up on all the K3 dts files, and that has to do with the pre-existing bindings
files in text and no yaml equivalent for them.

regards
Suman

> 
>> Do you mind pulling this patch out of this j7200 series ? I would
>> rather us cleanup the warnings a bit as well, and deal with yaml
>> conversion seperate from j7200 bindings?
> 
> Ill wait for Rob's view on this. He already asked to convert it during J721e
> binding update. Not sure we would like to delay any further.
> 
> Thanks and regards,
> Lokesh
> 
>>
>> We will need Rob's ack anyways, I would rather we look at things
>> independently.
>>




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux