From: Chen-Yu Tsai <wenst@chromium.org> To: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Cc: "Nícolas F. R. A. Prado" <nfraprado@collabora.com>, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, matthias.bgg@gmail.com, hsinyi@chromium.org, allen-kh.cheng@mediatek.com, gtk3@inbox.ru, luca@z3ntu.xyz, sam.shih@mediatek.com, sean.wang@mediatek.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH 02/11] arm64: dts: mediatek: Introduce MT8195 Cherry platform's Tomato Date: Mon, 4 Jul 2022 17:05:19 +0800 [thread overview] Message-ID: <CAGXv+5HUQ1pNSDqyiH8taec8ykeaztjNY5dQPKTcJPpYh5JS9g@mail.gmail.com> (raw) In-Reply-To: <91e60954-d44d-f99f-2b4f-c164fb33cc0e@collabora.com> On Mon, Jul 4, 2022 at 4:59 PM AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> wrote: > > Il 04/07/22 06:17, Chen-Yu Tsai ha scritto: > > On Sat, Jul 2, 2022 at 6:12 AM Nícolas F. R. A. Prado > > <nfraprado@collabora.com> wrote: > >> > >> On Thu, Jun 30, 2022 at 05:33:07PM +0200, AngeloGioacchino Del Regno wrote: > >>> Introduce the MT8195 Cherry Chromebook platform, including three > >>> revisions of Cherry Tomato boards. > >>> > >>> This basic configuration allows to boot Linux on all board revisions > >>> and get a serial console from a ramdisk. > >>> > >>> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > >>> --- > >>> arch/arm64/boot/dts/mediatek/Makefile | 3 +++ > >>> .../dts/mediatek/mt8195-cherry-tomato-r1.dts | 11 ++++++++ > >>> .../dts/mediatek/mt8195-cherry-tomato-r2.dts | 11 ++++++++ > >>> .../dts/mediatek/mt8195-cherry-tomato-r3.dts | 12 +++++++++ > >>> .../boot/dts/mediatek/mt8195-cherry.dtsi | 26 +++++++++++++++++++ > >>> 5 files changed, 63 insertions(+) > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r1.dts > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r2.dts > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r3.dts > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-cherry.dtsi > >> [..] > >>> diff --git a/arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r1.dts b/arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r1.dts > >>> new file mode 100644 > >>> index 000000000000..17e9e4d6f6ab > >>> --- /dev/null > >>> +++ b/arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r1.dts > >>> @@ -0,0 +1,11 @@ > >>> +// SPDX-License-Identifier: (GPL-2.0 OR MIT) > >>> +/* > >>> + * Copyright (C) 2021 MediaTek Inc. > >>> + */ > >>> +/dts-v1/; > >>> +#include "mt8195-cherry.dtsi" > >>> + > >>> +/ { > >>> + model = "MediaTek Tomato (rev1) board"; > >> > >> Given that the compatible is "google," I believe we'll want to rename the model > >> to "Google Tomato", much like was commented on the Asurada series [1], but > >> better to have confirmation from someone from Google. Chen-Yu? :) > > > > I asked for clarification internally and it turns out we didn't get the > > Asurada series quite right either. > > > > Google only owns the reference design, that is Asurada for MT8192 and > > Cherry for MT8195. The vendor own the end product design that is based > > off of Google's reference design. > > > > So for Tomato, the vendor is Acer. Note that "Tomato" and the other > > codenames seen in ChromeOS are public codenames that Google uses. > > > > The compatible string will likely stay "google,XXX", since this is set > > in firmware, and updating it after the product has shipped poses both > > a significant hurdle and risk. > > > > Yeah, makes sense. Should we call this "Acer Tomato (revX) board" then? > ...if we do that, though, we need to know if the other revisions of Tomato > are also from Acer, or we would be getting one right and all the others wrong. AFAIK each "device build" is its own project and gets its own codename, otherwise it would get real confusing for us internally really fast. ChenYu > Cheers, > Angelo > > > > > Regards > > ChenYu > > > >> Otherwise, > >> > >> Reviewed-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> > >> > >> Thanks, > >> Nícolas > >> > >> [1] https://lore.kernel.org/all/CAGXv+5Gv2pjPXynz6HCdgux+giPDC5qRk+KW1aFduVz82rM=+g@mail.gmail.com/ > >> > >>> + compatible = "google,tomato-rev1", "google,tomato", "mediatek,mt8195"; > >>> +}; > >> [..] > >>> -- > >>> 2.35.1 > >>> >
WARNING: multiple messages have this Message-ID (diff)
From: Chen-Yu Tsai <wenst@chromium.org> To: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Cc: "Nícolas F. R. A. Prado" <nfraprado@collabora.com>, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, matthias.bgg@gmail.com, hsinyi@chromium.org, allen-kh.cheng@mediatek.com, gtk3@inbox.ru, luca@z3ntu.xyz, sam.shih@mediatek.com, sean.wang@mediatek.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH 02/11] arm64: dts: mediatek: Introduce MT8195 Cherry platform's Tomato Date: Mon, 4 Jul 2022 17:05:19 +0800 [thread overview] Message-ID: <CAGXv+5HUQ1pNSDqyiH8taec8ykeaztjNY5dQPKTcJPpYh5JS9g@mail.gmail.com> (raw) In-Reply-To: <91e60954-d44d-f99f-2b4f-c164fb33cc0e@collabora.com> On Mon, Jul 4, 2022 at 4:59 PM AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> wrote: > > Il 04/07/22 06:17, Chen-Yu Tsai ha scritto: > > On Sat, Jul 2, 2022 at 6:12 AM Nícolas F. R. A. Prado > > <nfraprado@collabora.com> wrote: > >> > >> On Thu, Jun 30, 2022 at 05:33:07PM +0200, AngeloGioacchino Del Regno wrote: > >>> Introduce the MT8195 Cherry Chromebook platform, including three > >>> revisions of Cherry Tomato boards. > >>> > >>> This basic configuration allows to boot Linux on all board revisions > >>> and get a serial console from a ramdisk. > >>> > >>> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > >>> --- > >>> arch/arm64/boot/dts/mediatek/Makefile | 3 +++ > >>> .../dts/mediatek/mt8195-cherry-tomato-r1.dts | 11 ++++++++ > >>> .../dts/mediatek/mt8195-cherry-tomato-r2.dts | 11 ++++++++ > >>> .../dts/mediatek/mt8195-cherry-tomato-r3.dts | 12 +++++++++ > >>> .../boot/dts/mediatek/mt8195-cherry.dtsi | 26 +++++++++++++++++++ > >>> 5 files changed, 63 insertions(+) > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r1.dts > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r2.dts > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r3.dts > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-cherry.dtsi > >> [..] > >>> diff --git a/arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r1.dts b/arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r1.dts > >>> new file mode 100644 > >>> index 000000000000..17e9e4d6f6ab > >>> --- /dev/null > >>> +++ b/arch/arm64/boot/dts/mediatek/mt8195-cherry-tomato-r1.dts > >>> @@ -0,0 +1,11 @@ > >>> +// SPDX-License-Identifier: (GPL-2.0 OR MIT) > >>> +/* > >>> + * Copyright (C) 2021 MediaTek Inc. > >>> + */ > >>> +/dts-v1/; > >>> +#include "mt8195-cherry.dtsi" > >>> + > >>> +/ { > >>> + model = "MediaTek Tomato (rev1) board"; > >> > >> Given that the compatible is "google," I believe we'll want to rename the model > >> to "Google Tomato", much like was commented on the Asurada series [1], but > >> better to have confirmation from someone from Google. Chen-Yu? :) > > > > I asked for clarification internally and it turns out we didn't get the > > Asurada series quite right either. > > > > Google only owns the reference design, that is Asurada for MT8192 and > > Cherry for MT8195. The vendor own the end product design that is based > > off of Google's reference design. > > > > So for Tomato, the vendor is Acer. Note that "Tomato" and the other > > codenames seen in ChromeOS are public codenames that Google uses. > > > > The compatible string will likely stay "google,XXX", since this is set > > in firmware, and updating it after the product has shipped poses both > > a significant hurdle and risk. > > > > Yeah, makes sense. Should we call this "Acer Tomato (revX) board" then? > ...if we do that, though, we need to know if the other revisions of Tomato > are also from Acer, or we would be getting one right and all the others wrong. AFAIK each "device build" is its own project and gets its own codename, otherwise it would get real confusing for us internally really fast. ChenYu > Cheers, > Angelo > > > > > Regards > > ChenYu > > > >> Otherwise, > >> > >> Reviewed-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> > >> > >> Thanks, > >> Nícolas > >> > >> [1] https://lore.kernel.org/all/CAGXv+5Gv2pjPXynz6HCdgux+giPDC5qRk+KW1aFduVz82rM=+g@mail.gmail.com/ > >> > >>> + compatible = "google,tomato-rev1", "google,tomato", "mediatek,mt8195"; > >>> +}; > >> [..] > >>> -- > >>> 2.35.1 > >>> > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-07-04 9:05 UTC|newest] Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-06-30 15:33 [PATCH 00/11] MT8195 Chromebooks devicetrees - Google Tomato AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-06-30 15:33 ` [PATCH 01/11] dt-bindings: arm: mediatek: Add MT8195 Cherry Tomato Chromebooks AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-06-30 18:26 ` Krzysztof Kozlowski 2022-06-30 18:26 ` Krzysztof Kozlowski 2022-07-01 15:31 ` Nícolas F. R. A. Prado 2022-07-01 15:31 ` Nícolas F. R. A. Prado 2022-07-04 8:51 ` AngeloGioacchino Del Regno 2022-07-04 8:51 ` AngeloGioacchino Del Regno 2022-07-01 22:04 ` Nícolas F. R. A. Prado 2022-07-01 22:04 ` Nícolas F. R. A. Prado 2022-07-04 8:56 ` AngeloGioacchino Del Regno 2022-07-04 8:56 ` AngeloGioacchino Del Regno 2022-06-30 15:33 ` [PATCH 02/11] arm64: dts: mediatek: Introduce MT8195 Cherry platform's Tomato AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:11 ` Nícolas F. R. A. Prado 2022-07-01 22:11 ` Nícolas F. R. A. Prado 2022-07-04 4:17 ` Chen-Yu Tsai 2022-07-04 4:17 ` Chen-Yu Tsai 2022-07-04 8:59 ` AngeloGioacchino Del Regno 2022-07-04 8:59 ` AngeloGioacchino Del Regno 2022-07-04 9:05 ` Chen-Yu Tsai [this message] 2022-07-04 9:05 ` Chen-Yu Tsai 2022-07-04 9:06 ` AngeloGioacchino Del Regno 2022-07-04 9:06 ` AngeloGioacchino Del Regno 2022-06-30 15:33 ` [PATCH 03/11] arm64: dts: mediatek: cherry: Assign interrupt line to MT6359 PMIC AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:20 ` Nícolas F. R. A. Prado 2022-07-01 22:20 ` Nícolas F. R. A. Prado 2022-06-30 15:33 ` [PATCH 04/11] arm64: dts: mediatek: cherry: Add platform regulators layout and config AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:26 ` Nícolas F. R. A. Prado 2022-07-01 22:26 ` Nícolas F. R. A. Prado 2022-06-30 15:33 ` [PATCH 05/11] arm64: dts: mediatek: cherry: Add support for internal eMMC storage AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:36 ` Nícolas F. R. A. Prado 2022-07-01 22:36 ` Nícolas F. R. A. Prado 2022-06-30 15:33 ` [PATCH 06/11] arm64: dts: mediatek: cherry: Document gpios and add default pin config AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:39 ` Nícolas F. R. A. Prado 2022-07-01 22:39 ` Nícolas F. R. A. Prado 2022-06-30 15:33 ` [PATCH 07/11] arm64: dts: mediatek: cherry: Enable I2C and SPI controllers AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:42 ` Nícolas F. R. A. Prado 2022-07-01 22:42 ` Nícolas F. R. A. Prado 2022-06-30 15:33 ` [PATCH 08/11] arm64: dts: mediatek: cherry: Enable T-PHYs and USB XHCI controllers AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:43 ` Nícolas F. R. A. Prado 2022-07-01 22:43 ` Nícolas F. R. A. Prado 2022-06-30 15:33 ` [PATCH 09/11] arm64: dts: mediatek: cherry: Enable MT6360 sub-pmic on I2C7 AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:45 ` Nícolas F. R. A. Prado 2022-07-01 22:45 ` Nícolas F. R. A. Prado 2022-06-30 15:33 ` [PATCH 10/11] arm64: dts: mediatek: cherry: Enable support for the SPI NOR flash AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:47 ` Nícolas F. R. A. Prado 2022-07-01 22:47 ` Nícolas F. R. A. Prado 2022-06-30 15:33 ` [PATCH 11/11] arm64: dts: mediatek: cherry: Add I2C-HID touchscreen on I2C4 AngeloGioacchino Del Regno 2022-06-30 15:33 ` AngeloGioacchino Del Regno 2022-07-01 22:49 ` Nícolas F. R. A. Prado 2022-07-01 22:49 ` Nícolas F. R. A. Prado
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=CAGXv+5HUQ1pNSDqyiH8taec8ykeaztjNY5dQPKTcJPpYh5JS9g@mail.gmail.com \ --to=wenst@chromium.org \ --cc=allen-kh.cheng@mediatek.com \ --cc=angelogioacchino.delregno@collabora.com \ --cc=devicetree@vger.kernel.org \ --cc=gtk3@inbox.ru \ --cc=hsinyi@chromium.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=luca@z3ntu.xyz \ --cc=matthias.bgg@gmail.com \ --cc=nfraprado@collabora.com \ --cc=robh+dt@kernel.org \ --cc=sam.shih@mediatek.com \ --cc=sean.wang@mediatek.com \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.