Declare which infoframes are supported via the .hdmi_write_infoframe()
interface. Return -EOPNOTSUPP if the driver is asked to write or clear
the unsupported InfoFrame.

Reviewed-by: Liu Ying <victor....@nxp.com>
Acked-by: Daniel Stone <dani...@collabora.com>
Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@oss.qualcomm.com>
---
 drivers/gpu/drm/bridge/ite-it6263.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/gpu/drm/bridge/ite-it6263.c 
b/drivers/gpu/drm/bridge/ite-it6263.c
index 
2eb8fba7016cbf0dcb19aec4ca8849f1fffaa64c..691e2d8a721bdbf99ca2dd49a45ce508ee7d9591
 100644
--- a/drivers/gpu/drm/bridge/ite-it6263.c
+++ b/drivers/gpu/drm/bridge/ite-it6263.c
@@ -773,6 +773,7 @@ static int it6263_hdmi_clear_infoframe(struct drm_bridge 
*bridge,
                break;
        default:
                dev_dbg(it->dev, "unsupported HDMI infoframe 0x%x\n", type);
+               return -EOPNOTSUPP;
        }
 
        return 0;
@@ -813,6 +814,7 @@ static int it6263_hdmi_write_infoframe(struct drm_bridge 
*bridge,
                break;
        default:
                dev_dbg(it->dev, "unsupported HDMI infoframe 0x%x\n", type);
+               return -EOPNOTSUPP;
        }
 
        return 0;
@@ -899,6 +901,9 @@ 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 |
+               DRM_CONNECTOR_INFOFRAME_VENDOR;
 
        return devm_drm_bridge_add(dev, &it->bridge);
 }

-- 
2.47.3

Reply via email to