Skip to contentSkip to conversation

Support for airoha en7581 Soc

DISCUSSION Lorenzo Bianconi 5 participants 10 messages Jan 29, 2024

lore
Share & tools
lore
#1 lore raw

Il 29/01/24 12:20, Lorenzo Bianconi ha scritto:

Quoted 7 lines

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/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

#2 lore raw
Quoted 10 lines · AngeloGioacchino Del Regno

Il 29/01/24 12:20, Lorenzo Bianconi ha scritto:

Quoted 7 lines

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/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.

Quoted 7 lines

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

Actually that question is for both ARM and ARM64.

Cheers

#3 lore raw

Il 29/01/24 12:39, Conor Dooley ha scritto:

Quoted 7 lines · AngeloGioacchino Del Regno
Quoted 6 lines

Il 29/01/24 12:20, Lorenzo Bianconi ha scritto:

Quoted 5 lines

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/T/#rbafabf4bf2473327f35ce7d79623f63838630537

Quoted 8 lines
Quoted 4 lines
Quoted 2 lines

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 :-)

Quoted 7 lines
Quoted 7 lines

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

Actually that question is for both ARM and ARM64.

Cheers

#4 lore raw
Quoted 11 lines

Il 29/01/24 12:39, Conor Dooley ha scritto:

Quoted 10 lines · AngeloGioacchino Del Regno
Quoted 9 lines

Il 29/01/24 12:20, Lorenzo Bianconi ha scritto:

Quoted 7 lines

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/T/#rbafabf4bf2473327f35ce7d79623f63838630537

Do we need to respin them? Thx in advance.

Quoted 8 lines
Quoted 5 lines
Quoted 1 line

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

Quoted 9 lines
Quoted 8 lines
Quoted 8 lines

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

Actually that question is for both ARM and ARM64.

Cheers

#5 lore raw
In reply to Lorenzo Bianconi
Quoted 6 lines · Lorenzo Bianconi

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/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

#6 lore raw
Quoted 6 lines · Lorenzo Bianconi
Quoted 5 lines

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/T/#rbafabf4bf2473327f35ce7d79623f63838630537

Quoted 6 lines
Quoted 1 line

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)

Quoted 3 lines

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

Quoted 4 lines

Best regards,
Krzysztof

#7 lore raw
Quoted 8 lines · Lorenzo Bianconi
Quoted 8 lines · Lorenzo Bianconi
Quoted 6 lines

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/T/#rbafabf4bf2473327f35ce7d79623f63838630537

Do we need to respin them? Thx in advance.

Quoted 7 lines
Quoted 4 lines

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...

Quoted 7 lines
Quoted 4 lines

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

#8 lore raw
In reply to Krzysztof Kozlowski
Quoted 8 lines · Krzysztof Kozlowski
Quoted 6 lines · Lorenzo Bianconi

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/T/#rbafabf4bf2473327f35ce7d79623f63838630537

Do we need to respin them? Thx in advance.

Quoted 7 lines

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

#9 lore raw

Il 29/01/24 16:22, Arnd Bergmann ha scritto:

Quoted 14 lines · Krzysztof Kozlowski
Quoted 13 lines · Lorenzo Bianconi
Quoted 6 lines

Hi Matthias,

I was wondering about the status of the following patches:
https://lore.kernel.org/linux-arm-kernel/[email protected]/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.

Quoted 6 lines
Quoted 2 lines

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.

Quoted 1 line

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

Quoted 2 lines

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