Attention is currently required from: Patrick Rudolph, Christian Walter.

Angel Pons uploaded patch set #2 to this change.

View Change

mb/prodrive/hermes: Clean up some cosmetics

Use lowercase for hex numbers, sort includes alphabetically and avoid
relying on indirect inclusion. Include `<intelblocks/gpio.h>` instead
of `<intelblocks/gpio_defs.h>`, as the latter implcitly relies on one
definition from `<soc/gpio.h>`. Also drop useless dsdt.asl and fix up
the indentation of some includes.

Tested with BUILD_TIMELESS=1, Prodrive Hermes remains identical.

Change-Id: I3aeb9a644cf33cb4b1987174f40ef0fc7daccfa9
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
---
M src/mainboard/prodrive/hermes/dsdt.asl
M src/mainboard/prodrive/hermes/eeprom.c
M src/mainboard/prodrive/hermes/eeprom.h
M src/mainboard/prodrive/hermes/gpio.c
M src/mainboard/prodrive/hermes/hda_verb.c
M src/mainboard/prodrive/hermes/mainboard.c
M src/mainboard/prodrive/hermes/romstage.c
M src/mainboard/prodrive/hermes/smbios.c
8 files changed, 25 insertions(+), 26 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/59672/2

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3aeb9a644cf33cb4b1987174f40ef0fc7daccfa9
Gerrit-Change-Number: 59672
Gerrit-PatchSet: 2
Gerrit-Owner: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Christian Walter <christian.walter@9elements.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Attention: Christian Walter <christian.walter@9elements.com>
Gerrit-MessageType: newpatchset