Messages in this thread | | | Date | Wed, 30 Aug 2023 09:34:06 +0100 | From | Conor Dooley <> | Subject | Re: [PATCH v1 1/3] dt-bindings: mmc: Drop unused properties |
| |
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. [unhandled content-type:application/pgp-signature] | |