Marc Jones has posted comments on this change. ( https://review.coreboot.org/19724 ) Change subject: soc/amd/stoneyridge: Add northbridge support ...................................................................... Patch Set 4: Issues opened or updated based on the this review: Kahlee: coreboot -stoneyridge - move dram top setup 62240756 Kahlee: coreboot - fix src/soc/amd/common/agesawrapper_call.h 62240989 Kahlee: coreboot - stoneyridge - use coreboot register write functions 62241048 Kahlee: coreboot - stonyridge - fix northbridge.c 62240927 Kahlee: coreboot - stoneyridge - fix checkpatch warnings and errors 62240746 Kahlee: coreboot -stoneyridge - include headers for filetypes used. 62239898 Kahlee: coreboot - stoneyridge - fix soc/amd/common include files 62240201 Kahlee: coreboot -stoneyridge - fix agesa header file include paths / namespace 62239980 -- To view, visit https://review.coreboot.org/19724 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie86b4d744900f23502068517ece5bcea6c128993 Gerrit-PatchSet: 4 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Marc Jones <marc@marcjonesconsulting.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Marc Jones <marc@marcjonesconsulting.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-HasComments: No