All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wayne Lin <Wayne.Lin@amd.com>
To: <dri-devel@lists.freedesktop.org>
Cc: <ville.syrjala@linux.intel.com>, <jani.nikula@intel.com>,
	<mario.limonciello@amd.com>, <harry.wentland@amd.com>,
	Wayne Lin <Wayne.Lin@amd.com>
Subject: [PATCH 0/2] Fix I2C-Over-AUX handling
Date: Thu, 24 Apr 2025 11:07:32 +0800	[thread overview]
Message-ID: <20250424030734.873693-1-Wayne.Lin@amd.com> (raw)

These two patches are trying to fix how we handle I2C-Over_AUX
transactions when sink side can only complte partial data at one time.

We encounter EDID reading error with specific monitor and cause mode
list can only be lower resolutions. After analysis, the monitor will
reply I2C_ACK|AUX_ACK followed by a byte indicating the read data length
fewer than the requested one. Under this case, we should update the
request length and request again. Code we have today doesn't handle
such case.

Meanwhile, how we handle Write_Status_Update_Request for I2C write over
AUX transactions seems wrong as well. The request format should be address
only transaction without length and data, and we don't really send it out.
Further more, it should be constrained for I2C write case only. Not
for I2C read request.

1st patch to fix the I2C write problem and 2nd one to fix the I2C read.

---
Wayne Lin (2):
  drm/dp: Correct Write_Status_Update_Request handling
  drm/dp: Add handling for partially read under I2-readC-over-AUX

 drivers/gpu/drm/display/drm_dp_helper.c | 28 +++++++++++++++++++++----
 1 file changed, 24 insertions(+), 4 deletions(-)

-- 
2.43.0


             reply	other threads:[~2025-04-24  3:07 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-24  3:07 Wayne Lin [this message]
2025-04-24  3:07 ` [PATCH 1/2] drm/dp: Correct Write_Status_Update_Request handling Wayne Lin
2025-04-25 14:47   ` Ville Syrjälä
2025-04-27  9:47     ` Lin, Wayne
2025-04-24  3:07 ` [PATCH 2/2] drm/dp: Add handling for partially read under I2-readC-over-AUX Wayne Lin
2025-04-24  5:34 ` [PATCH 0/2] Fix I2C-Over-AUX handling Lin, Wayne

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250424030734.873693-1-Wayne.Lin@amd.com \
    --to=wayne.lin@amd.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=harry.wentland@amd.com \
    --cc=jani.nikula@intel.com \
    --cc=mario.limonciello@amd.com \
    --cc=ville.syrjala@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.