Declare which infoframes are supported via the .hdmi_write_infoframe()
interface. Audio infoframe is handled separately.

Reviewed-by: Liu Ying <victor....@nxp.com>
Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@oss.qualcomm.com>
---
 drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c 
b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
index 
26f8ef4824235a9a85b57a9a3a816fe26a59e45c..fe30567b56b4f600683f0545ed270665e692b12c
 100644
--- a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
+++ b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c
@@ -904,7 +904,7 @@ static int adv7511_bridge_hdmi_clear_infoframe(struct 
drm_bridge *bridge,
                break;
        default:
                drm_dbg_driver(adv7511->bridge.dev, "Unsupported HDMI InfoFrame 
%x\n", type);
-               break;
+               return -EOPNOTSUPP;
        }
 
        return 0;
@@ -938,7 +938,7 @@ static int adv7511_bridge_hdmi_write_infoframe(struct 
drm_bridge *bridge,
                break;
        default:
                drm_dbg_driver(adv7511->bridge.dev, "Unsupported HDMI InfoFrame 
%x\n", type);
-               break;
+               return -EOPNOTSUPP;
        }
 
        return 0;
@@ -1299,6 +1299,12 @@ static int adv7511_probe(struct i2c_client *i2c)
 
        adv7511->bridge.vendor = "Analog";
        adv7511->bridge.product = adv7511->info->name;
+       adv7511->bridge.software_infoframes =
+               DRM_CONNECTOR_INFOFRAME_AVI |
+               DRM_CONNECTOR_INFOFRAME_SPD |
+               DRM_CONNECTOR_INFOFRAME_VENDOR;
+       adv7511->bridge.autogenerated_infoframes =
+               DRM_CONNECTOR_INFOFRAME_AUDIO;
 
 #ifdef CONFIG_DRM_I2C_ADV7511_AUDIO
        adv7511->bridge.ops |= DRM_BRIDGE_OP_HDMI_AUDIO;

-- 
2.47.2

Reply via email to