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

[Dmitry Baryshkov]

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