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

[Roja Rani Yarubandi (Code Review)]

Roja Rani Yarubandi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35499 )

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

Patch Set 44:

(2 comments)

https://review.coreboot.org/c/coreboot/+/35499/44/src/soc/qualcomm/sc7180/qu... 
File src/soc/qualcomm/sc7180/qupv3_config.c:

https://review.coreboot.org/c/coreboot/+/35499/44/src/soc/qualcomm/sc7180/qu... 
PS44, Line 21: 							unsigned int mode)
> Why did the mode get added back here? Please explain what you're changing. […]

We added "mode" param back to be generic and portable in future. 
In future if other modes also supported, then it facilitates the application to change mode param alone from mainboard file.

https://review.coreboot.org/c/coreboot/+/35499/44/src/soc/qualcomm/sc7180/qu... 
PS44, Line 30: 	const char *filename[] = {
> I think you should follow this advice, it leads to slightly better code generation.

will make it static const