mbox series

[00/11] arm64: dts: qcom: msm8996: Introduce IFC6640

Message ID 20191021051322.297560-1-bjorn.andersson@linaro.org (mailing list archive)
Headers show
Series arm64: dts: qcom: msm8996: Introduce IFC6640 | expand

Message

Bjorn Andersson Oct. 21, 2019, 5:13 a.m. UTC 
Refactor msm8996 and db820c in order to make it follow the structure of newer
platforms, move db820c specific things to db820c.dtsi and then introduce the
Informace 6640 Single Board Computer.

Bjorn Andersson (11):
  arm64: dts: qcom: db820c: Move non-soc entries out of /soc
  arm64: dts: qcom: msm8996: Use node references in db820c
  arm64: dts: qcom: msm8996: Move regulator consumers to db820c
  arm64: dts: qcom: msm8996: Move regulators to db820c
  arm64: dts: qcom: db820c: Group root nodes
  arm64: dts: qcom: db820c: Sort all nodes
  arm64: dts: qcom: db820c: Remove pin specific files
  arm64: dts: qcom: msm8996: Pad addresses
  arm64: dts: qcom: msm8996: Sort all nodes in msm8996.dtsi
  arm64: dts: qcom: db820c: Use regulator names from schematics
  arm64: dts: qcom: msm8996: Introduce IFC6640

 arch/arm64/boot/dts/qcom/Makefile             |    1 +
 .../boot/dts/qcom/apq8096-db820c-pins.dtsi    |  109 -
 .../dts/qcom/apq8096-db820c-pmic-pins.dtsi    |   92 -
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi  | 1416 ++++---
 arch/arm64/boot/dts/qcom/apq8096-ifc6640.dts  |  412 ++
 arch/arm64/boot/dts/qcom/msm8996.dtsi         | 3505 ++++++++---------
 6 files changed, 2982 insertions(+), 2553 deletions(-)
 delete mode 100644 arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi
 delete mode 100644 arch/arm64/boot/dts/qcom/apq8096-db820c-pmic-pins.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/apq8096-ifc6640.dts

Comments

Vinod Koul Nov. 3, 2019, 8:13 a.m. UTC | #1 
On 20-10-19, 22:13, Bjorn Andersson wrote:
> Refactor msm8996 and db820c in order to make it follow the structure of newer
> platforms, move db820c specific things to db820c.dtsi and then introduce the
> Informace 6640 Single Board Computer.

This has patch 9/11 missing. But rest look good to me.

Acked-by: Vinod Koul <vkoul@kernel.org>
Bjorn Andersson Nov. 4, 2019, 4:55 a.m. UTC | #2 
On Sun 03 Nov 01:13 PDT 2019, Vinod Koul wrote:

> On 20-10-19, 22:13, Bjorn Andersson wrote:
> > Refactor msm8996 and db820c in order to make it follow the structure of newer
> > platforms, move db820c specific things to db820c.dtsi and then introduce the
> > Informace 6640 Single Board Computer.
> 
> This has patch 9/11 missing. But rest look good to me.
> 

That's really odd, I copy pasted the recipients into all the patches.
But I'm unable to find it under linux-arm-msm on lore as well.

It's under LKML though, can you please have a look and let me know if I
can extend your ack to patch 9/11 as well?

https://lore.kernel.org/lkml/20191021051322.297560-10-bjorn.andersson@linaro.org/

> Acked-by: Vinod Koul <vkoul@kernel.org>

Thank you for the review Vinod!

Regards,
Bjorn

> 
> -- 
> ~Vinod
Vinod Koul Nov. 4, 2019, 9:27 a.m. UTC | #3 
On 03-11-19, 20:55, Bjorn Andersson wrote:
> On Sun 03 Nov 01:13 PDT 2019, Vinod Koul wrote:
> 
> > On 20-10-19, 22:13, Bjorn Andersson wrote:
> > > Refactor msm8996 and db820c in order to make it follow the structure of newer
> > > platforms, move db820c specific things to db820c.dtsi and then introduce the
> > > Informace 6640 Single Board Computer.
> > 
> > This has patch 9/11 missing. But rest look good to me.
> > 
> 
> That's really odd, I copy pasted the recipients into all the patches.
> But I'm unable to find it under linux-arm-msm on lore as well.

Yup I can see it there but not on arm-msm. Do you use @linaro smtp to
send. Gmail is known to drop emails to lists on  vger..
I use @kernel.org one, havent seen issues on that yet

> It's under LKML though, can you please have a look and let me know if I
> can extend your ack to patch 9/11 as well?
> 
> https://lore.kernel.org/lkml/20191021051322.297560-10-bjorn.andersson@linaro.org/

Mostly looks good but the reserve memory doesnt seem sorted by node
(please recheck) and rest looks good so you can add it to that patch as
well :)