2023-05-26 16:13:35

by Kathiravan Thirumoorthy

Subject: [PATCH] arm64: dts: qcom: ipq6018: correct the qrng node name

qrng node address is mentioned incorrectly. Lets fix it.

Reported-by: Dmitry Baryshkov <[email protected]>
Signed-off-by: Kathiravan T <[email protected]>
---
Note: Only compilation and dtbs_check is checked. Also dependent on the
below series, no functional dependencies
https://lore.kernel.org/linux-arm-msm/[email protected]/T/#t

arch/arm64/boot/dts/qcom/ipq6018.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
index 0f6d6c6daed2..5d2cc0caf5a1 100644
--- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
@@ -309,7 +309,7 @@
#size-cells = <1>;
};

- prng: qrng@e1000 {
+ prng: qrng@e3000 {
compatible = "qcom,prng-ee";
reg = <0x0 0x000e3000 0x0 0x1000>;
clocks = <&gcc GCC_PRNG_AHB_CLK>;
--
2.17.1

2023-05-30 14:59:09

by Krzysztof Kozlowski

Subject: Re: [PATCH] arm64: dts: qcom: ipq6018: correct the qrng node name

On 26/05/2023 18:11, Kathiravan T wrote:
> qrng node address is mentioned incorrectly. Lets fix it.
>
> Reported-by: Dmitry Baryshkov <[email protected]>
> Signed-off-by: Kathiravan T <[email protected]>
> ---
> Note: Only compilation and dtbs_check is checked. Also dependent on the
> below series, no functional dependencies
> https://lore.kernel.org/linux-arm-msm/[email protected]/T/#t
>
> arch/arm64/boot/dts/qcom/ipq6018.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> index 0f6d6c6daed2..5d2cc0caf5a1 100644
> --- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> @@ -309,7 +309,7 @@
> #size-cells = <1>;
> };
>
> - prng: qrng@e1000 {
> + prng: qrng@e3000 {

This was fixed some time ago:

https://lore.kernel.org/all/[email protected]/

Best regards,
Krzysztof

2023-06-01 13:14:03

by Kathiravan Thirumoorthy

Subject: Re: [PATCH] arm64: dts: qcom: ipq6018: correct the qrng node name

On 5/30/2023 8:14 PM, Krzysztof Kozlowski wrote:
> On 26/05/2023 18:11, Kathiravan T wrote:
>> qrng node address is mentioned incorrectly. Lets fix it.
>>
>> Reported-by: Dmitry Baryshkov <[email protected]>
>> Signed-off-by: Kathiravan T <[email protected]>
>> ---
>> Note: Only compilation and dtbs_check is checked. Also dependent on the
>> below series, no functional dependencies
>> https://lore.kernel.org/linux-arm-msm/[email protected]/T/#t
>>
>> arch/arm64/boot/dts/qcom/ipq6018.dtsi | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
>> index 0f6d6c6daed2..5d2cc0caf5a1 100644
>> --- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
>> @@ -309,7 +309,7 @@
>> #size-cells = <1>;
>> };
>>
>> - prng: qrng@e1000 {
>> + prng: qrng@e3000 {
> This was fixed some time ago:
>
> https://lore.kernel.org/all/[email protected]/


Thanks, I didn't notice your patch.


>
> Best regards,
> Krzysztof
>