[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: [PATCH] powerdevil-remove-cpufreq.patch
From:       Dario Freddi <drf54321 () gmail ! com>
Date:       2010-03-19 12:39:47
Message-ID: 201003191339.51342.drf54321 () gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


(I'm dropping in now that the waters seems quiet)

Holger,

I have in mind a decent refactoring in powerdevil in the near future, due to 
some changes in Solid::Control that probably would be nice to have. I would 
appreciate if you could join in and helping out in it.

That said, my humble opinion.

This is "show the remaining time in the battery applet" all the way. And I 
think a similar solution could be taken. So: remove the control from the UI, 
but leave a "hidden" option in the profiles file letting the advanced user set 
his governor. What do you guys think? I think this should give out the best of 
both worlds.

On Friday 19 March 2010 13:32:52 Holger Macht wrote:
> On Mon 08. Mar - 13:39:16, Sebastian Kügler wrote:
> > On Sunday 07 March 2010 23:57:37 Holger Macht wrote:
> > > Remove all CPU frequency scaling code from powerdevil and the kcm
> > > module. You should never offer those options to the user because he
> > > might not understand what the impact is. You only want a dynamic
> > > frequency algorithm which is the default these days (ondemand
> > > governor). A good explanation is provided in [1].
> > 
> > +1. I agree that this option does not even remotely cover the intricacies
> > of CPU
> 
> So, are you willing to commit the patch on bahalf of me? ;-)
> 
> Thanks,
>  Holger
> 
> >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to
> >> unsubscribe <<

-- 
-------------------

Dario Freddi
KDE Developer
GPG Key Signature: 511A9A3B

["signature.asc" (application/pgp-signature)]

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


[prev in list] [next in list] [prev in thread] [next in thread]