On Fri Jul 10 13:07:32 2026 +0200, Hans Verkuil wrote:
> No need to call cec_post_state_event() if the next call
> is to cec_adap_unconfigure() since that will also call
> cec_post_state_event().
>
> This fixes the case when, when the physical address is
> invalidated, you get two CEC_EVENT_STATE_CHANGE events:
> one with a non-zero log_addr_mask and one with a zero
> log_addr_mask.
>
> You just want to see the event with phys_addr set to
> f.f.f.f and log_addr_mask set to 0.
>
> Signed-off-by: Hans Verkuil <[email protected]>
> Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Patch committed.
Thanks,
Mauro Carvalho Chehab
drivers/media/cec/core/cec-adap.c | 1 -
1 file changed, 1 deletion(-)
---
diff --git a/drivers/media/cec/core/cec-adap.c
b/drivers/media/cec/core/cec-adap.c
index bbd5395fa67d..5c7ede88c3df 100644
--- a/drivers/media/cec/core/cec-adap.c
+++ b/drivers/media/cec/core/cec-adap.c
@@ -1708,7 +1708,6 @@ void __cec_s_phys_addr(struct cec_adapter *adap, u16
phys_addr, bool block)
cec_phys_addr_exp(phys_addr));
if (becomes_invalid || !is_invalid) {
adap->phys_addr = CEC_PHYS_ADDR_INVALID;
- cec_post_state_event(adap);
cec_adap_unconfigure(adap);
if (becomes_invalid) {
cec_adap_enable(adap);
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
