Attention is currently required from: Angel Pons, Martin Roth.

Edward O'Callaghan uploaded patch set #2 to this change.

View Change

amd_imc.c: Cleanup C89ism and redundant NULL check

A bit of spring cleaning here to freshen up some C89ism's,
drop a redundant NULLality check in a static local function
and canonicalise branch predictions. Also const correct some
static locals and add missing include while here.

This makes the code overall shorter, more consistent and
succinct.

Change-Id: I1dbae500010fa50a67609efb3cbb5778b684ad04
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
---
M amd_imc.c
1 file changed, 29 insertions(+), 24 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/59/75359/2

To view, visit change 75359. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I1dbae500010fa50a67609efb3cbb5778b684ad04
Gerrit-Change-Number: 75359
Gerrit-PatchSet: 2
Gerrit-Owner: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Martin Roth <martin.roth@amd.corp-partner.google.com>
Gerrit-Attention: Angel Pons <th3fanbus@gmail.com>
Gerrit-Attention: Martin Roth <martin.roth@amd.corp-partner.google.com>
Gerrit-MessageType: newpatchset