Hi Dmitry, On 08/16/2025, Dmitry Baryshkov wrote: > Declare which infoframes are supported via the .hdmi_write_infoframe() > interface. > > Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@oss.qualcomm.com> > --- > drivers/gpu/drm/bridge/ite-it6263.c | 1 + > 1 file changed, 1 insertion(+)
For subject, s/it6232/it6263. > > diff --git a/drivers/gpu/drm/bridge/ite-it6263.c > b/drivers/gpu/drm/bridge/ite-it6263.c > index > cf813672b4ffb8ab5c524c6414ee7b414cebc018..b1956891a8388401c13cd2fc5c78f0779063adf4 > 100644 > --- a/drivers/gpu/drm/bridge/ite-it6263.c > +++ b/drivers/gpu/drm/bridge/ite-it6263.c > @@ -875,6 +875,7 @@ static int it6263_probe(struct i2c_client *client) > it->bridge.type = DRM_MODE_CONNECTOR_HDMIA; > it->bridge.vendor = "ITE"; > it->bridge.product = "IT6263"; > + it->bridge.supported_infoframes = DRM_CONNECTOR_INFOFRAME_AVI; With supported_infoframes set, the two lines setting vendor and product are dead code. I think it's worth dropping them and updating kerneldoc for vendor and product members because they don't have to be required if DRM_BRIDGE_OP_HDMI is set. But, this could be done with future patch(not in this patch series). Reviewed-by: Liu Ying <victor....@nxp.com> > > return devm_drm_bridge_add(dev, &it->bridge); > } > -- Regards, Liu Ying
