linux-kernel - Re: [PATCH v4 2/9] dt-bindings: serial: atmel,at91-usart: convert to json-schema

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <d05e20a6-6c4b-4e72-1abf-8ea5c7dfdaf6@microchip.com>
Date:   Tue, 20 Sep 2022 07:51:13 +0000
From:   <Sergiu.Moga@...rochip.com>
To:     <krzysztof.kozlowski@...aro.org>
CC:     <linux-arm-kernel@...ts.infradead.org>,
        <linux-serial@...r.kernel.org>, <alexandre.belloni@...tlin.com>,
        <Nicolas.Ferre@...rochip.com>, <richard.genoud@...il.com>,
        <linux-spi@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <jirislaby@...nel.org>, <devicetree@...r.kernel.org>,
        <Claudiu.Beznea@...rochip.com>, <lee@...nel.org>,
        <Kavyasree.Kotagiri@...rochip.com>, <robh+dt@...nel.org>,
        <gregkh@...uxfoundation.org>, <radu_nicolae.pirea@....ro>,
        <krzysztof.kozlowski+dt@...aro.org>
Subject: Re: [PATCH v4 2/9] dt-bindings: serial: atmel,at91-usart: convert to
 json-schema

On 19.09.2022 20:08, Krzysztof Kozlowski wrote:
> On Mon, 19 Sep 2022 18:08:40 +0300, Sergiu Moga wrote:
>> Convert at91 USART DT Binding for Atmel/Microchip SoCs to
>> json-schema format. Furthermore, move this binding to the
>> serial directory, since binding directories match hardware,
>> unlike the driver subsystems which match Linux convention.
>>
>> Signed-off-by: Sergiu Moga <sergiu.moga@...rochip.com>
>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>> ---
>>
>>
>> v1 -> v2:
>> - only do what the commit says, split the addition of other compatibles and
>> properties in other patches
>> - remove unnecessary "|"'s
>> - mention header in `atmel,usart-mode`'s description
>> - place `if:` under `allOf:`
>> - respect order of spi0's DT properties: compatible, then reg then the reset of properties
>>
>> v2 -> v3:
>> - Previously [PATCH 5]
>> - Check value of `atmel,usart-mode` instead of the node regex
>> - Define all properties top level and disallow them explicitly for other type,
>> since additionalProperties:false conflicts with referencing other schemas
>> - Remove useless else if: after else:
>>
>>
>> v3 -> v4:
>> - add R-b tag, this was previously [PATCH 6]
>>
>>
>>
>>
>>   .../devicetree/bindings/mfd/atmel-usart.txt   |  98 ----------
>>   .../bindings/serial/atmel,at91-usart.yaml     | 182 ++++++++++++++++++
>>   2 files changed, 182 insertions(+), 98 deletions(-)
>>   delete mode 100644 Documentation/devicetree/bindings/mfd/atmel-usart.txt
>>   create mode 100644 Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml
>>
> 
> Running 'make dtbs_check' with the schema in this patch gives the
> following warnings. Consider if they are expected or the schema is
> incorrect. These may not be new warnings.
> 
> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> This will change in the future.
> 
> Full log is available here: https://patchwork.ozlabs.org/patch/
> 
> 

Hello, thank you for checking :). The above link issues a 404. 
Everything should be fine though once all of the below are merged:
https://lore.kernel.org/linux-arm-kernel/53e72e5d-47fc-403d-c969-61b267a9ff15@microchip.com/
https://lore.kernel.org/linux-arm-kernel/1ae89854-74fa-6194-304f-db31d56d3674@microchip.com/

together with the patch series that I linked into the cover letter as a 
dependence:

https://lore.kernel.org/linux-arm-kernel/20220916075744.1879428-1-kavyasree.kotagiri@microchip.com/


Thank you,
	Sergiu