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

[Stef van Os (Code Review)]

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

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

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/40405/2/src/soc/intel/cannonlake/ac... 
File src/soc/intel/cannonlake/acpi/serialio.asl:

https://review.coreboot.org/c/coreboot/+/40405/2/src/soc/intel/cannonlake/ac... 
PS2, Line 72: 
double newline

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

https://review.coreboot.org/c/coreboot/+/40405/2/src/soc/intel/cannonlake/pc... 
PS2, Line 24: 	printk(BIOS_ERR, "ACPIGEN: %s\n", acpi_device_name(dev));
should this be BIOS_ERR?