Hi Andy, was just about to send v3. > > __be16 value; > > > ret = regmap(..., (...)value, sizeof(value)); > 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. Thanks, Tobias