diff options
author | Patrick Rudolph <siro@das-labor.org> | 2017-07-25 18:18:57 +0200 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-08-10 16:06:19 +0000 |
commit | 24680d0902f70d3b63f8d7b11f47ffac73697d94 (patch) | |
tree | 09005f9b40dbc99d25f89fe3ab61e858d02421e2 /src/mainboard | |
parent | d7dcc44eb97dcf99093e64b887b9dc2296a31d21 (diff) |
mb/lenovo/t400: Switch to new hybrid graphics driver
Use new hybrid graphics driver to get device state.
Move remaining code to romstage.c.
Tested on Lenovo T500:
* Linux 4.11.4 on Fedora 25
* Integrated (using NGI)
* Discrete (using VGA OpROM)
* Switchable (using NGI and VGA OpROM), tested with DRI_PRIME
No regressions found.
Change-Id: Iad2eccaab19c71f11308853ba9326d8186e67c93
Signed-off-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-on: https://review.coreboot.org/20793
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'src/mainboard')
-rw-r--r-- | src/mainboard/lenovo/t400/Kconfig | 1 | ||||
-rw-r--r-- | src/mainboard/lenovo/t400/Makefile.inc | 1 | ||||
-rw-r--r-- | src/mainboard/lenovo/t400/devicetree.cb | 19 | ||||
-rw-r--r-- | src/mainboard/lenovo/t400/hybrid_graphics.c | 91 | ||||
-rw-r--r-- | src/mainboard/lenovo/t400/romstage.c | 11 |
5 files changed, 30 insertions, 93 deletions
diff --git a/src/mainboard/lenovo/t400/Kconfig b/src/mainboard/lenovo/t400/Kconfig index decbab7f11..8eb5fc54dd 100644 --- a/src/mainboard/lenovo/t400/Kconfig +++ b/src/mainboard/lenovo/t400/Kconfig @@ -21,6 +21,7 @@ config BOARD_SPECIFIC_OPTIONS # dummy select INTEL_INT15 select SUPERIO_NSC_PC87382 select SUPERIO_NSC_PC87384 + select DRIVERS_LENOVO_HYBRID_GRAPHICS config MAINBOARD_DIR string diff --git a/src/mainboard/lenovo/t400/Makefile.inc b/src/mainboard/lenovo/t400/Makefile.inc index 8ac61bbb77..c2c82b6df7 100644 --- a/src/mainboard/lenovo/t400/Makefile.inc +++ b/src/mainboard/lenovo/t400/Makefile.inc @@ -17,5 +17,4 @@ romstage-y += dock.c ramstage-y += dock.c ramstage-y += cstates.c -romstage-y += hybrid_graphics.c ramstage-y += blc.c diff --git a/src/mainboard/lenovo/t400/devicetree.cb b/src/mainboard/lenovo/t400/devicetree.cb index 1adba8b17d..fd5b514c4a 100644 --- a/src/mainboard/lenovo/t400/devicetree.cb +++ b/src/mainboard/lenovo/t400/devicetree.cb @@ -206,6 +206,25 @@ chip northbridge/intel/gm45 end end + chip drivers/lenovo/hybrid_graphics + device pnp ff.f on end # dummy + + register "detect_gpio" = "21" + + register "has_panel_hybrid_gpio" = "1" + register "panel_hybrid_gpio" = "22" + register "panel_integrated_lvl" = "0" + + register "has_backlight_gpio" = "1" + register "backlight_gpio" = "19" + register "backlight_integrated_lvl" = "0" + + register "has_dgpu_power_gpio" = "1" + register "dgpu_power_gpio" = "49" + register "dgpu_power_off_lvl" = "0" + + register "has_thinker1" = "0" + end end device pci 1f.2 on # SATA/IDE 1 subsystemid 0x17aa 0x20f8 diff --git a/src/mainboard/lenovo/t400/hybrid_graphics.c b/src/mainboard/lenovo/t400/hybrid_graphics.c deleted file mode 100644 index b313ffa49c..0000000000 --- a/src/mainboard/lenovo/t400/hybrid_graphics.c +++ /dev/null @@ -1,91 +0,0 @@ -/* - * This file is part of the coreboot project. - * - * Copyright (C) 2012 secunet Security Networks AG - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License as - * published by the Free Software Foundation; version 2 of - * the License. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ -#include <stdint.h> -#include <string.h> -#include <southbridge/intel/common/gpio.h> -#include <northbridge/intel/gm45/gm45.h> -#include <console/console.h> -#include <option.h> - -#define HYBRID_GRAPHICS_INTEGRATED_ONLY 0 -#define HYBRID_GRAPHICS_DISCRETE_ONLY 1 -#define HYBRID_GRAPHICS_SWITCHABLE 2 - -#define MUX_GPIO 22 -#define BCL_CTL_GPIO 19 -#define GFX_PWR_EN_GPIO 49 - -#define HYBRID_DETECT_GPIO 21 - -void hybrid_graphics_init(sysinfo_t *sysinfo); - -static bool hybrid_graphics_installed(void) -{ - if (get_gpio(HYBRID_DETECT_GPIO)) - return false; - else - return true; -} - -void hybrid_graphics_init(sysinfo_t *sysinfo) -{ - /* Set default mode */ - uint8_t hybrid_graphics_mode = - HYBRID_GRAPHICS_INTEGRATED_ONLY; - - if (!hybrid_graphics_installed()) { - printk(BIOS_DEBUG, "Hybrid graphics not installed.\n"); - /* The display is not connected to a mux or switchable. */ - sysinfo->enable_igd = 1; - sysinfo->enable_peg = 0; - return; - } - - printk(BIOS_DEBUG, "Hybrid graphics available: "); - get_option(&hybrid_graphics_mode, "hybrid_graphics_mode"); - - /* Select appropriate hybrid graphics device */ - switch (hybrid_graphics_mode) { - default: - case HYBRID_GRAPHICS_INTEGRATED_ONLY: - printk(BIOS_DEBUG, "Activating Integrated Only.\n"); - set_gpio(MUX_GPIO, GPIO_LEVEL_LOW); - set_gpio(BCL_CTL_GPIO, GPIO_LEVEL_LOW); - set_gpio(GFX_PWR_EN_GPIO, GPIO_LEVEL_LOW); - - sysinfo->enable_igd = 1; - sysinfo->enable_peg = 0; - break; - case HYBRID_GRAPHICS_DISCRETE_ONLY: - printk(BIOS_DEBUG, "Activating Discrete Only.\n"); - set_gpio(MUX_GPIO, GPIO_LEVEL_HIGH); - set_gpio(BCL_CTL_GPIO, GPIO_LEVEL_HIGH); - set_gpio(GFX_PWR_EN_GPIO, GPIO_LEVEL_HIGH); - - sysinfo->enable_igd = 0; - sysinfo->enable_peg = 1; - break; - case HYBRID_GRAPHICS_SWITCHABLE: - printk(BIOS_DEBUG, "Activating Switchable (both GPUs).\n"); - set_gpio(MUX_GPIO, GPIO_LEVEL_LOW); - set_gpio(BCL_CTL_GPIO, GPIO_LEVEL_LOW); - set_gpio(GFX_PWR_EN_GPIO, GPIO_LEVEL_HIGH); - - sysinfo->enable_igd = 1; - sysinfo->enable_peg = 1; - break; - } -} diff --git a/src/mainboard/lenovo/t400/romstage.c b/src/mainboard/lenovo/t400/romstage.c index 2d35650b0a..f5d5dd1597 100644 --- a/src/mainboard/lenovo/t400/romstage.c +++ b/src/mainboard/lenovo/t400/romstage.c @@ -31,6 +31,7 @@ #include <console/console.h> #include <southbridge/intel/i82801ix/i82801ix.h> #include <northbridge/intel/gm45/gm45.h> +#include <drivers/lenovo/hybrid_graphics/hybrid_graphics.h> #include <timestamp.h> #include "dock.h" #include "gpio.h" @@ -38,7 +39,15 @@ #define LPC_DEV PCI_DEV(0, 0x1f, 0) #define MCH_DEV PCI_DEV(0, 0, 0) -void hybrid_graphics_init(sysinfo_t *sysinfo); +static void hybrid_graphics_init(sysinfo_t *sysinfo) +{ + bool peg, igd; + + early_hybrid_graphics(&igd, &peg); + + sysinfo->enable_igd = igd; + sysinfo->enable_peg = peg; +} static void early_lpc_setup(void) { |