Support for airoha en7581 Soc

Support for airoha en7581 Soc

[Lorenzo Bianconi]

[-- Attachment #1: Type: text/plain, Size: 260 bytes --]

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537

Do we need to respin them? Thx in advance.

Regards,
Lorenzo

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: Support for airoha en7581 Soc

[AngeloGioacchino Del Regno]

Il 29/01/24 12:20, Lorenzo Bianconi ha scritto:
> Hi Matthias,
> 
> I was wondering about the status of the following patches:
> https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
> 
> Do we need to respin them? Thx in advance.
> 

Lorenzo, I don't think you have to.

Matthias, should we update the MAINTAINERS file to include the MediaTek
Airoha folder?

Actually that question is for both ARM and ARM64.

Cheers

Re: Support for airoha en7581 Soc

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 749 bytes --]

On Mon, Jan 29, 2024 at 12:34:37PM +0100, AngeloGioacchino Del Regno wrote:
> Il 29/01/24 12:20, Lorenzo Bianconi ha scritto:
> > Hi Matthias,
> > 
> > I was wondering about the status of the following patches:
> > https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
> > 
> > Do we need to respin them? Thx in advance.
> > 
> 
> Lorenzo, I don't think you have to.

The compatibles in en7581-evb.dts appear to be undocumented as of
whatever linux-next I have checked out from late last week.

> 
> Matthias, should we update the MAINTAINERS file to include the MediaTek
> Airoha folder?
> 
> Actually that question is for both ARM and ARM64.
> 
> Cheers

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: Support for airoha en7581 Soc

[AngeloGioacchino Del Regno]

Il 29/01/24 12:39, Conor Dooley ha scritto:
> On Mon, Jan 29, 2024 at 12:34:37PM +0100, AngeloGioacchino Del Regno wrote:
>> Il 29/01/24 12:20, Lorenzo Bianconi ha scritto:
>>> Hi Matthias,
>>>
>>> I was wondering about the status of the following patches:
>>> https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
>>>
>>> Do we need to respin them? Thx in advance.
>>>
>>
>> Lorenzo, I don't think you have to.
> 
> The compatibles in en7581-evb.dts appear to be undocumented as of
> whatever linux-next I have checked out from late last week.
> 

Right, there's a 7523, but not a 7581.
Needs one more commit :-)

>>
>> Matthias, should we update the MAINTAINERS file to include the MediaTek
>> Airoha folder?
>>
>> Actually that question is for both ARM and ARM64.
>>
>> Cheers

Re: Support for airoha en7581 Soc

[Lorenzo Bianconi]

[-- Attachment #1: Type: text/plain, Size: 1024 bytes --]

> Il 29/01/24 12:39, Conor Dooley ha scritto:
> > On Mon, Jan 29, 2024 at 12:34:37PM +0100, AngeloGioacchino Del Regno wrote:
> > > Il 29/01/24 12:20, Lorenzo Bianconi ha scritto:
> > > > Hi Matthias,
> > > > 
> > > > I was wondering about the status of the following patches:
> > > > https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
> > > > 
> > > > Do we need to respin them? Thx in advance.
> > > > 
> > > 
> > > Lorenzo, I don't think you have to.
> > 
> > The compatibles in en7581-evb.dts appear to be undocumented as of
> > whatever linux-next I have checked out from late last week.
> > 
> 
> Right, there's a 7523, but not a 7581.
> Needs one more commit :-)

ack, I will respin the series.

Regards,
Lorenzo

> 
> > > 
> > > Matthias, should we update the MAINTAINERS file to include the MediaTek
> > > Airoha folder?
> > > 
> > > Actually that question is for both ARM and ARM64.
> > > 
> > > Cheers
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: Support for airoha en7581 Soc

[Krzysztof Kozlowski]

On 29/01/2024 12:20, Lorenzo Bianconi wrote:
> Hi Matthias,
> 
> I was wondering about the status of the following patches:
> https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
> 
> Do we need to respin them? Thx in advance.

I don't see anyone from SoC maintainers CCed on that patchset, so no one
will pick it up. Pinging makes sense if maintainers missed/ignored your
email. But if you do not send stuff to maintainers, you need to resend
to proper people, not ping.

Additionally, this is a new arch, but it seems to miss any maintainers
update.

Best regards,
Krzysztof

Re: Support for airoha en7581 Soc

[Lorenzo Bianconi]

[-- Attachment #1: Type: text/plain, Size: 1002 bytes --]

> On 29/01/2024 12:20, Lorenzo Bianconi wrote:
> > Hi Matthias,
> > 
> > I was wondering about the status of the following patches:
> > https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
> > 
> > Do we need to respin them? Thx in advance.
> 
> I don't see anyone from SoC maintainers CCed on that patchset, so no one
> will pick it up. Pinging makes sense if maintainers missed/ignored your
> email. But if you do not send stuff to maintainers, you need to resend
> to proper people, not ping.

I can't find any entry for airoha architecture in MAINTAINERS. Who is the
expected maintainer? (e.g. ARM variant)

> 
> Additionally, this is a new arch, but it seems to miss any maintainers
> update.

I am not the original author of the series, I have just seen it on the ML and I
was wondering about the status since it seems there are no replies.

Regards,
Lorenzo

> 
> Best regards,
> Krzysztof
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: Support for airoha en7581 Soc

[Krzysztof Kozlowski]

On 29/01/2024 16:20, Lorenzo Bianconi wrote:
>> On 29/01/2024 12:20, Lorenzo Bianconi wrote:
>>> Hi Matthias,
>>>
>>> I was wondering about the status of the following patches:
>>> https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
>>>
>>> Do we need to respin them? Thx in advance.
>>
>> I don't see anyone from SoC maintainers CCed on that patchset, so no one
>> will pick it up. Pinging makes sense if maintainers missed/ignored your
>> email. But if you do not send stuff to maintainers, you need to resend
>> to proper people, not ping.
> 
> I can't find any entry for airoha architecture in MAINTAINERS. Who is the
> expected maintainer? (e.g. ARM variant)

SoC maintainers are expected to be Cc'ed, even if they do not become
final maintainers. Get_maintainers, which should have been used there,
points it.

New platform should come with a new maintainer or should attach itself
to some existing platform. In both cases maintainers file will point to
platform maintainers...

> 
>>
>> Additionally, this is a new arch, but it seems to miss any maintainers
>> update.
> 
> I am not the original author of the series, I have just seen it on the ML and I
> was wondering about the status since it seems there are no replies.

Right. Lack of replies is the result of not Ccing anyone who would care.
get_maintainers.pl is the answer here.

Best regards,
Krzysztof

Re: Support for airoha en7581 Soc

[Arnd Bergmann]

On Mon, Jan 29, 2024, at 16:08, Krzysztof Kozlowski wrote:
> On 29/01/2024 12:20, Lorenzo Bianconi wrote:
>> Hi Matthias,
>> 
>> I was wondering about the status of the following patches:
>> https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
>> 
>> Do we need to respin them? Thx in advance.
>
> I don't see anyone from SoC maintainers CCed on that patchset, so no one
> will pick it up. Pinging makes sense if maintainers missed/ignored your
> email. But if you do not send stuff to maintainers, you need to resend
> to proper people, not ping.
>
> Additionally, this is a new arch, but it seems to miss any maintainers
> update.

I expect this to be picked up by the mediatek maintainers,
same as what we did for the arch/arm/ port that was merged
first. While airoha is a separate company these days,
I assume this is still closely related to mt7623, mt7683,
and mt7622, just like en7523 was.

     Arnd

Re: Support for airoha en7581 Soc

[AngeloGioacchino Del Regno]

Il 29/01/24 16:22, Arnd Bergmann ha scritto:
> On Mon, Jan 29, 2024, at 16:08, Krzysztof Kozlowski wrote:
>> On 29/01/2024 12:20, Lorenzo Bianconi wrote:
>>> Hi Matthias,
>>>
>>> I was wondering about the status of the following patches:
>>> https://lore.kernel.org/linux-arm-kernel/20231001122418.2688120-4-dd@embedd.com/T/#rbafabf4bf2473327f35ce7d79623f63838630537
>>>
>>> Do we need to respin them? Thx in advance.
>>
>> I don't see anyone from SoC maintainers CCed on that patchset, so no one
>> will pick it up. Pinging makes sense if maintainers missed/ignored your
>> email. But if you do not send stuff to maintainers, you need to resend
>> to proper people, not ping.
>>
>> Additionally, this is a new arch, but it seems to miss any maintainers
>> update.
> 
> I expect this to be picked up by the mediatek maintainers,
> same as what we did for the arch/arm/ port that was merged
> first.

Yes, we're doing that.

> While airoha is a separate company these days,

Not really... it's still a MediaTek subsidiary...

https://corp.mediatek.com/about/subsidiary-information

https://www.airoha.com/about

> I assume this is still closely related to mt7623, mt7683,
> and mt7622, just like en7523 was.

I assume the same. I'll add the airoha directory to the MAINTAINERS file
under ARM/Mediatek.

Cheers
Angelo