* Re: [Bug 29129] New: Cannot set refresh rate on nv50
[not found] ` <bug-29129-8800-V0hAGp6uBxMKqLRl/0Ahz6D7qz1kEfGD2LY78lusg7I@public.gmane.org/>
@ 2010-07-17 13:34 ` Ben Skeggs
2010-07-17 13:34 ` [Bug 29129] " bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Ben Skeggs @ 2010-07-17 13:34 UTC (permalink / raw)
To: bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ@public.gmane.org
Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
I *seriously* doubt this commit is responsible. It has nothing even remotely close to any changes that could cause what you're seeing.
Ben.
Sent from my iPhone
On 17/07/2010, at 22:39, bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ@public.gmane.org wrote:
> https://bugs.freedesktop.org/show_bug.cgi?id=29129
>
> Summary: Cannot set refresh rate on nv50
> Product: xorg
> Version: unspecified
> Platform: x86-64 (AMD64)
> URL: http://marc.info/?l=linux-kernel&m=127913459017236&w=2
> OS/Version: Linux (All)
> Status: NEW
> Severity: normal
> Priority: medium
> Component: Driver/nouveau
> AssignedTo: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> ReportedBy: svenjoac-Mmb7MZpHnFY@public.gmane.org
> QAContact: xorg-team-go0+a7rfsptAfugRpC6u6w@public.gmane.org
>
>
> Since Linux 2.6.34, it is no longer possible to set the refresh rate with the
> "video=...@nn" parameter, e.g. video=1280x1024@75 is ignored.
>
> I bisected this to the following commit:
>
> drm/nv50: enable hpd on any connector we know the gpio line for
>
> Signed-off-by: Ben Skeggs <bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
>
> diff --git a/drivers/gpu/drm/nouveau/nv50_display.c
> b/drivers/gpu/drm/nouveau/nv50_display.c
> index 61a89f2..ebcd8bf 100644
> --- a/drivers/gpu/drm/nouveau/nv50_display.c
> +++ b/drivers/gpu/drm/nouveau/nv50_display.c
> @@ -370,7 +370,9 @@ nv50_display_init(struct drm_device *dev)
> struct nouveau_connector *conn = nouveau_connector(connector);
> struct dcb_gpio_entry *gpio;
>
> - if (conn->dcb->gpio_tag == 0xff)
> + if (connector->connector_type != DRM_MODE_CONNECTOR_DVII &&
> + connector->connector_type != DRM_MODE_CONNECTOR_DVID &&
> + connector->connector_type != DRM_MODE_CONNECTOR_DisplayPort)
> continue;
>
> gpio = nouveau_bios_gpio_entry(dev, conn->dcb->gpio_tag);
>
> diff --git a/drivers/gpu/drm/nouveau/nv50_display.c
> b/drivers/gpu/drm/nouveau/nv50_display.c
> index ebcd8bf..61a89f2 100644
> --- a/drivers/gpu/drm/nouveau/nv50_display.c
> +++ b/drivers/gpu/drm/nouveau/nv50_display.c
> @@ -370,9 +370,7 @@ nv50_display_init(struct drm_device *dev)
> struct nouveau_connector *conn = nouveau_connector(connector);
> struct dcb_gpio_entry *gpio;
>
> - if (connector->connector_type != DRM_MODE_CONNECTOR_DVII &&
> - connector->connector_type != DRM_MODE_CONNECTOR_DVID &&
> - connector->connector_type != DRM_MODE_CONNECTOR_DisplayPort)
> + if (conn->dcb->gpio_tag == 0xff)
> continue;
>
> gpio = nouveau_bios_gpio_entry(dev, conn->dcb->gpio_tag);
>
>
> It should be noted that since this commit all the card data appear under
> /sys/class/drm/card0-DVI-I-1, e.g. /sys/class/drm/card0-DVI-I-1/enabled says
> "enabled" and /sys/class/drm/card0-VGA-1/enabled says "disabled", despite the
> monitor being connected via VGA.
>
> I only noticed this in 2.6.35-rc5 because the default refresh rate changed to
> 60 Hz there, see https://bugzilla.kernel.org/show_bug.cgi?id=16402.
>
> --
> Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are the assignee for the bug.
> _______________________________________________
> Nouveau mailing list
> Nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau
^ permalink raw reply [flat|nested] 5+ messages in thread* [Bug 29129] Cannot set refresh rate on nv50
[not found] ` <bug-29129-8800-V0hAGp6uBxMKqLRl/0Ahz6D7qz1kEfGD2LY78lusg7I@public.gmane.org/>
2010-07-17 13:34 ` Ben Skeggs
@ 2010-07-17 13:34 ` bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
2010-07-17 17:00 ` bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
2013-08-18 18:10 ` bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
3 siblings, 0 replies; 5+ messages in thread
From: bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ @ 2010-07-17 13:34 UTC (permalink / raw)
To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW
https://bugs.freedesktop.org/show_bug.cgi?id=29129
--- Comment #1 from Ben Skeggs <skeggsb-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2010-07-17 06:34:57 PDT ---
I *seriously* doubt this commit is responsible. It has nothing even remotely
close to any changes that could cause what you're seeing.
Ben.
Sent from my iPhone
On 17/07/2010, at 22:39, bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ@public.gmane.org wrote:
> https://bugs.freedesktop.org/show_bug.cgi?id=29129
>
> Summary: Cannot set refresh rate on nv50
> Product: xorg
> Version: unspecified
> Platform: x86-64 (AMD64)
> URL: http://marc.info/?l=linux-kernel&m=127913459017236&w=2
> OS/Version: Linux (All)
> Status: NEW
> Severity: normal
> Priority: medium
> Component: Driver/nouveau
> AssignedTo: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> ReportedBy: svenjoac-Mmb7MZpHnFY@public.gmane.org
> QAContact: xorg-team-go0+a7rfsptAfugRpC6u6w@public.gmane.org
>
>
> Since Linux 2.6.34, it is no longer possible to set the refresh rate with the
> "video=...@nn" parameter, e.g. video=1280x1024@75 is ignored.
>
> I bisected this to the following commit:
>
> drm/nv50: enable hpd on any connector we know the gpio line for
>
> Signed-off-by: Ben Skeggs <bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
>
> diff --git a/drivers/gpu/drm/nouveau/nv50_display.c
> b/drivers/gpu/drm/nouveau/nv50_display.c
> index 61a89f2..ebcd8bf 100644
> --- a/drivers/gpu/drm/nouveau/nv50_display.c
> +++ b/drivers/gpu/drm/nouveau/nv50_display.c
> @@ -370,7 +370,9 @@ nv50_display_init(struct drm_device *dev)
> struct nouveau_connector *conn = nouveau_connector(connector);
> struct dcb_gpio_entry *gpio;
>
> - if (conn->dcb->gpio_tag == 0xff)
> + if (connector->connector_type != DRM_MODE_CONNECTOR_DVII &&
> + connector->connector_type != DRM_MODE_CONNECTOR_DVID &&
> + connector->connector_type != DRM_MODE_CONNECTOR_DisplayPort)
> continue;
>
> gpio = nouveau_bios_gpio_entry(dev, conn->dcb->gpio_tag);
>
> diff --git a/drivers/gpu/drm/nouveau/nv50_display.c
> b/drivers/gpu/drm/nouveau/nv50_display.c
> index ebcd8bf..61a89f2 100644
> --- a/drivers/gpu/drm/nouveau/nv50_display.c
> +++ b/drivers/gpu/drm/nouveau/nv50_display.c
> @@ -370,9 +370,7 @@ nv50_display_init(struct drm_device *dev)
> struct nouveau_connector *conn = nouveau_connector(connector);
> struct dcb_gpio_entry *gpio;
>
> - if (connector->connector_type != DRM_MODE_CONNECTOR_DVII &&
> - connector->connector_type != DRM_MODE_CONNECTOR_DVID &&
> - connector->connector_type != DRM_MODE_CONNECTOR_DisplayPort)
> + if (conn->dcb->gpio_tag == 0xff)
> continue;
>
> gpio = nouveau_bios_gpio_entry(dev, conn->dcb->gpio_tag);
>
>
> It should be noted that since this commit all the card data appear under
> /sys/class/drm/card0-DVI-I-1, e.g. /sys/class/drm/card0-DVI-I-1/enabled says
> "enabled" and /sys/class/drm/card0-VGA-1/enabled says "disabled", despite the
> monitor being connected via VGA.
>
> I only noticed this in 2.6.35-rc5 because the default refresh rate changed to
> 60 Hz there, see https://bugzilla.kernel.org/show_bug.cgi?id=16402.
>
> --
> Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are the assignee for the bug.
> _______________________________________________
> Nouveau mailing list
> Nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread* [Bug 29129] Cannot set refresh rate on nv50
[not found] ` <bug-29129-8800-V0hAGp6uBxMKqLRl/0Ahz6D7qz1kEfGD2LY78lusg7I@public.gmane.org/>
2010-07-17 13:34 ` Ben Skeggs
2010-07-17 13:34 ` [Bug 29129] " bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
@ 2010-07-17 17:00 ` bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
2013-08-18 18:10 ` bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
3 siblings, 0 replies; 5+ messages in thread
From: bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ @ 2010-07-17 17:00 UTC (permalink / raw)
To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW
https://bugs.freedesktop.org/show_bug.cgi?id=29129
--- Comment #2 from Sven Joachim <svenjoac-Mmb7MZpHnFY@public.gmane.org> 2010-07-17 10:00:50 PDT ---
You're of course right, Ben, looks like I screwed up the last bisection step.
The actual change that introduced the buggy behavior was the previous one,
7f612d87f8b4b1ddbcee04264a93e5af7b9f21c7:
drm/nouveau: use dcb connector table for creating drm connectors
This makes this code common to both the nv04 and nv50 paths.
For the moment, we keep the previous behaviour with HDMI/eDP connectors
and report them as DVI-D/DP instead. This will be fixed once the rest
of the code has been fixed to deal with those types.
Signed-off-by: Ben Skeggs <bskeggs-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
I refrain from including the 274 lines "git diff" output.
In dmesg I see the following messages:
[ 0.896501] [drm] nouveau 0000:01:00.0: Detected a DVI-I connector
[ 0.896543] [drm] nouveau 0000:01:00.0: Detected a VGA connector
[ 0.896570] [drm] nouveau 0000:01:00.0: Detected a TV connector
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread* [Bug 29129] Cannot set refresh rate on nv50
[not found] ` <bug-29129-8800-V0hAGp6uBxMKqLRl/0Ahz6D7qz1kEfGD2LY78lusg7I@public.gmane.org/>
` (2 preceding siblings ...)
2010-07-17 17:00 ` bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
@ 2013-08-18 18:10 ` bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ
3 siblings, 0 replies; 5+ messages in thread
From: bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ @ 2013-08-18 18:10 UTC (permalink / raw)
To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW
[-- Attachment #1.1: Type: text/plain, Size: 1289 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=29129
Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |INVALID
--- Comment #3 from Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org> ---
It appears that this bug report has laid dormant for quite a while. Sorry we
haven't gotten to it. Since we fix bugs all the time, chances are pretty good
that your issue has been fixed with the latest software. Please give it a shot.
(Linux kernel 3.10.7, xf86-video-nouveau 1.0.9, mesa 9.1.6, or their git
versions.) If upgrading to the latest isn't an option for you, your distro's
bugzilla is probably the right destination for your bug report.
In an effort to clean up our bug list, we're pre-emptively closing all bugs
that haven't seen updates since 2011. If the original issue remains, please
make sure to provide fresh info, see http://nouveau.freedesktop.org/wiki/Bugs/
for what we need to see, and re-open this one.
Thanks,
The Nouveau Team
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 2704 bytes --]
[-- Attachment #2: Type: text/plain, Size: 181 bytes --]
_______________________________________________
Nouveau mailing list
Nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
http://lists.freedesktop.org/mailman/listinfo/nouveau
^ permalink raw reply [flat|nested] 5+ messages in thread