From 8a5884f561e88131d58e53b4d7a7d091ba13659c Mon Sep 17 00:00:00 2001 From: Alexander von Gluck IV Date: Wed, 8 Jul 2015 23:19:01 -0500 Subject: [PATCH] radeon_hd: Improve aux transaction tracing --- src/add-ons/accelerants/radeon_hd/displayport.cpp | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/add-ons/accelerants/radeon_hd/displayport.cpp b/src/add-ons/accelerants/radeon_hd/displayport.cpp index f510a6b50f..e651c033f9 100644 --- a/src/add-ons/accelerants/radeon_hd/displayport.cpp +++ b/src/add-ons/accelerants/radeon_hd/displayport.cpp @@ -200,9 +200,10 @@ dp_aux_transaction(uint32 connectorIndex, dp_aux_msg* message) message->reply = ack; if ((ack & AUX_NATIVE_REPLY_MASK) == AUX_NATIVE_REPLY_ACK) return B_OK; - else if ((ack & AUX_NATIVE_REPLY_MASK) == AUX_NATIVE_REPLY_DEFER) + else if ((ack & AUX_NATIVE_REPLY_MASK) == AUX_NATIVE_REPLY_DEFER) { + TRACE("%s: aux reply defer received. Snoozing.\n", __func__); snooze(400); - else + } else return B_IO_ERROR; } @@ -225,6 +226,7 @@ dpcd_reg_write(uint32 connectorIndex, uint16 address, uint8 value) ERROR("%s: error on DisplayPort aux write (0x%" B_PRIx32 ")\n", __func__, result); } + TRACE("%s: aux message reply: 0x%" B_PRIx8 "\n", __func__, message.reply); } @@ -245,6 +247,8 @@ dpcd_reg_read(uint32 connectorIndex, uint16 address) __func__, result); } + TRACE("%s: aux message reply: 0x%" B_PRIx8 "\n", __func__, message.reply); + return response; } @@ -739,7 +743,7 @@ dp_set_tp(uint32 connectorIndex, int trainingPattern) /* set training pattern on the source */ if (info.dceMajor >= 4 || !dp->trainingUseEncoder) { - TRACE("%s: Training with encoder...", __func__); + TRACE("%s: Training with encoder...\n", __func__); switch (trainingPattern) { case DP_TRAIN_PATTERN_1: rawTrainingPattern = ATOM_ENCODER_CMD_DP_LINK_TRAINING_PATTERN1; @@ -753,7 +757,7 @@ dp_set_tp(uint32 connectorIndex, int trainingPattern) } encoder_dig_setup(connectorIndex, pll->pixelClock, rawTrainingPattern); } else { - TRACE("%s: Training with encoder service...", __func__); + TRACE("%s: Training with encoder service...\n", __func__); switch (trainingPattern) { case DP_TRAIN_PATTERN_1: rawTrainingPattern = 0;