30 May
2021
30 May
'21
4:14 p.m.
Attention is currently required from: Patrick Rudolph. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55082 )
Change subject: soc/intel/elkhartlake: FSP-S storage related configs ......................................................................
Patch Set 1:
(1 comment)
File src/soc/intel/elkhartlake/fsp_params.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120328): https://review.coreboot.org/c/coreboot/+/55082/comment/b1851bb7_4cb79b6c PS1, Line 248: params->SataPortsDitoVal[i] = config->SataPortsDitoVal[i].enable ? line over 96 characters
--
To view, visit https://review.coreboot.org/c/coreboot/+/55082
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id4145fcf156756a610b8a9a705d4ab99fe7b0bf8
Gerrit-Change-Number: 55082
Gerrit-PatchSet: 1
Gerrit-Owner: Lean Sheng Tan lean.sheng.tan@intel.com
Gerrit-Reviewer: Patrick Rudolph siro@das-labor.org
Gerrit-CC: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Attention: Patrick Rudolph siro@das-labor.org
Gerrit-Comment-Date: Sun, 30 May 2021 16:14:43 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment