diff mbox series

[v3,8/8] MAINTAINERS: Add entry for QAIC driver

Message ID 1678138443-2760-9-git-send-email-quic_jhugo@quicinc.com (mailing list archive)
State New, archived
Headers show
Series QAIC accel driver | expand

Commit Message

Jeffrey Hugo March 6, 2023, 9:34 p.m. UTC 
Add MAINTAINERS entry for the Qualcomm Cloud AI 100 driver.

Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
Reviewed-by: Carl Vanderlip <quic_carlv@quicinc.com>
Reviewed-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
---
 MAINTAINERS | 9 +++++++++
 1 file changed, 9 insertions(+)

Comments

Jacek Lawrynowicz March 14, 2023, 9:59 a.m. UTC | #1 
Hi

On 06.03.2023 22:34, Jeffrey Hugo wrote:
> Add MAINTAINERS entry for the Qualcomm Cloud AI 100 driver.
> 
> Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
> Reviewed-by: Carl Vanderlip <quic_carlv@quicinc.com>
> Reviewed-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
> ---
>  MAINTAINERS | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b0db911..feb2974 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17253,6 +17253,15 @@ F:	Documentation/devicetree/bindings/clock/qcom,*
>  F:	drivers/clk/qcom/
>  F:	include/dt-bindings/clock/qcom,*
>  
> +QUALCOMM CLOUD AI (QAIC) DRIVER
> +M:	Jeffrey Hugo <quic_jhugo@quicinc.com>
> +L:	linux-arm-msm@vger.kernel.org
> +L:	dri-devel@lists.freedesktop.org
> +S:	Supported
> +F:	Documentation/accel/qaic/
> +F:	drivers/accel/qaic/
> +F:	include/uapi/drm/qaic_accel.h

Aren't you missing repo link?
T:	git git://anongit.freedesktop.org/drm/drm-misc

> +
>  QUALCOMM CORE POWER REDUCTION (CPR) AVS DRIVER
>  M:	Bjorn Andersson <andersson@kernel.org>
>  M:	Konrad Dybcio <konrad.dybcio@linaro.org>

Regards,
Jacek
Jeffrey Hugo March 14, 2023, 4:02 p.m. UTC | #2 
On 3/14/2023 3:59 AM, Jacek Lawrynowicz wrote:
> Hi
> 
> On 06.03.2023 22:34, Jeffrey Hugo wrote:
>> Add MAINTAINERS entry for the Qualcomm Cloud AI 100 driver.
>>
>> Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
>> Reviewed-by: Carl Vanderlip <quic_carlv@quicinc.com>
>> Reviewed-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
>> ---
>>   MAINTAINERS | 9 +++++++++
>>   1 file changed, 9 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index b0db911..feb2974 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -17253,6 +17253,15 @@ F:	Documentation/devicetree/bindings/clock/qcom,*
>>   F:	drivers/clk/qcom/
>>   F:	include/dt-bindings/clock/qcom,*
>>   
>> +QUALCOMM CLOUD AI (QAIC) DRIVER
>> +M:	Jeffrey Hugo <quic_jhugo@quicinc.com>
>> +L:	linux-arm-msm@vger.kernel.org
>> +L:	dri-devel@lists.freedesktop.org
>> +S:	Supported
>> +F:	Documentation/accel/qaic/
>> +F:	drivers/accel/qaic/
>> +F:	include/uapi/drm/qaic_accel.h
> 
> Aren't you missing repo link?
> T:	git git://anongit.freedesktop.org/drm/drm-misc

The long term repo is a bit of an open question.  I don't have access to 
that right now.

I'll try to get a conclusion on that for the next rev.
Jeffrey Hugo March 16, 2023, 5:04 p.m. UTC | #3 
On 3/14/2023 3:59 AM, Jacek Lawrynowicz wrote:
> Hi
> 
> On 06.03.2023 22:34, Jeffrey Hugo wrote:
>> Add MAINTAINERS entry for the Qualcomm Cloud AI 100 driver.
>>
>> Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
>> Reviewed-by: Carl Vanderlip <quic_carlv@quicinc.com>
>> Reviewed-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
>> ---
>>   MAINTAINERS | 9 +++++++++
>>   1 file changed, 9 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index b0db911..feb2974 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -17253,6 +17253,15 @@ F:	Documentation/devicetree/bindings/clock/qcom,*
>>   F:	drivers/clk/qcom/
>>   F:	include/dt-bindings/clock/qcom,*
>>   
>> +QUALCOMM CLOUD AI (QAIC) DRIVER
>> +M:	Jeffrey Hugo <quic_jhugo@quicinc.com>
>> +L:	linux-arm-msm@vger.kernel.org
>> +L:	dri-devel@lists.freedesktop.org
>> +S:	Supported
>> +F:	Documentation/accel/qaic/
>> +F:	drivers/accel/qaic/
>> +F:	include/uapi/drm/qaic_accel.h
> 
> Aren't you missing repo link?
> T:	git git://anongit.freedesktop.org/drm/drm-misc

Maarten/Maxime/Thomas are we ok to follow the iVPU example and use 
drm-misc for this, or would a separate tree be preferred?

-Jeff
Maxime Ripard March 17, 2023, 2:04 p.m. UTC | #4 
On Thu, Mar 16, 2023 at 11:04:05AM -0600, Jeffrey Hugo wrote:
> On 3/14/2023 3:59 AM, Jacek Lawrynowicz wrote:
> > Hi
> > 
> > On 06.03.2023 22:34, Jeffrey Hugo wrote:
> > > Add MAINTAINERS entry for the Qualcomm Cloud AI 100 driver.
> > > 
> > > Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
> > > Reviewed-by: Carl Vanderlip <quic_carlv@quicinc.com>
> > > Reviewed-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
> > > ---
> > >   MAINTAINERS | 9 +++++++++
> > >   1 file changed, 9 insertions(+)
> > > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index b0db911..feb2974 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -17253,6 +17253,15 @@ F:	Documentation/devicetree/bindings/clock/qcom,*
> > >   F:	drivers/clk/qcom/
> > >   F:	include/dt-bindings/clock/qcom,*
> > > +QUALCOMM CLOUD AI (QAIC) DRIVER
> > > +M:	Jeffrey Hugo <quic_jhugo@quicinc.com>
> > > +L:	linux-arm-msm@vger.kernel.org
> > > +L:	dri-devel@lists.freedesktop.org
> > > +S:	Supported
> > > +F:	Documentation/accel/qaic/
> > > +F:	drivers/accel/qaic/
> > > +F:	include/uapi/drm/qaic_accel.h
> > 
> > Aren't you missing repo link?
> > T:	git git://anongit.freedesktop.org/drm/drm-misc
> 
> Maarten/Maxime/Thomas are we ok to follow the iVPU example and use drm-misc
> for this, or would a separate tree be preferred?

Yeah, please go ahead with drm-misc

Do you have commit rights?

Maxime
Jeffrey Hugo March 17, 2023, 3:46 p.m. UTC | #5 
On 3/17/2023 8:04 AM, Maxime Ripard wrote:
> On Thu, Mar 16, 2023 at 11:04:05AM -0600, Jeffrey Hugo wrote:
>> On 3/14/2023 3:59 AM, Jacek Lawrynowicz wrote:
>>> Hi
>>>
>>> On 06.03.2023 22:34, Jeffrey Hugo wrote:
>>>> Add MAINTAINERS entry for the Qualcomm Cloud AI 100 driver.
>>>>
>>>> Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
>>>> Reviewed-by: Carl Vanderlip <quic_carlv@quicinc.com>
>>>> Reviewed-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
>>>> ---
>>>>    MAINTAINERS | 9 +++++++++
>>>>    1 file changed, 9 insertions(+)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index b0db911..feb2974 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -17253,6 +17253,15 @@ F:	Documentation/devicetree/bindings/clock/qcom,*
>>>>    F:	drivers/clk/qcom/
>>>>    F:	include/dt-bindings/clock/qcom,*
>>>> +QUALCOMM CLOUD AI (QAIC) DRIVER
>>>> +M:	Jeffrey Hugo <quic_jhugo@quicinc.com>
>>>> +L:	linux-arm-msm@vger.kernel.org
>>>> +L:	dri-devel@lists.freedesktop.org
>>>> +S:	Supported
>>>> +F:	Documentation/accel/qaic/
>>>> +F:	drivers/accel/qaic/
>>>> +F:	include/uapi/drm/qaic_accel.h
>>>
>>> Aren't you missing repo link?
>>> T:	git git://anongit.freedesktop.org/drm/drm-misc
>>
>> Maarten/Maxime/Thomas are we ok to follow the iVPU example and use drm-misc
>> for this, or would a separate tree be preferred?
> 
> Yeah, please go ahead with drm-misc
> 
> Do you have commit rights?

No.  My operating assumption is this series will get merged first, which 
will then justify having commit rights.  I'm new to DRM, so please 
educate me if I'm missing something.

Thanks

-Jeff
Oded Gabbay March 20, 2023, 7:03 a.m. UTC | #6 
On Fri, Mar 17, 2023 at 5:46 PM Jeffrey Hugo <quic_jhugo@quicinc.com> wrote:
>
> On 3/17/2023 8:04 AM, Maxime Ripard wrote:
> > On Thu, Mar 16, 2023 at 11:04:05AM -0600, Jeffrey Hugo wrote:
> >> On 3/14/2023 3:59 AM, Jacek Lawrynowicz wrote:
> >>> Hi
> >>>
> >>> On 06.03.2023 22:34, Jeffrey Hugo wrote:
> >>>> Add MAINTAINERS entry for the Qualcomm Cloud AI 100 driver.
> >>>>
> >>>> Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
> >>>> Reviewed-by: Carl Vanderlip <quic_carlv@quicinc.com>
> >>>> Reviewed-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
> >>>> ---
> >>>>    MAINTAINERS | 9 +++++++++
> >>>>    1 file changed, 9 insertions(+)
> >>>>
> >>>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>>> index b0db911..feb2974 100644
> >>>> --- a/MAINTAINERS
> >>>> +++ b/MAINTAINERS
> >>>> @@ -17253,6 +17253,15 @@ F:        Documentation/devicetree/bindings/clock/qcom,*
> >>>>    F:      drivers/clk/qcom/
> >>>>    F:      include/dt-bindings/clock/qcom,*
> >>>> +QUALCOMM CLOUD AI (QAIC) DRIVER
> >>>> +M:        Jeffrey Hugo <quic_jhugo@quicinc.com>
> >>>> +L:        linux-arm-msm@vger.kernel.org
> >>>> +L:        dri-devel@lists.freedesktop.org
> >>>> +S:        Supported
> >>>> +F:        Documentation/accel/qaic/
> >>>> +F:        drivers/accel/qaic/
> >>>> +F:        include/uapi/drm/qaic_accel.h
> >>>
> >>> Aren't you missing repo link?
> >>> T:  git git://anongit.freedesktop.org/drm/drm-misc
> >>
> >> Maarten/Maxime/Thomas are we ok to follow the iVPU example and use drm-misc
> >> for this, or would a separate tree be preferred?
> >
> > Yeah, please go ahead with drm-misc
> >
> > Do you have commit rights?
>
> No.  My operating assumption is this series will get merged first, which
> will then justify having commit rights.  I'm new to DRM, so please
> educate me if I'm missing something.
>
> Thanks
>
> -Jeff

Your assumption is correct. Once it gets merged, I think that's enough
for you to get commit rights.
And drm-misc is the place for most of the drivers in drm, so that's
also completely fine.

Thanks,
Oded
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index b0db911..feb2974 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17253,6 +17253,15 @@  F:	Documentation/devicetree/bindings/clock/qcom,*
 F:	drivers/clk/qcom/
 F:	include/dt-bindings/clock/qcom,*
 
+QUALCOMM CLOUD AI (QAIC) DRIVER
+M:	Jeffrey Hugo <quic_jhugo@quicinc.com>
+L:	linux-arm-msm@vger.kernel.org
+L:	dri-devel@lists.freedesktop.org
+S:	Supported
+F:	Documentation/accel/qaic/
+F:	drivers/accel/qaic/
+F:	include/uapi/drm/qaic_accel.h
+
 QUALCOMM CORE POWER REDUCTION (CPR) AVS DRIVER
 M:	Bjorn Andersson <andersson@kernel.org>
 M:	Konrad Dybcio <konrad.dybcio@linaro.org>