Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45396 )
Change subject: soc/mediatek/mt8192: Add usb host support ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/45396/1/src/soc/mediatek/mt8192/inc... File src/soc/mediatek/mt8192/include/soc/usb.h:
https://review.coreboot.org/c/coreboot/+/45396/1/src/soc/mediatek/mt8192/inc... PS1, Line 1: /* SPDX-License-Identifier: GPL-2.0-only */ Please add a blank line below.
https://review.coreboot.org/c/coreboot/+/45396/1/src/soc/mediatek/mt8192/usb... File src/soc/mediatek/mt8192/usb.c:
https://review.coreboot.org/c/coreboot/+/45396/1/src/soc/mediatek/mt8192/usb... PS1, Line 1: /* SPDX-License-Identifier: GPL-2.0-only */ Ditto.
https://review.coreboot.org/c/coreboot/+/45396/1/src/soc/mediatek/mt8192/usb... PS1, Line 16: Please remove the blank line at the end of the file.