Re: [PATCH v3 02/29] media: MAINTAINERS: Add Qualcomm Iris video accelerator driver

From: Dmitry Baryshkov
Date: Thu Sep 05 2024 - 07:03:35 EST


On Thu, 5 Sept 2024 at 14:02, Dikshita Agarwal
<quic_dikshita@xxxxxxxxxxx> wrote:
>
>
>
> On 9/5/2024 3:40 PM, Dmitry Baryshkov wrote:
> > On Thu, Sep 05, 2024 at 11:17:55AM GMT, Dikshita Agarwal wrote:
> >>
> >>
> >> On 8/27/2024 4:12 PM, Krzysztof Kozlowski wrote:
> >>> On 27/08/2024 12:05, Dikshita Agarwal via B4 Relay wrote:
> >>>> From: Dikshita Agarwal <quic_dikshita@xxxxxxxxxxx>
> >>>>
> >>>> Add an entry for Iris video decoder accelerator driver.
> >>>>
> >>>> Signed-off-by: Vikash Garodia <quic_vgarodia@xxxxxxxxxxx>
> >>>> Signed-off-by: Dikshita Agarwal <quic_dikshita@xxxxxxxxxxx>
> >>>> ---
> >>>> MAINTAINERS | 11 +++++++++++
> >>>> 1 file changed, 11 insertions(+)
> >>>>
> >>>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>>> index 8766f3e5e87e..105e67fca308 100644
> >>>> --- a/MAINTAINERS
> >>>> +++ b/MAINTAINERS
> >>>> @@ -18898,6 +18898,17 @@ S: Maintained
> >>>> F: Documentation/devicetree/bindings/regulator/vqmmc-ipq4019-regulator.yaml
> >>>> F: drivers/regulator/vqmmc-ipq4019-regulator.c
> >>>>
> >>>> +QUALCOMM IRIS VIDEO ACCELERATOR DRIVER
> >>>> +M: Vikash Garodia <quic_vgarodia@xxxxxxxxxxx>
> >>>> +M: Dikshita Agarwal <quic_dikshita@xxxxxxxxxxx>
> >>>> +R: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
> >>>> +L: linux-media@xxxxxxxxxxxxxxx
> >>>> +L: linux-arm-msm@xxxxxxxxxxxxxxx
> >>>> +S: Maintained
> >>>> +T: git git://linuxtv.org/media_tree.git
> >>>
> >>> Drop, you do not maintain that Git tree.
> >> Sure, will remove
> >>>
> >>>> +F: Documentation/devicetree/bindings/media/qcom,*-iris.yaml
> >>>> +F: drivers/media/platform/qcom/iris/
> >>>
> >>> Drop, does not exist. Or fix your patch order.
> >> Are you suggesting to add this patch as the last patch of the series?
> >> or remove just below entry and add one more patch at the end to update
> >> MAINTAINERS file with the same?
> >
> > Adding it after the patch that adds the driver should be fine. Likewise
> > adding it at the end is also fine.
> >
> sure, so should I add the whole patch once driver is introduced or have a
> separate patch for just below?

I'd say a single patch is better.

> >> +F: drivers/media/platform/qcom/iris/
> >>
> >> Thanks,
> >> Dikshita
> >>>
> >>>
> >>> Best regards,
> >>> Krzysztof
> >>>
> >>>
> >



--
With best wishes
Dmitry