Change in coreboot[master]: acpi: Bump FADT Minor Version to 3

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/42250 )

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

acpi: Bump FADT Minor Version to 3

Change-Id: I3c7803bcab3a4054280e352fd154cc28bf62f723
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/acpi.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/42250/1

diff --git a/src/acpi/acpi.c b/src/acpi/acpi.c
index a4ea24b..15794f1 100644
--- a/src/acpi/acpi.c
+++ b/src/acpi/acpi.c
@@ -1239,7 +1239,7 @@
    memcpy(header->asl_compiler_id, ASLC, 4);
    header->asl_compiler_revision = asl_revision;

-	fadt->FADT_MinorVersion = 0;
+	fadt->FADT_MinorVersion = 3;

fadt->firmware_ctrl = (unsigned long) facs;
    fadt->x_firmware_ctl_l = (unsigned long)facs;

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/42250 )

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

Abandoned

Merge Conflict

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has restored this change. ( https://review.coreboot.org/c/coreboot/+/42250 )

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

Restored

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/42250

to look at the new patch set (#2).

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

acpi: Bump FADT Minor Version to 3

Change-Id: I3c7803bcab3a4054280e352fd154cc28bf62f723
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/acpi.c
M src/soc/amd/picasso/acpi.c
M src/soc/amd/stoneyridge/acpi.c
M src/southbridge/amd/agesa/hudson/fadt.c
M src/southbridge/amd/cimx/sb800/fadt.c
M src/southbridge/amd/pi/hudson/fadt.c
6 files changed, 1 insertion(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/42250/2

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42250 )

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42250/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/42250/2//COMMIT_MSG@8 
PS2, Line 8: 
What is the current default? Please add a reference for the description of the changes between the current minor version and 3.

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Nico Huber,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/42250

to look at the new patch set (#3).

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

acpi: Bump FADT Minor Version to 3

Regarding ACPI specification 6.3, page #133 current minor version FADT table is 3.

Change-Id: I3c7803bcab3a4054280e352fd154cc28bf62f723
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/acpi.c
M src/soc/amd/picasso/acpi.c
M src/soc/amd/stoneyridge/acpi.c
M src/southbridge/amd/agesa/hudson/fadt.c
M src/southbridge/amd/cimx/sb800/fadt.c
M src/southbridge/amd/pi/hudson/fadt.c
6 files changed, 1 insertion(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/42250/3

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Nico Huber,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/42250

to look at the new patch set (#4).

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

acpi: Bump FADT Minor Version to 3

Regarding ACPI specification 6.3, page #133 current FADT table minor version is 3.

Change-Id: I3c7803bcab3a4054280e352fd154cc28bf62f723
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/acpi.c
M src/soc/amd/picasso/acpi.c
M src/soc/amd/stoneyridge/acpi.c
M src/southbridge/amd/agesa/hudson/fadt.c
M src/southbridge/amd/cimx/sb800/fadt.c
M src/southbridge/amd/pi/hudson/fadt.c
6 files changed, 1 insertion(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/42250/4

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42250 )

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

Patch Set 4:

The last bump of FADT triggered additional checks in windows 10. This commit needs on tests on real hardware to see if it still boots.

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Nico Huber,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/42250

to look at the new patch set (#5).

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

acpi: Bump FADT Minor Version to 3

Regarding ACPI specification 6.3, page #133 current FADT table minor version is 3.

Change-Id: I3c7803bcab3a4054280e352fd154cc28bf62f723
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/acpi.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/42250/5

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/42250 )

Change subject: acpi: Bump FADT Minor Version to 3
......................................................................

Abandoned