drm/amd/display: clean code for transition event log.
authorChiawen Huang <chiawen.huang@amd.com>
Tue, 28 Aug 2018 05:38:34 +0000 (13:38 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 11 Sep 2018 03:44:16 +0000 (22:44 -0500)
[Why]
There are same purpose transition events.

[How]
remove the redundant event log.

Signed-off-by: Chiawen Huang <chiawen.huang@amd.com>
Reviewed-by: Tony Cheng <Tony.Cheng@amd.com>
Acked-by: Bhawanpreet Lakha <Bhawanpreet.Lakha@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dm_event_log.h
drivers/gpu/drm/amd/display/dc/i2caux/aux_engine.c
drivers/gpu/drm/amd/display/dc/i2caux/i2c_hw_engine.c

index c1ce2dd52f9b12c0f34c466fb606e353292de1e4..00a275dfa472946d252136d0d8d1909907bc4ee6 100644 (file)
@@ -31,8 +31,6 @@
 
 #define __DM_EVENT_LOG_H__
 
-#define EVENT_LOG_I2CAUX_READ(transType, dcc, address, status, len, data)
-#define EVENT_LOG_I2CAUX_WRITE(transType, dcc, address, status, len, data)
 #define EVENT_LOG_AUX_REQ(dcc, type, action, address, len, data)
 #define EVENT_LOG_AUX_Reply(dcc, type, swStatus, replyStatus, len, data)
 
index 03292c52b18dbbbcd4a9e4cdaffab171039db4ad..8cbf38b2470d08212977a09608e63854773cdddb 100644 (file)
@@ -297,12 +297,6 @@ static bool read_command(
 
        if (request->payload.address_space ==
                I2CAUX_TRANSACTION_ADDRESS_SPACE_DPCD) {
-               EVENT_LOG_I2CAUX_READ(request->payload.address_space,
-                               engine->base.ddc->pin_data->en,
-                               request->payload.address,
-                               request->status,
-                               request->payload.length,
-                               request->payload.data);
                DC_LOG_I2C_AUX("READ: addr:0x%x  value:0x%x Result:%d",
                                request->payload.address,
                                request->payload.data[0],
@@ -519,12 +513,6 @@ static bool write_command(
 
        if (request->payload.address_space ==
                I2CAUX_TRANSACTION_ADDRESS_SPACE_DPCD) {
-               EVENT_LOG_I2CAUX_WRITE(request->payload.address_space,
-                               engine->base.ddc->pin_data->en,
-                               request->payload.address,
-                               request->status,
-                               request->payload.length,
-                               request->payload.data);
                DC_LOG_I2C_AUX("WRITE: addr:0x%x  value:0x%x Result:%d",
                                request->payload.address,
                                request->payload.data[0],
index 1747b9f5f10e52be9864292cc8de4d946de7c0e0..c995ef4ea5a43086bbade5f6b2fa90bc71026867 100644 (file)
@@ -171,21 +171,6 @@ bool dal_i2c_hw_engine_submit_request(
                        process_channel_reply(&hw_engine->base, &reply);
        }
 
-       if (i2caux_request->operation == I2CAUX_TRANSACTION_READ) {
-               EVENT_LOG_I2CAUX_READ(i2caux_request->payload.address_space,
-                                                       engine->ddc->pin_data->en,
-                                                       i2caux_request->payload.address,
-                                                       i2caux_request->status,
-                                                       i2caux_request->payload.length,
-                                                       i2caux_request->payload.data);
-       } else if (i2caux_request->operation == I2CAUX_TRANSACTION_WRITE) {
-               EVENT_LOG_I2CAUX_WRITE(i2caux_request->payload.address_space,
-                                                       engine->ddc->pin_data->en,
-                                                       i2caux_request->payload.address,
-                                                       i2caux_request->status,
-                                                       i2caux_request->payload.length,
-                                                       i2caux_request->payload.data);
-       }
 
 
        return result;