Change in ...coreboot[master]: sdm845: Add QUPv3 FW load & config

[Akash Asthana (Code Review)]

Akash Asthana has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/25372 )

Change subject: sdm845: Add QUPv3 FW load & config
......................................................................

Patch Set 73:

(1 comment)

https://review.coreboot.org/#/c/25372/6/src/mainboard/google/cheza/qupv3_con... 
File src/mainboard/google/cheza/qupv3_config.c:

https://review.coreboot.org/#/c/25372/6/src/mainboard/google/cheza/qupv3_con... 
PS6, Line 18: struct se_cfg se_mappings[QUPV3_SE_MAX] =
> This is the first SoC we ever had that has to load firmware to get basic peripherals working, so the […]
> > > > To load QUPV3 firmware from respective driver init function

We have been discussing with the Firmware/HW team and came to the conclusion that this can’t be supported.
Right now, the FW loading sequence is fixed and doesn’t allow initialization of few Registers on every FW load separately. It’s recommended to be done only once and hence forces to load firmware at one go.