Re: Rule for bridge yaml dt binding maintainers?
From: Sam Ravnborg
Date: Mon Apr 20 2020 - 13:59:19 EST
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:
> 1. Is there a general rule for assigning yaml file owners/maintainers?
>
> 2. Is this vagueness specific to the bridge dt bindings only?
>
> 3. 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