* [PATCH] misc/pca9552: Fix LED status register indexing in pca955x_get_led()
@ 2021-07-23 4:36 Andrew Jeffery
2021-07-23 7:53 ` Cédric Le Goater
0 siblings, 1 reply; 2+ messages in thread
From: Andrew Jeffery @ 2021-07-23 4:36 UTC (permalink / raw)
To: clg; +Cc: peter.maydell, qemu-arm, joel, qemu-devel
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>
---
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.30.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] misc/pca9552: Fix LED status register indexing in pca955x_get_led()
2021-07-23 4:36 [PATCH] misc/pca9552: Fix LED status register indexing in pca955x_get_led() Andrew Jeffery
@ 2021-07-23 7:53 ` Cédric Le Goater
0 siblings, 0 replies; 2+ messages in thread
From: Cédric Le Goater @ 2021-07-23 7:53 UTC (permalink / raw)
To: Andrew Jeffery; +Cc: peter.maydell, qemu-arm, joel, qemu-devel
On 7/23/21 6:36 AM, Andrew Jeffery wrote:
> 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>
I don't think we need this fix for 6.1 since it has been there since
January 2020. Tell me otherwise.
Thanks,
C.
> ---
> 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);
> }
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-07-23 7:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-23 4:36 [PATCH] misc/pca9552: Fix LED status register indexing in pca955x_get_led() Andrew Jeffery
2021-07-23 7:53 ` Cédric Le Goater
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.