On Wed, 28 Jun 2023, Mitul Golani <mitulkumar.ajitkumar.golani@xxxxxxxxx> wrote: > Currently we do not check if there is enough bandwidth for > audio, and what channels and freq it can really support. > Also sometimes there can be HW constraints e.g. GLK where audio > channels supported are only 2. Why are you sending this so many times? What are the changes between the versions? People lose track. BR, Jani. > > https://patchwork.freedesktop.org/series/107647/ > > Obtain the optimal audio rate and channel based on available display > timing constraints. > > This can be achieved by: > - Retrieve the supported channel and rate information from SADs > - Adding audio-related config parameters in the CRTC state, such > as audio support, rate, and channel. > - Initializing the audio config parameters with the maximum supported > rate and channel by the audio source. > - Computing the SADs based on the audio source's capabilities. > > Signed-off-by: Mitul Golani <mitulkumar.ajitkumar.golani@xxxxxxxxx> > > Mitul Golani (3): > drm/i915: Add has_audio to separate audio parameter in crtc_state > drm/i915/display: Configure and initialize HDMI audio capabilities > drm/i915/display: Add wrapper to Compute SAD > > drivers/gpu/drm/i915/display/g4x_dp.c | 4 +- > drivers/gpu/drm/i915/display/g4x_hdmi.c | 16 +-- > drivers/gpu/drm/i915/display/intel_audio.c | 115 +++++++++++++++++- > drivers/gpu/drm/i915/display/intel_cdclk.c | 6 +- > .../drm/i915/display/intel_crtc_state_dump.c | 4 +- > drivers/gpu/drm/i915/display/intel_ddi.c | 2 +- > drivers/gpu/drm/i915/display/intel_display.c | 4 +- > .../drm/i915/display/intel_display_types.h | 12 +- > drivers/gpu/drm/i915/display/intel_dp.c | 2 +- > drivers/gpu/drm/i915/display/intel_dp_mst.c | 2 +- > drivers/gpu/drm/i915/display/intel_hdmi.c | 2 +- > drivers/gpu/drm/i915/display/intel_sdvo.c | 10 +- > 12 files changed, 147 insertions(+), 32 deletions(-) -- Jani Nikula, Intel Open Source Graphics Center
