Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-18 Thread Rafael J. Wysocki
On Tuesday, June 18, 2013 10:20:33 AM Viresh Kumar wrote: > On 5 June 2013 18:38, Viresh Kumar wrote: > > To be honest with the amount of experience I have now, my log was > > poor :( > > > > I have used following log in the attached patch: > > > > Subject: [PATCH] cpufreq: userspace: Simplify gov

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-17 Thread Viresh Kumar
On 5 June 2013 18:38, Viresh Kumar wrote: > To be honest with the amount of experience I have now, my log was > poor :( > > I have used following log in the attached patch: > > Subject: [PATCH] cpufreq: userspace: Simplify governor > > Userspace governor has got more code than what it needs for it

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-08 Thread Xiaoguang Chen
2013/6/5 Viresh Kumar : > On 5 June 2013 18:07, Rafael J. Wysocki wrote: >> On Wednesday, June 05, 2013 02:34:33 PM Viresh Kumar wrote: >>> Userspace governor has got more code than what it needs for its functioning. >>> Lets simplify it. >> >> OK >> >> Why exactly is the code you're removing unne

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-05 Thread Viresh Kumar
On 5 June 2013 18:07, Rafael J. Wysocki wrote: > On Wednesday, June 05, 2013 02:34:33 PM Viresh Kumar wrote: >> Userspace governor has got more code than what it needs for its functioning. >> Lets simplify it. > > OK > > Why exactly is the code you're removing unnecessary? > >> Over that it will f

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-05 Thread Rafael J. Wysocki
On Wednesday, June 05, 2013 02:34:33 PM Viresh Kumar wrote: > Userspace governor has got more code than what it needs for its functioning. > Lets simplify it. OK Why exactly is the code you're removing unnecessary? > Over that it will fix issues in cpufreq_governor_userspace(), which isn't > do