diff mbox

drm/radeon/kms: set a default max_pixel_clock

Message ID 1314732186-13571-1-git-send-email-airlied@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Dave Airlie Aug. 30, 2011, 7:23 p.m. UTC
From: Dave Airlie <airlied@redhat.com>

On some Power rv100 cards, we have no ATY OF table, but we have
no combios table either, and hence we refuse all modes on VGA-0
since we end up with a 0 max pixel clock.

Signed-off-by: Dave Airlie <airlied@redhat.com>
---
 drivers/gpu/drm/radeon/radeon_clocks.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

Comments

Alex Deucher Aug. 30, 2011, 7:39 p.m. UTC | #1
On Tue, Aug 30, 2011 at 3:23 PM, Dave Airlie <airlied@gmail.com> wrote:
> From: Dave Airlie <airlied@redhat.com>
>
> On some Power rv100 cards, we have no ATY OF table, but we have
> no combios table either, and hence we refuse all modes on VGA-0
> since we end up with a 0 max pixel clock.
>
> Signed-off-by: Dave Airlie <airlied@redhat.com>

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

> ---
>  drivers/gpu/drm/radeon/radeon_clocks.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c b/drivers/gpu/drm/radeon/radeon_clocks.c
> index dcd0863e..b6e18c8 100644
> --- a/drivers/gpu/drm/radeon/radeon_clocks.c
> +++ b/drivers/gpu/drm/radeon/radeon_clocks.c
> @@ -219,6 +219,9 @@ void radeon_get_clock_info(struct drm_device *dev)
>                } else {
>                        DRM_INFO("Using generic clock info\n");
>
> +                       /* may need to be per card */
> +                       rdev->clock.max_pixel_clock = 35000;
> +
>                        if (rdev->flags & RADEON_IS_IGP) {
>                                p1pll->reference_freq = 1432;
>                                p2pll->reference_freq = 1432;
> --
> 1.7.6
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
Jerome Glisse Aug. 30, 2011, 7:41 p.m. UTC | #2
On Tue, Aug 30, 2011 at 3:23 PM, Dave Airlie <airlied@gmail.com> wrote:
> From: Dave Airlie <airlied@redhat.com>
>
> On some Power rv100 cards, we have no ATY OF table, but we have
> no combios table either, and hence we refuse all modes on VGA-0
> since we end up with a 0 max pixel clock.
>
> Signed-off-by: Dave Airlie <airlied@redhat.com>
Reviewed-by: Jerome Glisse <jglisse@redhat.com>
> ---
>  drivers/gpu/drm/radeon/radeon_clocks.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c b/drivers/gpu/drm/radeon/radeon_clocks.c
> index dcd0863e..b6e18c8 100644
> --- a/drivers/gpu/drm/radeon/radeon_clocks.c
> +++ b/drivers/gpu/drm/radeon/radeon_clocks.c
> @@ -219,6 +219,9 @@ void radeon_get_clock_info(struct drm_device *dev)
>                } else {
>                        DRM_INFO("Using generic clock info\n");
>
> +                       /* may need to be per card */
> +                       rdev->clock.max_pixel_clock = 35000;
> +
>                        if (rdev->flags & RADEON_IS_IGP) {
>                                p1pll->reference_freq = 1432;
>                                p2pll->reference_freq = 1432;
> --
> 1.7.6
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
diff mbox

Patch

diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c b/drivers/gpu/drm/radeon/radeon_clocks.c
index dcd0863e..b6e18c8 100644
--- a/drivers/gpu/drm/radeon/radeon_clocks.c
+++ b/drivers/gpu/drm/radeon/radeon_clocks.c
@@ -219,6 +219,9 @@  void radeon_get_clock_info(struct drm_device *dev)
 		} else {
 			DRM_INFO("Using generic clock info\n");
 
+			/* may need to be per card */
+			rdev->clock.max_pixel_clock = 35000;
+
 			if (rdev->flags & RADEON_IS_IGP) {
 				p1pll->reference_freq = 1432;
 				p2pll->reference_freq = 1432;