Skip to content

Conversation

@filipleple
Copy link
Owner

Add support for the Lenovo ThinkPad T460s.

Known issues:

  • Missing onboard RAM SPD binaries and appropriate logic (currently only uses the DIMM slot). Currently learning how to obtain them properly.

  • Bad CLKREQ and SRCCLK settings for every device in overridetree - not crucial, just that power saving on idle is a bit less optimal. It should take a quick look at the schematics to correct them.

Add support for the Lenovo ThinkPad T460s.

Known issues:

* Missing onboard RAM SPD binaries and appropriate logic (currently only
  uses the DIMM slot). Currently learning how to obtain them properly.

* Bad CLKREQ and SRCCLK settings for every device in overridetree - not
  crucial, just that power saving on idle is a bit less optimal. It
  should take a quick look at the schematics to correct them.

Signed-off-by: Filip Lewiński <filip.lewinski@3mdeb.com>
filipleple pushed a commit that referenced this pull request Nov 14, 2025
Reconfigure I2C controller settings to disable I2C1 and enable I2C6
for the mc_rpl1 mainboard. This change reflects the updated hardware
configuration requirements.

Changes:
- Disable I2C controller 1
- Enable I2C controller 6

TEST=Build and boot tested on mc_rpl1 mainboard.
     Verified I2C6 functionality and confirmed I2C1 is disabled with
     `lspci -v | grep -A 5 "Serial bus controller"`. The output
     confirms that I2C6 (PCI 00:10.0) is enabled and I2C1 (PCI 00:15.1)
     is disabled because it is absent.

     ```
     00:10.0 Serial bus controller: Intel Corporation Alder Lake-P Serial IO I2C Controller #2 (rev 01)
        Subsystem: Intel Corporation Alder Lake-P Serial IO I2C Controller
        Flags: bus master, fast devsel, latency 0, IRQ 24, IOMMU group 4
        Memory at 80a12000 (64-bit, non-prefetchable) [size=4K]
        Capabilities: [80] Power Management version 3
        Capabilities: [90] Vendor Specific Information: Len=14 <?>
     ```

Change-Id: I4867062743ee10b34f94a1e588a10115b553a16e
Signed-off-by: Kilian Krause <kilian.krause@siemens.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/89690
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Uwe Poeche <uwe.poeche@siemens.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants