[Intel-gfx] [PATCH v3] Support for ns2501 DVO.

[Paul Menzel]

Dear Thomas,


you put Daniel address twice into the CC field.


Am Sonntag, den 17.06.2012, 20:59 +0200 schrieb Thomas Richter:
> Includes now proper DPMS support.
>   Includes switching between resolutions - from 640x480 to 1024x768.
>   Currently assumes that the native display resolution is 1024x768.

Please put that after the next paragraph.

> The ns2501 seems to be rather critical - if the output PLL is not
> running, the chip doesn't seem to be clocked and then doesn't react
> on i2c messages. Thus, a quick'n-dirty trick ensures that the DVO
> is active before submitting any i2c messages to it. This is
> probably to be reviewed.

Such for the commit message unnecessary information goes below the first
`---` below.

> Signed-Off-by: Thomas Richter <thor at math.tu-berlin.de>
> ---

Here goes email message information.

>   drivers/gpu/drm/i915/Makefile        |    1 +
>   drivers/gpu/drm/i915/dvo.h           |    1 +
>   drivers/gpu/drm/i915/dvo_ns2501.c    |  566 
> ++++++++++++++++++++++++++++++++++

Unfortunately you have to resend as v4 since your mailer line wrapped
this line and others.

[…]


Thanks,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120617/c1e50705/attachment.sig>