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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



W dniu 7 czerwca 2010 18:09 użytkownik Alex Deucher
<alexdeucher@xxxxxxxxx> napisał:
> 2010/6/7 Rafał Miłecki <zajec5@xxxxxxxxx>:
>> W dniu 7 czerwca 2010 07:00 użytkownik Alex Deucher
>> <alexdeucher@xxxxxxxxx> napisał:
>>> 2010/6/6 Rafał Miłecki <zajec5@xxxxxxxxx>:
>>>> 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@xxxxxxxxx>
>>>
>>> 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ł
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux