Re: [PATCH] ARM: dts: qcom: msm8974: correct qfprom node reg

From: Konrad Dybcio
Date: Mon Jan 30 2023 - 13:31:33 EST




On 30.01.2023 19:20, luca@xxxxxxxxx wrote:
> From: Craig Tatlor <ctatlor97@xxxxxxxxx>
>
> The qfprom actually starts at 0xfc4b8000 instead of 0xfc4bc000 as
> defined previously. Adjust the tsens offsets accordingly.
>
> [luca@xxxxxxxxx: extract to standalone patch]
>
> Fixes: c59ffb519357 ("arm: dts: msm8974: Add thermal zones, tsens and qfprom nodes")
> Signed-off-by: Craig Tatlor <ctatlor97@xxxxxxxxx>
> Signed-off-by: Luca Weiss <luca@xxxxxxxxx>
> ---
Isn't this a raw vs ecc-corrected values problem?

Konrad
> arch/arm/boot/dts/qcom-msm8974.dtsi | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi
> index 8d216a3c0851..922d235c6065 100644
> --- a/arch/arm/boot/dts/qcom-msm8974.dtsi
> +++ b/arch/arm/boot/dts/qcom-msm8974.dtsi
> @@ -1132,16 +1132,16 @@ restart@fc4ab000 {
> reg = <0xfc4ab000 0x4>;
> };
>
> - qfprom: qfprom@fc4bc000 {
> + qfprom: qfprom@fc4b8000 {
> compatible = "qcom,msm8974-qfprom", "qcom,qfprom";
> - reg = <0xfc4bc000 0x1000>;
> + reg = <0xfc4b8000 0x7000>;
> #address-cells = <1>;
> #size-cells = <1>;
> - tsens_calib: calib@d0 {
> - reg = <0xd0 0x18>;
> + tsens_calib: calib@40d0 {
> + reg = <0x40d0 0x18>;
> };
> - tsens_backup: backup@440 {
> - reg = <0x440 0x10>;
> + tsens_backup: backup@4440 {
> + reg = <0x4440 0x10>;
> };
> };
>
>
> ---
> base-commit: 6d796c50f84ca79f1722bb131799e5a5710c4700
> change-id: 20230130-msm8974-qfprom-619c0e8f26eb
>
> Best regards,