From 5e21cd450c13176450e817fdf470925d45daaf05 Mon Sep 17 00:00:00 2001 From: Werner Zeh Date: Fri, 30 Jun 2017 10:05:06 +0200 Subject: vendorcode/siemens: Fix bug in hwilib for FF_FanReq and FF_FreezeDis The two flags FF_FanReq and FF_Freeze_Dis should be treated as 8 bit values and not as 32 bit. Change the length of these fields to 1 byte and adjust the offset and mask for FF_FanReq. Change-Id: If5e9e8b16491a33997cc39c3dd9b80f97682adc6 Signed-off-by: Werner Zeh Reviewed-on: https://review.coreboot.org/20431 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Stefan Reinauer --- src/vendorcode/siemens/hwilib/hwilib.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/vendorcode/siemens/hwilib/hwilib.c b/src/vendorcode/siemens/hwilib/hwilib.c index c1533e80e2..08e8da98ff 100644 --- a/src/vendorcode/siemens/hwilib/hwilib.c +++ b/src/vendorcode/siemens/hwilib/hwilib.c @@ -143,14 +143,14 @@ static const struct param_info params[] = { .pos[0] = {.blk_type = BLK_HIB, .offset = 0xe0, .len = 0x80}, .get_field = hwilib_read_bytes }, [FF_FreezeDis] = { - .pos[0] = {.blk_type = BLK_HIB, .offset = 0x1b8, .len = 4}, + .pos[0] = {.blk_type = BLK_HIB, .offset = 0x1b8, .len = 1}, .mask = 0x10, .mask_offset = 4, .get_field = hwilib_read_bytes }, [FF_FanReq] = { - .pos[0] = {.blk_type = BLK_HIB, .offset = 0x1b8, .len = 4}, - .mask = 0x400, - .mask_offset = 10, + .pos[0] = {.blk_type = BLK_HIB, .offset = 0x1b9, .len = 1}, + .mask = 0x04, + .mask_offset = 2, .get_field = hwilib_read_bytes }, [NvramVirtTimeDsaveReset] = { .pos[0] = {.blk_type = BLK_HIB, .offset = 0x1be, .len = 2}, -- cgit v1.2.3