Attachment 86296 Details for Bug 68235 – patch 1/2

[patch] patch 1/2

0001-drm-radeon-dpm-fetch-the-max-clk-from-voltage-dep-ta.patch (text/plain), 2.37 KB, created by Alex Deucher on 2013-09-21 19:45:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Deucher

Created: 2013-09-21 19:45:06 UTC

Size: 2.37 KB

patch

obsolete

>From 465ff12be96159ef7173ae7a03ed7d24a3b37990 Mon Sep 17 00:00:00 2001
>From: Alex Deucher <alexander.deucher@amd.com>
>Date: Sat, 21 Sep 2013 13:47:57 -0400
>Subject: [PATCH 1/6] drm/radeon/dpm: fetch the max clk from voltage dep tables
> helper
>
>This patch adds a helper function to fetch the max clock
>from the voltage clock dependecy tables.  Clocks above that
>level tend to be unstable and may require voltage tweaks.
>
>This patch implemented the helper function to fetch the max clocks
>from the dependency tables.  The following patches implement the
>per-asic clock filtering.
>
>See bug:
>https://bugs.freedesktop.org/show_bug.cgi?id=68235
>
>Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
>---
> drivers/gpu/drm/radeon/btc_dpm.c | 17 +++++++++++++++++
> drivers/gpu/drm/radeon/btc_dpm.h |  2 ++
> 2 files changed, 19 insertions(+)
>
>diff --git a/drivers/gpu/drm/radeon/btc_dpm.c b/drivers/gpu/drm/radeon/btc_dpm.c
>index 05ff315..0d0f065 100644
>--- a/drivers/gpu/drm/radeon/btc_dpm.c
>+++ b/drivers/gpu/drm/radeon/btc_dpm.c
>@@ -1168,6 +1168,23 @@ static const struct radeon_blacklist_clocks btc_blacklist_clocks[] =
>         { 25000, 30000, RADEON_SCLK_UP }
> };
> 
>+void btc_get_max_clock_from_voltage_dependency_table(struct radeon_clock_voltage_dependency_table *table,
>+						     u32 *max_clock)
>+{
>+	u32 i, clock = 0;
>+
>+	if ((table == NULL) || (table->count == 0)) {
>+		*max_clock = clock;
>+		return;
>+	}
>+
>+	for (i = 0; i < table->count; i++) {
>+		if (clock < table->entries[i].clk)
>+			clock = table->entries[i].clk;
>+	}
>+	*max_clock = clock;
>+}
>+
> void btc_apply_voltage_dependency_rules(struct radeon_clock_voltage_dependency_table *table,
> 					u32 clock, u16 max_voltage, u16 *voltage)
> {
>diff --git a/drivers/gpu/drm/radeon/btc_dpm.h b/drivers/gpu/drm/radeon/btc_dpm.h
>index 1a15e0e..3b6f12b 100644
>--- a/drivers/gpu/drm/radeon/btc_dpm.h
>+++ b/drivers/gpu/drm/radeon/btc_dpm.h
>@@ -46,6 +46,8 @@ void btc_adjust_clock_combinations(struct radeon_device *rdev,
> 				   struct rv7xx_pl *pl);
> void btc_apply_voltage_dependency_rules(struct radeon_clock_voltage_dependency_table *table,
> 					u32 clock, u16 max_voltage, u16 *voltage);
>+void btc_get_max_clock_from_voltage_dependency_table(struct radeon_clock_voltage_dependency_table *table,
>+						     u32 *max_clock);
> void btc_apply_voltage_delta_rules(struct radeon_device *rdev,
> 				   u16 max_vddc, u16 max_vddci,
> 				   u16 *vddc, u16 *vddci);
>-- 
>1.8.3.1
>

Actions: View | Splinter Review

Attachments on bug 68235: 84186 | 85157 | 85578 | 85798 | 85989 | 86111 | 86112 | 86147 | 86168 | 86296 | 86297

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.