From d60abfcb74ab4580f95cba1f8bfd71824acd7de6 Mon Sep 17 00:00:00 2001 From: Felix Singer Date: Sun, 6 Dec 2020 11:51:10 +0100 Subject: soc/intel/skylake: Shorten SATA mode enum value names MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Skylake FSP isn't used by coreboot anymore. Therefore, drop the misleading comment and the "KBLFSP" extension from the names of these enums. Also, drop the "MODE" extension to make their names shorter in general, since it doesn't add any more value. Built clevo/n130wu with BUILD_TIMELESS=1, coreboot.rom remains identical. Change-Id: If37d40e4e1dfd11e9315039acde7cafee0ac60f0 Signed-off-by: Felix Singer Reviewed-on: https://review.coreboot.org/c/coreboot/+/48377 Reviewed-by: Michael Niewöhner Tested-by: build bot (Jenkins) --- src/soc/intel/skylake/chip.h | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'src/soc/intel/skylake') diff --git a/src/soc/intel/skylake/chip.h b/src/soc/intel/skylake/chip.h index 0bab45ab22..58d172f074 100644 --- a/src/soc/intel/skylake/chip.h +++ b/src/soc/intel/skylake/chip.h @@ -142,11 +142,8 @@ struct soc_intel_skylake_config { /* SATA related */ enum { - /* Documentation and header files of Skylake FSP disagree on - the values, Kaby Lake FSP (KabylakeFsp0001 on github) uses - these: */ - KBLFSP_SATA_MODE_AHCI = 0, - KBLFSP_SATA_MODE_RAID = 1, + SATA_AHCI = 0, + SATA_RAID = 1, } SataMode; u8 SataSalpSupport; u8 SataPortsEnable[8]; -- cgit v1.2.3