✓ CI.checkpatch: success for drm/i915/xe3lpd: Map POWER_DOMAIN_AUDIO_PLAYBACK to DC_off (rev2)

Patchwork patchwork at emeril.freedesktop.org
Wed Mar 5 22:51:06 UTC 2025


== Series Details ==

Series: drm/i915/xe3lpd: Map POWER_DOMAIN_AUDIO_PLAYBACK to DC_off (rev2)
URL   : https://patchwork.freedesktop.org/series/145594/
State : success

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
cbb4e4a079d89106c2736adc3c7de6f9dc56da07
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit a154118913e51b158b889e53e8216ef5a94beee1
Author: Gustavo Sousa <gustavo.sousa at intel.com>
Date:   Thu Feb 27 16:14:25 2025 -0300

    drm/i915/xe3lpd: Map POWER_DOMAIN_AUDIO_PLAYBACK to DC_off
    
    In Xe3_LPD, display audio has the core audio logic located in PG0 and
    per-transcoder logic in the same power well that provides power for the
    transcoder [1].
    
    For stuff like audio device enumeration, we need to ensure that PG0 is
    turned on. For playback, we additionally need the transcoder's power
    well to be enabled.
    
    That essentially means that, for audio playback, there isn't a special
    power well that needs to be enabled, because modeset sequences will
    ensure that the required power wells are enabled.
    
    That said, there might be cases where PG0 could be disabled due to
    display entering DC6 while the audio driver tries to interact with the
    graphics driver for stuff like audio device enumeration.
    
    We recently hit that kind of scenario, where "aplay -l" was being used
    to enumerate audio devices on a PTL machine with PSR enabled and no
    external displays attached.
    
    Since intel_audio_component_get_power() uses
    POWER_DOMAIN_AUDIO_PLAYBACK, make sure to map that power domain to
    DC_off power well, so that we disable dynamic DC states (which includes
    DC6) while the audio driver needs display audio power.
    
    [1] The core-audio vs per-transcoder logic split is not really new in
        Xe3_LPD. This is also true for previous display generations. We need
        to figure out the correct version where this split happened so that
        we can apply fixes in the current power domain mapping.
    
    Bspec: 72519
    Signed-off-by: Gustavo Sousa <gustavo.sousa at intel.com>
    Reviewed-by: Kai Vehmanen <kai.vehmanen at linux.intel.com>
+ /mt/dim checkpatch 3ccfd8016ad8283ecef0dfcf0e4f2132ade9bf4c drm-intel
a154118913e5 drm/i915/xe3lpd: Map POWER_DOMAIN_AUDIO_PLAYBACK to DC_off




More information about the Intel-xe mailing list