2023-07-04 10:53:16

by Komal Bajaj

Subject: [PATCH] arm64: dts: qcom: qdu1000: Add reserved gpio list

Add reserved gpio list for QDU1000 and QRU1000 SoCs
which will not be used by HLOS.

Signed-off-by: Komal Bajaj <[email protected]>
---
arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 4 ++++
arch/arm64/boot/dts/qcom/qru1000-idp.dts | 4 ++++
2 files changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
index 1d22f87fd238..0496e87ddfd5 100644
--- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
@@ -471,6 +471,10 @@ &sdhc {
status = "okay";
};

+&tlmm {
+ gpio-reserved-ranges = <28 2>;
+};
+
&uart7 {
status = "okay";
};
diff --git a/arch/arm64/boot/dts/qcom/qru1000-idp.dts b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
index 2cc893ae4d10..80dadd2f30a3 100644
--- a/arch/arm64/boot/dts/qcom/qru1000-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
@@ -448,6 +448,10 @@ &qupv3_id_0 {
status = "okay";
};

+&tlmm {
+ gpio-reserved-ranges = <28 2>;
+};
+
&uart7 {
status = "okay";
};
--
2.40.1

2023-07-04 11:14:35

by Konrad Dybcio

Subject: Re: [PATCH] arm64: dts: qcom: qdu1000: Add reserved gpio list

On 4.07.2023 12:22, Komal Bajaj wrote:
> Add reserved gpio list for QDU1000 and QRU1000 SoCs
> which will not be used by HLOS.
>
> Signed-off-by: Komal Bajaj <[email protected]>
> ---
Generally such changes to board files are split into per-board
commits.

Can you explain why these GPIOs will be inaccessible? Some secure
I2C peripheral, I'd guess?

Konrad
> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 4 ++++
> arch/arm64/boot/dts/qcom/qru1000-idp.dts | 4 ++++
> 2 files changed, 8 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> index 1d22f87fd238..0496e87ddfd5 100644
> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> @@ -471,6 +471,10 @@ &sdhc {
> status = "okay";
> };
>
> +&tlmm {
> + gpio-reserved-ranges = <28 2>;
> +};
> +
> &uart7 {
> status = "okay";
> };
> diff --git a/arch/arm64/boot/dts/qcom/qru1000-idp.dts b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
> index 2cc893ae4d10..80dadd2f30a3 100644
> --- a/arch/arm64/boot/dts/qcom/qru1000-idp.dts
> +++ b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
> @@ -448,6 +448,10 @@ &qupv3_id_0 {
> status = "okay";
> };
>
> +&tlmm {
> + gpio-reserved-ranges = <28 2>;
> +};
> +
> &uart7 {
> status = "okay";
> };

2023-07-04 12:03:49

by Komal Bajaj

Subject: Re: [PATCH] arm64: dts: qcom: qdu1000: Add reserved gpio list

On 7/4/2023 4:38 PM, Konrad Dybcio wrote:
> On 4.07.2023 12:22, Komal Bajaj wrote:
>> Add reserved gpio list for QDU1000 and QRU1000 SoCs
>> which will not be used by HLOS.
>>
>> Signed-off-by: Komal Bajaj <[email protected]>
>> ---
> Generally such changes to board files are split into per-board
> commits.

Sure, will split into per-board commits.

>
> Can you explain why these GPIOs will be inaccessible? Some secure
> I2C peripheral, I'd guess?

These GPIOs are needed by Modem subsystem.

Thanks
Komal

>
> Konrad
>> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 4 ++++
>> arch/arm64/boot/dts/qcom/qru1000-idp.dts | 4 ++++
>> 2 files changed, 8 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> index 1d22f87fd238..0496e87ddfd5 100644
>> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> @@ -471,6 +471,10 @@ &sdhc {
>> status = "okay";
>> };
>>
>> +&tlmm {
>> + gpio-reserved-ranges = <28 2>;
>> +};
>> +
>> &uart7 {
>> status = "okay";
>> };
>> diff --git a/arch/arm64/boot/dts/qcom/qru1000-idp.dts b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
>> index 2cc893ae4d10..80dadd2f30a3 100644
>> --- a/arch/arm64/boot/dts/qcom/qru1000-idp.dts
>> +++ b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
>> @@ -448,6 +448,10 @@ &qupv3_id_0 {
>> status = "okay";
>> };
>>
>> +&tlmm {
>> + gpio-reserved-ranges = <28 2>;
>> +};
>> +
>> &uart7 {
>> status = "okay";
>> };