* 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