linux-kernel - Re: [PATCH v1 1/3] dt-bindings: mmc: Drop unused properties

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 for Android: free password hash cracker in your pocket

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

Message-Id: <9EF26965-10E5-4BCA-AC5E-93C5AA55A0DF@jrtc27.com>
Date:   Fri, 1 Sep 2023 18:20:38 +0100
From:   Jessica Clarke <jrtc27@...c27.com>
To:     Conor Dooley <conor@...nel.org>
Cc:     William Qiu <william.qiu@...rfivetech.com>,
        Conor Dooley <conor.dooley@...rochip.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>, LKML <linux-kernel@...r.kernel.org>,
        linux-riscv <linux-riscv@...ts.infradead.org>,
        linux-mmc@...r.kernel.org, Emil Renner Berthing <kernel@...il.dk>,
        Rob Herring <robh+dt@...nel.org>,
        Jaehoon Chung <jh80.chung@...sung.com>,
        Ulf Hansson <ulf.hansson@...aro.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Paul Walmsley <paul.walmsley@...ive.com>,
        Palmer Dabbelt <palmer@...belt.com>,
        Albert Ou <aou@...s.berkeley.edu>
Subject: Re: [PATCH v1 1/3] dt-bindings: mmc: Drop unused properties

On 1 Sep 2023, at 16:42, Conor Dooley <conor@...nel.org> wrote:
> 
> On Fri, Sep 01, 2023 at 10:33:13AM +0800, William Qiu wrote:
>> 
>> 
>> On 2023/8/30 16:34, Conor Dooley wrote:
>>> On Wed, Aug 30, 2023 at 09:29:20AM +0200, Krzysztof Kozlowski wrote:
>>>> On 30/08/2023 08:50, Conor Dooley wrote:
>>>>> On Wed, Aug 30, 2023 at 11:18:44AM +0800, William Qiu wrote:
>>>>>> Due to the change of tuning implementation, it's no longer necessary to
>>>>>> use the "starfive,sysreg" property in dts, so drop the relevant
>>>>>> description in dt-bindings here.
>>>>> 
>>>>> How does changing your software implantation invalidate a description of
>>>>> the hardware?
>>>>> 
>>>> 
>>>> Which is kind of proof that this syscon was just to substitute
>>>> incomplete hardware description (e.g. missing clocks and phys). We
>>>> should have rejected it. Just like we should reject them in the future.
>>> 
>>> :s I dunno what to do with this... I'm inclined to say not to remove it
>>> from the binding or dts at all & only change the software.
>>> 
>>>> There are just few cases where syscon is reasonable. All others is just
>>>> laziness. It's not only starfivetech, of course. Several other
>>>> contributors do the same.
>>> 
>>> I'm not sure if laziness is fair, lack of understanding is usually more
>>> likely.
>> 
>> For this, I tend to keep it in binding, but remove it from required. Because
>> we only modify the tuning implementation, it doesn't mean that this property
>> need to be removed, it's just no longer be the required one.
> 
> Please only remove it from required if the current driver doesn't break
> if the regmap is removed.

Either way please make sure the documentation clearly states “never use
this, if you’re using it you’re doing it wrong, this only exists
because it was wrongly used in the past”. Otherwise people writing
drivers for other OSes will probably use it too thinking they need to.

Jess