lkml.org 
[lkml]   [2014]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] cpufreq: create another field .flags in cpufreq_frequency_table
From
On 28 March 2014 15:49, Lukasz Majewski <l.majewski@samsung.com> wrote:
> One little remark - since cpufreq_frequency_tables are defined as
> static it seems like we don't need to explicitly specify the 0, for
> flags field.
>
> Something like {, L13, 200 * 1000}, shall be enough.

This looks awkward, writing zero is better. The other option was to do
it like this:

{.driver_data = L13, .frequency = 200 * 1000}

But I didn't wanted to make such changes in this patch.

>> +/* Special Values of .flags field */
>> +#define CPUFREQ_BOOST_FREQ 0x1
>
> Maybe (1 << 0) to explicitly show that those are flags.

Will do that in case I resend this patch, otherwise it wouldn't
harm for now.

>> struct cpufreq_frequency_table {
>> + unsigned int flags;
>> unsigned int driver_data; /* driver specific data, not
>
> The driver_data shall be changed to int. However I suppose that it will
> be done at separate patch.

I am not planning to change it for now. People storing 'int' into this
field would read it back into 'int' and so things would work as normal
I suppose.


\
 
 \ /
  Last update: 2014-03-28 12:01    [W:0.079 / U:26.244 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site