lkml.org 
[lkml]   [2019]   [Jun]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 21/53] drm/i915/sdvo: Implement proper HDMI audio support for SDVO
    Date
    From: Ville Syrjälä <ville.syrjala@linux.intel.com>

    commit d74408f528261f900dddb9778f61b5c5a7a6249c upstream.

    Our SDVO audio support is pretty bogus. We can't push audio over the
    SDVO bus, so trying to enable audio in the SDVO control register doesn't
    do anything. In fact it looks like the SDVO encoder will always mix in
    the audio coming over HDA, and there's no (at least documented) way to
    disable that from our side. So HDMI audio does work currently on gen4
    but only by luck really. On gen3 it got broken by the referenced commit.
    And what has always been missing on every platform is the ELD.

    To pass the ELD to the audio driver we need to write it to magic buffer
    in the SDVO encoder hardware which then gets pulled out via HDA in the
    other end. Ie. pretty much the same thing we had for native HDMI before
    we started to just pass the ELD between the drivers. This sort of
    explains why we even have that silly hardware buffer with native HDMI.

    $ cat /proc/asound/card0/eld#1.0
    -monitor_present 0
    -eld_valid 0
    +monitor_present 1
    +eld_valid 1
    +monitor_name LG TV
    +connection_type HDMI
    +...

    This also fixes our state readout since we can now query the SDVO
    encoder about the state of the "ELD valid" and "presence detect"
    bits. As mentioned those don't actually control whether audio
    gets sent over the HDMI cable, but it's the best we can do. And with
    the state checker appeased we can re-enable HDMI audio for gen3.

    Cc: stable@vger.kernel.org
    Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
    Cc: zardam@gmail.com
    Tested-by: zardam@gmail.com
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108976
    Fixes: de44e256b92c ("drm/i915/sdvo: Shut up state checker with hdmi cards on gen3")
    Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20190409144054.24561-3-ville.syrjala@linux.intel.com
    Reviewed-by: Imre Deak <imre.deak@intel.com>
    (cherry picked from commit dc49a56bd43bb04982e64b44436831da801d0237)
    Signed-off-by: Jani Nikula <jani.nikula@intel.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/gpu/drm/i915/intel_sdvo.c | 58 ++++++++++++++++++++++++++-------
    drivers/gpu/drm/i915/intel_sdvo_regs.h | 3 +
    2 files changed, 50 insertions(+), 11 deletions(-)

    --- a/drivers/gpu/drm/i915/intel_sdvo.c
    +++ b/drivers/gpu/drm/i915/intel_sdvo.c
    @@ -928,6 +928,13 @@ static bool intel_sdvo_set_colorimetry(s
    return intel_sdvo_set_value(intel_sdvo, SDVO_CMD_SET_COLORIMETRY, &mode, 1);
    }

    +static bool intel_sdvo_set_audio_state(struct intel_sdvo *intel_sdvo,
    + u8 audio_state)
    +{
    + return intel_sdvo_set_value(intel_sdvo, SDVO_CMD_SET_AUDIO_STAT,
    + &audio_state, 1);
    +}
    +
    #if 0
    static void intel_sdvo_dump_hdmi_buf(struct intel_sdvo *intel_sdvo)
    {
    @@ -1359,11 +1366,6 @@ static void intel_sdvo_pre_enable(struct
    else
    sdvox |= SDVO_PIPE_SEL(crtc->pipe);

    - if (crtc_state->has_audio) {
    - WARN_ON_ONCE(INTEL_GEN(dev_priv) < 4);
    - sdvox |= SDVO_AUDIO_ENABLE;
    - }
    -
    if (INTEL_GEN(dev_priv) >= 4) {
    /* done in crtc_mode_set as the dpll_md reg must be written early */
    } else if (IS_I945G(dev_priv) || IS_I945GM(dev_priv) ||
    @@ -1492,8 +1494,13 @@ static void intel_sdvo_get_config(struct
    if (sdvox & HDMI_COLOR_RANGE_16_235)
    pipe_config->limited_color_range = true;

    - if (sdvox & SDVO_AUDIO_ENABLE)
    - pipe_config->has_audio = true;
    + if (intel_sdvo_get_value(intel_sdvo, SDVO_CMD_GET_AUDIO_STAT,
    + &val, 1)) {
    + u8 mask = SDVO_AUDIO_ELD_VALID | SDVO_AUDIO_PRESENCE_DETECT;
    +
    + if ((val & mask) == mask)
    + pipe_config->has_audio = true;
    + }

    if (intel_sdvo_get_value(intel_sdvo, SDVO_CMD_GET_ENCODE,
    &val, 1)) {
    @@ -1506,6 +1513,32 @@ static void intel_sdvo_get_config(struct
    pipe_config->pixel_multiplier, encoder_pixel_multiplier);
    }

    +static void intel_sdvo_disable_audio(struct intel_sdvo *intel_sdvo)
    +{
    + intel_sdvo_set_audio_state(intel_sdvo, 0);
    +}
    +
    +static void intel_sdvo_enable_audio(struct intel_sdvo *intel_sdvo,
    + const struct intel_crtc_state *crtc_state,
    + const struct drm_connector_state *conn_state)
    +{
    + const struct drm_display_mode *adjusted_mode =
    + &crtc_state->base.adjusted_mode;
    + struct drm_connector *connector = conn_state->connector;
    + u8 *eld = connector->eld;
    +
    + eld[6] = drm_av_sync_delay(connector, adjusted_mode) / 2;
    +
    + intel_sdvo_set_audio_state(intel_sdvo, 0);
    +
    + intel_sdvo_write_infoframe(intel_sdvo, SDVO_HBUF_INDEX_ELD,
    + SDVO_HBUF_TX_DISABLED,
    + eld, drm_eld_size(eld));
    +
    + intel_sdvo_set_audio_state(intel_sdvo, SDVO_AUDIO_ELD_VALID |
    + SDVO_AUDIO_PRESENCE_DETECT);
    +}
    +
    static void intel_disable_sdvo(struct intel_encoder *encoder,
    struct intel_crtc_state *old_crtc_state,
    struct drm_connector_state *conn_state)
    @@ -1515,6 +1548,9 @@ static void intel_disable_sdvo(struct in
    struct intel_crtc *crtc = to_intel_crtc(encoder->base.crtc);
    u32 temp;

    + if (old_crtc_state->has_audio)
    + intel_sdvo_disable_audio(intel_sdvo);
    +
    intel_sdvo_set_active_outputs(intel_sdvo, 0);
    if (0)
    intel_sdvo_set_encoder_power_state(intel_sdvo,
    @@ -1598,6 +1634,9 @@ static void intel_enable_sdvo(struct int
    intel_sdvo_set_encoder_power_state(intel_sdvo,
    DRM_MODE_DPMS_ON);
    intel_sdvo_set_active_outputs(intel_sdvo, intel_sdvo->attached_output);
    +
    + if (pipe_config->has_audio)
    + intel_sdvo_enable_audio(intel_sdvo, pipe_config, conn_state);
    }

    static enum drm_mode_status
    @@ -2468,7 +2507,6 @@ static bool
    intel_sdvo_dvi_init(struct intel_sdvo *intel_sdvo, int device)
    {
    struct drm_encoder *encoder = &intel_sdvo->base.base;
    - struct drm_i915_private *dev_priv = to_i915(encoder->dev);
    struct drm_connector *connector;
    struct intel_encoder *intel_encoder = to_intel_encoder(encoder);
    struct intel_connector *intel_connector;
    @@ -2504,9 +2542,7 @@ intel_sdvo_dvi_init(struct intel_sdvo *i
    encoder->encoder_type = DRM_MODE_ENCODER_TMDS;
    connector->connector_type = DRM_MODE_CONNECTOR_DVID;

    - /* gen3 doesn't do the hdmi bits in the SDVO register */
    - if (INTEL_GEN(dev_priv) >= 4 &&
    - intel_sdvo_is_hdmi_connector(intel_sdvo, device)) {
    + if (intel_sdvo_is_hdmi_connector(intel_sdvo, device)) {
    connector->connector_type = DRM_MODE_CONNECTOR_HDMIA;
    intel_sdvo->is_hdmi = true;
    }
    --- a/drivers/gpu/drm/i915/intel_sdvo_regs.h
    +++ b/drivers/gpu/drm/i915/intel_sdvo_regs.h
    @@ -707,6 +707,9 @@ struct intel_sdvo_enhancements_arg {
    #define SDVO_CMD_GET_AUDIO_ENCRYPT_PREFER 0x90
    #define SDVO_CMD_SET_AUDIO_STAT 0x91
    #define SDVO_CMD_GET_AUDIO_STAT 0x92
    + #define SDVO_AUDIO_ELD_VALID (1 << 0)
    + #define SDVO_AUDIO_PRESENCE_DETECT (1 << 1)
    + #define SDVO_AUDIO_CP_READY (1 << 2)
    #define SDVO_CMD_SET_HBUF_INDEX 0x93
    #define SDVO_HBUF_INDEX_ELD 0
    #define SDVO_HBUF_INDEX_AVI_IF 1

    \
     
     \ /
      Last update: 2019-06-17 23:32    [W:3.663 / U:0.132 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site