[PATCH 1/5] arch: arm: dts: k3-j784s4-r5-evm: Add missing fss range

[Andrew Davis]

On 8/8/24 4:29 PM, Jonathan Humphreys wrote:
> Upstream DTS added explicit ranges to the fss node.  It did not include the
> 32 bit memory space needed by the R5 to access OSPI.  With the upstream DTS
> sync, OSPI boot no longer works.
> 
> Adding the missing range here. It is also being added in the upstream DTS,
> so after the next upstream DTS sync, this patch can be removed. See
> f062a015f4 (arm64: dts: ti: k3-j784s4: Use exact ranges for FSS node)
> 
> Fixes: 5024a96db8e (Subtree merge tag 'v6.10-dts' of devicetree-rebasing
> repo [1] into dts/upstream)
> 
> Signed-off-by: Jonathan Humphreys <j-humphreys at ti.com>
> ---
>   arch/arm/dts/k3-j784s4-r5-evm.dts | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/arch/arm/dts/k3-j784s4-r5-evm.dts b/arch/arm/dts/k3-j784s4-r5-evm.dts
> index bef4573d3d2..26af72d73f4 100644
> --- a/arch/arm/dts/k3-j784s4-r5-evm.dts
> +++ b/arch/arm/dts/k3-j784s4-r5-evm.dts
> @@ -97,6 +97,12 @@
>   	      <0x0 0x58000000 0x0 0x8000000>;
>   };
>   
> +&fss {
> +	/* enable ranges missing from the fss node */
> +	ranges = <0x0 0x47000000 0x0 0x47000000 0x0 0x00068400>,
> +					 <0x0 0x50000000 0x0 0x50000000 0x0 0x08000000>;

Looks like way too many tabs, do you have your editor set to 8 space tabs for DTS files?

Otherwise, LGTM,

Reviewed-by: Andrew Davis <afd at ti.com>

> +};
> +
>   &mcu_ringacc {
>   	ti,sci = <&dm_tifs>;
>   };