Hi Cristian,
On Mon, 25 Aug 2025 at 10:57, Cristian Ciocaltea
<cristian.ciocal...@collabora.com> wrote:
> +static int dw_hdmi_qp_cec_init(struct drm_bridge *bridge,
> + struct drm_connector *connector)
> +{
> + struct dw_hdmi_qp *hdmi = dw_hdmi_qp_from_bridge(bridge);
> + struct dw_hdmi_qp_cec *cec = hdmi->cec;
> + int ret;
> +
> + if (cec->irq < 0) {
> + dev_err(hdmi->dev, "Invalid cec irq: %d\n", cec->irq);
> + return -EINVAL;
> + }
There is a bisect break here until patch 4/6 as nothing provides the
CEC IRQ, so the whole connector init will fail.
You should either plumb the IRQ through first, or just make it
optional to retain compatibility.
Cheers,
Daniel
