[coreboot-gerrit] Change in coreboot[master]: siemens/mc_apl1: Set coreboot ready LED

[Mario Scheithauer (Code Review)]

Mario Scheithauer has posted comments on this change. ( https://review.coreboot.org/22136 )

Change subject: siemens/mc_apl1: Set coreboot ready LED
......................................................................


Patch Set 2:

(2 comments)

https://review.coreboot.org/#/c/22136/1/src/mainboard/siemens/mc_apl1/mainboard.c
File src/mainboard/siemens/mc_apl1/mainboard.c:

https://review.coreboot.org/#/c/22136/1/src/mainboard/siemens/mc_apl1/mainboard.c@177
PS1, Line 177: 
> Do we really need this info in the console? If yes, then set the level upo 
No, you're right. We don't need this info additional.


https://review.coreboot.org/#/c/22136/1/src/mainboard/siemens/mc_apl1/mainboard.c@180
PS1, Line 180: BOOT_STATE_INIT_ENTRY(BS_PAYLOAD_BOOT, BS_ON_ENTRY, finalize_boot, NULL);
> Skip this additional empty line.
Done



-- 
To view, visit https://review.coreboot.org/22136
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I179d013746c1334337dc9e6b7f09ac54eff0cd77
Gerrit-Change-Number: 22136
Gerrit-PatchSet: 2
Gerrit-Owner: Mario Scheithauer <mario.scheithauer at siemens.com>
Gerrit-Reviewer: Mario Scheithauer <mario.scheithauer at siemens.com>
Gerrit-Reviewer: Werner Zeh <werner.zeh at siemens.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-Comment-Date: Wed, 25 Oct 2017 10:59:44 +0000
Gerrit-HasComments: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20171025/06ccdffe/attachment.html>