On Thu, Dec 10, 2020 at 11:08:03AM +0100, Marco Felsch wrote:
> On 20-12-09 15:09, Dmitry Torokhov wrote:

...

> >     case EV_FT:
> >             reg_addr->reg_threshold = EV_REGISTER_THRESHOLD;
> > +           reg_addr->reg_report_rate = NO_REGISTER;
> >             reg_addr->reg_gain = EV_REGISTER_GAIN;
> >             reg_addr->reg_offset = NO_REGISTER;
> >             reg_addr->reg_offset_x = EV_REGISTER_OFFSET_X;
> >             reg_addr->reg_offset_y = EV_REGISTER_OFFSET_Y;
> >             reg_addr->reg_num_x = NO_REGISTER;
> >             reg_addr->reg_num_y = NO_REGISTER;
> > -           reg_addr->reg_report_rate = NO_REGISTER;
> >             break;
> 
> Nit:
> Unrelated change.

I guess the motive is to get these assignments consistent between the cases.
Documentation actually allows this kind of modifications of code in one change
when them are toughly related.

-- 
With Best Regards,
Andy Shevchenko


Reply via email to