✓ CI.checkpatch: success for drm/xe: Eliminate usage of TIMESTAMP_OVERRIDE

Patchwork patchwork at emeril.freedesktop.org
Tue Feb 25 22:54:54 UTC 2025


== Series Details ==

Series: drm/xe: Eliminate usage of TIMESTAMP_OVERRIDE
URL   : https://patchwork.freedesktop.org/series/145437/
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 6758b8fc825d74a5ccbaf463654ba090e2dd8736
Author: Matt Roper <matthew.d.roper at intel.com>
Date:   Tue Feb 25 14:49:09 2025 -0800

    drm/xe: Eliminate usage of TIMESTAMP_OVERRIDE
    
    Recent discussions with the hardware architects have revealed that
    the TIMESTAMP_OVERRIDE register is never expected to hold a valid/useful
    value on production hardware.  That register would only get used by
    hardware workarounds (although there are none that use it today) or
    during early internal hardware testing.
    
    Due to lack of documentation it's not clear exactly what the driver
    should be doing if CTC_MODE[0] is set (or even whether that's a setting
    that would ever be encountered on real hardware), but it's definitely
    not what Xe and i915 have been doing.  So drop the incorrect code trying
    to use TIMESTAMP_REGISTER.  If the driver does encounter CTC_MODE[0] in
    the wild, we'll print a warning and just continue trying to use the
    crystal clock frequency since that's probably less incorrect than what
    we're doing today.
    
    Cc: Lucas De Marchi <lucas.demarchi at intel.com>
    Cc: Balasubramani Vivekanandan <balasubramani.vivekanandan at intel.com>
    Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
+ /mt/dim checkpatch 533a24b8fa29455518aed9ac72209c834d814238 drm-intel
6758b8fc825d drm/xe: Eliminate usage of TIMESTAMP_OVERRIDE




More information about the Intel-xe mailing list