[coreboot-gerrit] Change in ...coreboot[master]: mb/lenovo/x201/smihandler.c: Remove useless smihandler code

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/33508

Change subject: mb/lenovo/x201/smihandler.c: Remove useless smihandler code
......................................................................

mb/lenovo/x201/smihandler.c: Remove useless smihandler code

This code to handle the brightness from SMM is copied from the Lenovo
Thinkpad X60 code, but does not work on later generation. The PCI
device it tries to address does not even exist on those devices.

Change-Id: I0c25c3e5bec651b27158a84cc91289639a04ceb6
Signed-off-by: Arthur Heymans 
---
M src/mainboard/lenovo/x201/smihandler.c
1 file changed, 0 insertions(+), 25 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/33508/1

diff --git a/src/mainboard/lenovo/x201/smihandler.c b/src/mainboard/lenovo/x201/smihandler.c
index 9043f8e..8d2a966 100644
--- a/src/mainboard/lenovo/x201/smihandler.c
+++ b/src/mainboard/lenovo/x201/smihandler.c
@@ -71,23 +71,6 @@
    return 1;
 }

-static void mainboard_smi_brightness_up(void)
-{
-	u8 value;
-
-	if ((value = pci_read_config8(PCI_DEV(0, 2, 1), 0xf4)) < 0xf0)
-		pci_write_config8(PCI_DEV(0, 2, 1), 0xf4, (value + 0x10) | 0xf);
-}
-
-static void mainboard_smi_brightness_down(void)
-{
-	u8 value;
-
-	if ((value = pci_read_config8(PCI_DEV(0, 2, 1), 0xf4)) > 0x10)
-		pci_write_config8(PCI_DEV(0, 2, 1), 0xf4,
-				  (value - 0x10) & 0xf0);
-}
-
 static void mainboard_smi_handle_ec_sci(void)
 {
    u8 status = inb(EC_SC);
@@ -100,14 +83,6 @@
    printk(BIOS_DEBUG, "EC event %02x\n", event);

switch (event) {
-	case 0x14:
-		/* brightness up */
-		mainboard_smi_brightness_up();
-		break;
-	case 0x15:
-		/* brightness down */
-		mainboard_smi_brightness_down();
-		break;
    case 0x18:
    	/* Fn-F9 key */
    case 0x27: