On Wed, May 27, 2015 at 02:52:36PM +0530, Viresh Kumar wrote: > > + timer->flags = (timer->flags & ~TIMER_BASEMASK) | cpu; > > Because 'cpu' is used only once in this routine, maybe we can use > 'new_base->cpu' here and the line will still be exactly 80 columns > long. > > Not sure, but maybe we can create a inline helper for this operation > as it is repeated at multiple places.
I suggested bitfields at some point I think. That gets the compiler to generate those helpers for you. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
