[PATCH v2 5/5] rockchip: rock-pi-4: dts: spi: Make the index of the spi flash the same in SPL and U-Boot proper

[Quentin Schulz]

Hi Xavier,

On 7/15/22 18:30, Xavier Drudis Ferran wrote:
> Spi0 is not needed in SPL and SPL could be a little smaller without it,
> but then the SF_DEFAULT_BOOT would have to be 0 to refer to spi1, and
> that's confusing, because once U-Boot proper runs, it numbers the bus 1.
> 
> Add spi0 to the pre-reloc and spl trees so that the flash is always
> connected to bus 1.
> 

Mmmm... Could we instead make U-Boot use the bus number from the alias 
in the aliases DT node? I think the mmc subsystem does this already and 
it would mean we don't need to enable unnecessary devices. Also, relying 
on boot order for the bus number is brittle in Linux, I don't know about 
U-Boot, but if we can avoid this assumption, it'd be great :)

See: 
https://source.denx.de/u-boot/u-boot/-/commit/2243d19e5618122d9d7aba23eb51f63f2719dba5 
for how to do it today?

> Changes since v1:
>     - new patch in v2.
> 

Your patch series got sent with each commit in their individual thread 
instead of the usual way, with 1/X patch as the first mail in the 
thread. I think this could make this difficult to merge for the 
maintainer, could you please try to put all your patch into a single 
thread for the next version/patch series you will send? `git send-email 
*.patch` should do that for you by default I'm pretty sure.

What happened with your patches:
https://lore.kernel.org/u-boot/20220715162712.GB2143@begut/T/#u
https://lore.kernel.org/u-boot/20220715162802.GC2143@begut/T/#u

What should have had happened:
https://lore.kernel.org/u-boot/20220715163435.1725-1-afd@ti.com/T/#m6339d543cec31cf6a30355d0e44fd3b99492a30d

Cheers,
Quentin