[PATCH] drm/radeon/kms: add support for internal thermal sensors (v2)

[Alex Deucher]

On Wed, Jun 2, 2010 at 7:55 AM, Julien Cristau <jcristau at debian.org> wrote:
> On Tue, Jun  1, 2010 at 18:52:30 -0400, Alex Deucher wrote:
>
>> diff --git a/drivers/gpu/drm/radeon/Kconfig b/drivers/gpu/drm/radeon/Kconfig
>> index 80c5b3e..2e3896d 100644
>> --- a/drivers/gpu/drm/radeon/Kconfig
>> +++ b/drivers/gpu/drm/radeon/Kconfig
>> @@ -2,6 +2,7 @@ config DRM_RADEON_KMS
>>       bool "Enable modesetting on radeon by default - NEW DRIVER"
>>       depends on DRM_RADEON
>>       depends on POWER_SUPPLY
>> +     depends on HWMON
>>       help
>>         Choose this option if you want kernel modesetting enabled by default.
>>
> Shouldn't these be under CONFIG_DRM_RADEON, with _KMS only changing the
> default value of the modeset parameter?  Otherwise you could still try
> to build the driver with _KMS off, and CONFIG_HWMON disabled (which I
> assume is going to fail).

Yes, I patched an old version of the tree without this patch:
http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;a=commitdiff;h=5d9b7e2d2db95549aeab129c95d588236137a4c8

I'll update the patch.

>
> Cheers,
> Julien
>