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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <8772c48f-348b-8a68-2099-562a29b9dd8d@quicinc.com>
Date: Tue, 15 Jul 2025 11:21:17 +0530
From: Vikash Garodia <quic_vgarodia@...cinc.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        <bryan.odonoghue@...aro.org>,
        Dikshita Agarwal <quic_dikshita@...cinc.com>,
        Abhinav Kumar <abhinav.kumar@...ux.dev>,
        Mauro Carvalho Chehab
	<mchehab@...nel.org>,
        <linux-media@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
CC: Hans Verkuil <hverkuil@...all.nl>
Subject: Re: [PATCH] media: iris: MAINTAINERS: Document actual maintainership
 by Bryan O'Donoghue


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)

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.

Regards,
Vikash
> 
> [1] https://lore.kernel.org/linux-media/20250630121704.260831-1-bod@kernel.org/T/#u
> 
> Cc: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
> Cc: Vikash Garodia <quic_vgarodia@...cinc.com>
> Cc: Dikshita Agarwal <quic_dikshita@...cinc.com>
> Cc: Abhinav Kumar <abhinav.kumar@...ux.dev>
> Cc: Hans Verkuil <hverkuil@...all.nl>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> ---
>  MAINTAINERS | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 40831ae42296..f56e40d9b3c7 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -20678,10 +20678,10 @@ F:	Documentation/devicetree/bindings/regulator/vqmmc-ipq4019-regulator.yaml
>  F:	drivers/regulator/vqmmc-ipq4019-regulator.c
>  
>  QUALCOMM IRIS VIDEO ACCELERATOR DRIVER
> +M:	Bryan O'Donoghue <bryan.odonoghue@...aro.org>
>  M:	Vikash Garodia <quic_vgarodia@...cinc.com>
>  M:	Dikshita Agarwal <quic_dikshita@...cinc.com>
>  R:	Abhinav Kumar <abhinav.kumar@...ux.dev>
> -R:	Bryan O'Donoghue <bryan.odonoghue@...aro.org>
>  L:	linux-media@...r.kernel.org
>  L:	linux-arm-msm@...r.kernel.org
>  S:	Maintained
> @@ -20750,9 +20750,9 @@ F:	Documentation/devicetree/bindings/usb/qcom,pmic-*.yaml
>  F:	drivers/usb/typec/tcpm/qcom/
>  
>  QUALCOMM VENUS VIDEO ACCELERATOR DRIVER
> +M:	Bryan O'Donoghue <bryan.odonoghue@...aro.org>
>  M:	Vikash Garodia <quic_vgarodia@...cinc.com>
>  M:	Dikshita Agarwal <quic_dikshita@...cinc.com>
> -R:	Bryan O'Donoghue <bryan.odonoghue@...aro.org>
>  L:	linux-media@...r.kernel.org
>  L:	linux-arm-msm@...r.kernel.org
>  S:	Maintained