[PATCH 1/2] drm/radeon/kms/r600+: do not set the same voltage as current one

[Rafał Miłecki]

W dniu 7 czerwca 2010 18:09 użytkownik Alex Deucher
<alexdeucher at gmail.com> napisał:
> 2010/6/7 Rafał Miłecki <zajec5 at gmail.com>:
>> W dniu 7 czerwca 2010 07:00 użytkownik Alex Deucher
>> <alexdeucher at gmail.com> napisał:
>>> 2010/6/6 Rafał Miłecki <zajec5 at gmail.com>:
>>>> We have similar tracking for engine and memory. We could think about some
>>>> solution for older GPUs as well, however there are not strict voltage values
>>>> set.
>>>>
>>>> Signed-off-by: Rafał Miłecki <zajec5 at gmail.com>
>>>
>>> I actually had almost the exact same patch in my tree (broken out from
>>> my voltage step patch), I just hadn't gotten around to sending it.
>>> Mine is against drm-linus and has the default voltage level set.  See
>>> attached.
>>
>> I know, actually I've stolen your name (curr_vddc).
>>
>> Do you plan to include this in .35? If yes, it's fine. If not, I'd
>> prefer to split your patch into two. The one I've sent and second that
>> adds you step setting.
>
> I'm not planning to include the whole step setting patch, but I did
> break out the current_vddc part and planned to push that (attached to
> the last email) for 2.6.35.

Ahh, indeed, sorry. I just wonder about setting current_vddc even when
we do not call radeon_atom_set_voltage. This way we will have false
value in current_vddc, if voltage change is not available (type !=
VOLTAGE_SW).

-- 
Rafał