[PATCH 2/2] arm: dts: k3-am642-evm/sk: Enable OSPI support in SPL

[Jon Humphreys]

Tom Rini <trini at konsulko.com> writes:

> On Thu, Feb 22, 2024 at 11:24:38PM -0600, Jonathan Humphreys wrote:
>> Add bootph DT tags to enable OSPI in SPL.
>> Set OSPI regs for R5 SPL to address OSPI's boot region.
>> 
>> Signed-off-by: Jonathan Humphreys <j-humphreys at ti.com>
>> ---
>>  arch/arm/dts/k3-am642-evm-u-boot.dtsi | 24 ++++++++++++++++++++++++
>>  arch/arm/dts/k3-am642-r5-evm.dts      |  5 +++++
>>  arch/arm/dts/k3-am642-r5-sk.dts       |  5 +++++
>>  arch/arm/dts/k3-am642-sk-u-boot.dtsi  | 24 ++++++++++++++++++++++++
>>  4 files changed, 58 insertions(+)
>> 
>> diff --git a/arch/arm/dts/k3-am642-evm-u-boot.dtsi b/arch/arm/dts/k3-am642-evm-u-boot.dtsi
>> index b843078243..f7ab9f9a30 100644
>> --- a/arch/arm/dts/k3-am642-evm-u-boot.dtsi
>> +++ b/arch/arm/dts/k3-am642-evm-u-boot.dtsi
>> @@ -182,3 +182,27 @@
>>  &cpsw_port2 {
>>  	status = "disabled";
>>  };
>> +
>> +&ospi0_pins_default {
>> +	bootph-all;
>> +};
>> +
>> +&fss {
>> +	bootph-all;
>> +};
>> +
>> +&ospi0 {
>> +	bootph-all;
>> +
>> +	flash at 0 {
>> +		bootph-all;
>> +
>> +		partitions {
>> +			bootph-all;
>> +
>> +			partition at 3fc0000 {
>> +				bootph-all;
>> +			};
>> +		};
>> +	};
>> +};
>
> Do we really need these tags on the partition table? And if so, do we
> need them in main U-Boot, or just SPL?
>

Tom, these are the OSPI phy calibration patterns.  I added them as they
are in the J721e u-boot.dtsi and both use the same OSPI interface.

However, I dug into this more and the upstream Cadence OSPI driver isn't
calibrating the phys.

I will respin this series to remove the .dtsi nodes, and add a patch to
the J721e series to remove it from there as well.

Thanks for catching.

> -- 
> Tom