Re: [PATCH V2 0/5] Add PMIC DT files for sc7280

Bjorn Andersson
2021-04-04T17:41:53Z

On Thu 01 Apr 04:13 CDT 2021, satya priya wrote:

> Changes in V2:
> - As per Matthias's comments:
> - I've Split the patch into per-PMIC patches and one sc7280 patch
> - Removed 2nd critical point, thermal-governer property
> - s/pm8325_tz/pm7325_temp_alarm and s/pm7325_temp_alarm/pm7325_thermal
> - Fixed few other minor errors.
>
> - As per Bjorn's comments, replaced '_' with '-' in node names and moved
> DT files inclusion to board dts.
>
> This series is dependent on below series which adds DT files for SC7280 SoC
> https://lore.kernel.org/patchwork/project/lkml/list/?series=488871

No need to mention this dependency, as you posted this after said series
had been picked up.

However, also picked up are patches from Vinod adding initial pm8350c
and pmk8350 files, so please rebase you changes onto linux-next - in
addition to follow up on Matthias feedback.

Thanks,
Bjorn

>
> satya priya (5):
> arm64: dts: qcom: pm7325: Add PMIC peripherals for pm7325
> arm64: dts: qcom: pm8350c: Add PMIC peripherals for pm8350c
> arm64: dts: qcom: pmk8350: Add PMIC peripherals for pmk8350
> arm64: dts: qcom: pmr735a: Add PMIC peripherals for pmr735a
> arm64: dts: sc7280: Include PMIC DT files for sc7280
>
> arch/arm64/boot/dts/qcom/pm7325.dtsi | 53 +++++++++++++++++
> arch/arm64/boot/dts/qcom/pm8350c.dtsi | 53 +++++++++++++++++
> arch/arm64/boot/dts/qcom/pmk8350.dtsi | 100 ++++++++++++++++++++++++++++++++
> arch/arm64/boot/dts/qcom/pmr735a.dtsi | 53 +++++++++++++++++
> arch/arm64/boot/dts/qcom/sc7280-idp.dts | 4 ++
> arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +
> 6 files changed, 266 insertions(+)
> create mode 100644 arch/arm64/boot/dts/qcom/pm7325.dtsi
> create mode 100644 arch/arm64/boot/dts/qcom/pm8350c.dtsi
> create mode 100644 arch/arm64/boot/dts/qcom/pmk8350.dtsi
> create mode 100644 arch/arm64/boot/dts/qcom/pmr735a.dtsi
>
> --
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation
>