HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31612
Change subject: ACPI: Fix error on asl_compiler_revision format ......................................................................
ACPI: Fix error on asl_compiler_revision format
dmesg report wrong asl compiler version as this value wasn't in hexadecimal: FACP 0x00000000CFB5D6C0 0000F4 (v04 COREv4 COREBOOT 00000000 CORE 01341407) DSDT 0x00000000CFB5B280 002436 (v02 COREv4 COREBOOT 20190112 INTL 20190215)
This error was introduced on change-Id: I91b54b43c8b
Tested on 945G-M4 board, dmesg now report the right value: FACP 0x00000000CFB5D6C0 0000F4 (v04 COREv4 COREBOOT 00000000 CORE 20190215) DSDT 0x00000000CFB5B280 002436 (v02 COREv4 COREBOOT 20190112 INTL 20190215)
Change-Id: I5494bb1e7af17eba0c2da8a0c2eb9e40ffd8d065 Signed-off-by: Elyes HAOUAS ehaouas@noos.fr --- M util/genbuild_h/genbuild_h.sh 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/31612/1
diff --git a/util/genbuild_h/genbuild_h.sh b/util/genbuild_h/genbuild_h.sh index 3c02b56..340de35 100755 --- a/util/genbuild_h/genbuild_h.sh +++ b/util/genbuild_h/genbuild_h.sh @@ -67,5 +67,5 @@ printf "#define COREBOOT_DMI_DATE "$(our_date "$DATE" +%m/%d/%Y)"\n" printf "\n" printf "#define COREBOOT_COMPILE_TIME "$(our_date "$DATE" +%T)"\n" -printf "#define ASL_VERSION %d\n" `./util/crossgcc/xgcc/bin/iasl -v | grep version | sed 's/.*version //'` +printf "#define ASL_VERSION 0x%d\n" `./util/crossgcc/xgcc/bin/iasl -v | grep version | sed 's/.*version //'` printf "#endif\n"
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31612 )
Change subject: ACPI: Fix error on asl_compiler_revision format ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/31612/1/util/genbuild_h/genbuild_h.sh File util/genbuild_h/genbuild_h.sh:
https://review.coreboot.org/#/c/31612/1/util/genbuild_h/genbuild_h.sh@70 PS1, Line 70: printf "#define ASL_VERSION 0x%d\n" `./util/crossgcc/xgcc/bin/iasl -v | grep version | sed 's/.*version //'` line over 80 characters
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31612 )
Change subject: ACPI: Fix error on asl_compiler_revision format ......................................................................
Patch Set 1: Code-Review+2
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31612 )
Change subject: ACPI: Fix error on asl_compiler_revision format ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
https://review.coreboot.org/#/c/31612/1/util/genbuild_h/genbuild_h.sh File util/genbuild_h/genbuild_h.sh:
https://review.coreboot.org/#/c/31612/1/util/genbuild_h/genbuild_h.sh@70 PS1, Line 70: 0x%d LOL, this looks like something to fix xD
Nico Huber has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/31612 )
Change subject: ACPI: Fix error on asl_compiler_revision format ......................................................................
ACPI: Fix error on asl_compiler_revision format
dmesg report wrong asl compiler version as this value wasn't in hexadecimal: FACP 0x00000000CFB5D6C0 0000F4 (v04 COREv4 COREBOOT 00000000 CORE 01341407) DSDT 0x00000000CFB5B280 002436 (v02 COREv4 COREBOOT 20190112 INTL 20190215)
This error was introduced on change-Id: I91b54b43c8b
Tested on 945G-M4 board, dmesg now report the right value: FACP 0x00000000CFB5D6C0 0000F4 (v04 COREv4 COREBOOT 00000000 CORE 20190215) DSDT 0x00000000CFB5B280 002436 (v02 COREv4 COREBOOT 20190112 INTL 20190215)
Change-Id: I5494bb1e7af17eba0c2da8a0c2eb9e40ffd8d065 Signed-off-by: Elyes HAOUAS ehaouas@noos.fr Reviewed-on: https://review.coreboot.org/c/31612 Reviewed-by: Patrick Georgi pgeorgi@google.com Reviewed-by: Nico Huber nico.h@gmx.de Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M util/genbuild_h/genbuild_h.sh 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved Nico Huber: Looks good to me, approved
diff --git a/util/genbuild_h/genbuild_h.sh b/util/genbuild_h/genbuild_h.sh index 3c02b56..340de35 100755 --- a/util/genbuild_h/genbuild_h.sh +++ b/util/genbuild_h/genbuild_h.sh @@ -67,5 +67,5 @@ printf "#define COREBOOT_DMI_DATE "$(our_date "$DATE" +%m/%d/%Y)"\n" printf "\n" printf "#define COREBOOT_COMPILE_TIME "$(our_date "$DATE" +%T)"\n" -printf "#define ASL_VERSION %d\n" `./util/crossgcc/xgcc/bin/iasl -v | grep version | sed 's/.*version //'` +printf "#define ASL_VERSION 0x%d\n" `./util/crossgcc/xgcc/bin/iasl -v | grep version | sed 's/.*version //'` printf "#endif\n"