[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: Add max uart config

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31213 )

Change subject: soc/intel/cannonlake: Add max uart config
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/31213/1/src/soc/intel/cannonlake/Kconfig 
File src/soc/intel/cannonlake/Kconfig:

https://review.coreboot.org/#/c/31213/1/src/soc/intel/cannonlake/Kconfig@190 
PS1, Line 190: SOC_INTEL_UART_DEV_MAX
> its almost same kind of FSP UPD change as ICL where we will fill those LPSS UPDs as below […]

Right. I meant does it really need to be a Kconfig option though? Doing something like "#define SOC_INTEL_UART_DEV_MAX 3" would be good enough?