Nicholas Chin has uploaded a new patch set (#4). ( https://review.coreboot.org/c/coreboot/+/82152?usp=email )
Change subject: mb/dell: Add Latitude E6330 (Ivy Bridge)
......................................................................
mb/dell: Add Latitude E6330 (Ivy Bridge)
Mainboard is QAL70/LA-7741P. I do not physically have this system;
someone with physical access to one sent me the output of autoport which
I then modified to produce this port. I was also sent the VBT binary,
which was obtained from `/sys/kernel/debug/dri/0/i915_vbt` while running
version A21 of the vendor firmware. This port has not been tested.
The EC is the SMSC MEC5055, which seems to be compatible with the
existing MEC5035 code. As with the other Dell systems with this EC, this
board is assumed to be internally flashable using an EC command that
tells it to pull the FDO pin low on the next boot, which also tells the
vendor firmware to disable all write protections to the flash [1].
[1] https://gitlab.com/nic3-14159/dell-flash-unlock
Change-Id: I827826e9ff8a9a534c50250458b399104478e06c
Signed-off-by: Nicholas Chin <nic.c3.14(a)gmail.com>
---
M src/mainboard/dell/snb_ivb_latitude/Kconfig
M src/mainboard/dell/snb_ivb_latitude/Kconfig.name
A src/mainboard/dell/snb_ivb_latitude/variants/e6330/data.vbt
A src/mainboard/dell/snb_ivb_latitude/variants/e6330/early_init.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6330/gpio.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6330/hda_verb.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6330/overridetree.cb
7 files changed, 288 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/52/82152/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/82152?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I827826e9ff8a9a534c50250458b399104478e06c
Gerrit-Change-Number: 82152
Gerrit-PatchSet: 4
Gerrit-Owner: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Attention is currently required from: Felix Singer.
Nicholas Chin has posted comments on this change by Nicholas Chin. ( https://review.coreboot.org/c/coreboot/+/82125?usp=email )
Change subject: mb/dell: Add Latitude E5530 (Ivy Bridge)
......................................................................
Patch Set 3:
(2 comments)
File src/mainboard/dell/snb_ivb_latitude/variants/e5530/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/82125/comment/7231132a_e8652439?us… :
PS2, Line 4: register "gpu_cpu_backlight" = "0x00000000"
: register "gpu_pch_backlight" = "0x03d003d0"
:
> Move into igpu device scope
Done
https://review.coreboot.org/c/coreboot/+/82125/comment/0303d908_e3262185?us… :
PS2, Line 11: register "superspeed_capable_ports" = "0x0000000f"
: register "xhci_overcurrent_mapping" = "0x00000c03"
:
> Move into XHCI scope
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/82125?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Idaf6618df70aa19d8e60b2263088737712dec5f0
Gerrit-Change-Number: 82125
Gerrit-PatchSet: 3
Gerrit-Owner: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Attention: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Comment-Date: Wed, 29 May 2024 20:57:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Attention is currently required from: Felix Singer, Paul Menzel.
Nicholas Chin has posted comments on this change by Nicholas Chin. ( https://review.coreboot.org/c/coreboot/+/79012?usp=email )
Change subject: mb/dell: Add Latitude E6530 (Ivy Bridge)
......................................................................
Patch Set 4:
(2 comments)
File src/mainboard/dell/snb_ivb_latitude/variants/e6530/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/79012/comment/282fc053_6e31e72b?us… :
PS3, Line 4: register "gpu_cpu_backlight" = "0x00000251"
: r
> Move into igpu device scope
Done
https://review.coreboot.org/c/coreboot/+/79012/comment/4a3c61b7_0cead54a?us… :
PS3, Line 11: register "superspeed_capable_ports" = "0x0000000f"
: register "xhci_overcurrent_mapping" = "0x00000c03"
:
> Move into xhci device scope
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/79012?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I9fcd73416018574f8934962f92c8222d0101cb71
Gerrit-Change-Number: 79012
Gerrit-PatchSet: 4
Gerrit-Owner: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Comment-Date: Wed, 29 May 2024 20:57:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Attention is currently required from: Martin L Roth, Nicholas Chin, Paul Menzel.
Hello Martin L Roth, Paul Menzel, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/77444?usp=email
to look at the new patch set (#13).
The following approvals got outdated and were removed:
Verified+1 by build bot (Jenkins)
Change subject: mb/dell: Add Latitude E6430 (Ivy Bridge)
......................................................................
mb/dell: Add Latitude E6430 (Ivy Bridge)
Mainboard is QAL80/LA-7781P (UMA). The version with an Nvidia dGPU was
not tested. This is based on the autoport output with some manual fixes.
The VBT was obtained using `intelvbttool --inlegacy --outvbt data.vbt`
while running version A24 (latest version) of the vendor firmware.
The flash is 8MiB + 4MiB, and can be easily accessed by removing the
keyboard. It can also be internally flashed by sending a command to the
EC, which causes the EC to pull the FDO pin low and the firmware to skip
setting up any chipset based write protections [1]. The EC is the SMSC
MEC5055, which seems to be compatible with the existing MEC5035 code.
Working:
- Libgfxinit
- USB EHCI debug (left side usb port is HCD index 2, middle port on the
right side is HCD index 1) with the CH347
- Keyboard
- Touchpad/trackpoint
- ExpressCard (tested with USB 3.0 card)
- Audio
- Ethernet
- SD card reader
- mPCIe WiFi
- SeaBIOS 1.16.3
- edk2 (MrChromebox's fork, uefipayload_202309)
- Internal flashing using dell-flash-unlock
Not working:
- S3 suspend: Possibly EC related, DRAM power is getting cut when
entering S3
- Physical wireless switch: this triggers an SMI handler in the vendor
firmware which sends commands to the EC to enable/disable wireless
devices, and has not been reimplemented
- Battery reporting: needs ACPI code for the EC
- Brightness hotkeys: probably EC related
- The system reports that the power button was pressed and shuts down
when the CPU hits around 86 degrees Celsius, before the CPU can
thermal throttle. Likely EC and possibly PECI related.
- Integrated keyboard with upstream GRUB 2.12 payload: Upstream GRUB
initializes the 8042 PS/2 controller in a way that is incompatible
with how the EC firmware emulates it. GRUB tries to initialize the
controller with scan code set 2 without translation, but the EC only
ever returns set 1 scan codes to the system and thus is only works as
an untranslated set 1 keyboard or a translated set 2 keyboard,
regardless of commands to set the scan code. A USB keyboard works
fine.
Unknown/untested:
- Dock
- eSATA
- TPM
- dGPU on non-UMA model
- Bluetooth module (not included on my system)
[1] https://gitlab.com/nic3-14159/dell-flash-unlock
Change-Id: I93c6622fc5da1d0d61a5b2c197ac7227d9525908
Signed-off-by: Nicholas Chin <nic.c3.14(a)gmail.com>
---
A src/mainboard/dell/snb_ivb_latitude/Kconfig
A src/mainboard/dell/snb_ivb_latitude/Kconfig.name
A src/mainboard/dell/snb_ivb_latitude/Makefile.mk
A src/mainboard/dell/snb_ivb_latitude/acpi/ec.asl
A src/mainboard/dell/snb_ivb_latitude/acpi/platform.asl
A src/mainboard/dell/snb_ivb_latitude/acpi/superio.asl
A src/mainboard/dell/snb_ivb_latitude/board_info.txt
A src/mainboard/dell/snb_ivb_latitude/cmos.default
A src/mainboard/dell/snb_ivb_latitude/cmos.layout
A src/mainboard/dell/snb_ivb_latitude/devicetree.cb
A src/mainboard/dell/snb_ivb_latitude/dsdt.asl
A src/mainboard/dell/snb_ivb_latitude/gma-mainboard.ads
A src/mainboard/dell/snb_ivb_latitude/mainboard.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6430/data.vbt
A src/mainboard/dell/snb_ivb_latitude/variants/e6430/early_init.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6430/gpio.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6430/hda_verb.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6430/overridetree.cb
18 files changed, 585 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/77444/13
--
To view, visit https://review.coreboot.org/c/coreboot/+/77444?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I93c6622fc5da1d0d61a5b2c197ac7227d9525908
Gerrit-Change-Number: 77444
Gerrit-PatchSet: 13
Gerrit-Owner: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-CC: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-CC: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Attention is currently required from: Nicholas Chin, Paul Menzel.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/79012?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified+1 by build bot (Jenkins)
Change subject: mb/dell: Add Latitude E6530 (Ivy Bridge)
......................................................................
mb/dell: Add Latitude E6530 (Ivy Bridge)
Mainboard is QALA0/LA-7761P (UMA). The version with a Nvidia dGPU was
not tested. I do not physically have this system; someone with physical
access to one sent me the output of autoport which I then modified to
produce this port.
I was also sent the vbios obtained using intel_bios_dumper while running
version A22 of the vendor firmware, which I then processed using
`intelvbttool --inoprom vbios.bin --outvbt data.vbt` to obtain data.vbt.
This was originally tested and found to be working as a standalone board
port in Libreboot, though this variant based port in upstream coreboot
has not been tested.
This can be internally flashed by sending a command to the EC, which
causes the EC to pull the FDO pin low and the firmware to skip setting
up any chipset based write protections [1]. The EC is the SMSC MEC5055,
which seems to be compatible with the existing MEC5035 code.
[1] https://gitlab.com/nic3-14159/dell-flash-unlock
Change-Id: I9fcd73416018574f8934962f92c8222d0101cb71
Signed-off-by: Nicholas Chin <nic.c3.14(a)gmail.com>
---
M src/mainboard/dell/snb_ivb_latitude/Kconfig
M src/mainboard/dell/snb_ivb_latitude/Kconfig.name
A src/mainboard/dell/snb_ivb_latitude/variants/e6530/data.vbt
A src/mainboard/dell/snb_ivb_latitude/variants/e6530/early_init.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6530/gpio.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6530/hda_verb.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6530/overridetree.cb
7 files changed, 287 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/79012/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/79012?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I9fcd73416018574f8934962f92c8222d0101cb71
Gerrit-Change-Number: 79012
Gerrit-PatchSet: 4
Gerrit-Owner: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Attention is currently required from: Nicholas Chin.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/82125?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by build bot (Jenkins)
Change subject: mb/dell: Add Latitude E5530 (Ivy Bridge)
......................................................................
mb/dell: Add Latitude E5530 (Ivy Bridge)
Mainboard is QXW10/LA-7902P (UMA). I do not physically have this board;
someone with physical access to one sent me the output of autoport which
I then modified to produce this port. I was also sent the VBT binary,
which was obtained from `/sys/kernel/debug/dri/0/i915_vbt` while running
version A21 of the vendor firmware.
This was originally tested and found to be working as a standalone board
port in Libreboot, but this variant based port in upstream coreboot has
not been tested.
This can be internally flashed by sending a command to the EC, which
causes the EC to pull the FDO pin low and the firmware to skip setting
up any chipset based write protections [1]. The EC is the SMSC MEC5055,
which seems to be compatible with the existing MEC5035 code.
Change-Id: Idaf6618df70aa19d8e60b2263088737712dec5f0
Signed-off-by: Nicholas Chin <nic.c3.14(a)gmail.com>
---
M src/mainboard/dell/snb_ivb_latitude/Kconfig
M src/mainboard/dell/snb_ivb_latitude/Kconfig.name
A src/mainboard/dell/snb_ivb_latitude/variants/e5530/data.vbt
A src/mainboard/dell/snb_ivb_latitude/variants/e5530/early_init.c
A src/mainboard/dell/snb_ivb_latitude/variants/e5530/gpio.c
A src/mainboard/dell/snb_ivb_latitude/variants/e5530/hda_verb.c
A src/mainboard/dell/snb_ivb_latitude/variants/e5530/overridetree.cb
7 files changed, 290 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/25/82125/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/82125?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Idaf6618df70aa19d8e60b2263088737712dec5f0
Gerrit-Change-Number: 82125
Gerrit-PatchSet: 3
Gerrit-Owner: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Attention: Nicholas Chin <nic.c3.14(a)gmail.com>
Nicholas Chin has uploaded a new patch set (#3). ( https://review.coreboot.org/c/coreboot/+/82126?usp=email )
Change subject: mb/dell: Add Latitude E6420 (Sandy Bridge)
......................................................................
mb/dell: Add Latitude E6420 (Sandy Bridge)
Mainboard is PAL50/LA-6591P (UMA). The version with an Nvidia dGPU was
not tested. I do not physically have this system; someone with physical
access to one sent me the output of autoport which I then modified to
produce this port. I was also sent the VBT binary, which was obtained
from `/sys/kernel/debug/dri/0/i915_vbt` while running version A25 of the
vendor firmware.
This was originally tested and found to be working as a standalone board
port in Libreboot, but this variant based port in upstream coreboot has
not been tested.
This can be internally flashed by sending a command to the EC, which
causes the EC to pull the FDO pin low and the firmware to skip setting
up any chipset based write protections [1]. The EC is the SMSC MEC5055,
which seems to be compatible with the existing MEC5035 code.
[1] https://gitlab.com/nic3-14159/dell-flash-unlock
Change-Id: Ic48d9ea58172a5b13958c8afebcb19c8929c4394
Signed-off-by: Nicholas Chin <nic.c3.14(a)gmail.com>
---
M src/mainboard/dell/snb_ivb_latitude/Kconfig
M src/mainboard/dell/snb_ivb_latitude/Kconfig.name
A src/mainboard/dell/snb_ivb_latitude/variants/e6420/data.vbt
A src/mainboard/dell/snb_ivb_latitude/variants/e6420/early_init.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6420/gpio.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6420/hda_verb.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6420/overridetree.cb
7 files changed, 288 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/82126/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/82126?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ic48d9ea58172a5b13958c8afebcb19c8929c4394
Gerrit-Change-Number: 82126
Gerrit-PatchSet: 3
Gerrit-Owner: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Nicholas Chin has uploaded a new patch set (#3). ( https://review.coreboot.org/c/coreboot/+/82127?usp=email )
Change subject: mb/dell: Add Latitude E6520 (Sandy Bridge)
......................................................................
mb/dell: Add Latitude E6520 (Sandy Bridge)
Mainboard is PAL60/LA-6562P (UMA). The version with an Nvidia dGPU was
not tested. I do not physically have this system; someone with physical
access to one sent me the output of autoport which I then modified to
produce this port. I was also sent the VBT binary, which was obtained
from `/sys/kernel/debug/dri/0/i915_vbt` while running version A08 of the
vendor firmware.
This was originally tested and found to be working as a standalone board
port in Libreboot, but this variant based port in upstream coreboot has
not been tested.
This can be internally flashed by sending a command to the EC, which
causes the EC to pull the FDO pin low and the firmware to skip setting
up any chipset based write protections [1]. The EC is the SMSC MEC5055,
which seems to be compatible with the existing MEC5035 code.
[1] https://gitlab.com/nic3-14159/dell-flash-unlock
Change-Id: Ibdd40cc15642b8d404159d5962670ccc4167a9ec
Signed-off-by: Nicholas Chin <nic.c3.14(a)gmail.com>
---
M src/mainboard/dell/snb_ivb_latitude/Kconfig
M src/mainboard/dell/snb_ivb_latitude/Kconfig.name
A src/mainboard/dell/snb_ivb_latitude/variants/e6520/data.vbt
A src/mainboard/dell/snb_ivb_latitude/variants/e6520/early_init.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6520/gpio.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6520/hda_verb.c
A src/mainboard/dell/snb_ivb_latitude/variants/e6520/overridetree.cb
7 files changed, 284 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/82127/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/82127?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ibdd40cc15642b8d404159d5962670ccc4167a9ec
Gerrit-Change-Number: 82127
Gerrit-PatchSet: 3
Gerrit-Owner: Nicholas Chin <nic.c3.14(a)gmail.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>