On Wed, Mar 11, 2020 at 09:48:30AM +0100, Tobias Schramm wrote: > was just about to send v3. > > > > > __be16 value; > > > > > > ret = regmap(..., (...)value, sizeof(value)); Just checked this, simple &value works. > > if (ret) > > return ret; // note, you missed this in above. > > > > *val = be16_to_cpu(value); > > return 0; > > > > That implementation looks pretty clean. I'll take it. Thank you. -- With Best Regards, Andy Shevchenko