Attachment 49724 Details for Bug 39513 – fix

[patch] fix

0001-drm-radeon-kms-add-thermal-chip-quirk-for-asus-9600x.patch (text/plain), 2.23 KB, created by Alex Deucher on 2011-07-29 09:18:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Deucher

Created: 2011-07-29 09:18:22 UTC

Size: 2.23 KB

patch

obsolete

>From e6226b01975a4660f1eebd3d0ea4e423c3dc880d Mon Sep 17 00:00:00 2001
>From: Alex Deucher <alexander.deucher@amd.com>
>Date: Fri, 29 Jul 2011 12:15:53 -0400
>Subject: [PATCH] drm/radeon/kms: add thermal chip quirk for asus 9600xt
>
>The board has an lm63 compatible thermal chip, but no
>thermal chip entry in the vbios tables.
>
>Fixes:
>https://bugs.freedesktop.org/show_bug.cgi?id=39513
>
>Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
>---
> drivers/gpu/drm/radeon/radeon_combios.c |   21 ++++++++++++++++++++-
> 1 files changed, 20 insertions(+), 1 deletions(-)
>
>diff --git a/drivers/gpu/drm/radeon/radeon_combios.c b/drivers/gpu/drm/radeon/radeon_combios.c
>index a74217c..767a91f 100644
>--- a/drivers/gpu/drm/radeon/radeon_combios.c
>+++ b/drivers/gpu/drm/radeon/radeon_combios.c
>@@ -2557,6 +2557,7 @@ void radeon_combios_get_power_modes(struct radeon_device *rdev)
> 	u16 offset, misc, misc2 = 0;
> 	u8 rev, blocks, tmp;
> 	int state_index = 0;
>+	struct radeon_i2c_bus_rec i2c_bus;
> 
> 	rdev->pm.default_power_state_index = -1;
> 
>@@ -2575,7 +2576,6 @@ void radeon_combios_get_power_modes(struct radeon_device *rdev)
> 	offset = combios_get_table_offset(dev, COMBIOS_OVERDRIVE_INFO_TABLE);
> 	if (offset) {
> 		u8 thermal_controller = 0, gpio = 0, i2c_addr = 0, clk_bit = 0, data_bit = 0;
>-		struct radeon_i2c_bus_rec i2c_bus;
> 
> 		rev = RBIOS8(offset);
> 
>@@ -2617,6 +2617,25 @@ void radeon_combios_get_power_modes(struct radeon_device *rdev)
> 				i2c_new_device(&rdev->pm.i2c_bus->adapter, &info);
> 			}
> 		}
>+	} else {
>+		/* boards with a thermal chip, but no overdrive table */
>+
>+		/* Asus 9600xt has an lm63/f75375s on the MONID bus */
>+		if ((dev->pdev->device == 0x4152) &&
>+		    (dev->pdev->subsystem_vendor == 0x1043) &&
>+		    (dev->pdev->subsystem_device == 0xc002)) {
>+			i2c_bus = combios_setup_i2c_bus(rdev, DDC_MONID, 0, 0);
>+			rdev->pm.i2c_bus = radeon_i2c_lookup(rdev, &i2c_bus);
>+			if (rdev->pm.i2c_bus) {
>+				struct i2c_board_info info = { };
>+				const char *name = "f75375s";
>+				info.addr = 0x2e;
>+				strlcpy(info.type, name, sizeof(info.type));
>+				i2c_new_device(&rdev->pm.i2c_bus->adapter, &info);
>+				DRM_INFO("Possible %s thermal controller at 0x%02x\n",
>+					 name, info.addr);
>+			}
>+		}
> 	}
> 
> 	if (rdev->flags & RADEON_IS_MOBILITY) {
>-- 
>1.7.1.1
>

Actions: View | Splinter Review

Attachments on bug 39513: 49685 | 49686 | 49724 | 49727 | 49728

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.