[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 06/10] misc/pca9552: Fix LED status register indexing in pca955x_
From: |
Cédric Le Goater |
Subject: |
[PATCH 06/10] misc/pca9552: Fix LED status register indexing in pca955x_get_led() |
Date: |
Mon, 9 Aug 2021 15:15:52 +0200 |
From: Andrew Jeffery <andrew@aj.id.au>
There was a bit of a thinko in the state calculation where every odd pin
in was reported in e.g. "pwm0" mode rather than "off". This was the
result of an incorrect bit shift for the 2-bit field representing each
LED state.
Fixes: a90d8f84674d ("misc/pca9552: Add qom set and get")
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <20210723043624.348158-1-andrew@aj.id.au>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
---
hw/misc/pca9552.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/misc/pca9552.c b/hw/misc/pca9552.c
index b7686e27d7fa..fff19e369a39 100644
--- a/hw/misc/pca9552.c
+++ b/hw/misc/pca9552.c
@@ -272,7 +272,7 @@ static void pca955x_get_led(Object *obj, Visitor *v, const
char *name,
* reading the INPUTx reg
*/
reg = PCA9552_LS0 + led / 4;
- state = (pca955x_read(s, reg) >> (led % 8)) & 0x3;
+ state = (pca955x_read(s, reg) >> ((led % 4) * 2)) & 0x3;
visit_type_str(v, name, (char **)&led_state[state], errp);
}
--
2.31.1
- [PATCH 03/10] watchdog: aspeed: Fix sequential control writes, (continued)
- [PATCH 06/10] misc/pca9552: Fix LED status register indexing in pca955x_get_led(),
Cédric Le Goater <=
- [PATCH 07/10] arm/aspeed: rainier: Add i2c eeproms and muxes, Cédric Le Goater, 2021/08/09
- [PATCH 10/10] arm/aspeed: Add DPS310 to Witherspoon and Rainier, Cédric Le Goater, 2021/08/09
- [PATCH 05/10] hw: aspeed_gpio: Clarify GPIO controller name, Cédric Le Goater, 2021/08/09
- [PATCH 08/10] aspeed: Emulate the AST2600A3, Cédric Le Goater, 2021/08/09