Change in ...coreboot[master]: mb/lenovo/x201/acpi: Update _OSI strings and OSYS values

[HAOUAS Elyes (Code Review)]

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

Change subject: mb/lenovo/x201/acpi: Update _OSI strings and OSYS values
......................................................................

mb/lenovo/x201/acpi: Update _OSI strings and OSYS values

Change-Id: I503bd92f2a1fa56e3f97c8090431abe92a51fd0e
Signed-off-by: Elyes HAOUAS 
---
M src/mainboard/lenovo/x201/acpi/platform.asl
1 file changed, 68 insertions(+), 17 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/30469/1

diff --git a/src/mainboard/lenovo/x201/acpi/platform.asl b/src/mainboard/lenovo/x201/acpi/platform.asl
index 7d7013b..1f55624 100644
--- a/src/mainboard/lenovo/x201/acpi/platform.asl
+++ b/src/mainboard/lenovo/x201/acpi/platform.asl
@@ -111,52 +111,103 @@
    	 *       2001    | Windows XP(+SP1)
    	 *       2002    | Windows XP SP2
    	 *       2006    | Windows Vista
-		 *       ????    | Windows 7
+		 *       2009    | Windows 7
+		 *       2012    | Windows 8
+		 *       2013    | Windows 8.1
+		 *       2015    | Windows 10
+		 *       2016    | Windows 10, version 1607
+		 *       2017    | Windows 10, version 1703
+		 *       2017.2  | Windows 10, version 1709
+		 *       2018    | Windows 10, version 1803
+		 *       2018.2  | Windows 10, version 1809
    	 */

/* Let's assume we're running at least Windows 2000 */
    	Store (2000, OSYS)

If (CondRefOf(_OSI)) {
+			/* Linux answers _OSI with "True" for a couple of
+			 * Windows version queries. But unlike Windows it
+			 * needs a Video repost, so let's determine whether
+			 * we're running Linux.
+			 */
+
+			If (_OSI("Linux")) {
+				Store (1, LINX)
+			}
+
+			If (_OSI("Windows 2000")) {
+				Store (0x07D0, OSYS)
+			}
+
    		If (_OSI("Windows 2001")) {
-				Store (2001, OSYS)
+				Store (0x07D1, OSYS)
    		}

If (_OSI("Windows 2001 SP1")) {
-				Store (2001, OSYS)
-			}
-
-			If (_OSI("Windows 2001 SP2")) {
-				Store (2002, OSYS)
+				Store (0x07D1, OSYS)
    		}

If (_OSI("Windows 2001.1")) {
-				Store (2001, OSYS)
+				Store (0x07D1, OSYS)
+			}
+
+			If (_OSI("Windows 2001 SP2")) {
+				Store (0x07D1, OSYS)
    		}

If (_OSI("Windows 2001.1 SP1")) {
-				Store (2001, OSYS)
+				Store (0x07D1, OSYS)
    		}

If (_OSI("Windows 2006")) {
-				Store (2006, OSYS)
-			}
-
-			If (_OSI("Windows 2006.1")) {
-				Store (2006, OSYS)
+				Store (0x07D6, OSYS)
    		}

If (_OSI("Windows 2006 SP1")) {
-				Store (2006, OSYS)
+				Store (0x07D6, OSYS)
+			}
+
+			If (_OSI("Windows 2006.1")) {
+				Store (0x07D6, OSYS)
    		}

If (_OSI("Windows 2009")) {
-				Store (2009, OSYS)
+				Store (0x07D9, OSYS)
    		}

If (_OSI("Windows 2012")) {
-				Store (2012, OSYS)
+				Store (0x07DC, OSYS)
    		}
+
+			If (_OSI("Windows 2013")) {
+				Store (0x07DD, OSYS)
+			}
+
+			If (_OSI("Windows 2015")) {
+				Store (0x07DF, OSYS)
+			}
+
+			If (_OSI("Windows 2016")) {
+				Store (0x07E0, OSYS)
+			}
+
+			If (_OSI("Windows 2017")) {
+				Store (0x07E1, OSYS)
+			}
+
+			If (_OSI("Windows 2017.2")) {
+				Store (0x07E1, OSYS)
+			}
+
+			If (_OSI("Windows 2018")) {
+				Store (0x07E2, OSYS)
+			}
+
+			If (_OSI("Windows 2018.2")) {
+				Store (0x07E2, OSYS)
+			}
+
    	}
    }
 }

[HAOUAS Elyes (Code Review)]

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

Change subject: mb/lenovo/x201/acpi: Update _OSI strings and OSYS values
......................................................................

Abandoned