[coreboot-gerrit] Change in coreboot[master]: sc7180: Add QUPv3 FW load & config

[Taniya Das (Code Review)]

Taniya Das has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35499 )

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

Patch Set 41:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35499/40/src/soc/qualcomm/sc7180/in... 
File src/soc/qualcomm/sc7180/include/soc/qupv3_config.h:

https://review.coreboot.org/c/coreboot/+/35499/40/src/soc/qualcomm/sc7180/in... 
PS40, Line 54: #define QUPV3_UART_SRC_HZ					7372800
> Again, this should also be used for where 7372800 appears in clock.c. […]

Hi Julius,

In the cases where the frequency needs to be updated for UART operation the clock driver needs to be updated.
Do you think it is a good idea/way to define it in clock code?

Please let me know your comments.