• Arnd Bergmann's avatar
    drm/amd/display: fix empty debug macros · c48977f0
    Arnd Bergmann authored
    Using an empty macro expansion as a conditional expression
    produces a W=1 warning:
    
    drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_aux.c: In function 'dce_aux_transfer_with_retries':
    drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_aux.c:775:156: error: suggest braces around empty body in an 'if' statement [-Werror=empty-body]
      775 |                                                                 "dce_aux_transfer_with_retries: AUX_RET_SUCCESS: AUX_TRANSACTION_REPLY_I2C_OVER_AUX_DEFER");
          |                                                                                                                                                            ^
    drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_aux.c:783:155: error: suggest braces around empty body in an 'if' statement [-Werror=empty-body]
      783 |                                                                 "dce_aux_transfer_with_retries: AUX_RET_SUCCESS: AUX_TRANSACTION_REPLY_I2C_OVER_AUX_NACK");
          |                                                                                                                                                           ^
    
    Expand it to "do { } while (0)" instead to make the expression
    more robust and avoid the warning.
    
    Fixes: 56aca230 ("drm/amd/display: Add AUX I2C tracing.")
    Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
    Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
    c48977f0
dce_aux.c 26.8 KB