W dniu 11 września 2009 20:41 użytkownik Rafał Miłecki
<[email protected]> napisał:
> This patchset implements features needed to implement real power management.
>
> Last one (0005) even shows that it's possible to implement
> downclocking on DPMS_OFF with previous patches. It's just prove of
> working infrastructure, nothing acceptable to commit. However I hope
> it makes you decide commiting 0001-0004.
>
> Please review/test/commit. If you decide to apply last patch, just use:
> xset dpms force off

I was thinking over weekend about possible bad decisions made here
that will hit us later.

It's quite possible we won't use only limited amount of power modes
but quite big amount of dynamically generated modes between "minimum"
and "maximum" (with some step for clocks and voltage). Anyway even
then we will need to keep "minimum" and "maximum" (and "default"?).
And keeping that in 1 struct properties will be pain:
radeon_pm->minimum_engine_clock
radeon_pm->minimum_memory_clock
radeon_pm->minimum_voltage
*3

So I believe introducing radeom_pm_state is still a good idea.
Eventually we will modify state setting function, but probably nothing
more.

-- 
Rafał

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to