RE: [PATCH 1/1] arm64: dts: imx93: Fix anatop node size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> Subject: [PATCH 1/1] arm64: dts: imx93: Fix anatop node size
> 
> Although the memory map of i.MX93 reference manual rev. 2 claims that
> analog top has start address of 0x44480000 and end address of 0x4448ffff,
> this overlaps with TMU memory area starting at 0x44482000, as stated in
> section 73.6.1.
> As PLL configuration registers start at addresses up to 0x44481400, as used by
> clk-imx93, reduce the anatop size to 0x2000, so exclude the TMU area but
> keep all PLL registers inside.
> 
> Fixes: ec8b5b5058ea ("arm64: dts: freescale: Add i.MX93 dtsi support")
> Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx>

Yes, the anatop size should be changed. Found the same issue after rebase our downstream kernel to 6.5.0-rc1.

Reviewed-by: Jacky Bai <ping.bai@xxxxxxx>
BR
> ---
> Unfortunately TMU isn't listed in memory map, so it was a bit of guess work.
> 
> This fixes the probe failure of qoriq_thermal:
> qoriq_thermal 44482000.tmu: can't request region for resource [mem
> 0x44482000-0x44482fff] qoriq_thermal 44482000.tmu: Failed to get memory
> region
> qoriq_thermal: probe of 44482000.tmu failed with error -16
> 
> /proc/iomem also shows it's claimed by anatop:
> > 44480000-4448ffff : 44450000.clock-controller anatop@44480000
> 
>  arch/arm64/boot/dts/freescale/imx93.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx93.dtsi
> b/arch/arm64/boot/dts/freescale/imx93.dtsi
> index 4ec9df78f2050..6f85a05ee7e1a 100644
> --- a/arch/arm64/boot/dts/freescale/imx93.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx93.dtsi
> @@ -379,7 +379,7 @@ mediamix: power-domain@44462400 {
> 
>  			anatop: anatop@44480000 {
>  				compatible = "fsl,imx93-anatop", "syscon";
> -				reg = <0x44480000 0x10000>;
> +				reg = <0x44480000 0x2000>;
>  			};
> 
>  			tmu: tmu@44482000 {
> --
> 2.34.1





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux