Commit 8c7a075d authored by Russell King's avatar Russell King

drm/i2c: tda998x: use drm_hdmi_avi_infoframe_from_display_mode()

Make use of the DRM HDMI AVI infoframe helper to construct the AVI
infoframe, rather than coding this up ourselves.  This allows DRM
to supply proper aspect ratio information derived from the DRM
display mode structure.
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent 07259f8b
...@@ -610,19 +610,21 @@ tda998x_write_aif(struct tda998x_priv *priv, struct tda998x_encoder_params *p) ...@@ -610,19 +610,21 @@ tda998x_write_aif(struct tda998x_priv *priv, struct tda998x_encoder_params *p)
static void static void
tda998x_write_avi(struct tda998x_priv *priv, struct drm_display_mode *mode) tda998x_write_avi(struct tda998x_priv *priv, struct drm_display_mode *mode)
{ {
u8 buf[PB(HDMI_AVI_INFOFRAME_SIZE) + 1]; struct hdmi_avi_infoframe frame;
u8 buf[HDMI_INFOFRAME_HEADER_SIZE + HDMI_AVI_INFOFRAME_SIZE];
ssize_t len;
memset(buf, 0, sizeof(buf)); drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
buf[HB(0)] = HDMI_INFOFRAME_TYPE_AVI;
buf[HB(1)] = 0x02; frame.quantization_range = HDMI_QUANTIZATION_RANGE_FULL;
buf[HB(2)] = HDMI_AVI_INFOFRAME_SIZE;
buf[PB(1)] = HDMI_SCAN_MODE_UNDERSCAN; len = hdmi_avi_infoframe_pack(&frame, buf, sizeof(buf));
buf[PB(2)] = HDMI_ACTIVE_ASPECT_PICTURE; if (len < 0) {
buf[PB(3)] = HDMI_QUANTIZATION_RANGE_FULL << 2; dev_err(&priv->hdmi->dev, "hdmi_avi_infoframe_pack() failed: %d\n", len);
buf[PB(4)] = drm_match_cea_mode(mode); return;
}
tda998x_write_if(priv, DIP_IF_FLAGS_IF2, REG_IF2_HB0, buf,
sizeof(buf)); tda998x_write_if(priv, DIP_IF_FLAGS_IF2, REG_IF2_HB0, buf, len);
} }
static void tda998x_audio_mute(struct tda998x_priv *priv, bool on) static void tda998x_audio_mute(struct tda998x_priv *priv, bool on)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment