[PATCH v2 6/6] board: mediatek: Add Genio 520/720 EVK defconfigs

[Julien Stephan]

Le lun. 16 mars 2026 à 15:34, David Lechner <dlechner at baylibre.com> a écrit :
>
> On 3/16/26 7:38 AM, Julien Stephan wrote:
> > Le mer. 11 mars 2026 à 01:17, David Lechner <dlechner at baylibre.com> a écrit :
> >>
> >> From: Chris-QJ Chen <chris-qj.chen at mediatek.com>
> >>
> >> Add basic defconfigs for Genio 520 and 720 EVKs.
> >>
> >> Signed-off-by: Chris-QJ Chen <chris-qj.chen at mediatek.com>
> >> Signed-off-by: Macpaul Lin <macpaul.lin at mediatek.com>
> >> Signed-off-by: David Lechner <dlechner at baylibre.com>
> >> ---
> >>  board/mediatek/MAINTAINERS             |  7 +++++++
> >>  configs/mt8189.config                  | 34 ++++++++++++++++++++++++++++++++++
> >>  configs/mt8371_genio_520_evk_defconfig |  4 ++++
> >>  configs/mt8391_genio_720_evk_defconfig |  4 ++++
> >>  4 files changed, 49 insertions(+)
> >>
> >> diff --git a/board/mediatek/MAINTAINERS b/board/mediatek/MAINTAINERS
> >> index 446a9e8e53c..4f371592059 100644
> >> --- a/board/mediatek/MAINTAINERS
> >> +++ b/board/mediatek/MAINTAINERS
> >> @@ -1,3 +1,10 @@
> >> +MT8189/MT8371/MT8391 EVK
> >
> > Hi David,
> > I would not put MT8189 here since there is no MT8189 EVK
> >
> >> +M:     Macpaul Lin <macpaul.lin at mediatek.com>
> >> +S:     Maintained
> >> +F:     configs/mt8189.config
> >> +F:     configs/mt8371_genio_520_evk_defconfig
> >> +F:     configs/mt8391_genio_720_evk_defconfig
> >
> > Any reason to have genio_520/genio720 in the defconfig name?
>
> The upstream devicetree name will have "genio-720" in the name, so
> find it strange to call the defconfig something that doesn't match
> the devicetree.
>
> If there is a (not Genio) MT8391 EVK I would expect it to have a
> different devicetree and therefore a different defconfig.
>
> > All other boards we are currently upstreaming don't specify it in
> > their defconfig name:
> > mt8365_evk_defconfig/mt8370_evk_defconfig/mt8390_evk_defconfig ...
> >

I get your point.. From my understanding, mt8371_evk IS genio_520, so
IMHO there will be no non genio MT8371_EVK.
But, I understand that it makes sense to have the name of the
defconfig matching the name of the device tree.. In that case should
we rename all other defconfigs for mt board?
(8365, 8370,8390,8395..)