Attention is currently required from: Paul Menzel, Angel Pons, Name of user not set #1004271.
Stefan Ott has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40678 )
Change subject: mb/lenovo/t60: Add support for ThinkLight
......................................................................
Patch Set 7:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/40678/comment/48ced88c_4910c8b0
PS2, Line 20: Unfortunately I do not own a T60 to test this.
I think it’s formally not needed, and you can just tag the comment as resolved.
Ack
Commit Message:
https://review.coreboot.org/c/coreboot/+/40678/comment/09e028fb_eb2c091f
PS4, Line 21: Signed-off-by: Stefan Ott stefan@ott.net
Yes, you can add it yourself
Ack
Patchset:
PS7:
Okay, thanks
--
To view, visit
https://review.coreboot.org/c/coreboot/+/40678
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I47f878533d36857d002d2e2605cc8bc7e1d960c9
Gerrit-Change-Number: 40678
Gerrit-PatchSet: 7
Gerrit-Owner: Stefan Ott
coreboot@desire.ch
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Name of user not set #1004271
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Name of user not set #1004271
Gerrit-Comment-Date: Sat, 07 May 2022 23:00:39 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Stefan Ott
coreboot@desire.ch
Comment-In-Reply-To: Felix Singer
felixsinger@posteo.net
Comment-In-Reply-To: Paul Menzel
paulepanter@mailbox.org
Comment-In-Reply-To: Angel Pons
th3fanbus@gmail.com
Comment-In-Reply-To: Name of user not set #1004271
Comment-In-Reply-To: Alexander Couzens
lynxis@fe80.eu
Comment-In-Reply-To: Cuno
Gerrit-MessageType: comment