<html><body><p> <pre> On Tue, 2025-02-11 at 12:34 +0100, AngeloGioacchino Del Regno wrote: > External email : Please do not click links or open attachments until you have verified the sender or the content. > > > In preparation for adding the HDMI TX v2 driver, and to allow > a future modernization of the HDMI v1 one, perform changes > that enable the usage of the HDMI Helpers provided by DRM. > > Check if the HDMI driver provides the function pointers to > hdmi_{clear,write}_infoframe used by the HDMI Helper API and, > if present, add DRM_BRIDGE_OP_HDMI to the drm_bridge ops, > enabling the drm API to register the bridge as HDMI and to use > the HDMI Helper functions. > > If the hdmi_{write,clear}_infoframe pointers are not assigned, > vendor and product strings and HDMI helpers will not be used, > hence this commit brings no functional changes to drivers that > have not been refactored to use the new helpers. > > This also means that, in the current state, there is effectively > no functional change to mtk_hdmi and its other components. Reviewed-by: CK Hu <ck.hu@mediatek.com> > > Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > --- > drivers/gpu/drm/mediatek/mtk_hdmi_common.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/gpu/drm/mediatek/mtk_hdmi_common.c b/drivers/gpu/drm/mediatek/mtk_hdmi_common.c > index 2c91f65f26fa..d58752b772e8 100644 > --- a/drivers/gpu/drm/mediatek/mtk_hdmi_common.c > +++ b/drivers/gpu/drm/mediatek/mtk_hdmi_common.c > @@ -410,6 +410,11 @@ struct mtk_hdmi *mtk_hdmi_common_probe(struct platform_device *pdev) > hdmi->bridge.of_node = pdev->dev.of_node; > hdmi->bridge.ops = DRM_BRIDGE_OP_DETECT | DRM_BRIDGE_OP_EDID > | DRM_BRIDGE_OP_HPD; > + > + if (ver_conf->bridge_funcs->hdmi_write_infoframe && > + ver_conf->bridge_funcs->hdmi_clear_infoframe) > + hdmi->bridge.ops |= DRM_BRIDGE_OP_HDMI; > + > hdmi->bridge.type = DRM_MODE_CONNECTOR_HDMIA; > hdmi->bridge.ddc = hdmi->ddc_adpt; > hdmi->bridge.vendor = "MediaTek"; > -- > 2.48.1 > </pre> </p></body></html><pre>************* MEDIATEK Confidentiality Notice ******************** The information contained in this e-mail message (including any attachments) may be confidential, proprietary, privileged, or otherwise exempt from disclosure under applicable laws. It is intended to be conveyed only to the designated recipient(s). Any use, dissemination, distribution, printing, retaining or copying of this e-mail (including its attachments) by unintended recipient(s) is strictly prohibited and may be unlawful. If you are not an intended recipient of this e-mail, or believe that you have received this e-mail in error, please notify the sender immediately (by replying to this e-mail), delete any and all copies of this e-mail (including any attachments) from your system, and do not disclose the content of this e-mail to any other person. Thank you! </pre>