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

Ki-Seok Jo kiseok.jo at irondevice.com
Thu Feb 2 02:54:40 CET 2023


> On Wed, Feb 01, 2023 at 03:43:12PM +0100, Krzysztof Kozlowski wrote:
> 
> > I tried. I started writing patch to fix few things in this binding and
> > then noticed that it is entirely empty and documents nothing.
> 
> I really don't see an empty binding as a major problem in and of itself,
> we can always add properties later.  Again, I can't tell what the problems
> you're seeing are.
> 
> > The trouble is that soon you will send it to Linus and then it becomes
> > the ABI even though no one ever approved or reviewed the actual ABI.
> 
> So send a patch to delete the property parsing code then, like I say
> removing the entire driver is very much an overraction.  The properties are
> all optional in the code.

Ok. I'm sorry for not checking correctly.
I only reviewed using the full source build and checkpatch.pl.
But I missed the config setting...

So, could I get the information how to test the binding files?
>From what I've checked now, using make dt_binding_check, right?

I'll try to read again like submitting-patches.rst and writing-bindings.rst.
And then re-write the patch. I'm sorry again for not checking properly.


Thanks Mark and Krzysztof to feedback.

Best Regards,
Kiseok Jo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 499 bytes
Desc: signature.asc
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20230202/c20a2aa3/attachment.sig>


More information about the Alsa-devel mailing list