✓ CI.checkpatch: success for drm/dp: Fix Write_Status_Update_Request AUX request format

Patchwork patchwork at emeril.freedesktop.org
Mon May 12 14:04:33 UTC 2025


== Series Details ==

Series: drm/dp: Fix Write_Status_Update_Request AUX request format
URL   : https://patchwork.freedesktop.org/series/148889/
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
202708c00696422fd217223bb679a353a5936e23
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 37b40ea7444f24177df52a986cb6a51cc8789ce2
Author: Wayne Lin <Wayne.Lin at amd.com>
Date:   Fri May 9 17:22:18 2025 +0800

    drm/dp: Fix Write_Status_Update_Request AUX request format
    
    [Why]
    Notice AUX request format of I2C-over-AUX with
    Write_Status_Update_Request flag set is incorrect. It should
    be address only request without length and data like:
    "SYNC->COM3:0 (= 0110)|0000-> 0000|0000->
    0|7-bit I2C address (the same as the last)-> STOP->".
    
    [How]
    Refer to DP v2.1 Table 2-178, correct the
    Write_Status_Update_Request to be address only request.
    
    Note that we might receive 0 returned by aux->transfer() when
    receive reply I2C_ACK|AUX_ACK of Write_Status_Update_Request
    transaction. Which indicating all data bytes get written.
    We should avoid to return 0 bytes get transferred under this
    case.
    
    V2:
    - Add checking condition before restoring msg->buffer and
      msg->size. (Limonciello Mario)
    - Revise unclear comment to appropriately describe the idea.
      (Jani Nikula)
    
    Fixes: 68ec2a2a2481 ("drm/dp: Use I2C_WRITE_STATUS_UPDATE to drain partial I2C_WRITE requests")
    Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
    Cc: Jani Nikula <jani.nikula at intel.com>
    Cc: Mario Limonciello <mario.limonciello at amd.com>
    Cc: Harry Wentland <harry.wentland at amd.com>
    Cc: stable at vger.kernel.org
    Signed-off-by: Wayne Lin <Wayne.Lin at amd.com>
+ /mt/dim checkpatch 3db1bb52113808f8bc72a7049e2d36e76acc882c drm-intel
37b40ea7444f drm/dp: Fix Write_Status_Update_Request AUX request format




More information about the Intel-xe mailing list