drivers/i2c/at24rf08c: Disable DRIVER_LENOVO_SERIALS by default

This should be the sane default, as having this option enabled when the
AT24RF08C (Asset Identification EEPROM) is not present on the mainboard
can cause SMBIOS table entries to become "*INVALID*". This can,
for example, result in strange hostnames when an OS installer uses
SMBIOS information to automatically generate one. On the other hand,
the coreboot generated SMBIOS tables will at least always contain basic
information. Therefore, this driver should be treated as an enhancement
rather than a default.

Currently, the following mainboards have this option disabled:

  - ThinkCentre M710s
  - ThinkCentre M700 / M900 Tiny
  - Haswell ThinkPads
    - ThinkPad T440p
    - ThinkPad W541

Therefore, we can remove this option entirely on these mainboards. Note
that there may be other mainboards which do not have this chip present
but still have the option enabled. However, this requires a more
detailed investigation which would be out of scope of this change.

TESTS=Timeless builds on lenovo mainboards produce the same binary.

config INCLUDE_CONFIG_FILE
	default n

Was temporarily added to `mb/lenovo/Kconfig` during these tests, as
while the configuration does not change, the order of entries do.
Therefore, technically producing a different binary when included.

Change-Id: I5bb101bd6696c39718ee779426d0ec3e721e1b51
Signed-off-by: Nicholas Sudsgaard <devel+coreboot@nsudsgaard.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84544
Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Nicholas Sudsgaard 2024-09-25 01:21:19 +00:00 committed by Felix Held
commit 16fe2eec08
23 changed files with 20 additions and 12 deletions

View file

@ -2,5 +2,4 @@
config DRIVER_LENOVO_SERIALS
bool
default y if VENDOR_LENOVO
select SMBIOS_PROVIDED_BY_MOBO

View file

@ -72,10 +72,6 @@ config USBDEBUG_HCD_INDEX
int
default 2
config DRIVER_LENOVO_SERIALS
bool
default n
config PS2K_EISAID
default "LEN0071"

View file

@ -5,6 +5,7 @@ if BOARD_LENOVO_L520
config BOARD_SPECIFIC_OPTIONS
def_bool y
select BOARD_ROMSIZE_KB_4096
select DRIVER_LENOVO_SERIALS
select EC_LENOVO_H8
select EC_LENOVO_PMH7
select HAVE_ACPI_RESUME

View file

@ -2,6 +2,7 @@
if BOARD_LENOVO_THINKCENTRE_M900_TINY
# TODO: fix and enable DRIVER_LENOVO_SERIALS
config BOARD_SPECIFIC_OPTIONS
def_bool y
select BOARD_ROMSIZE_KB_16384
@ -37,8 +38,4 @@ config PRERAM_CBMEM_CONSOLE_SIZE
config DIMM_SPD_SIZE
default 512 #DDR4
# TODO: fix and enable
config DRIVER_LENOVO_SERIALS
default n
endif

View file

@ -5,6 +5,7 @@ if BOARD_LENOVO_S230U
config BOARD_SPECIFIC_OPTIONS
def_bool y
select BOARD_ROMSIZE_KB_12288
select DRIVER_LENOVO_SERIALS
select EC_ACPI
select EC_COMPAL_ENE932
select GFX_GMA_PANEL_1_ON_LVDS

View file

@ -23,6 +23,7 @@ config BOARD_SPECIFIC_OPTIONS
select SUPERIO_NSC_PC87382
select SUPERIO_NSC_PC87384
select DRIVERS_LENOVO_HYBRID_GRAPHICS
select DRIVER_LENOVO_SERIALS
select MAINBOARD_HAS_LIBGFXINIT
select MAINBOARD_USES_IFD_GBE_REGION if !BOARD_LENOVO_R500
select INTEL_GMA_HAVE_VBT

View file

@ -25,6 +25,7 @@ config BOARD_SPECIFIC_OPTIONS
select H8_HAS_BAT_THRESHOLDS_IMPL
select MAINBOARD_HAS_LIBGFXINIT
select DRIVERS_RICOH_RCE822
select DRIVER_LENOVO_SERIALS
config VBOOT
select VBOOT_VBNV_FLASH

View file

@ -19,6 +19,7 @@ config BOARD_SPECIFIC_OPTIONS
select HAVE_ACPI_RESUME
select INTEL_INT15
select DRIVERS_RICOH_RCE822
select DRIVER_LENOVO_SERIALS
select MEMORY_MAPPED_TPM
select MAINBOARD_HAS_TPM1
select MAINBOARD_HAS_LIBGFXINIT

View file

@ -6,6 +6,7 @@ config BOARD_SPECIFIC_OPTIONS
def_bool y
select BOARD_ROMSIZE_KB_8192
select DRIVERS_LENOVO_HYBRID_GRAPHICS
select DRIVER_LENOVO_SERIALS
select EC_LENOVO_H8
select EC_LENOVO_PMH7
select GFX_GMA_PANEL_1_ON_LVDS

View file

@ -6,6 +6,7 @@ config BOARD_SPECIFIC_OPTIONS
def_bool y
select BOARD_ROMSIZE_KB_12288
select DRIVERS_LENOVO_HYBRID_GRAPHICS
select DRIVER_LENOVO_SERIALS
select DRIVERS_RICOH_RCE822
select EC_LENOVO_H8
select EC_LENOVO_PMH7

View file

@ -26,6 +26,7 @@ config BOARD_SPECIFIC_OPTIONS
select INTEL_GMA_HAVE_VBT
select MAINBOARD_USES_IFD_GBE_REGION
select DRIVERS_RICOH_RCE822 if BOARD_LENOVO_T431S
select DRIVER_LENOVO_SERIALS
select HAVE_SPD_IN_CBFS if BOARD_LENOVO_T431S
# Workaround for EC/KBC IRQ1.

View file

@ -23,6 +23,7 @@ config BOARD_LENOVO_BASEBOARD_T520
select DRIVERS_LENOVO_HYBRID_GRAPHICS
select INTEL_GMA_HAVE_VBT if BOARD_LENOVO_T520
select MAINBOARD_USES_IFD_GBE_REGION
select DRIVER_LENOVO_SERIALS
# Workaround for EC/KBC IRQ1.
select SERIRQ_CONTINUOUS_MODE

View file

@ -4,6 +4,7 @@ config BOARD_LENOVO_BASEBOARD_T530
def_bool n
select BOARD_ROMSIZE_KB_12288
select DRIVERS_LENOVO_HYBRID_GRAPHICS
select DRIVER_LENOVO_SERIALS
select EC_LENOVO_H8
select EC_LENOVO_PMH7
select GFX_GMA_PANEL_1_ON_LVDS

View file

@ -15,6 +15,7 @@ config BOARD_SPECIFIC_OPTIONS
select EC_LENOVO_PMH7
select EC_LENOVO_H8
select DRIVERS_I2C_CK505
select DRIVER_LENOVO_SERIALS
select HAVE_OPTION_TABLE
select INTEL_INT15
select HAVE_MP_TABLE

View file

@ -14,6 +14,7 @@ config BOARD_SPECIFIC_OPTIONS
select HAVE_CMOS_DEFAULT
select HAVE_ACPI_RESUME
select DRIVERS_I2C_CK505
select DRIVER_LENOVO_SERIALS
select INTEL_GMA_HAVE_VBT
select MAINBOARD_HAS_LIBGFXINIT

View file

@ -23,7 +23,4 @@ config MAINBOARD_DIR
config MAINBOARD_PART_NUMBER
default "ThinkCentre M710s"
config DRIVER_LENOVO_SERIALS
default n
endif # BOARD_LENOVO_THINKCENTRE_M710S

View file

@ -5,6 +5,7 @@ if BOARD_LENOVO_X131E
config BOARD_SPECIFIC_OPTIONS
def_bool y
select BOARD_ROMSIZE_KB_12288
select DRIVER_LENOVO_SERIALS
select EC_LENOVO_H8
select GFX_GMA_PANEL_1_ON_LVDS
select HAVE_ACPI_RESUME

View file

@ -6,6 +6,7 @@ config BOARD_SPECIFIC_OPTIONS
def_bool y
select BOARD_ROMSIZE_KB_12288
select DRIVERS_RICOH_RCE822
select DRIVER_LENOVO_SERIALS
select EC_LENOVO_H8
select EC_LENOVO_PMH7
select GFX_GMA_PANEL_1_ON_LVDS

View file

@ -22,6 +22,7 @@ config BOARD_SPECIFIC_OPTIONS
select INTEL_INT15
select SUPERIO_NSC_PC87382
select DRIVERS_LENOVO_WACOM
select DRIVER_LENOVO_SERIALS
select MAINBOARD_HAS_LIBGFXINIT
select MAINBOARD_USES_IFD_GBE_REGION

View file

@ -19,6 +19,7 @@ config BOARD_SPECIFIC_OPTIONS
select MAINBOARD_HAS_LIBGFXINIT
select SUPERIO_NSC_PC87382
select DRIVERS_LENOVO_WACOM
select DRIVER_LENOVO_SERIALS
select MEMORY_MAPPED_TPM
select MAINBOARD_HAS_TPM1
select INTEL_GMA_HAVE_VBT

View file

@ -6,6 +6,7 @@ config BOARD_SPECIFIC_OPTIONS
def_bool y
select BOARD_ROMSIZE_KB_8192
select DRIVERS_RICOH_RCE822
select DRIVER_LENOVO_SERIALS
select EC_LENOVO_H8
select EC_LENOVO_PMH7
select GFX_GMA_PANEL_1_ON_LVDS

View file

@ -21,6 +21,7 @@ config BOARD_SPECIFIC_OPTIONS
select HAVE_ACPI_RESUME
select INTEL_INT15
select DRIVERS_RICOH_RCE822
select DRIVER_LENOVO_SERIALS
select MEMORY_MAPPED_TPM
select MAINBOARD_HAS_TPM1 if BOARD_LENOVO_X230 || BOARD_LENOVO_X230T || BOARD_LENOVO_X230_EDP
select MAINBOARD_HAS_LIBGFXINIT

View file

@ -15,6 +15,7 @@ config BOARD_SPECIFIC_OPTIONS
select EC_LENOVO_PMH7
select EC_LENOVO_H8
select DRIVERS_I2C_CK505
select DRIVER_LENOVO_SERIALS
select HAVE_OPTION_TABLE
select INTEL_INT15
select HAVE_CMOS_DEFAULT