Re: [PATCH 00/17] Multiple addition to ipq8064 dtsi

From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Ansuel Smith <ansuelsmth@gmail.com>
Cc: Andy Gross <agross@kernel.org>, Rob Herring <robh+dt@kernel.org>,
	linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/17] Multiple addition to ipq8064 dtsi
Date: Mon, 31 Jan 2022 16:51:11 -0600	[thread overview]
Message-ID: <Yfhn3x6SYzlzZORx@builder.lan> (raw)
In-Reply-To: <20220118012051.21691-1-ansuelsmth@gmail.com>

On Mon 17 Jan 19:20 CST 2022, Ansuel Smith wrote:

> This try to complete the ipq8064 dtsi and introduce 2 new dtsi
> ipq8064-v2 and ipq8065. While some node are still missing (cpufreq node,
> l2 scale node, fab scale node) this would add most of the missing node
> to make ipq8064 actually usable.
> 
> Some of the changes are the fix for the pci IO that cause any secondary
> wifi card with ath10k to fail init.
> Adds regulators definition for RPM.
> Adds many missing gsbi nodes used by all the devices.
> Enable the usb phy by default as they are actually enabled internally by
> xlate only if the dwc3 driver is used.
> Add opp table and declare idle state for ipq8064.
> Fix some dtc warning.
> 
> This also add the ipq8064-v2.0 dtsi and the ipq8065 dtsi used by more
> recent devices based on this SoC.
> 

From the look of the series I suspect that you format-patch'ed a series
with 16 patches, then fixed something and re-ran it now with 17 patches.

Can you please resubmit the series to make it clear what you would like
me to apply. (Preferably with that one commit message clarified)

Regards,
Bjorn

> Ansuel Smith (17):
>   ARM: dts: qcom: add multiple missing pin definition for ipq8064
>   ARM: dts: qcom: add gsbi6 missing definition for ipq8064
>   ARM: dts: qcom: add missing rpm regulators and cells for ipq8064
>   ARM: dts: qcom: add missing snps,dwmac compatible for gmac ipq8064
>   ARM: dts: qcom: enable usb phy by default for ipq8064
>   ARM: dts: qcom: reduce pci IO size to 64K for ipq8064
>   ARM: dts: qcom: fix dtc warning for missing #address-cells for ipq8064
>   ARM: dts: qcom: add smem node for ipq8064
>   ARM: dts: qcom: add saw for l2 cache and kraitcc for ipq8064
>   ARM: dts: qcom: add sic non secure node for ipq8064
>   ARM: dts: qcom: fix and add some missing gsbi node for ipq8064
>   ARM: dts: qcom: add opp table for cpu and l2 for ipq8064
>   ARM: dts: qcom: add speedbin efuse nvmem binding
>   ARM: dts: qcom: add multiple missing binding for cpu and l2 for
>     ipq8064
>   ARM: dts: qcom: remove redundant binding from ipq8064 rb3011 dts
>   ARM: dts: qcom: add ipq8064-v2.0 dtsi
>   ARM: dts: qcom: add ipq8065 dtsi
> 
>  arch/arm/boot/dts/qcom-ipq8064-rb3011.dts |  17 --
>  arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi  |  70 +++++
>  arch/arm/boot/dts/qcom-ipq8064.dtsi       | 344 +++++++++++++++++++++-
>  arch/arm/boot/dts/qcom-ipq8065.dtsi       | 168 +++++++++++
>  4 files changed, 568 insertions(+), 31 deletions(-)
>  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
>  create mode 100644 arch/arm/boot/dts/qcom-ipq8065.dtsi
> 
> -- 
> 2.33.1
> 