build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25105 )
Change subject: drivers/spi: Winbond specific write-protection enable ......................................................................
Patch Set 18:
(7 comments)
https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c File src/drivers/spi/winbond.c:
https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@33 PS18, Line 33: #define CMD_VOLATILE_SREG_WREN 0x50 /* Write Enable for Volatile Status Register */ line over 80 characters
https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@494 PS18, Line 494: val = (union status_reg1_bp3){ .bp = bp, .tb = tb, .sec = 0 }.u; space required after that close brace '}'
https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@495 PS18, Line 495: mask = (union status_reg1_bp3){ .bp = ~0, .tb = 1, .sec = 1 }.u; space required after that close brace '}'
https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@497 PS18, Line 497: val = (union status_reg1_bp4){ .bp = bp, .tb = tb }.u; space required after that close brace '}'
https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@498 PS18, Line 498: mask = (union status_reg1_bp4){ .bp = ~0, .tb = 1 }.u; space required after that close brace '}'
https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@505 PS18, Line 505: val = (union status_reg2){ .cmp = cmp }.u; space required after that close brace '}'
https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@506 PS18, Line 506: mask = (union status_reg2){ .cmp = 1 }.u; space required after that close brace '}'
-
build bot (Jenkins) (Code Review)