MAINTAINERS: Add video maintainer #90083
Conversation
| @@ -2176,11 +2176,12 @@ Release Notes: | |||
| - "area: Timer" | |||
|
|
|||
| "Drivers: Video": | |||
| status: odd fixes | |||
| status: maintained | |||
| maintainers: | |||
There was a problem hiding this comment.
Fine, video subsystem is clearly active these days. I think It would make sense to add everyone below under maintainers, with them as reviewer of this PR.
There was a problem hiding this comment.
Hi @loicpoulain, I don't know exactly what the process is for reviewing but it's fine to me to follow.
It's also fine to me to add other people who want to step up as video maintainers in this same PR if needed (but I am not sure if it's the right way or it should be done independently).
There was a problem hiding this comment.
I am volonter as well as maintainers if that is ok. If ok with that way you can also add me to the list.
There was a problem hiding this comment.
I am willing to join the maintenance as well:
- if it even helps to have 4 maintainers rather than 2,
- if occasional misses to the "2 day before reviews" are acceptable, as I missed that rule for instance today as I just came back from a conference. I am around a PR 24h max usually.
There was a problem hiding this comment.
I will open separate PRs for @avolmat-st, @loicpoulain and @josuah. :)
There was a problem hiding this comment.
The rule is simply that a PR cannot be merged until it has been approved by the right people AND has been in review for a minimum of 2 business days.
Good to know thank you!
There was a problem hiding this comment.
It makes things easier that I stay within the collaborators section as currently.
There was a problem hiding this comment.
I think we should have (at least) two maintainters. I propose to add josuah to the list (I didn't know that we can add multiple people in the same PR) and wish to retain my candidature and motivation too.
There was a problem hiding this comment.
@loicpoulain did you also want being listed in maintainers? There are typically 1~2 maintainers per area, but ~10 areas have 3.
There was a problem hiding this comment.
@josuah yes, I would like to be involved as maintainer for that video subsystem/framework.
There was a problem hiding this comment.
The video subsystem has be very active for a good period of time. All of these collaborators have been working well together to move things along. They have also been growing this sub-community which is very helpful.
Also, and most importantly for me, having a maintainer means the dang bot will stop assigning these PRs to me ;-)...
But seriously, keep working together as you have been. You can have more than one maintainer if it helps ensure proper coverage of assignees for PRs.
Add Josuah Demangeon "josuah" and Phi Bang Nguyen "ngphibang" as maintainers for video drivers. Signed-off-by: Phi Bang Nguyen <[email protected]>
ngphibang
dismissed stale reviews from dleach02, josuah, and avolmat-st
via
5466a2a
ngphibang
force-pushed
the
video_maintainers
branch
from
8350a80 to
5466a2a
Compare
|
|
ping @nashif @MaureenHelm -- this is good to go I think |
The video section has evolved but lacks maintainers, which affects the process of merging video-related PRs. As initially discussed by @josuah and @avolmat-st, add myself "ngphibang" as maintainer for video drivers.
It would be better to have more maintainers to consolidate the review process.
@loicpoulain @josuah @avolmat-st