linux-kernel - Re: [PATCH] ASoC: dt-bindings: Drop broken irondevice,sma1303 binding

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: <Y9pxGUMWyMeXQpZM@sirena.org.uk>
Date:   Wed, 1 Feb 2023 14:03:05 +0000
From:   Mark Brown <broonie@...nel.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc:     Liam Girdwood <lgirdwood@...il.com>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Jaroslav Kysela <perex@...ex.cz>,
        Takashi Iwai <tiwai@...e.com>, alsa-devel@...a-project.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        Kiseok Jo <kiseok.jo@...ndevice.com>
Subject: Re: [PATCH] ASoC: dt-bindings: Drop broken irondevice,sma1303 binding

On Wed, Feb 01, 2023 at 02:13:46PM +0100, Krzysztof Kozlowski wrote:
> On 01/02/2023 14:10, Krzysztof Kozlowski wrote:

> > Because the binding:
> > 1. Was never tested,
> > 2. Was never sent to Devicetree maintainers,
> > 3. Is entirely broken and wrong, so it would have to be almost rewritten
> >    from scratch,
> > 4. It does not match the driver, IOW, the binding is fake.

> I understand that in general we tend to fix, not just to revert. But the
> poor quality of this binding and the next patch, which was suppose to
> fix it, plus complete lack of testing, means I do not believe the author
> will send correct binding.

> More over, fixing binding might require dropping incorrect properties,
> thus changing the driver. I am not willing to do that, I doubt that
> anyone has the time for it.

It is an absolutely trivial binding as is, it is utterly
disproportionate to delete both the binding and the driver to fix
whatever it is that the issues you're seeing are (I can't really tell
TBH).  Undocumented properties are a separate thing but again a revert
is obviously disproportionate here, glancing at the driver the code is
all well enough separated and can have default values.  Looking again I
did miss the sysclk selection which should be dropped, clocks should use
the clock bindings.

> It's the job of submitter to work on it.

It's also not the end of the world if we have a driver that isn't
perfect.

Please, try to keep things constructive.

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)