Re: [PATCH] media: iris: MAINTAINERS: Document actual maintainership by Bryan O'Donoghue

[Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>]

On 15/07/2025 07:51, Vikash Garodia wrote:
> 
> On 7/14/2025 8:46 PM, Krzysztof Kozlowski wrote:
>> Bryan O'Donoghue reviews and applies patches for both Iris and Venus
>> Qualcomm SoC video codecs (visible in git log as his Signed-off-by and
>> in pull requests like [1]), so he is de facto the maintainer responsible
>> for the code.  Reflect this actual state my changing his entry from
>> reviewer to maintainer and moving the entry to alphabetical position by
>> first name.
> 
> NAK.
> 
> The roles and responsibilities are well agreed by media maintainer(Hans), with
> Bryan part of that discussion, w.r.t code contributions to iris and sending
> patches to media tree. The only reason Bryan post the patches is that Hans wants
> single PR for patches across Qualcomm media drivers (Camss/Videoss)

That's the maintainer role, so Bryan is the maintainer. I am documenting
actual status and your NAK is naking what? That Bryan cannot handle patches?

Sorry, this is already happening.

Your push back here is odd, impolite and really disappointing. You
actually should be happy that person outside wants to care about this
driver...

> Hi Hans,
> 
> Incase you would like to split sending PRs, as the contributions for Venus/Iris
> would be significantly higher, let us know, we can pick that up separately.

Considering quality of the code you sent as Iris upstreaming, you are
not there yet.

https://lore.kernel.org/all/1690550624-14642-1-git-send-email-quic_vgarodia@quicinc.com/

Please learn, read how the process works, what is the responsibility of
maintainers first.


Best regards,
Krzysztof