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

Patchwork patchwork at emeril.freedesktop.org
Thu Feb 27 20:10:33 UTC 2025


== Series Details ==

Series: drm/i915/xe3lpd: Map POWER_DOMAIN_AUDIO_PLAYBACK to DC_off
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
22f9cda3436b4fe965b5c5f31d2f2c1bcb483189
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit ae751764c8f1b7ad89c4b3097e4545de450a3db8
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>
+ /mt/dim checkpatch 6badede10d92c242241bd7bf59a5488b5eba5aa7 drm-intel
ae751764c8f1 drm/i915/xe3lpd: Map POWER_DOMAIN_AUDIO_PLAYBACK to DC_off




More information about the Intel-xe mailing list