Marco Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40836 )
Change subject: mb/google/dedede: Read DRAM part number from CBI
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40836/2/src/mainboard/google/dedede...
File src/mainboard/google/dedede/romstage.c:
https://review.coreboot.org/c/coreboot/+/40836/2/src/mainboard/google/dedede...
PS2, Line 42: strlen(part_num_store)
Did you mean to use sizeof(part_num_store) instead of strlen?
Just think why do we need to return a fixed size which normally is bigger then size of part number instead of a real length?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/40836
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I40780a35e04efb279591e9db179cb86b5e907c0d
Gerrit-Change-Number: 40836
Gerrit-PatchSet: 2
Gerrit-Owner: Marco Chen
marcochen@google.com
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Justin TerAvest
teravest@chromium.org
Gerrit-Reviewer: Karthik Ramasubramanian
kramasub@google.com
Gerrit-Reviewer: Marco Chen
marcochen@chromium.org
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Thu, 30 Apr 2020 23:34:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Karthik Ramasubramanian
kramasub@google.com
Gerrit-MessageType: comment