On 2018-02-12 04:31 PM, Andrey Grodzovsky wrote:
> With this logger you should probably remove the Linux specific logger in amdgpu_dm_mst_types.c, check log_dpcd function.
>
This currently only logs the first byte of the response. Once that's fixed you're right, we should rip out the one from amdgpu_dm_mst_types.c.
Harry
> Andrey
>
>
> On 02/12/2018 12:16 PM, Harry Wentland wrote:
>> From: Eric Yang <Eric.Yang2 at amd.com>
>>
>> Add basic logging for DPCD access. Does not print
>> by default.
>>
>> Currently only prints first byte of the data accessed.
>>
>> Technical debt: Need to make it so that the entire
>> data block accessed is printed. Also need to log
>> address space that's not DPCD.
>>
>> Change-Id: I10ef7042c14d70508845ef827ebec2432d8d8176
>> Signed-off-by: Eric Yang <Eric.Yang2 at amd.com>
>> Reviewed-by: Tony Cheng <Tony.Cheng at amd.com>
>> Acked-by: Harry Wentland <harry.wentland at amd.com>
>> ---
>> Â drivers/gpu/drm/amd/display/dc/i2caux/aux_engine.c | 16 ++++++++++++++++
>> Â 1 file changed, 16 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/amd/display/dc/i2caux/aux_engine.c b/drivers/gpu/drm/amd/display/dc/i2caux/aux_engine.c
>> index fc7a7d4ebca5..0b1db48fef36 100644
>> --- a/drivers/gpu/drm/amd/display/dc/i2caux/aux_engine.c
>> +++ b/drivers/gpu/drm/amd/display/dc/i2caux/aux_engine.c
>> @@ -284,6 +284,14 @@ static bool read_command(
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â msleep(engine->delay);
>> Â Â Â Â Â } while (ctx.operation_succeeded && !ctx.transaction_complete);
>> Â +Â Â Â if (request->payload.address_space ==
>> +Â Â Â Â Â Â Â I2CAUX_TRANSACTION_ADDRESS_SPACE_DPCD) {
>> +       dm_logger_write(engine->base.ctx->logger, LOG_I2C_AUX, "READ: addr:0x%x value:0x%x Result:%d",
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â request->payload.address,
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â request->payload.data[0],
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ctx.operation_succeeded);
>> +Â Â Â }
>> +
>> Â Â Â Â Â return ctx.operation_succeeded;
>> Â }
>> Â @@ -484,6 +492,14 @@ static bool write_command(
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â msleep(engine->delay);
>> Â Â Â Â Â } while (ctx.operation_succeeded && !ctx.transaction_complete);
>> Â +Â Â Â if (request->payload.address_space ==
>> +Â Â Â Â Â Â Â I2CAUX_TRANSACTION_ADDRESS_SPACE_DPCD) {
>> +       dm_logger_write(engine->base.ctx->logger, LOG_I2C_AUX, "WRITE: addr:0x%x value:0x%x Result:%d",
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â request->payload.address,
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â request->payload.data[0],
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ctx.operation_succeeded);
>> +Â Â Â }
>> +
>> Â Â Â Â Â return ctx.operation_succeeded;
>> Â }
>> Â
>
