[PATCH 1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant

[PATCH 1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant

[Christian Marangi]

Add ipq8064-v2.0 dtsi variant that differ from original ipq8064 SoC for
some additional pcie, sata and usb configuration values, additional
reserved memory and serial output.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
---
 .../boot/dts/qcom-ipq8064-v2.0-smb208.dtsi    | 37 ++++++++++
 arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi      | 69 +++++++++++++++++++
 2 files changed, 106 insertions(+)
 create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
 create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi

diff --git a/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
new file mode 100644
index 000000000000..0442580b22de
--- /dev/null
+++ b/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
@@ -0,0 +1,37 @@
+// SPDX-License-Identifier: GPL-2.0
+
+#include "qcom-ipq8064-v2.0.dtsi"
+
+&rpm {
+	smb208_regulators: regulators {
+		compatible = "qcom,rpm-smb208-regulators";
+
+		smb208_s1a: s1a {
+			regulator-min-microvolt = <1050000>;
+			regulator-max-microvolt = <1150000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s1b: s1b {
+			regulator-min-microvolt = <1050000>;
+			regulator-max-microvolt = <1150000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s2a: s2a {
+			regulator-min-microvolt = < 800000>;
+			regulator-max-microvolt = <1250000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s2b: s2b {
+			regulator-min-microvolt = < 800000>;
+			regulator-max-microvolt = <1250000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+	};
+};
diff --git a/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
new file mode 100644
index 000000000000..2f117d576daf
--- /dev/null
+++ b/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
@@ -0,0 +1,69 @@
+// SPDX-License-Identifier: GPL-2.0
+
+#include "qcom-ipq8064.dtsi"
+
+/ {
+	model = "Qualcomm Technologies, Inc. IPQ8064-v2.0";
+
+	aliases {
+		serial0 = &gsbi4_serial;
+	};
+
+	chosen {
+		stdout-path = "serial0:115200n8";
+	};
+
+	reserved-memory {
+		#address-cells = <1>;
+		#size-cells = <1>;
+		ranges;
+
+		rsvd@41200000 {
+			reg = <0x41200000 0x300000>;
+			no-map;
+		};
+	};
+};
+
+&gsbi4 {
+	qcom,mode = <GSBI_PROT_I2C_UART>;
+	status = "okay";
+
+	serial@16340000 {
+		status = "okay";
+	};
+	/*
+	 * The i2c device on gsbi4 should not be enabled.
+	 * On ipq806x designs gsbi4 i2c is meant for exclusive
+	 * RPM usage. Turning this on in kernel manifests as
+	 * i2c failure for the RPM.
+	 */
+};
+
+&pcie0 {
+	compatible = "qcom,pcie-ipq8064-v2";
+};
+
+&pcie1 {
+	compatible = "qcom,pcie-ipq8064-v2";
+};
+
+&pcie2 {
+	compatible = "qcom,pcie-ipq8064-v2";
+};
+
+&sata {
+	ports-implemented = <0x1>;
+};
+
+&ss_phy_0 {
+	qcom,rx-eq = <2>;
+	qcom,tx-deamp_3_5db = <32>;
+	qcom,mpll = <5>;
+};
+
+&ss_phy_1 {
+	qcom,rx-eq = <2>;
+	qcom,tx-deamp_3_5db = <32>;
+	qcom,mpll = <5>;
+};
-- 
2.36.1

[PATCH 2/3] ARM: dts: qcom: ipq8064: add ipq8062 variant

[Christian Marangi]

ipq8062 SoC is based on ipq8064-v2.0 with lower supported freq, lack of
usb port and a reduced voltage output with the smb208 regulators.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
---
 arch/arm/boot/dts/qcom-ipq8062-smb208.dtsi | 37 ++++++++++++++++++++++
 arch/arm/boot/dts/qcom-ipq8062.dtsi        |  8 +++++
 2 files changed, 45 insertions(+)
 create mode 100644 arch/arm/boot/dts/qcom-ipq8062-smb208.dtsi
 create mode 100644 arch/arm/boot/dts/qcom-ipq8062.dtsi

diff --git a/arch/arm/boot/dts/qcom-ipq8062-smb208.dtsi b/arch/arm/boot/dts/qcom-ipq8062-smb208.dtsi
new file mode 100644
index 000000000000..9d06255104c7
--- /dev/null
+++ b/arch/arm/boot/dts/qcom-ipq8062-smb208.dtsi
@@ -0,0 +1,37 @@
+// SPDX-License-Identifier: GPL-2.0-only
+
+#include "qcom-ipq8062.dtsi"
+
+&rpm {
+	smb208_regulators: regulators {
+		compatible = "qcom,rpm-smb208-regulators";
+
+		smb208_s1a: s1a {
+			regulator-min-microvolt = <1050000>;
+			regulator-max-microvolt = <1150000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s1b: s1b {
+			regulator-min-microvolt = <1050000>;
+			regulator-max-microvolt = <1150000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s2a: s2a {
+			regulator-min-microvolt = < 800000>;
+			regulator-max-microvolt = <1150000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s2b: s2b {
+			regulator-min-microvolt = < 800000>;
+			regulator-max-microvolt = <1150000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+	};
+};
diff --git a/arch/arm/boot/dts/qcom-ipq8062.dtsi b/arch/arm/boot/dts/qcom-ipq8062.dtsi
new file mode 100644
index 000000000000..5d3ebd3e2e51
--- /dev/null
+++ b/arch/arm/boot/dts/qcom-ipq8062.dtsi
@@ -0,0 +1,8 @@
+// SPDX-License-Identifier: GPL-2.0-only
+
+#include "qcom-ipq8064-v2.0.dtsi"
+
+/ {
+	model = "Qualcomm Technologies, Inc. IPQ8062";
+	compatible = "qcom,ipq8062", "qcom,ipq8064";
+};
-- 
2.36.1

[PATCH 3/3] ARM: dts: qcom: ipq8064: add ipq8065 variant

[Christian Marangi]

ipq8065 SoC is based on ipq8064-v2.0 with a more clocked CPU and
an increased voltage output with the smb208 regulators.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
---
 arch/arm/boot/dts/qcom-ipq8065-smb208.dtsi | 37 ++++++++++++++++++++++
 arch/arm/boot/dts/qcom-ipq8065.dtsi        |  8 +++++
 2 files changed, 45 insertions(+)
 create mode 100644 arch/arm/boot/dts/qcom-ipq8065-smb208.dtsi
 create mode 100644 arch/arm/boot/dts/qcom-ipq8065.dtsi

diff --git a/arch/arm/boot/dts/qcom-ipq8065-smb208.dtsi b/arch/arm/boot/dts/qcom-ipq8065-smb208.dtsi
new file mode 100644
index 000000000000..803e6ff99ef8
--- /dev/null
+++ b/arch/arm/boot/dts/qcom-ipq8065-smb208.dtsi
@@ -0,0 +1,37 @@
+// SPDX-License-Identifier: GPL-2.0
+
+#include "qcom-ipq8065.dtsi"
+
+&rpm {
+	smb208_regulators: regulators {
+		compatible = "qcom,rpm-smb208-regulators";
+
+		smb208_s1a: s1a {
+			regulator-min-microvolt = <1050000>;
+			regulator-max-microvolt = <1150000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s1b: s1b {
+			regulator-min-microvolt = <1050000>;
+			regulator-max-microvolt = <1150000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s2a: s2a {
+			regulator-min-microvolt = <775000>;
+			regulator-max-microvolt = <1275000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+
+		smb208_s2b: s2b {
+			regulator-min-microvolt = <775000>;
+			regulator-max-microvolt = <1275000>;
+
+			qcom,switch-mode-frequency = <1200000>;
+		};
+	};
+};
diff --git a/arch/arm/boot/dts/qcom-ipq8065.dtsi b/arch/arm/boot/dts/qcom-ipq8065.dtsi
new file mode 100644
index 000000000000..ea49f6cc416d
--- /dev/null
+++ b/arch/arm/boot/dts/qcom-ipq8065.dtsi
@@ -0,0 +1,8 @@
+// SPDX-License-Identifier: GPL-2.0
+
+#include "qcom-ipq8064-v2.0.dtsi"
+
+/ {
+	model = "Qualcomm Technologies, Inc. IPQ8065";
+	compatible = "qcom,ipq8065", "qcom,ipq8064";
+};
-- 
2.36.1

Re: [PATCH 1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant

[Christian Marangi]

On Mon, Jul 18, 2022 at 06:18:24PM +0200, Christian Marangi wrote:
> Add ipq8064-v2.0 dtsi variant that differ from original ipq8064 SoC for
> some additional pcie, sata and usb configuration values, additional
> reserved memory and serial output.
> 
> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

Any news for this?

> ---
>  .../boot/dts/qcom-ipq8064-v2.0-smb208.dtsi    | 37 ++++++++++
>  arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi      | 69 +++++++++++++++++++
>  2 files changed, 106 insertions(+)
>  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
>  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
> 
> diff --git a/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
> new file mode 100644
> index 000000000000..0442580b22de
> --- /dev/null
> +++ b/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
> @@ -0,0 +1,37 @@
> +// SPDX-License-Identifier: GPL-2.0
> +
> +#include "qcom-ipq8064-v2.0.dtsi"
> +
> +&rpm {
> +	smb208_regulators: regulators {
> +		compatible = "qcom,rpm-smb208-regulators";
> +
> +		smb208_s1a: s1a {
> +			regulator-min-microvolt = <1050000>;
> +			regulator-max-microvolt = <1150000>;
> +
> +			qcom,switch-mode-frequency = <1200000>;
> +		};
> +
> +		smb208_s1b: s1b {
> +			regulator-min-microvolt = <1050000>;
> +			regulator-max-microvolt = <1150000>;
> +
> +			qcom,switch-mode-frequency = <1200000>;
> +		};
> +
> +		smb208_s2a: s2a {
> +			regulator-min-microvolt = < 800000>;
> +			regulator-max-microvolt = <1250000>;
> +
> +			qcom,switch-mode-frequency = <1200000>;
> +		};
> +
> +		smb208_s2b: s2b {
> +			regulator-min-microvolt = < 800000>;
> +			regulator-max-microvolt = <1250000>;
> +
> +			qcom,switch-mode-frequency = <1200000>;
> +		};
> +	};
> +};
> diff --git a/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
> new file mode 100644
> index 000000000000..2f117d576daf
> --- /dev/null
> +++ b/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
> @@ -0,0 +1,69 @@
> +// SPDX-License-Identifier: GPL-2.0
> +
> +#include "qcom-ipq8064.dtsi"
> +
> +/ {
> +	model = "Qualcomm Technologies, Inc. IPQ8064-v2.0";
> +
> +	aliases {
> +		serial0 = &gsbi4_serial;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	reserved-memory {
> +		#address-cells = <1>;
> +		#size-cells = <1>;
> +		ranges;
> +
> +		rsvd@41200000 {
> +			reg = <0x41200000 0x300000>;
> +			no-map;
> +		};
> +	};
> +};
> +
> +&gsbi4 {
> +	qcom,mode = <GSBI_PROT_I2C_UART>;
> +	status = "okay";
> +
> +	serial@16340000 {
> +		status = "okay";
> +	};
> +	/*
> +	 * The i2c device on gsbi4 should not be enabled.
> +	 * On ipq806x designs gsbi4 i2c is meant for exclusive
> +	 * RPM usage. Turning this on in kernel manifests as
> +	 * i2c failure for the RPM.
> +	 */
> +};
> +
> +&pcie0 {
> +	compatible = "qcom,pcie-ipq8064-v2";
> +};
> +
> +&pcie1 {
> +	compatible = "qcom,pcie-ipq8064-v2";
> +};
> +
> +&pcie2 {
> +	compatible = "qcom,pcie-ipq8064-v2";
> +};
> +
> +&sata {
> +	ports-implemented = <0x1>;
> +};
> +
> +&ss_phy_0 {
> +	qcom,rx-eq = <2>;
> +	qcom,tx-deamp_3_5db = <32>;
> +	qcom,mpll = <5>;
> +};
> +
> +&ss_phy_1 {
> +	qcom,rx-eq = <2>;
> +	qcom,tx-deamp_3_5db = <32>;
> +	qcom,mpll = <5>;
> +};
> -- 
> 2.36.1
> 

-- 
	Ansuel

Re: [PATCH 1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant

[Konrad Dybcio]

On 7.08.2022 15:00, Christian Marangi wrote:
> On Mon, Jul 18, 2022 at 06:18:24PM +0200, Christian Marangi wrote:
>> Add ipq8064-v2.0 dtsi variant that differ from original ipq8064 SoC for
>> some additional pcie, sata and usb configuration values, additional
>> reserved memory and serial output.
>>
>> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
> 
> Any news for this?Unless Qualcomm naming was different back then, you should simply merge
all of these changes into ipq8064.dtsi, as v1 is often the pre-production,
internal chip revision and only the last one (or the last and second-last)
are shipped in production devices.

Konrad
> 
>> ---
>>  .../boot/dts/qcom-ipq8064-v2.0-smb208.dtsi    | 37 ++++++++++
>>  arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi      | 69 +++++++++++++++++++
>>  2 files changed, 106 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
>>  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
>>
>> diff --git a/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
>> new file mode 100644
>> index 000000000000..0442580b22de
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
>> @@ -0,0 +1,37 @@
>> +// SPDX-License-Identifier: GPL-2.0
>> +
>> +#include "qcom-ipq8064-v2.0.dtsi"
>> +
>> +&rpm {
>> +	smb208_regulators: regulators {
>> +		compatible = "qcom,rpm-smb208-regulators";
>> +
>> +		smb208_s1a: s1a {
>> +			regulator-min-microvolt = <1050000>;
>> +			regulator-max-microvolt = <1150000>;
>> +
>> +			qcom,switch-mode-frequency = <1200000>;
>> +		};
>> +
>> +		smb208_s1b: s1b {
>> +			regulator-min-microvolt = <1050000>;
>> +			regulator-max-microvolt = <1150000>;
>> +
>> +			qcom,switch-mode-frequency = <1200000>;
>> +		};
>> +
>> +		smb208_s2a: s2a {
>> +			regulator-min-microvolt = < 800000>;
>> +			regulator-max-microvolt = <1250000>;
>> +
>> +			qcom,switch-mode-frequency = <1200000>;
>> +		};
>> +
>> +		smb208_s2b: s2b {
>> +			regulator-min-microvolt = < 800000>;
>> +			regulator-max-microvolt = <1250000>;
>> +
>> +			qcom,switch-mode-frequency = <1200000>;
>> +		};
>> +	};
>> +};
>> diff --git a/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
>> new file mode 100644
>> index 000000000000..2f117d576daf
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
>> @@ -0,0 +1,69 @@
>> +// SPDX-License-Identifier: GPL-2.0
>> +
>> +#include "qcom-ipq8064.dtsi"
>> +
>> +/ {
>> +	model = "Qualcomm Technologies, Inc. IPQ8064-v2.0";
>> +
>> +	aliases {
>> +		serial0 = &gsbi4_serial;
>> +	};
>> +
>> +	chosen {
>> +		stdout-path = "serial0:115200n8";
>> +	};
>> +
>> +	reserved-memory {
>> +		#address-cells = <1>;
>> +		#size-cells = <1>;
>> +		ranges;
>> +
>> +		rsvd@41200000 {
>> +			reg = <0x41200000 0x300000>;
>> +			no-map;
>> +		};
>> +	};
>> +};
>> +
>> +&gsbi4 {
>> +	qcom,mode = <GSBI_PROT_I2C_UART>;
>> +	status = "okay";
>> +
>> +	serial@16340000 {
>> +		status = "okay";
>> +	};
>> +	/*
>> +	 * The i2c device on gsbi4 should not be enabled.
>> +	 * On ipq806x designs gsbi4 i2c is meant for exclusive
>> +	 * RPM usage. Turning this on in kernel manifests as
>> +	 * i2c failure for the RPM.
>> +	 */
>> +};
>> +
>> +&pcie0 {
>> +	compatible = "qcom,pcie-ipq8064-v2";
>> +};
>> +
>> +&pcie1 {
>> +	compatible = "qcom,pcie-ipq8064-v2";
>> +};
>> +
>> +&pcie2 {
>> +	compatible = "qcom,pcie-ipq8064-v2";
>> +};
>> +
>> +&sata {
>> +	ports-implemented = <0x1>;
>> +};
>> +
>> +&ss_phy_0 {
>> +	qcom,rx-eq = <2>;
>> +	qcom,tx-deamp_3_5db = <32>;
>> +	qcom,mpll = <5>;
>> +};
>> +
>> +&ss_phy_1 {
>> +	qcom,rx-eq = <2>;
>> +	qcom,tx-deamp_3_5db = <32>;
>> +	qcom,mpll = <5>;
>> +};
>> -- 
>> 2.36.1
>>
> 

Re: [PATCH 1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant

[Christian Marangi]

On Mon, Aug 08, 2022 at 12:55:36PM +0200, Konrad Dybcio wrote:
> 
> 
> On 7.08.2022 15:00, Christian Marangi wrote:
> > On Mon, Jul 18, 2022 at 06:18:24PM +0200, Christian Marangi wrote:
> >> Add ipq8064-v2.0 dtsi variant that differ from original ipq8064 SoC for
> >> some additional pcie, sata and usb configuration values, additional
> >> reserved memory and serial output.
> >>
> >> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
> > 
> > Any news for this?Unless Qualcomm naming was different back then, you should simply merge
> all of these changes into ipq8064.dtsi, as v1 is often the pre-production,
> internal chip revision and only the last one (or the last and second-last)
> are shipped in production devices.
>

Mhh, this is not the case, there are dev board based on v1 and we also
have some device based on v1 (that have some difference for pci and usb)
One example is a Netgear r7500 where we have 2 revision one based on
ipq8064-v1.0 and one based on ipq8064-v2.0.

> > 
> >> ---
> >>  .../boot/dts/qcom-ipq8064-v2.0-smb208.dtsi    | 37 ++++++++++
> >>  arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi      | 69 +++++++++++++++++++
> >>  2 files changed, 106 insertions(+)
> >>  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
> >>  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
> >>
> >> diff --git a/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
> >> new file mode 100644
> >> index 000000000000..0442580b22de
> >> --- /dev/null
> >> +++ b/arch/arm/boot/dts/qcom-ipq8064-v2.0-smb208.dtsi
> >> @@ -0,0 +1,37 @@
> >> +// SPDX-License-Identifier: GPL-2.0
> >> +
> >> +#include "qcom-ipq8064-v2.0.dtsi"
> >> +
> >> +&rpm {
> >> +	smb208_regulators: regulators {
> >> +		compatible = "qcom,rpm-smb208-regulators";
> >> +
> >> +		smb208_s1a: s1a {
> >> +			regulator-min-microvolt = <1050000>;
> >> +			regulator-max-microvolt = <1150000>;
> >> +
> >> +			qcom,switch-mode-frequency = <1200000>;
> >> +		};
> >> +
> >> +		smb208_s1b: s1b {
> >> +			regulator-min-microvolt = <1050000>;
> >> +			regulator-max-microvolt = <1150000>;
> >> +
> >> +			qcom,switch-mode-frequency = <1200000>;
> >> +		};
> >> +
> >> +		smb208_s2a: s2a {
> >> +			regulator-min-microvolt = < 800000>;
> >> +			regulator-max-microvolt = <1250000>;
> >> +
> >> +			qcom,switch-mode-frequency = <1200000>;
> >> +		};
> >> +
> >> +		smb208_s2b: s2b {
> >> +			regulator-min-microvolt = < 800000>;
> >> +			regulator-max-microvolt = <1250000>;
> >> +
> >> +			qcom,switch-mode-frequency = <1200000>;
> >> +		};
> >> +	};
> >> +};
> >> diff --git a/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi b/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
> >> new file mode 100644
> >> index 000000000000..2f117d576daf
> >> --- /dev/null
> >> +++ b/arch/arm/boot/dts/qcom-ipq8064-v2.0.dtsi
> >> @@ -0,0 +1,69 @@
> >> +// SPDX-License-Identifier: GPL-2.0
> >> +
> >> +#include "qcom-ipq8064.dtsi"
> >> +
> >> +/ {
> >> +	model = "Qualcomm Technologies, Inc. IPQ8064-v2.0";
> >> +
> >> +	aliases {
> >> +		serial0 = &gsbi4_serial;
> >> +	};
> >> +
> >> +	chosen {
> >> +		stdout-path = "serial0:115200n8";
> >> +	};
> >> +
> >> +	reserved-memory {
> >> +		#address-cells = <1>;
> >> +		#size-cells = <1>;
> >> +		ranges;
> >> +
> >> +		rsvd@41200000 {
> >> +			reg = <0x41200000 0x300000>;
> >> +			no-map;
> >> +		};
> >> +	};
> >> +};
> >> +
> >> +&gsbi4 {
> >> +	qcom,mode = <GSBI_PROT_I2C_UART>;
> >> +	status = "okay";
> >> +
> >> +	serial@16340000 {
> >> +		status = "okay";
> >> +	};
> >> +	/*
> >> +	 * The i2c device on gsbi4 should not be enabled.
> >> +	 * On ipq806x designs gsbi4 i2c is meant for exclusive
> >> +	 * RPM usage. Turning this on in kernel manifests as
> >> +	 * i2c failure for the RPM.
> >> +	 */
> >> +};
> >> +
> >> +&pcie0 {
> >> +	compatible = "qcom,pcie-ipq8064-v2";
> >> +};
> >> +
> >> +&pcie1 {
> >> +	compatible = "qcom,pcie-ipq8064-v2";
> >> +};
> >> +
> >> +&pcie2 {
> >> +	compatible = "qcom,pcie-ipq8064-v2";
> >> +};
> >> +
> >> +&sata {
> >> +	ports-implemented = <0x1>;
> >> +};
> >> +
> >> +&ss_phy_0 {
> >> +	qcom,rx-eq = <2>;
> >> +	qcom,tx-deamp_3_5db = <32>;
> >> +	qcom,mpll = <5>;
> >> +};
> >> +
> >> +&ss_phy_1 {
> >> +	qcom,rx-eq = <2>;
> >> +	qcom,tx-deamp_3_5db = <32>;
> >> +	qcom,mpll = <5>;
> >> +};
> >> -- 
> >> 2.36.1
> >>
> > 

-- 
	Ansuel

Re: [PATCH 1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant

[Konrad Dybcio]

On 8.08.2022 15:35, Christian Marangi wrote:
> On Mon, Aug 08, 2022 at 12:55:36PM +0200, Konrad Dybcio wrote:
>>
>>
>> On 7.08.2022 15:00, Christian Marangi wrote:
>>> On Mon, Jul 18, 2022 at 06:18:24PM +0200, Christian Marangi wrote:
>>>> Add ipq8064-v2.0 dtsi variant that differ from original ipq8064 SoC for
>>>> some additional pcie, sata and usb configuration values, additional
>>>> reserved memory and serial output.
>>>>
>>>> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
>>>
>>> Any news for this?Unless Qualcomm naming was different back then, you should simply merge
>> all of these changes into ipq8064.dtsi, as v1 is often the pre-production,
>> internal chip revision and only the last one (or the last and second-last)
>> are shipped in production devices.
>>
> 
> Mhh, this is not the case, there are dev board based on v1 and we also
> have some device based on v1 (that have some difference for pci and usb)
> One example is a Netgear r7500 where we have 2 revision one based on
> ipq8064-v1.0 and one based on ipq8064-v2.0.
Very interesting.. but if there really are devices shipping with v1, I guess
it's the correct thing to keep both.

Konrad

Re: [PATCH 1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant

[Christian Marangi]

On Thu, Aug 11, 2022 at 02:48:29PM +0200, Konrad Dybcio wrote:
> 
> 
> On 8.08.2022 15:35, Christian Marangi wrote:
> > On Mon, Aug 08, 2022 at 12:55:36PM +0200, Konrad Dybcio wrote:
> >>
> >>
> >> On 7.08.2022 15:00, Christian Marangi wrote:
> >>> On Mon, Jul 18, 2022 at 06:18:24PM +0200, Christian Marangi wrote:
> >>>> Add ipq8064-v2.0 dtsi variant that differ from original ipq8064 SoC for
> >>>> some additional pcie, sata and usb configuration values, additional
> >>>> reserved memory and serial output.
> >>>>
> >>>> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
> >>>
> >>> Any news for this?Unless Qualcomm naming was different back then, you should simply merge
> >> all of these changes into ipq8064.dtsi, as v1 is often the pre-production,
> >> internal chip revision and only the last one (or the last and second-last)
> >> are shipped in production devices.
> >>
> > 
> > Mhh, this is not the case, there are dev board based on v1 and we also
> > have some device based on v1 (that have some difference for pci and usb)
> > One example is a Netgear r7500 where we have 2 revision one based on
> > ipq8064-v1.0 and one based on ipq8064-v2.0.
> Very interesting.. but if there really are devices shipping with v1, I guess
> it's the correct thing to keep both.
> 
> Konrad

Yep, that is the case. Main concern here is the fact that we have to
have smb208 dtsi variant for each revision and we can't reuse v2 for
ipq8065 and ipq8062 as they have different opp for the CPU core.
(currently not present but they will be added in the future when we will
finally have a correct krait cache devfreq driver)

-- 
	Ansuel

Re: (subset) [PATCH 1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant

[Bjorn Andersson]

On Mon, 18 Jul 2022 18:18:24 +0200, Christian Marangi wrote:
> Add ipq8064-v2.0 dtsi variant that differ from original ipq8064 SoC for
> some additional pcie, sata and usb configuration values, additional
> reserved memory and serial output.
> 
> 

Applied, thanks!

[1/3] ARM: dts: qcom: ipq8064: add v2 dtsi variant
      commit: a9f2cd80ee4669c851d6953fbbb592a3ba44df9f
[2/3] ARM: dts: qcom: ipq8064: add ipq8062 variant
      commit: 58907a1cae53dd5f91a7dfb17ac8de10c60c32fd
[3/3] ARM: dts: qcom: ipq8064: add ipq8065 variant
      commit: 12e621362be39350167ede99542256e768ecbfd6

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>