[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
> Also kconfig has better readability over macro, hence in my personal choice i really like kconfig. […]

Common code implementation is different from this. In common code, you want each SoC to provide this info to common code using Kconfig. IIUC, here, the usage is pretty much restricted to defining size of SerialIoUartMode in chip.h.