GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ec/lenovo/pmh7: use read/write function in clear_bit/set_bit 

Make the code simpler and improve readability.

Change-Id: Ifa9308c32e4646c122254931b55fb83541a10a3c
Signed-off-by: Alexander Couzens <lynxis@fe80.eu>
Reviewed-on: https://review.coreboot.org/28195
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
Alexander Couzens 2018-08-17 18:36:56 +02:00 committed by Felix Held
parent b0622de878
commit 74ab031ba1
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/ec/lenovo/pmh7/pmh7.c
View File

@ -68,20 +68,16 @@ void pmh7_register_set_bit(int reg, int bit)
{
char val;
outb(reg, EC_LENOVO_PMH7_ADDR);
val = inb(EC_LENOVO_PMH7_DATA);
outb(reg, EC_LENOVO_PMH7_ADDR);
outb(val | (1 << bit), EC_LENOVO_PMH7_DATA);
val = pmh7_register_read(reg);
pmh7_register_write(reg, val | (1 << bit));
}
void pmh7_register_clear_bit(int reg, int bit)
{
char val;
outb(reg, EC_LENOVO_PMH7_ADDR);
val = inb(EC_LENOVO_PMH7_DATA);
outb(reg, EC_LENOVO_PMH7_ADDR);
outb(val &= ~(1 << bit), EC_LENOVO_PMH7_DATA);
val = pmh7_register_read(reg);
pmh7_register_write(reg, val & ~(1 << bit));
}
char pmh7_register_read(int reg)