FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu Kernel Team

 
 
LinkBack Thread Tools
 
Old 03-29-2010, 12:58 PM
Stefan Bader
 
Default drm/nv04-nv40: Fix up the programmed horizontal sync pulse delay.

Chris got back to me directly and said that this only were two patches to be
sent. He also wanted to pass them on to stable.
@Chris, this is done by just passing on that patch (with the sha1 reference of
the upstream commit to stable@kernel.org. Usually cc'ing maintainers and other
people on the s-o-b list and asking for inclusion.
This one looks ok to me.

Chris Halse Rogers wrote:
> From: Christopher James Halse Rogers <raof@ubuntu.com>
>
> The calculated values were a little bit off (~16 clocks), the only
> effect it could have had is a slightly offset image with respect to
> the blob on analog outputs (bug 26790).
>
> BugLink: http://bugs.launchpad.net/bugs/529130
>
> Signed-off-by: Francisco Jerez <currojerez@riseup.net>
> Signed-off-by: Christopher James Halse Rogers <raof@ubuntu.com>
Acked-by: Stefan Bader <stefan.bader@canonical.com>
> ---
> drivers/gpu/drm/nouveau/nv04_crtc.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/nouveau/nv04_crtc.c b/drivers/gpu/drm/nouveau/nv04_crtc.c
> index d2f143e..9986aba 100644
> --- a/drivers/gpu/drm/nouveau/nv04_crtc.c
> +++ b/drivers/gpu/drm/nouveau/nv04_crtc.c
> @@ -230,9 +230,9 @@ nv_crtc_mode_set_vga(struct drm_crtc *crtc, struct drm_display_mode *mode)
> struct drm_framebuffer *fb = crtc->fb;
>
> /* Calculate our timings */
> - int horizDisplay = (mode->crtc_hdisplay >> 3) - 1;
> - int horizStart = (mode->crtc_hsync_start >> 3) - 1;
> - int horizEnd = (mode->crtc_hsync_end >> 3) - 1;
> + int horizDisplay = (mode->crtc_hdisplay >> 3) - 1;
> + int horizStart = (mode->crtc_hsync_start >> 3) + 1;
> + int horizEnd = (mode->crtc_hsync_end >> 3) + 1;
> int horizTotal = (mode->crtc_htotal >> 3) - 5;
> int horizBlankStart = (mode->crtc_hdisplay >> 3) - 1;
> int horizBlankEnd = (mode->crtc_htotal >> 3) - 1;


--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 03-30-2010, 01:18 PM
Andy Whitcroft
 
Default drm/nv04-nv40: Fix up the programmed horizontal sync pulse delay.

On Fri, Mar 26, 2010 at 09:25:55AM +1100, Chris Halse Rogers wrote:
> From: Christopher James Halse Rogers <raof@ubuntu.com>
>
> The calculated values were a little bit off (~16 clocks), the only
> effect it could have had is a slightly offset image with respect to
> the blob on analog outputs (bug 26790).

I would love to know what the heck a blob is in this context and how I
could tell my image was offset from it .

> BugLink: http://bugs.launchpad.net/bugs/529130
>
> Signed-off-by: Francisco Jerez <currojerez@riseup.net>
> Signed-off-by: Christopher James Halse Rogers <raof@ubuntu.com>
> ---
> drivers/gpu/drm/nouveau/nv04_crtc.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/nouveau/nv04_crtc.c b/drivers/gpu/drm/nouveau/nv04_crtc.c
> index d2f143e..9986aba 100644
> --- a/drivers/gpu/drm/nouveau/nv04_crtc.c
> +++ b/drivers/gpu/drm/nouveau/nv04_crtc.c
> @@ -230,9 +230,9 @@ nv_crtc_mode_set_vga(struct drm_crtc *crtc, struct drm_display_mode *mode)
> struct drm_framebuffer *fb = crtc->fb;
>
> /* Calculate our timings */
> - int horizDisplay = (mode->crtc_hdisplay >> 3) - 1;
> - int horizStart = (mode->crtc_hsync_start >> 3) - 1;
> - int horizEnd = (mode->crtc_hsync_end >> 3) - 1;
> + int horizDisplay = (mode->crtc_hdisplay >> 3) - 1;
> + int horizStart = (mode->crtc_hsync_start >> 3) + 1;
> + int horizEnd = (mode->crtc_hsync_end >> 3) + 1;
> int horizTotal = (mode->crtc_htotal >> 3) - 5;
> int horizBlankStart = (mode->crtc_hdisplay >> 3) - 1;
> int horizBlankEnd = (mode->crtc_htotal >> 3) - 1;

Looks like it might do what it says, and its upstreaming.

Acked-by: Andy Whitcroft <apw@canonical.com>

-apw

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 03-30-2010, 11:26 PM
Christopher James Halse Rogers
 
Default drm/nv04-nv40: Fix up the programmed horizontal sync pulse delay.

On Tue, 2010-03-30 at 14:18 +0100, Andy Whitcroft wrote:
> On Fri, Mar 26, 2010 at 09:25:55AM +1100, Chris Halse Rogers wrote:
> > From: Christopher James Halse Rogers <raof@ubuntu.com>
> >
> > The calculated values were a little bit off (~16 clocks), the only
> > effect it could have had is a slightly offset image with respect to
> > the blob on analog outputs (bug 26790).
>
> I would love to know what the heck a blob is in this context and how I
> could tell my image was offset from it .
>
In this context the blob is the nvidia binary drivers. The difference
in timing meant that for users of VGA outputs the display was offset
with nouveau after fiddling with the monitor to align it correctly when
run with the nvidia drivers. This would be particularly annoying for
dual-booters, who'd need to re-align their monitor on each OS switch.

I don't miss having to fiddle with image alignment on analogue outputs!

> > BugLink: http://bugs.launchpad.net/bugs/529130
> >
> > Signed-off-by: Francisco Jerez <currojerez@riseup.net>
> > Signed-off-by: Christopher James Halse Rogers <raof@ubuntu.com>
> > ---
> > drivers/gpu/drm/nouveau/nv04_crtc.c | 6 +++---
> > 1 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/nouveau/nv04_crtc.c b/drivers/gpu/drm/nouveau/nv04_crtc.c
> > index d2f143e..9986aba 100644
> > --- a/drivers/gpu/drm/nouveau/nv04_crtc.c
> > +++ b/drivers/gpu/drm/nouveau/nv04_crtc.c
> > @@ -230,9 +230,9 @@ nv_crtc_mode_set_vga(struct drm_crtc *crtc, struct drm_display_mode *mode)
> > struct drm_framebuffer *fb = crtc->fb;
> >
> > /* Calculate our timings */
> > - int horizDisplay = (mode->crtc_hdisplay >> 3) - 1;
> > - int horizStart = (mode->crtc_hsync_start >> 3) - 1;
> > - int horizEnd = (mode->crtc_hsync_end >> 3) - 1;
> > + int horizDisplay = (mode->crtc_hdisplay >> 3) - 1;
> > + int horizStart = (mode->crtc_hsync_start >> 3) + 1;
> > + int horizEnd = (mode->crtc_hsync_end >> 3) + 1;
> > int horizTotal = (mode->crtc_htotal >> 3) - 5;
> > int horizBlankStart = (mode->crtc_hdisplay >> 3) - 1;
> > int horizBlankEnd = (mode->crtc_htotal >> 3) - 1;
>
> Looks like it might do what it says, and its upstreaming.
>
> Acked-by: Andy Whitcroft <apw@canonical.com>
>
> -apw


--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 

Thread Tools




All times are GMT. The time now is 11:19 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org