Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34603 ) Change subject: mb/asus: Add ASUS H110M-E/M.2 mainboard ...................................................................... Patch Set 83: (12 comments) Please, don't just copy and paste things around... https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/Makefile.inc: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: GPL-2.0-or-later GPL-2.0-only https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/acpi/dptf.asl: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: GPL-2.0-or-later GPL-2.0-only https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/bootblock.c: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: GPL-2.0-or-later GPL-2.0-only https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/cmos.layout: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: GPL-2.0-or-later GPL-2.0-only https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/devicetree.cb: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: GPL-2.0-or-later GPL-2.0-only https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/dsdt.asl: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: GPL-2.0-or-later GPL-2.0-only https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/hda_verb.c: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 27: AZALIA_SUBVENDOR(0, 0x10438445), AZALIA_PIN_CFG(0, 0x11, 0x40000000), What happened here? Why aren't the AZALIA_PIN_CFG statements on a single line? https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/include/gpio.h: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: later The original file was never GPL-2.0-or-later, it was GPL-2.0-only https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 4: This file is part of the coreboot project. Only keep this line of the entire comment block. /* This file is part of the coreboot project. */ Please do this for ALL files https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 19: _GPIO_DVS_H This symbol was from the H110M-DVS. Maybe change it to match your board? https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/ramstage.c: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: GPL-2.0-or-later GPL-2.0-only https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/romstage.c: https://review.coreboot.org/c/coreboot/+/34603/83/src/mainboard/asus/h110m-e... PS83, Line 1: GPL-2.0-or-later GPL-2.0-only -- To view, visit https://review.coreboot.org/c/coreboot/+/34603 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Id8fa41ecccaa8dba8dc2158ce62d328c7928e05c Gerrit-Change-Number: 34603 Gerrit-PatchSet: 83 Gerrit-Owner: Pavlushka Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com> Gerrit-Reviewer: Martin Roth <martinroth@google.com> Gerrit-Reviewer: Maxim Polyakov <m.poliakov@yahoo.com> Gerrit-Reviewer: Maxim Polyakov <max.senia.poliak@gmail.com> Gerrit-Reviewer: Michael Niewöhner Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com> Gerrit-Reviewer: Pavlushka Gerrit-Reviewer: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-CC: Patrick Rudolph <siro@das-labor.org> Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net> Gerrit-Comment-Date: Sun, 29 Mar 2020 18:32:04 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment