On Wed, Nov 4, 2020 at 9:35 PM Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > > On Wed, Nov 4, 2020 at 9:34 PM Bartosz Golaszewski <brgl@xxxxxxxx> wrote: > > ... > > > +static const struct regmap_config exar_regmap_config = { > > + .name = "exar-gpio", > > + .reg_bits = 8, > > + .val_bits = 8, > > +}; > > Looking at gpio-pca953xx regmap conversion I'm wondering shouldn't you > provide a callback to define volatile registers (such as GPIO input > bits)? > > -- > With Best Regards, > Andy Shevchenko I think this was done in pca953x due to weird calculations of banks and registers. For a rather simple driver like this one I don't think this is needed. Bartosz