[coreboot-gerrit] [XS] Change in coreboot[main]: mb/google/skyrim/var/skyrim: Hide fingerprint reader from Windows OS

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/80711?usp=email )

Change subject: mb/google/skyrim/var/skyrim: Hide fingerprint reader from Windows OS
......................................................................

mb/google/skyrim/var/skyrim: Hide fingerprint reader from Windows OS

No Windows driver exists or is needed, so hide to prevent an unknown
device from being listed in Windows Device Manager. Same change was
made for frostflow variant previously.

TEST=build/boot Win11 on skyrim, verify unknown device for the
fingerprint reader no longer present.

Change-Id: Ia700aa4ccd478bc734db012e1419e566a5dcf493
Signed-off-by: Matt DeVillier 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80711
Reviewed-by: Nico Huber 
Reviewed-by: Felix Held 
Tested-by: build bot (Jenkins) 
---
M src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Nico Huber: Looks good to me, but someone else must approve
  Felix Held: Looks good to me, approved
  build bot (Jenkins): Verified

diff --git a/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb b/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
index eb16120..be498fc 100644
--- a/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
+++ b/src/mainboard/google/skyrim/variants/skyrim/overridetree.cb
@@ -238,7 +238,7 @@
    		register "reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPIO_12)"
    		register "enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_4)"
    		register "enable_delay_ms" = "3"
-			device generic 0 alias fpmcu on
+			device generic 0 alias fpmcu hidden
    			probe FP FP_PRESENT
    		end
    	end