From 24680d0902f70d3b63f8d7b11f47ffac73697d94 Mon Sep 17 00:00:00 2001 From: Patrick Rudolph Date: Tue, 25 Jul 2017 18:18:57 +0200 Subject: 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 Reviewed-on: https://review.coreboot.org/20793 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber Reviewed-by: Arthur Heymans --- src/drivers/lenovo/hybrid_graphics.c | 11 ---- src/mainboard/lenovo/t400/Kconfig | 1 + src/mainboard/lenovo/t400/Makefile.inc | 1 - src/mainboard/lenovo/t400/devicetree.cb | 19 ++++++ src/mainboard/lenovo/t400/hybrid_graphics.c | 91 ----------------------------- src/mainboard/lenovo/t400/romstage.c | 11 +++- 6 files changed, 30 insertions(+), 104 deletions(-) delete mode 100644 src/mainboard/lenovo/t400/hybrid_graphics.c diff --git a/src/drivers/lenovo/hybrid_graphics.c b/src/drivers/lenovo/hybrid_graphics.c index 9b46646e4b..802823acdc 100644 --- a/src/drivers/lenovo/hybrid_graphics.c +++ b/src/drivers/lenovo/hybrid_graphics.c @@ -112,14 +112,3 @@ static const struct pci_driver hybrid_peg_nvidia __pci_driver = { .vendor = PCI_VENDOR_ID_NVIDIA, .devices = pci_device_ids_nvidia, }; - -static const unsigned short pci_device_ids_amd[] = { - 0x9591, /* ATI Mobility Radeon HD 3650 Lenovo T500/W500 */ - 0x95c4, /* ATI Mobility Radeon HD 3470 Lenovo T400/R400 */ - 0 }; - -static const struct pci_driver hybrid_peg_amd __pci_driver = { - .ops = &hybrid_graphics_ops, - .vendor = PCI_VENDOR_ID_ATI, - .devices = pci_device_ids_amd, -}; 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 -#include -#include -#include -#include -#include - -#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 #include #include +#include #include #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) { -- cgit v1.2.3