Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38545 )
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
util/lint: enforce SPDX license headers in src/superio
Change-Id: Iae8d4f0470f75b47e53c50790f06902acb9a24cc Signed-off-by: Felix Held felix-coreboot@felixheld.de --- M util/lint/lint-stable-000-license-headers 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/45/38545/1
diff --git a/util/lint/lint-stable-000-license-headers b/util/lint/lint-stable-000-license-headers index fcaf32c..bee87a0 100755 --- a/util/lint/lint-stable-000-license-headers +++ b/util/lint/lint-stable-000-license-headers @@ -6,6 +6,7 @@
# Directories requiring SPDX Identifiers only util/lint/lint-000-license-headers "src/acpi" SPDX_ONLY +util/lint/lint-000-license-headers "src/superio" SPDX_ONLY
# Top level util/lint/lint-000-license-headers "src/arch src/commonlib src/console \
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38545 )
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
Patch Set 2: Code-Review+1
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38545 )
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
Patch Set 2: Code-Review+2
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38545 )
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38545/2/util/lint/lint-stable-000-l... File util/lint/lint-stable-000-license-headers:
https://review.coreboot.org/c/coreboot/+/38545/2/util/lint/lint-stable-000-l... PS2, Line 14: src/superio remove here?
Hello Angel Pons, Paul Menzel, build bot (Jenkins), Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/38545
to look at the new patch set (#3).
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
util/lint: enforce SPDX license headers in src/superio
Change-Id: Iae8d4f0470f75b47e53c50790f06902acb9a24cc Signed-off-by: Felix Held felix-coreboot@felixheld.de --- M util/lint/lint-stable-000-license-headers 1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/45/38545/3
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38545 )
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38545/2/util/lint/lint-stable-000-l... File util/lint/lint-stable-000-license-headers:
https://review.coreboot.org/c/coreboot/+/38545/2/util/lint/lint-stable-000-l... PS2, Line 14: src/superio
remove here?
missed that one. done.
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38545 )
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
Patch Set 3: Code-Review+2
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/38545 )
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
util/lint: enforce SPDX license headers in src/superio
Change-Id: Iae8d4f0470f75b47e53c50790f06902acb9a24cc Signed-off-by: Felix Held felix-coreboot@felixheld.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/38545 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Patrick Georgi pgeorgi@google.com --- M util/lint/lint-stable-000-license-headers 1 file changed, 2 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved
diff --git a/util/lint/lint-stable-000-license-headers b/util/lint/lint-stable-000-license-headers index fcaf32c..99622cc 100755 --- a/util/lint/lint-stable-000-license-headers +++ b/util/lint/lint-stable-000-license-headers @@ -6,11 +6,12 @@
# Directories requiring SPDX Identifiers only util/lint/lint-000-license-headers "src/acpi" SPDX_ONLY +util/lint/lint-000-license-headers "src/superio" SPDX_ONLY
# Top level util/lint/lint-000-license-headers "src/arch src/commonlib src/console \ src/cpu src/device src/ec src/mainboard src/northbridge src/soc \ - src/southbridge src/superio" + src/southbridge"
# src/drivers util/lint/lint-000-license-headers "src/drivers/ams src/drivers/aspeed src/drivers/dec src/drivers/elog \
9elements QA has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38545 )
Change subject: util/lint: enforce SPDX license headers in src/superio ......................................................................
Patch Set 4:
Automatic boot test returned (PASS/FAIL/TOTAL): 3/0/3 Emulation targets: EMULATION_QEMU_X86_Q35 using payload TianoCore : SUCCESS : https://lava.9esec.io/r/353 EMULATION_QEMU_X86_Q35 using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/352 EMULATION_QEMU_X86_I440FX using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/351
Please note: This test is under development and might not be accurate at all!
-
9elements QA (Code Review) -
Angel Pons (Code Review)
-
Felix Held (Code Review)
-
Patrick Georgi (Code Review)
-
Paul Menzel (Code Review)