diff options
author | Nico Huber <nico.h@gmx.de> | 2020-07-06 21:14:02 +0200 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2020-07-07 19:30:47 +0000 |
commit | 3452cb1359229b0457e9e1f6282c67fd459d4c90 (patch) | |
tree | f53081633f4c282b9c99cd28e5aa3d90bfaf7f84 /src/soc/intel/skylake/i2c.c | |
parent | 5d16a25e0cece666e9275a1c627050bfd918b6ac (diff) |
mb/lenovo/t60: Fix override devicetrees
When converting to override trees in commit c1dc2d5e68 (mb/lenovo/t60:
Switch to override tree), some device nodes were missed. These are
essential, as `chip` configuration data is always tied to device
nodes. The resulting `static.c` contained multiple copies of the
`chip` configuration structs, but the wrong ones were hooked up.
The therefore missing configuration of the clock gen led to general
instability, especially with SMP under Linux (probably due to the
attempt to enter lower C states on an idle core). Passing `maxcpus=1`
to the Linux kernel served as a workaround.
Change-Id: I6c26d633d1860cf9a5415994444e75ae1c2e59ad
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43150
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/intel/skylake/i2c.c')
0 files changed, 0 insertions, 0 deletions