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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux