On Wed, Nov 4, 2020 at 9:35 PM Andy Shevchenko <[email protected]> wrote: > > On Wed, Nov 4, 2020 at 9:34 PM Bartosz Golaszewski <[email protected]> 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

