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

Dikshita Agarwal via B4 Relay posted 29 patches 1 year, 3 months ago
There is a newer version of this series
[PATCH v3 02/29] media: MAINTAINERS: Add Qualcomm Iris video accelerator driver
Posted by Dikshita Agarwal via B4 Relay 1 year, 3 months ago
From: Dikshita Agarwal <quic_dikshita@quicinc.com>

Add an entry for Iris video decoder accelerator driver.

Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com>
---
 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@quicinc.com>
+M:	Dikshita Agarwal <quic_dikshita@quicinc.com>
+R:	Abhinav Kumar <quic_abhinavk@quicinc.com>
+L:	linux-media@vger.kernel.org
+L:	linux-arm-msm@vger.kernel.org
+S:	Maintained
+T:	git git://linuxtv.org/media_tree.git
+F:	Documentation/devicetree/bindings/media/qcom,*-iris.yaml
+F:	drivers/media/platform/qcom/iris/
+
 QUALCOMM NAND CONTROLLER DRIVER
 M:	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
 L:	linux-mtd@lists.infradead.org

-- 
2.34.1
Re: [PATCH v3 02/29] media: MAINTAINERS: Add Qualcomm Iris video accelerator driver
Posted by Krzysztof Kozlowski 1 year, 3 months ago
On 27/08/2024 12:05, Dikshita Agarwal via B4 Relay wrote:
> From: Dikshita Agarwal <quic_dikshita@quicinc.com>
> 
> Add an entry for Iris video decoder accelerator driver.
> 
> Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
> Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com>
> ---
>  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@quicinc.com>
> +M:	Dikshita Agarwal <quic_dikshita@quicinc.com>
> +R:	Abhinav Kumar <quic_abhinavk@quicinc.com>
> +L:	linux-media@vger.kernel.org
> +L:	linux-arm-msm@vger.kernel.org
> +S:	Maintained
> +T:	git git://linuxtv.org/media_tree.git

Drop, you do not maintain that Git tree.

> +F:	Documentation/devicetree/bindings/media/qcom,*-iris.yaml
> +F:	drivers/media/platform/qcom/iris/

Drop, does not exist. Or fix your patch order.


Best regards,
Krzysztof
Re: [PATCH v3 02/29] media: MAINTAINERS: Add Qualcomm Iris video accelerator driver
Posted by Dikshita Agarwal 1 year, 3 months ago

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@quicinc.com>
>>
>> Add an entry for Iris video decoder accelerator driver.
>>
>> Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
>> Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com>
>> ---
>>  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@quicinc.com>
>> +M:	Dikshita Agarwal <quic_dikshita@quicinc.com>
>> +R:	Abhinav Kumar <quic_abhinavk@quicinc.com>
>> +L:	linux-media@vger.kernel.org
>> +L:	linux-arm-msm@vger.kernel.org
>> +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?
+F:	drivers/media/platform/qcom/iris/

Thanks,
Dikshita
> 
> 
> Best regards,
> Krzysztof
> 
>
Re: [PATCH v3 02/29] media: MAINTAINERS: Add Qualcomm Iris video accelerator driver
Posted by Dmitry Baryshkov 1 year, 3 months ago
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@quicinc.com>
> >>
> >> Add an entry for Iris video decoder accelerator driver.
> >>
> >> Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
> >> Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com>
> >> ---
> >>  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@quicinc.com>
> >> +M:	Dikshita Agarwal <quic_dikshita@quicinc.com>
> >> +R:	Abhinav Kumar <quic_abhinavk@quicinc.com>
> >> +L:	linux-media@vger.kernel.org
> >> +L:	linux-arm-msm@vger.kernel.org
> >> +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.

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

-- 
With best wishes
Dmitry
Re: [PATCH v3 02/29] media: MAINTAINERS: Add Qualcomm Iris video accelerator driver
Posted by Dikshita Agarwal 1 year, 3 months ago

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@quicinc.com>
>>>>
>>>> Add an entry for Iris video decoder accelerator driver.
>>>>
>>>> Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
>>>> Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com>
>>>> ---
>>>>  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@quicinc.com>
>>>> +M:	Dikshita Agarwal <quic_dikshita@quicinc.com>
>>>> +R:	Abhinav Kumar <quic_abhinavk@quicinc.com>
>>>> +L:	linux-media@vger.kernel.org
>>>> +L:	linux-arm-msm@vger.kernel.org
>>>> +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?
>> +F:	drivers/media/platform/qcom/iris/
>>
>> Thanks,
>> Dikshita
>>>
>>>
>>> Best regards,
>>> Krzysztof
>>>
>>>
>
Re: [PATCH v3 02/29] media: MAINTAINERS: Add Qualcomm Iris video accelerator driver
Posted by Dmitry Baryshkov 1 year, 3 months ago
On Thu, 5 Sept 2024 at 14:02, Dikshita Agarwal
<quic_dikshita@quicinc.com> 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@quicinc.com>
> >>>>
> >>>> Add an entry for Iris video decoder accelerator driver.
> >>>>
> >>>> Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
> >>>> Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com>
> >>>> ---
> >>>>  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@quicinc.com>
> >>>> +M:        Dikshita Agarwal <quic_dikshita@quicinc.com>
> >>>> +R:        Abhinav Kumar <quic_abhinavk@quicinc.com>
> >>>> +L:        linux-media@vger.kernel.org
> >>>> +L:        linux-arm-msm@vger.kernel.org
> >>>> +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
Re: [PATCH v3 02/29] media: MAINTAINERS: Add Qualcomm Iris video accelerator driver
Posted by Dikshita Agarwal 1 year, 3 months ago

On 9/5/2024 4:32 PM, Dmitry Baryshkov wrote:
> On Thu, 5 Sept 2024 at 14:02, Dikshita Agarwal
> <quic_dikshita@quicinc.com> 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@quicinc.com>
>>>>>>
>>>>>> Add an entry for Iris video decoder accelerator driver.
>>>>>>
>>>>>> Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
>>>>>> Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com>
>>>>>> ---
>>>>>>  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@quicinc.com>
>>>>>> +M:        Dikshita Agarwal <quic_dikshita@quicinc.com>
>>>>>> +R:        Abhinav Kumar <quic_abhinavk@quicinc.com>
>>>>>> +L:        linux-media@vger.kernel.org
>>>>>> +L:        linux-arm-msm@vger.kernel.org
>>>>>> +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
>>>>>
>>>>>
>>>
> 
> 
>