On Mon, Feb 23, 2026 at 05:20:45PM +0800, Hermes Wu via B4 Relay wrote:
> From: Hermes Wu <[email protected]>
> 
> Add support for the ITE IT6162 MIPI DSI to HDMI 2.0 bridge chip.
> The IT6162 is an I2C-controlled bridge that supports the following
> configurations:
> 
>   - Single MIPI DSI input: up to 4K @ 30Hz
>   - Dual MIPI DSI input (combined): up to 4K @ 60Hz
> 
> The driver implements the DRM bridge and connector frameworks,
> including mode setting, EDID retrieval, and HPD support.
> 
> Signed-off-by: Hermes Wu <[email protected]>
> ---
>  drivers/gpu/drm/bridge/Kconfig      |   17 +
>  drivers/gpu/drm/bridge/Makefile     |    1 +
>  drivers/gpu/drm/bridge/ite-it6162.c | 1876 
> +++++++++++++++++++++++++++++++++++
>  3 files changed, 1894 insertions(+)
> 
> +
> +     mipicfg->pn_swap = of_property_present(endpoint,
> +                                            "ite,mipi-dsi-phy-pn-swap");

None of the properties you use here are described in the bindings.

What is the actual meaning of this value?


> +     mipicfg->lane_swap = of_property_present(endpoint,
> +                                              "ite,mipi-dsi-phy-link-swap");

The same. If I understand the name, you should use the order of lanes in
data-lanes.

> +
> +     if (of_property_present(endpoint, 
> "ite,mipi-dsi-mode-video-sync-pulse")) {
> +             mipicfg->mode_flags |= MIPI_DSI_MODE_VIDEO_SYNC_PULSE;
> +             mipicfg->mode = SYNC_PULSE;

What for?

> +     }
> +
> +     if (of_property_present(endpoint, "ite,mipi-dsi-clock-non-continous")) {
> +             mipicfg->mode_flags |= MIPI_DSI_CLOCK_NON_CONTINUOUS;
> +             mipicfg->continuous_clk = false;
> +     }

"clock-noncontinuous" from video-interfaces.yaml

> +
> +     of_node_put(endpoint);
> +
> +     dev_info(dev, "lanes: %d pn_swap: %d, lane_swap: %d, mode_flags: %lu",
> +              mipicfg->lane_num, mipicfg->pn_swap,
> +              mipicfg->lane_swap, mipicfg->mode_flags);
> +}
> +



> +
> +     /* get hdcp support*/
> +     ret = of_property_read_u16(np, "ite,hdcp-version", &reg_val);

This one is also not defined.

> +     if (ret == 0) {
> +             it6162->en_hdcp = true;
> +             it6162->hdcp_version = reg_val;
> +             dev_info(dev, "HDCP version %x", it6162->hdcp_version);
> +

> +MODULE_AUTHOR("Pet Weng <[email protected]>");
> +MODULE_AUTHOR("Hermes Wu <[email protected]>");
> +MODULE_DESCRIPTION("it6162 mipi to hdmi driver");

MIPI, HDMI.

> +MODULE_LICENSE("GPL");
> 
> -- 
> 2.34.1
> 
> 

-- 
With best wishes
Dmitry

Reply via email to