Re: [PATCH v11 0/3] Add basic node support for MediaTek MT8186 SoC - allen-kh.cheng

From: allen-kh.cheng <allen-kh.cheng@mediatek.com>
To: Matthias Brugger <matthias.bgg@gmail.com>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: <Project_Global_Chrome_Upstream_Group@mediatek.com>,
	<devicetree@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-mediatek@lists.infradead.org>,
	"Chen-Yu Tsai" <wenst@chromium.org>,
	Ryder Lee <ryder.lee@kernel.org>, <hsinyi@chromium.org>
Subject: Re: [PATCH v11 0/3] Add basic node support for MediaTek MT8186 SoC
Date: Thu, 7 Jul 2022 18:15:38 +0800	[thread overview]
Message-ID: <75855b7c8784909086db4e773c13a8e2eb591292.camel@mediatek.com> (raw)
In-Reply-To: <0f758c3d-a1cd-5e28-d3bb-70833b53c14b@gmail.com>

Hi Matthias,

On Thu, 2022-07-07 at 11:40 +0200, Matthias Brugger wrote:
> 
> On 22/06/2022 17:40, Matthias Brugger wrote:
> > 
> > 
> > On 20/05/2022 14:22, Allen-KH Cheng wrote:
> > > MT8186 is a SoC based on 64bit ARMv8 architecture.
> > > It contains 6 CA55 and 2 CA76 cores.
> > > MT8186 share many HW IP with MT65xx series.
> > > 
> > > This patchset was tested on MT8186 evaluation board to shell.
> > > 
> > 
> > Applied, thanks!
> > 
> 
> Based on the review from Angelo I dropped patch 3 from the tree.
> Please address 
> the issues and submit again.
> 
> Regards,
> Matthias
> 

Sure, thanks for the comments from Angelo.

I will update in next version.

BTW, could you please take a look into this series ?

https://lore.kernel.org/all/20220622130824.29143-1-nancy.lin@mediatek.com/

This series is blocked for long time and it's in verion 24.

I would appreciate your help.

Best regards,
Allen

> > > Based on next-20220519, linux-next/master
> > > 
> > > changes since v9:
> > >   - remove some merged PATCHs from series
> > >   - reorder nodes in dts (cpu-map)
> > >   - remove okay status in auxadc
> > >   - remove unnecessary suffix node name for i2c
> > >   - add pwm node
> > >   - add dsi-phy node
> > >   - add dpi node
> > > 
> > > changes since v9:
> > >   - add one space before equal sign of drive-strength-adv
> > >   - corect compatible name for big cores (ca76)
> > >   - use upper case of address in pinctrl
> > >   - add pwrap node
> > >   - add pwm node
> > > 
> > > changes since v8:
> > >   - change name from pins_bus to pins-sda-scl
> > >   - correct email address
> > >   - add capacity-dmips-mhz for each CPU
> > >   - add ppi-partitions in gic node
> > >   - change name to power-domain
> > >   - remove status "okay" in scp node
> > >   - update timer and pericfg compatible in series
> > > 
> > > changes since v7:
> > >   - add scp&auxadc node
> > > 
> > > changes since v6:
> > >   - remove unnecessary blank line
> > > 
> > > changes since v5:
> > >   - replace Mediatek a to MediaTek
> > >   - use GPL-2.0-only OR BSD-2-Clause
> > > 
> > > changes since v4:
> > >   - correct driver clock of mt8186
> > >   - add power domains controller and clock controllers
> > >   - add pinctrl, usb host, spi and i2c nodes
> > >   - add node status in mt8186-evb.dts
> > >   - correct some dtbs_check warnings
> > > 
> > > changes since v3:
> > >   - remove serial, mmc and phy patch from series. (already
> > > merged)
> > >   - remove mcusysoff node
> > >   - move oscillator nodes at the head of dts
> > >   - change name from usb-phy to t-phy
> > > 
> > > changes since v2:
> > >   - add soc {} in mt8186.dtsi
> > > 
> > > changes since v1:
> > >   - add dt-bindings: arm: Add compatible for MediaTek MT8186
> > > 
> > > Allen-KH Cheng (3):
> > >    dt-bindings: arm: mediatek: Add mt8186 pericfg compatible
> > >    dt-bindings: arm: Add compatible for MediaTek MT8186
> > >    arm64: dts: Add MediaTek SoC MT8186 dts and evaluation board
> > > and
> > >      Makefile
> > > 
> > >   .../devicetree/bindings/arm/mediatek.yaml     |    4 +
> > >   .../arm/mediatek/mediatek,pericfg.yaml        |    1 +
> > >   arch/arm64/boot/dts/mediatek/Makefile         |    1 +
> > >   arch/arm64/boot/dts/mediatek/mt8186-evb.dts   |  232 ++++
> > >   arch/arm64/boot/dts/mediatek/mt8186.dtsi      | 1016
> > > +++++++++++++++++
> > >   5 files changed, 1254 insertions(+)
> > >   create mode 100644 arch/arm64/boot/dts/mediatek/mt8186-evb.dts
> > >   create mode 100644 arch/arm64/boot/dts/mediatek/mt8186.dtsi
> > > 