linux-kernel - Re: [PATCH] spi: dt-bindings: Fix unevaluatedProperties warnings in examples

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]

Date:   Thu, 26 May 2022 14:08:06 +0000
From:   <Conor.Dooley@...rochip.com>
To:     <robh@...nel.org>
CC:     <broonie@...nel.org>, <krzysztof.kozlowski+dt@...aro.org>,
        <agross@...nel.org>, <bjorn.andersson@...aro.org>,
        <krzysztof.kozlowski@...aro.org>, <linux-spi@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <linux-arm-msm@...r.kernel.org>
Subject: Re: [PATCH] spi: dt-bindings: Fix unevaluatedProperties warnings in
 examples

On 26/05/2022 15:01, Rob Herring wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On Thu, May 26, 2022 at 06:52:23AM +0000, Conor.Dooley@...rochip.com wrote:
>> On 26/05/2022 02:41, Rob Herring wrote:
>>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>>
>>> The 'unevaluatedProperties' schema checks is not fully working and doesn't
>>> catch some cases where there's a $ref to another schema. A fix is pending,
>>> but results in new warnings in examples.
>>>
>>> 'spi-max-frequency' is supposed to be a per SPI peripheral device property,
>>> not a SPI controller property, so drop it.
>>
>> Reviewed-by: Conor Dooley <conor.dooley@...rochip.com>
>>
>> As this property is in the mpfs devicetree & I'll need to fix it, is this
>> patch targeted at 5.19? (I assume given the window is open and there's no
>> fixes tag that it isn't).
> 
> 5.18 ideally because warnings are going to appear when I push the

18? That a typo or am I missing something? Either way, will send the dt
fix shortly.
Thanks,
Conor.

> dtschema fix for 'unevaluatedProperties'.
> 
> Rob