[coreboot-gerrit] Change in coreboot[master]: amd/pi/hudson: Add LPC IO decode enable function
Martin Roth (Code Review)
gerrit at coreboot.org
Tue Apr 25 18:26:47 CEST 2017
Martin Roth has posted comments on this change. ( https://review.coreboot.org/19160 )
Change subject: amd/pi/hudson: Add LPC IO decode enable function
......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/#/c/19160/1/src/southbridge/amd/pi/hudson/early_setup.c
File src/southbridge/amd/pi/hudson/early_setup.c:
PS1, Line 184: assert
> Doesn't it die and tell you the file function and line? This should always
No, by default, assert is NOT fatal. It's a Kconfig option.
https://review.coreboot.org/cgit/coreboot.git/tree/src/Kconfig#n693
PS1, Line 204: assert(0);
> either the developer finds this and it stops or we just continue on if asse
As mentioned above, assert is not fatal by default, so it seems likely that the developer will never see this.
https://review.coreboot.org/#/c/19160/12/src/southbridge/amd/pi/hudson/early_setup.c
File src/southbridge/amd/pi/hudson/early_setup.c:
PS12, Line 190: bas
Aaron earlier mentioned getting rid of this.
--
To view, visit https://review.coreboot.org/19160
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2bed3a99180188101e00b4431d634227e488cbda
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Marc Jones <marc at marcjonesconsulting.com>
Gerrit-Reviewer: Aaron Durbin <adurbin at chromium.org>
Gerrit-Reviewer: Marc Jones <marc at marcjonesconsulting.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd at gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins)
Gerrit-HasComments: Yes
More information about the coreboot-gerrit
mailing list