From: Arnd Bergmann <firstname.lastname@example.org> To: John Crispin <email@example.com> Cc: "Arnd Bergmann" <firstname.lastname@example.org>, "Bert Vermeulen" <email@example.com>, DTML <firstname.lastname@example.org>, "Linux Kernel Mailing List" <email@example.com>, "Linux ARM" <firstname.lastname@example.org>, "Russell King" <email@example.com>, "Linus Walleij" <firstname.lastname@example.org>, "Andrew Morton" <email@example.com>, "Geert Uytterhoeven" <firstname.lastname@example.org>, "Anshuman Khandual" <email@example.com>, "Krzysztof Kozlowski" <firstname.lastname@example.org>, "Ard Biesheuvel" <email@example.com>, "YiFei Zhu" <firstname.lastname@example.org>, "Mike Rapoport" <email@example.com>, "Uwe Kleine-König" <firstname.lastname@example.org>, "Nick Desaulniers" <email@example.com>, "Kees Cook" <firstname.lastname@example.org>, "Masahiro Yamada" <email@example.com>, "Nathan Chancellor" <firstname.lastname@example.org>, "Viresh Kumar" <email@example.com>, "Felix Fietkau" <firstname.lastname@example.org>, "Matthias Brugger" <email@example.com> Subject: Re: [PATCH 4/5] ARM: Add basic support for EcoNet EN7523 SoC Date: Fri, 30 Jul 2021 18:55:02 +0200 [thread overview] Message-ID: <CAK8P3a1c9bVjACjAN=2d+XiBn+yWgHHs9CEUmik7U61+3MzFhw@mail.gmail.com> (raw) In-Reply-To: <firstname.lastname@example.org> On Fri, Jul 30, 2021 at 5:16 PM John Crispin <email@example.com> wrote: > On 30.07.21 16:48, Arnd Bergmann wrote: > > Given how closely related this probably is to MT7623/MT7622, should this > > perhaps just be part of arch/arm/mach-mediatek? According to > > https://wikidevi.wi-cat.ru/MediaTek#xPON, the older (mips based) MT752x > > chips are apparently just rebranded to EN752x after the business unit > > was spun off, but I guess they are still in the same family. > > Hi, > > ECNT (what was once known as trendchip) is now a subsidary of MTK (and > not a BU if I am understanding it correctly). > > the EN7523 is rather similar to the MT7622 for some parts, other parts > (spi, flash, wdt, gpio, .. drivers all needed to be rewritten and will > be part of the next series). > > the older MIPS silicon shares almost no IP with the current ARM silicon. Ah, so I guess the old Trendchip parts were separately developed separately from the Ralink parts before the original acquisition, and then Ralink/Mediatek combined the two product lines before spinning off the dsl products into a new subsidiary. > not really my call to decide which folder this should live in. it seemed > natural to just give it its own folder, as ECNT is not a BU of MTK. > > we can change that however if required. My preference would be to have a common directory for both, but I'm not going to require that. From the kernel perspective the main question is actually not who makes the parts but who is going to maintain the code. Matthias is doing a good job taking care of the Mediatek parts, and he's familiar with the arm-soc process. If there is enough overlap between the Mediatek and EcoNet devices that we would expect either conflicts between binding/driver patches, or that you would want each other to review the patches for related parts, then it would make most sense to have a common directory and maintainer entry for both, with all patches going through the same git tree. It would also be nice to have someone listed as second maintainer for mediatek, since Matthias is currently the only one listed there. (Doesn't have to be you, I don't mean to drag you into taking up more work if you don't want to). Please discuss this between the three of you (Bert, John and Matthias), and let me know what you think works best for all of you. Arnd
next prev parent reply other threads:[~2021-07-30 16:55 UTC|newest] Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-07-30 13:45 [PATCH 0/4] Add " Bert Vermeulen 2021-07-30 13:45 ` [PATCH 1/5] dt-bindings: Add vendor prefix for EcoNet Bert Vermeulen 2021-08-06 21:02 ` Rob Herring 2021-07-30 13:45 ` [PATCH 2/5] dt-bindings: arm: econet: Add binding for EN7523 SoC and EVB Bert Vermeulen 2021-08-06 21:04 ` Rob Herring 2021-07-30 13:45 ` [PATCH 3/5] ARM: dts: Add basic support for EcoNet EN7523 Bert Vermeulen 2021-07-30 14:31 ` Linus Walleij [not found] ` <firstname.lastname@example.org> 2021-08-01 9:07 ` Bert Vermeulen 2021-08-01 9:40 ` Arnd Bergmann 2021-07-30 14:45 ` Daniel Palmer 2021-07-30 14:46 ` Mark Rutland 2021-08-04 16:41 ` Bert Vermeulen 2021-08-06 20:59 ` Rob Herring 2021-07-30 14:59 ` Mark Rutland 2021-08-06 20:52 ` Rob Herring 2021-07-30 16:47 ` Andre Przywara 2021-07-30 13:45 ` [PATCH 4/5] ARM: Add basic support for EcoNet EN7523 SoC Bert Vermeulen 2021-07-30 14:48 ` Arnd Bergmann 2021-07-30 15:15 ` John Crispin 2021-07-30 16:55 ` Arnd Bergmann [this message] 2021-08-01 16:44 ` Ard Biesheuvel 2021-09-03 16:20 ` Felix Fietkau 2021-09-03 17:47 ` Ard Biesheuvel 2021-08-04 16:43 ` Bert Vermeulen 2021-08-09 12:46 ` Geert Uytterhoeven 2021-08-09 12:49 ` Ard Biesheuvel 2021-08-09 13:00 ` Bert Vermeulen 2021-07-30 13:45 ` [PATCH 5/5] ARM: multi_v7_defconfig: Add " Bert Vermeulen 2021-07-30 14:26 ` [PATCH 0/4] " Linus Walleij
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAK8P3a1c9bVjACjAN=2d+XiBn+yWgHHs9CEUmik7U61+3MzFhw@mail.gmail.com' \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCH 4/5] ARM: Add basic support for EcoNet EN7523 SoC' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).