[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Add support for UART2 on PCH-H

[Justin van Son (Code Review)]

Justin van Son has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40405 )

Change subject: soc/intel/cannonlake: Add support for UART2 on PCH-H
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/40405/4/src/soc/intel/cannonlake/pc... 
File src/soc/intel/cannonlake/pch_h.c:

https://review.coreboot.org/c/coreboot/+/40405/4/src/soc/intel/cannonlake/pc... 
PS4, Line 20: static void pch_h_uart2_fill_ssdt(struct device *dev)
Shouldn't this be const struct device?