* Thomas Gleixner <t...@linutronix.de> wrote:

> Now that HPET clockevent support is integrated into the channel data, reuse
> the cached boot configuration instead of copying the same information into
> a flags field.
> 
> This also allows to consolidate the reservation code into one place, which
> can now solely depend on the mode information.
> 
> Signed-off-by: Thomas Gleixner <t...@linutronix.de>
> ---
>  arch/x86/kernel/hpet.c |   76 
> ++++++++++++++-----------------------------------
>  1 file changed, 23 insertions(+), 53 deletions(-)
> 
> --- a/arch/x86/kernel/hpet.c
> +++ b/arch/x86/kernel/hpet.c
> @@ -25,8 +25,8 @@ struct hpet_channel {
>       unsigned int                    num;
>       unsigned int                    cpu;
>       unsigned int                    irq;
> +     unsigned int                    inuse;
>       enum hpet_mode                  mode;
> -     unsigned int                    flags;
>       unsigned int                    boot_cfg;
>       char                            name[10];

Let's fight entropy name by name and s/inuse/in_use ?

Because in_use is closer to harmony:

 dagon:~/tip> git grep -w in_use | wc -l
 518
 dagon:~/tip> git grep -w inuse | wc -l
 318

and also easier to read.

Reviewed-by: Ingo Molnar <mi...@kernel.org>

Thanks,

        Ingo

Reply via email to