On Sun, May 18, 2025 at 11:13:20AM +0000, Lothar Rubusch wrote: > Link activity and inactivity to indicate the internal power-saving state. > Add auto-sleep to be linked to inactivity. ... > + en = en && act_en && inact_en; > + > + return regmap_update_bits(data->regmap, > + ADXL313_REG_POWER_CTL, > + ADXL313_POWER_CTL_INACT_MSK, > + en ? (ADXL313_POWER_CTL_AUTO_SLEEP | ADXL313_POWER_CTL_LINK) > + : 0); Make it a logical split en ? (ADXL313_POWER_CTL_AUTO_SLEEP | ADXL313_POWER_CTL_LINK) : 0); -- With Best Regards, Andy Shevchenko