Re: [PATCH v8 8/8] arm64: dts: mt6359: add PMIC MT6359 related nodes

[Macpaul Lin]

On 1/25/22 4:32 PM, Macpaul Lin wrote:
> On 6/16/21 10:07 PM, Hsin-hsiung Wang wrote:
> > Hi,
> >
> > On Fri, 2021-06-11 at 16:09 +0200, Matthias Brugger wrote:
> > > On 26/05/2021 08:52, Hsin-Hsiung Wang wrote:
> > > > From: Wen Su <wen.su@xxxxxxxxxxxx>
> > > >
> > > > add PMIC MT6359 related nodes which is for MT6779 platform
> > > >
> > > > Signed-off-by: Wen Su <wen.su@xxxxxxxxxxxx>
> > > > Signed-off-by: Hsin-Hsiung Wang <hsin-hsiung.wang@xxxxxxxxxxxx>
> > > > ---
> > > > changes since v7:
> > > > - no change.
> > > > ---
> > > >   arch/arm64/boot/dts/mediatek/mt6359.dtsi    | 298 
> > > > ++++++++++++++++++++++++++++
> > > >   arch/arm64/boot/dts/mediatek/mt8192-evb.dts |   1 +
> > > >   2 files changed, 299 insertions(+)
> > > >   create mode 100644 arch/arm64/boot/dts/mediatek/mt6359.dtsi
> > > >
> > > > diff --git a/arch/arm64/boot/dts/mediatek/mt6359.dtsi 
> > > > b/arch/arm64/boot/dts/mediatek/mt6359.dtsi
> > > > new file mode 100644
> > > > index 0000000..18c0d53
> > > > --- /dev/null
> > > > +++ b/arch/arm64/boot/dts/mediatek/mt6359.dtsi
> > > > @@ -0,0 +1,298 @@
> > > > +// SPDX-License-Identifier: GPL-2.0
> > >
> > > Any specific reason for not setting it "SPDX-License-Identifier: 
> > > (GPL-2.0+ OR
> > > MIT)" ?
> > >
> > > Other then that, looks good.
> > >
> > > Matthias
> >
> > Thanks for the review, there is no special reason for the writing.
> > I will update it in the next patch.
> [The content of the patch has been deleted...]
>
> According to the reviewing note of PMIC wrap, URL:
> https://patchwork.kernel.org/project/linux-mediatek/cover/1615563286-22126-1-git-send-email-hsin-hsiung.wang@xxxxxxxxxxxx 
>
>
> Dear Hui and Sen,
> could you help update and upstream the mt6359.dtsi and mt8192-evb.dts 
> with the necessary fixes?
>
> Thanks!
> Macpaul Lin

Dear Hui,

I've found patch v8 will cause build fail because pwrap node hasn't been 
merged according to [1] and [2] (mt8192). However, the pwrap node for 
mt8195 is already in [3] mt8195.dtsi. The mt8195 boards using mt6359 
will need mt6359.dtsi for enabling more peripherals.

I think to split the mt8192 part into the other patch in next version 
could reduce the dependencies between mt6359.dtsi and mt8192.dtsi at 
this stage. Hence we don't need to wait pwrap node be merged in mt8192.dtsi.

[1] 
https://lore.kernel.org/all/1615563286-22126-6-git-send-email-hsin-hsiung.wang@xxxxxxxxxxxx/
[2] 
https://lore.kernel.org/all/e33053cf-f337-d7f6-fbf6-93d385f7e683@xxxxxxxxx/
[3] 
https://lore.kernel.org/all/20220112114724.1953-4-tinghan.shen@xxxxxxxxxxxx/

After that, you can have my Reviewed-by tag in next version.

Reviewed-by: Macpaul Lin <macpaul.lin@xxxxxxxxxxxx>

Thanks :)
Macpaul Lin