Thread (13 messages) 13 messages, 3 authors, 2020-06-20

Re: [PATCH v5 2/2] media: cec: i2c: ch7322: Add ch7322 CEC controller driver

From: Jeff Chase <hidden>
Date: 2020-06-20 04:56:55
Also in: linux-media

quoted
quoted
quoted
This particular chip can actually be used both with DMI matching but also
on an ARM with device tree, but since you can't test this on an ARM board,
there is no point in adding support for that.

However, compared to the cros-ec and seco drivers you can do something a bit
different here: those drivers just return -ENODEV if there is no match, but
since this driver reads the EDID it can just continue as long as it does not
set the CEC_CAP_CONNECTOR_INFO capability.
Hi Hans,

After looking into this I don't think I want to actually register the
cec adapter with the hdmi port's cec notifier because then it will
start receiving physical address updates from the port. Since the
driver already updates the physical address itself it seems like that
would cause some issues.

I'm looking at just adding the connector to the adapter directly using
`cec_s_conn_info()`. But it doesn't look like I can get the conn_info
without registering with the notifier. Do you have any thoughts here
before I try tackling changes to the cec framework?

Thanks,
Jeff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help