[coreboot-gerrit] [M] Change in coreboot[main]: soc/mediatek: PCI: Fix translation window

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Hung-Te Lin, Jianjun Wang, Nico Huber, Paul Menzel, Yidi Lin.

Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78044?usp=email )

The change is no longer submittable: All-Comments-Resolved is unsatisfied now.

Change subject: soc/mediatek: PCI: Fix translation window
......................................................................

Patch Set 14:

(1 comment)

File src/soc/mediatek/common/pcie.c:

https://review.coreboot.org/c/coreboot/+/78044/comment/146a4d8e_c31b80ce :
PS14, Line 240: 	if (mtk_pcie_set_trans_window(&count, table_base, &conf->mmio_res_io)
              : 	    < 0) {
              : 		printk(BIOS_ERR, "%s: Failed to set IO window, ignore it\n",
              : 		       __func__);
              : 		count = 0;
              : 	}
              : 
              : 	/* Initialize memory resources constraints. */
              : 	if (mtk_pcie_set_trans_window(&count, table_base, &conf->mmio_res_mem)
              : 	    < 0)
> Nit: there is no need to break lines here. 96 chars is our (soft) limit.

Please fix it.