Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/19774 )
Change subject: soc/marvell/armada38x: Move spi driver to use spi_bus_map ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/#/c/19774/3/src/soc/marvell/armada38x/spi.c File src/soc/marvell/armada38x/spi.c:
Line 492: .bus_end = 1,
How do you actually know there are 2 buses? I can't for the life of me figu
Yeah, it is a bit tricky since none of the mainboards actually use this SoC. I looked up the datasheet for the SoC here:https://www.marvell.com/embedded-processors/armada-38x/ and found that it refers to two SPI ports 0 and 1 in the hardware spec and the functional spec provides register definitions for two SPI controllers SPI0 and SPI1.
-
Furquan Shaikh (Code Review)