[Intel-gfx] [PATCH 1/2] drm/i915: Read the response after issuing DDC bus switch command

Eric Anholt eric at anholt.net
Tue Jan 12 23:46:10 CET 2010


On Fri,  8 Jan 2010 10:58:19 +0800, yakui.zhao at intel.com wrote:
> From: Zhao Yakui <yakui.zhao at intel.com>
> 
> For some SDVO cards based on conextant chip, we can't read the EDID if we don't
> read the response after issuing SDVO DDC bus switch command. In order to read
> the EDID we need read the response after issuing the DDC bus switch command.
> 
> From the SDVO spec once when another I2C transaction is finished after completing
> the I2C transaction of issuing the bus switch command, it will be switched back
> to the SDVO internal state again. So we can't initiate a new I2C transaction to
> read the response after issuing the DDC bus switch command. Instead we should
> issue DDC bus switch command and read the response in the same I2C transaction.
> 
> https://bugs.freedesktop.org/show_bug.cgi?id=23842
> https://bugs.freedesktop.org/show_bug.cgi?id=24458
> https://bugs.freedesktop.org/show_bug.cgi?id=24522
> https://bugs.freedesktop.org/show_bug.cgi?id=24282
> 
> Signed-off-by: Zhao Yakui <yakui.zhao at intel.com>
> Tested-by: Sebastien Caty <sebastien.caty at mrnf.gouv.qc.ca>

Much nicer commit text!  I re-wrapped your commit messages, since they
didn't fit in 80 columns in git log.  If you're using emacs, M-q will
reflow your text nicely so that it does.

Applied to for-linus.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20100112/b37d2b47/attachment.sig>


More information about the Intel-gfx mailing list