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