build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68685 )
Change subject: src/drivers/ipmi: accomodate BMC revision being 0
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161025):
https://review.coreboot.org/c/coreboot/+/68685/comment/ea8f3e53_c41bdbda
PS1, Line 9: BMC major/minor revision may be 0. Get the value directly from
'accomodate' may be misspelled - perhaps 'accommodate'?
--
To view, visit https://review.coreboot.org/c/coreboot/+/68685
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0e08c6d02de8f6efceb69b6d6cebad9d61cfd20e
Gerrit-Change-Number: 68685
Gerrit-PatchSet: 1
Gerrit-Owner: Jonathan Zhang <jonzhang(a)fb.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sat, 22 Oct 2022 01:12:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61307 )
Change subject: console: Add ANSI escape sequences for highlighting
......................................................................
Patch Set 8:
(1 comment)
File src/console/Kconfig:
https://review.coreboot.org/c/coreboot/+/61307/comment/21c14311_dc90779c
PS8, Line 400: default y
> Out of curiosity, is there a specific reason to enable this option by default instead of using site- […]
I was just trying to guess what would be the preferred option for most users. I understand that there may be some use cases where this is annoying, but on the other side if it is off by default most users would probably never discover it in the first place. I would assume that we have a lot more developers reading their UART in a terminal window than those who pipe it into a file, but if you disagree I don't know what's the best way to find out for sure... maybe we should hold a poll on the mailing list or something?
Of course, for use cases where this is causing issues it's easy to disable (that's why I made it a separate Kconfig). The tokens are also easy to strip out after the fact, e.g. by piping through `sed -e "s/\x1b\[[^m]\{1,9\}m//g"`.
--
To view, visit https://review.coreboot.org/c/coreboot/+/61307
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I868f4026918bc0e967c32e14bcf3ac05816415e8
Gerrit-Change-Number: 61307
Gerrit-PatchSet: 8
Gerrit-Owner: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: 9elements QA <hardwaretestrobot(a)gmail.com>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Comment-Date: Sat, 22 Oct 2022 00:08:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Martin Roth.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68684 )
Change subject: soc/amd: Add framework for Glinda SoC
......................................................................
Patch Set 1:
(8 comments)
File src/soc/amd/glinda/fsp_m_params.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161024):
https://review.coreboot.org/c/coreboot/+/68684/comment/8c946d52_4a05c0fc
PS1, Line 32: for (i = 0; i < num; i++) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161024):
https://review.coreboot.org/c/coreboot/+/68684/comment/f7e93037_20b23ae5
PS1, Line 45: for (i = 0; i < num; i++) {
braces {} are not necessary for single statement blocks
File src/soc/amd/glinda/root_complex.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161024):
https://review.coreboot.org/c/coreboot/+/68684/comment/a5109b80_4b8c26ea
PS1, Line 86: * | | X86_RESET_VECTOR = BOOTBLOCK_END - 0x10
line length of 98 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161024):
https://review.coreboot.org/c/coreboot/+/68684/comment/630f5419_b034a6d2
PS1, Line 89: * +--------------------------------+ BOOTBLOCK_ADDR = BOOTBLOCK_END - C_ENV_BOOTBLOCK_SIZE
line length of 111 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161024):
https://review.coreboot.org/c/coreboot/+/68684/comment/40e94b98_7dc695f8
PS1, Line 94: * +--------------------------------+ PSP_SHAREDMEM_BASE + PSP_SHAREDMEM_SIZE + PRERAM_CBMEM_CONSOLE_SIZE + 0x200
line length of 133 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161024):
https://review.coreboot.org/c/coreboot/+/68684/comment/e7485398_6260aee8
PS1, Line 96: * +--------------------------------+ PSP_SHAREDMEM_BASE + PSP_SHAREDMEM_SIZE + PRERAM_CBMEM_CONSOLE_SIZE
line length of 125 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161024):
https://review.coreboot.org/c/coreboot/+/68684/comment/8852d7bd_49692a55
PS1, Line 99: * +--------------------------------+ PSP_SHAREDMEM_BASE + PSP_SHAREDMEM_SIZE
line length of 97 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161024):
https://review.coreboot.org/c/coreboot/+/68684/comment/4707cdf7_1e0b4eed
PS1, Line 181: reserved_ram_resource_kb(dev, idx++, res->addr / KiB, res->length / KiB);
line length of 97 exceeds 96 columns
--
To view, visit https://review.coreboot.org/c/coreboot/+/68684
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I43d0fdb711c441dc410a14f6bb04b808abefe920
Gerrit-Change-Number: 68684
Gerrit-PatchSet: 1
Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-Comment-Date: Fri, 21 Oct 2022 22:44:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Elyes Haouas.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68651 )
Change subject: Update arm-trusted-firmware submodule to upstream master
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/68651
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic32c0889961f529e1762b208ef118a94369c34e6
Gerrit-Change-Number: 68651
Gerrit-PatchSet: 1
Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Comment-Date: Fri, 21 Oct 2022 22:30:14 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment