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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> > > 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

So, was the entire thing dropped, or was only the binding dropped?
If they are not also, can I just patch in the current state?

And I tested the rewritten file with the following command.

  make dt_binding_check DT_SCHEMA_FILES=irondevice,sma1303.yaml

Is there anything else I should do?

Thanks to your help, I think I am becoming a better developer.
I'll try harder. Thanks so much!

Best regards,
Kiseok Jo

Attachment: signature.asc
Description: signature.asc


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux