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