diff options
| author | Hans Verkuil <hverkuil+cisco@kernel.org> | 2026-07-10 13:07:32 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2026-07-17 10:14:06 +0200 |
| commit | 2ff4b91e82bd76e1a6e1ff22bfc9aeda08e12a61 (patch) | |
| tree | f1abfb04403fee2afac6dc65303146bb8e87bff8 | |
| parent | 67d95987201591a91067a3e49ebef265fd968df6 (diff) | |
| download | linux-2ff4b91e82bd76e1a6e1ff22bfc9aeda08e12a61.tar.gz linux-2ff4b91e82bd76e1a6e1ff22bfc9aeda08e12a61.zip | |
media: cec/core: drop cec_post_state_event call
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 <hverkuil+cisco@kernel.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
| -rw-r--r-- | drivers/media/cec/core/cec-adap.c | 1 |
1 files changed, 0 insertions, 1 deletions
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); |