Attention is currently required from: Nico Huber, Angel Pons, Arthur Heymans.

Angel Pons uploaded patch set #7 to this change.

View Change

[TESTME] drivers/pc80/rtc/option.c: Allow integer options to cross byte boundaries

Redesign CMOS option handling code to allow accessing integer options
whose bit offset crosses byte boundaries. This allows cramming more CMOS
settings in the same space without having to play Tetris.

The new system is most likely not backwards-compatible for options that
cross byte boundaries (it was not meant to be backwards-compatible).
However, this is a non-issue, as all CMOS options used in coreboot never
cross byte boundaries.

Change-Id: Id1e0ed05efeaaa999223bd209f977a096fa39e15
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
---
M src/drivers/pc80/rtc/option.c
1 file changed, 64 insertions(+), 71 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/52647/7

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id1e0ed05efeaaa999223bd209f977a096fa39e15
Gerrit-Change-Number: 52647
Gerrit-PatchSet: 7
Gerrit-Owner: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-CC: Stefan Reinauer <stefan.reinauer@coreboot.org>
Gerrit-Attention: Nico Huber <nico.h@gmx.de>
Gerrit-Attention: Angel Pons <th3fanbus@gmail.com>
Gerrit-Attention: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-MessageType: newpatchset