Hi Adrian
On Mon, Apr 20, 2020 at 02:19:24PM +0300, Adrian Ratiu wrote:
Hello,
I got confused while doing the txt -> yaml conversion at [1] and it's still not clear to me who should be added in the "maintainers" field. Clearly not the maintainers as returned by get_maintainer.pl. :)
Rob mentioned that "owners" should be manintainers but I also have trouble picking the persons who should be owners / yaml maintainers.
Looking at the completed bridge conversions in the latest linux-next, I couldn't find a rule and the majority of bindings are still txt:
$ find ./devicetree/bindings/display/bridge/ -name *txt | wc -l 23 $ find ./devicetree/bindings/display/bridge/ -name *yaml | wc -l 5
So my questions are:
Is there a general rule for assigning yaml file owners/maintainers?
Is this vagueness specific to the bridge dt bindings only?
Who should step up and maintain these bindings? Original/new authors,
SoC, bridge, DRM maintainers etc.?
It would be useful to have a rule to make it easier to do these conversions. We (Collabora) are considering doing the conversion work.
For the panel conversion I did recently it was simple: 1) If listed in MAINTAINERS - use this info 2) Otherwise use the person(s) that authored the original .txt file. Using git log --follow foo.txt 3) In a few cases I may have decided otherwise, but the above covers the majority.
I would also be great if you or someone else could: - teach get_maintainers about .yaml file listed maintainers - teach checkpatch that it is OK to convert .txt to .yaml - teach checkpatch about some simple yaml validation (maybe)
I am looking forward to the day we have more .yaml files than .txt files in Documentation/devicetree/binding/*
Sam