On Fri, Feb 21, 2025 at 08:19:40AM -0600, Tom Rini wrote: > On Thu, Feb 20, 2025 at 06:30:18PM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 20 Feb 2025 at 13:40, Tom Rini <[email protected]> wrote: > > > > > > On Thu, Feb 20, 2025 at 11:13:34AM -0700, Simon Glass wrote: [snip] > > > I will look at "splg4" once it's somewhere on source.denx.de and I can > > > look at it, and refrain from otherwise assuming how it solves the > > > problems I had seen previously. > > > > I pushed an updated version to dm/splg-working but it is not very > > updated. Still needs more work. > > Thanks.
So, after doing the remaining CONFIG_TEXT_BASE -> CONFIG_PPL_TEXT_BASE
changes, here's another example of the problem with your approach. What
stops xilinx_zynqmp_kria from building in splg-working is that
BUTTON was missing from scripts/conf_nospl. Annoyingly, a mrproper (or
since I always use O=, rm -rf) is needed for changes there to be picked
up, but that's maybe just a missing Makefile dependency line. But that
just makes it easier to see the next problem, which I don't see the
answer to. For PPL, we can build drivers/spi/zynqmp_gqspi.o just fine.
For SPL however:
CC spl/drivers/spi/zynqmp_gqspi.o
/home/trini/work/u-boot/u-boot/drivers/spi/zynqmp_gqspi.c: In function
'zynqmp_qspi_of_to_plat':
/home/trini/work/u-boot/u-boot/drivers/spi/zynqmp_gqspi.c:203:22: warning: cast
to pointer from integer of different size [-Wint-to-pointer-cast]
203 | plat->regs = (struct zynqmp_qspi_regs *)(dev_read_addr(bus) +
| ^
/home/trini/work/u-boot/u-boot/drivers/spi/zynqmp_gqspi.c:205:26: warning: cast
to pointer from integer of different size [-Wint-to-pointer-cast]
205 | plat->dma_regs = (struct zynqmp_qspi_dma_regs *)
| ^
And I don't see, really, what's even getting us down this error path.
--
Tom
signature.asc
Description: PGP signature
