[coreboot-gerrit] Change in coreboot[master]: drivers/storage: Fix formatting issues detected by checkpatch

Lee Leahy (Code Review) gerrit at coreboot.org
Mon Mar 27 23:56:11 CEST 2017


Hello Andrey Petrov, Aaron Durbin, Bora Guvendik, build bot (Jenkins), Martin Roth,

I'd like you to reexamine a change.  Please visit

    https://review.coreboot.org/19009

to look at the new patch set (#2).

Change subject: drivers/storage: Fix formatting issues detected by checkpatch
......................................................................

drivers/storage: Fix formatting issues detected by checkpatch

Fix the following errors and warnings detected by checkpatch.pl:

ERROR: switch and case should be at the same indent
ERROR: spaces prohibited around that '->' (ctx:VxW)
WARNING: line over 80 characters
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
WARNING: suspect code indent for conditional statements (24, 28)

These issues should not change the binary.

TEST=None

Change-Id: I6df6bc5f5342bbdd024a96f2396af558d8cdbc49
Signed-off-by: Lee Leahy <Leroy.P.Leahy at intel.com>
---
M src/drivers/storage/mmc.c
M src/drivers/storage/sdhci.c
M src/include/device/sdhci.h
3 files changed, 50 insertions(+), 49 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/09/19009/2
-- 
To view, visit https://review.coreboot.org/19009
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6df6bc5f5342bbdd024a96f2396af558d8cdbc49
Gerrit-PatchSet: 2
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Lee Leahy <leroy.p.leahy at intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin at chromium.org>
Gerrit-Reviewer: Andrey Petrov <andrey.petrov at intel.com>
Gerrit-Reviewer: Bora Guvendik <bora.guvendik at intel.com>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: build bot (Jenkins)



More information about the coreboot-gerrit mailing list