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

[Ki-Seok Jo <kiseok.jo@irondevice.com>]

[-- Attachment #1: Type: text/plain, Size: 1238 bytes --]

> 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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 499 bytes --]