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

From: Dikshita Agarwal
Date: Thu Sep 05 2024 - 07:14:54 EST




On 9/5/2024 4:32 PM, Dmitry Baryshkov wrote:
> 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.
Noted.
>
>>>> +F: drivers/media/platform/qcom/iris/
>>>>
>>>> Thanks,
>>>> Dikshita
>>>>>
>>>>>
>>>>> Best regards,
>>>>> Krzysztof
>>>>>
>>>>>
>>>
>
>
>