Nico Huber submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved
mb/hp/folio_9470m: Transform into variant

Tested with BUILD_TIMELESS=1, binary does not change.

Change-Id: I65696c5739469b33253c22c1d5a65cc31ef3a421
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38095
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
---
D src/mainboard/hp/folio_9470m/Kconfig
D src/mainboard/hp/folio_9470m/Kconfig.name
D src/mainboard/hp/folio_9470m/Makefile.inc
D src/mainboard/hp/folio_9470m/acpi/ec.asl
D src/mainboard/hp/folio_9470m/acpi/platform.asl
D src/mainboard/hp/folio_9470m/acpi/superio.asl
D src/mainboard/hp/folio_9470m/acpi_tables.c
D src/mainboard/hp/folio_9470m/cmos.default
D src/mainboard/hp/folio_9470m/cmos.layout
D src/mainboard/hp/folio_9470m/dsdt.asl
D src/mainboard/hp/folio_9470m/mainboard.c
M src/mainboard/hp/snb_ivb_laptops/Kconfig
M src/mainboard/hp/snb_ivb_laptops/Kconfig.name
R src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/board_info.txt
R src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/data.vbt
R src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/devicetree.cb
R src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/early_init.c
R src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/gma-mainboard.ads
R src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/gpio.c
R src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/hda_verb.c
20 files changed, 16 insertions(+), 334 deletions(-)

diff --git a/src/mainboard/hp/folio_9470m/Kconfig b/src/mainboard/hp/folio_9470m/Kconfig
deleted file mode 100644
index b40ca25..0000000
--- a/src/mainboard/hp/folio_9470m/Kconfig
+++ /dev/null
@@ -1,46 +0,0 @@
-if BOARD_HP_FOLIO_9470M
-
-config BOARD_SPECIFIC_OPTIONS
- def_bool y
- select BOARD_ROMSIZE_KB_16384
- select EC_HP_KBC1126
- select GFX_GMA_INTERNAL_IS_LVDS
- select HAVE_ACPI_RESUME
- select HAVE_ACPI_TABLES
- select HAVE_CMOS_DEFAULT
- select HAVE_OPTION_TABLE
- select INTEL_GMA_HAVE_VBT
- select INTEL_INT15
- select MAINBOARD_HAS_LIBGFXINIT
- select MAINBOARD_HAS_LPC_TPM
- select MAINBOARD_USES_IFD_GBE_REGION
- select NORTHBRIDGE_INTEL_SANDYBRIDGE
- select SERIRQ_CONTINUOUS_MODE
- select SOUTHBRIDGE_INTEL_C216
- select SYSTEM_TYPE_LAPTOP
- select USE_NATIVE_RAMINIT
-
-config MAINBOARD_DIR
- string
- default "hp/folio_9470m"
-
-config MAINBOARD_PART_NUMBER
- string
- default "EliteBook Folio 9470m"
-
-config VGA_BIOS_FILE
- string
- default "pci8086,0166.rom"
-
-config VGA_BIOS_ID
- string
- default "8086,0166"
-
-config MAX_CPUS
- int
- default 8
-
-config USBDEBUG_HCD_INDEX
- int
- default 0
-endif
diff --git a/src/mainboard/hp/folio_9470m/Kconfig.name b/src/mainboard/hp/folio_9470m/Kconfig.name
deleted file mode 100644
index 92dd985..0000000
--- a/src/mainboard/hp/folio_9470m/Kconfig.name
+++ /dev/null
@@ -1,2 +0,0 @@
-config BOARD_HP_FOLIO_9470M
- bool "EliteBook Folio 9470m"
diff --git a/src/mainboard/hp/folio_9470m/Makefile.inc b/src/mainboard/hp/folio_9470m/Makefile.inc
deleted file mode 100644
index 1d25875..0000000
--- a/src/mainboard/hp/folio_9470m/Makefile.inc
+++ /dev/null
@@ -1,21 +0,0 @@
-##
-## This file is part of the coreboot project.
-##
-## Copyright (C) 2011 The ChromiumOS Authors. All rights reserved.
-##
-## 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.
-##
-
-bootblock-y += gpio.c
-romstage-y += gpio.c
-
-ramstage-$(CONFIG_MAINBOARD_USE_LIBGFXINIT) += gma-mainboard.ads
-bootblock-y += early_init.c
-romstage-y += early_init.c
diff --git a/src/mainboard/hp/folio_9470m/acpi/ec.asl b/src/mainboard/hp/folio_9470m/acpi/ec.asl
deleted file mode 100644
index ac65fb3..0000000
--- a/src/mainboard/hp/folio_9470m/acpi/ec.asl
+++ /dev/null
@@ -1,16 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2017 Iru Cai <mytbk920423@gmail.com>
- *
- * 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 <ec/hp/kbc1126/acpi/ec.asl>
diff --git a/src/mainboard/hp/folio_9470m/acpi/platform.asl b/src/mainboard/hp/folio_9470m/acpi/platform.asl
deleted file mode 100644
index fe0f936..0000000
--- a/src/mainboard/hp/folio_9470m/acpi/platform.asl
+++ /dev/null
@@ -1,27 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2011-2012 The Chromium OS Authors. All rights reserved.
- *
- * 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.
- */
-
-Method(_WAK,1)
-{
- \_SB.PCI0.LPCB.EC0.ACPI = 1
- \_SB.PCI0.LPCB.EC0.SLPT = 0
-
- Return(Package(){0,0})
-}
-
-Method(_PTS,1)
-{
- \_SB.PCI0.LPCB.EC0.SLPT = Arg0
-}
diff --git a/src/mainboard/hp/folio_9470m/acpi/superio.asl b/src/mainboard/hp/folio_9470m/acpi/superio.asl
deleted file mode 100644
index f2b35ba..0000000
--- a/src/mainboard/hp/folio_9470m/acpi/superio.asl
+++ /dev/null
@@ -1 +0,0 @@
-#include <drivers/pc80/pc/ps2_controller.asl>
diff --git a/src/mainboard/hp/folio_9470m/acpi_tables.c b/src/mainboard/hp/folio_9470m/acpi_tables.c
deleted file mode 100644
index 114f6e1..0000000
--- a/src/mainboard/hp/folio_9470m/acpi_tables.c
+++ /dev/null
@@ -1,27 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008-2009 coresystems GmbH
- * Copyright (C) 2014 Vladimir Serbinenko
- *
- * 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 <southbridge/intel/bd82x6x/nvs.h>
-
-void acpi_create_gnvs(global_nvs_t *gnvs)
-{
- // the lid is open by default.
- gnvs->lids = 1;
-
- gnvs->tcrt = 100;
- gnvs->tpsv = 90;
-}
diff --git a/src/mainboard/hp/folio_9470m/cmos.default b/src/mainboard/hp/folio_9470m/cmos.default
deleted file mode 100644
index 455dc1cd..0000000
--- a/src/mainboard/hp/folio_9470m/cmos.default
+++ /dev/null
@@ -1,6 +0,0 @@
-boot_option=Fallback
-debug_level=Debug
-power_on_after_fail=Disable
-nmi=Enable
-volume=0x3
-gfx_uma_size=32M
diff --git a/src/mainboard/hp/folio_9470m/cmos.layout b/src/mainboard/hp/folio_9470m/cmos.layout
deleted file mode 100644
index f1526f3..0000000
--- a/src/mainboard/hp/folio_9470m/cmos.layout
+++ /dev/null
@@ -1,116 +0,0 @@
-##
-## This file is part of the coreboot project.
-##
-## Copyright (C) 2007-2008 coresystems GmbH
-## Copyright (C) 2014 Vladimir Serbinenko
-##
-## 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.
-##
-
-# -----------------------------------------------------------------
-entries
-
-# -----------------------------------------------------------------
-# Status Register A
-# -----------------------------------------------------------------
-# Status Register B
-# -----------------------------------------------------------------
-# Status Register C
-#96 4 r 0 status_c_rsvd
-#100 1 r 0 uf_flag
-#101 1 r 0 af_flag
-#102 1 r 0 pf_flag
-#103 1 r 0 irqf_flag
-# -----------------------------------------------------------------
-# Status Register D
-#104 7 r 0 status_d_rsvd
-#111 1 r 0 valid_cmos_ram
-# -----------------------------------------------------------------
-# Diagnostic Status Register
-#112 8 r 0 diag_rsvd1
-
-# -----------------------------------------------------------------
-0 120 r 0 reserved_memory
-#120 264 r 0 unused
-
-# -----------------------------------------------------------------
-# RTC_BOOT_BYTE (coreboot hardcoded)
-384 1 e 4 boot_option
-388 4 h 0 reboot_counter
-#390 2 r 0 unused?
-
-# -----------------------------------------------------------------
-# coreboot config options: console
-#392 3 r 0 unused
-395 4 e 6 debug_level
-#399 1 r 0 unused
-
-#400 8 r 0 reserved for century byte
-
-# coreboot config options: southbridge
-408 1 e 1 nmi
-409 2 e 7 power_on_after_fail
-
-421 1 e 9 sata_mode
-
-# coreboot config options: cpu
-#424 8 r 0 unused
-
-# coreboot config options: northbridge
-432 3 e 11 gfx_uma_size
-#435 5 r 0 unused
-
-440 8 h 0 volume
-
-# SandyBridge MRC Scrambler Seed values
-896 32 r 0 mrc_scrambler_seed
-928 32 r 0 mrc_scrambler_seed_s3
-960 16 r 0 mrc_scrambler_seed_chk
-
-# coreboot config options: check sums
-984 16 h 0 check_sum
-
-# -----------------------------------------------------------------
-
-enumerations
-
-#ID value text
-1 0 Disable
-1 1 Enable
-2 0 Enable
-2 1 Disable
-4 0 Fallback
-4 1 Normal
-6 0 Emergency
-6 1 Alert
-6 2 Critical
-6 3 Error
-6 4 Warning
-6 5 Notice
-6 6 Info
-6 7 Debug
-6 8 Spew
-7 0 Disable
-7 1 Enable
-7 2 Keep
-9 0 AHCI
-9 1 Compatible
-11 0 32M
-11 1 64M
-11 2 96M
-11 3 128M
-11 4 160M
-11 5 192M
-11 6 224M
-
-# -----------------------------------------------------------------
-checksums
-
-checksum 392 447 984
diff --git a/src/mainboard/hp/folio_9470m/dsdt.asl b/src/mainboard/hp/folio_9470m/dsdt.asl
deleted file mode 100644
index 81f45c1..0000000
--- a/src/mainboard/hp/folio_9470m/dsdt.asl
+++ /dev/null
@@ -1,44 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007-2009 coresystems GmbH
- *
- * 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.
- */
-
-#define BRIGHTNESS_UP \_SB.PCI0.GFX0.INCB
-#define BRIGHTNESS_DOWN \_SB.PCI0.GFX0.DECB
-#define ACPI_VIDEO_DEVICE \_SB.PCI0.GFX0
-#include <arch/acpi.h>
-DefinitionBlock(
- "dsdt.aml",
- "DSDT",
- 0x02, // DSDT revision: ACPI v2.0 and up
- OEM_ID,
- ACPI_TABLE_CREATOR,
- 0x20141018 // OEM revision
-)
-{
- #include "acpi/platform.asl"
- #include <cpu/intel/common/acpi/cpu.asl>
- #include <southbridge/intel/common/acpi/platform.asl>
- /* global NVS and variables. */
- #include <southbridge/intel/bd82x6x/acpi/globalnvs.asl>
- #include <southbridge/intel/common/acpi/sleepstates.asl>
-
- Scope (\_SB) {
- Device (PCI0)
- {
- #include <northbridge/intel/sandybridge/acpi/sandybridge.asl>
- #include <drivers/intel/gma/acpi/default_brightness_levels.asl>
- #include <southbridge/intel/bd82x6x/acpi/pch.asl>
- }
- }
-}
diff --git a/src/mainboard/hp/folio_9470m/mainboard.c b/src/mainboard/hp/folio_9470m/mainboard.c
deleted file mode 100644
index 058701e..0000000
--- a/src/mainboard/hp/folio_9470m/mainboard.c
+++ /dev/null
@@ -1,28 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2017 Bill Xie <persmule@gmail.com>
- *
- * 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 <device/device.h>
-#include <drivers/intel/gma/int15.h>
-
-static void mainboard_enable(struct device *dev)
-{
- install_intel_vga_int15_handler(GMA_INT15_ACTIVE_LFP_INT_LVDS,
- GMA_INT15_PANEL_FIT_DEFAULT,
- GMA_INT15_BOOT_DISPLAY_DEFAULT, 0);
-}
-
-struct chip_operations mainboard_ops = {
- .enable_dev = mainboard_enable,
-};
diff --git a/src/mainboard/hp/snb_ivb_laptops/Kconfig b/src/mainboard/hp/snb_ivb_laptops/Kconfig
index 62489cb..c6b218f 100644
--- a/src/mainboard/hp/snb_ivb_laptops/Kconfig
+++ b/src/mainboard/hp/snb_ivb_laptops/Kconfig
@@ -38,6 +38,7 @@
default "8460p" if BOARD_HP_8460P
default "8470p" if BOARD_HP_8470P
default "8770w" if BOARD_HP_8770W
+ default "folio_9470m" if BOARD_HP_FOLIO_9470M

config MAINBOARD_PART_NUMBER
string
@@ -46,6 +47,7 @@
default "EliteBook 8460p" if BOARD_HP_8460P
default "EliteBook 8470p" if BOARD_HP_8470P
default "EliteBook 8770w" if BOARD_HP_8770W
+ default "EliteBook Folio 9470m" if BOARD_HP_FOLIO_9470M

config DEVICETREE
string
@@ -72,5 +74,6 @@
default 1 if BOARD_HP_8460P
default 2 if BOARD_HP_8470P
default 2 if BOARD_HP_8770W
+ default 0 if BOARD_HP_FOLIO_9470M

endif
diff --git a/src/mainboard/hp/snb_ivb_laptops/Kconfig.name b/src/mainboard/hp/snb_ivb_laptops/Kconfig.name
index 2eedbf1..4ae7b9b 100644
--- a/src/mainboard/hp/snb_ivb_laptops/Kconfig.name
+++ b/src/mainboard/hp/snb_ivb_laptops/Kconfig.name
@@ -69,3 +69,16 @@
select MAINBOARD_USES_IFD_GBE_REGION
select SOUTHBRIDGE_INTEL_C216
select SUPERIO_SMSC_LPC47N217
+
+config BOARD_HP_FOLIO_9470M
+ bool "EliteBook Folio 9470m"
+
+ select BOARD_HP_SNB_IVB_LAPTOPS
+ select BOARD_ROMSIZE_KB_16384
+ select GFX_GMA_INTERNAL_IS_LVDS
+ select INTEL_GMA_HAVE_VBT
+ select INTEL_INT15
+ select MAINBOARD_HAS_LIBGFXINIT
+ select MAINBOARD_HAS_LPC_TPM
+ select MAINBOARD_USES_IFD_GBE_REGION
+ select SOUTHBRIDGE_INTEL_C216
diff --git a/src/mainboard/hp/folio_9470m/board_info.txt b/src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/board_info.txt
similarity index 100%
rename from src/mainboard/hp/folio_9470m/board_info.txt
rename to src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/board_info.txt
diff --git a/src/mainboard/hp/folio_9470m/data.vbt b/src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/data.vbt
similarity index 100%
rename from src/mainboard/hp/folio_9470m/data.vbt
rename to src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/data.vbt
Binary files differ
diff --git a/src/mainboard/hp/folio_9470m/devicetree.cb b/src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/devicetree.cb
similarity index 100%
rename from src/mainboard/hp/folio_9470m/devicetree.cb
rename to src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/devicetree.cb
diff --git a/src/mainboard/hp/folio_9470m/early_init.c b/src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/early_init.c
similarity index 100%
rename from src/mainboard/hp/folio_9470m/early_init.c
rename to src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/early_init.c
diff --git a/src/mainboard/hp/folio_9470m/gma-mainboard.ads b/src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/gma-mainboard.ads
similarity index 100%
rename from src/mainboard/hp/folio_9470m/gma-mainboard.ads
rename to src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/gma-mainboard.ads
diff --git a/src/mainboard/hp/folio_9470m/gpio.c b/src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/gpio.c
similarity index 100%
rename from src/mainboard/hp/folio_9470m/gpio.c
rename to src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/gpio.c
diff --git a/src/mainboard/hp/folio_9470m/hda_verb.c b/src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/hda_verb.c
similarity index 100%
rename from src/mainboard/hp/folio_9470m/hda_verb.c
rename to src/mainboard/hp/snb_ivb_laptops/variants/folio_9470m/hda_verb.c

To view, visit change 38095. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I65696c5739469b33253c22c1d5a65cc31ef3a421
Gerrit-Change-Number: 38095
Gerrit-PatchSet: 4
Gerrit-Owner: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Bill XIE <persmule@hardenedlinux.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: merged