Tom Rini <tr...@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-humphr...@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@0 {
>> +            bootph-all;
>> +
>> +            partitions {
>> +                    bootph-all;
>> +
>> +                    partition@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

Reply via email to