Re: Re: [PATCH] PM / devfreq: Use freq_table for available_frequencies

2014-04-18 Thread MyungJoo Ham
> On 04/15/2014 11:41 AM, Saravana Kannan wrote: > > Ah, I misunderstood your previous email. I thought you Nack-ed my patch > > and decided to send your own patch to replace mine. Ok, I'll fix up mine > > and send it out. > > MyungJoo/Kyungmin, > > I sent out an updated patch. Can you please tak

Re: [PATCH] PM / devfreq: Use freq_table for available_frequencies

2014-04-16 Thread Saravana Kannan
On 04/15/2014 11:41 AM, Saravana Kannan wrote: > Ah, I misunderstood your previous email. I thought you Nack-ed my patch > and decided to send your own patch to replace mine. Ok, I'll fix up mine > and send it out. MyungJoo/Kyungmin, I sent out an updated patch. Can you please take a look? Thank

Re: [PATCH] PM / devfreq: Use freq_table for available_frequencies

2014-04-15 Thread Saravana Kannan
On 04/14/2014 11:30 PM, 함명주 wrote: >> On 04/14/2014 06:36 PM, 함명주 wrote: MyungJoo/Kyungmin, Bump. Can we accept this patch please? -Saravana >>> >>> Nack. >>> >>> Please note that freq_table is also an optional value, which may >>> be null. >> >> Ah, I saw that the max_freq

Re: Re: [PATCH] PM / devfreq: Use freq_table for available_frequencies

2014-04-14 Thread 함명주
> On 04/14/2014 06:36 PM, 함명주 wrote: > >> MyungJoo/Kyungmin, > >> > >> Bump. Can we accept this patch please? > >> > >> -Saravana > > > > Nack. > > > > Please note that freq_table is also an optional value, which may > > be null. > > Ah, I saw that the max_freq would be zero if freq_table was NU

Re: [PATCH] PM / devfreq: Use freq_table for available_frequencies

2014-04-14 Thread Saravana Kannan
On 04/14/2014 06:36 PM, 함명주 wrote: >> MyungJoo/Kyungmin, >> >> Bump. Can we accept this patch please? >> >> -Saravana > > Nack. > > Please note that freq_table is also an optional value, which may > be null. Ah, I saw that the max_freq would be zero if freq_table was NULL and I assumed that it c

Re: Re: [PATCH] PM / devfreq: Use freq_table for available_frequencies

2014-04-14 Thread 함명주
> MyungJoo/Kyungmin, > > Bump. Can we accept this patch please? > > -Saravana Nack. Please note that freq_table is also an optional value, which may be null. Besides, please be aware that your code is under rcu_read_lock(). Cheers, MyungJoo. ps. I'll send a related patch (avoid accessing nu

Re: [PATCH] PM / devfreq: Use freq_table for available_frequencies

2014-04-14 Thread Saravana Kannan
MyungJoo/Kyungmin, Bump. Can we accept this patch please? -Saravana On 04/10/2014 07:54 PM, Saravana Kannan wrote: Some devices use freq_table instead of OPP. For those devices, the available_frequencies file shows up empty. Fix that by using freq_table to generate the available_frequencies d

[PATCH] PM / devfreq: Use freq_table for available_frequencies

2014-04-10 Thread Saravana Kannan
Some devices use freq_table instead of OPP. For those devices, the available_frequencies file shows up empty. Fix that by using freq_table to generate the available_frequencies data when OPP is not present. Signed-off-by: Saravana Kannan --- drivers/devfreq/devfreq.c | 15 +++ 1 file