Messages in this thread | | | Date | Tue, 17 Mar 2020 12:42:23 +0200 | From | Andy Shevchenko <> | Subject | Re: [PATCH] Input: edt-ft5x06 - add fw_version debugfs file to read |
| |
On Tue, Mar 17, 2020 at 11:25:39AM +0100, Martin Kepplinger wrote: > On 16.03.20 15:40, Andy Shevchenko wrote: > > On Mon, Mar 16, 2020 at 03:27:56PM +0100, Martin Kepplinger wrote: > >> Add simple fw_version file in debugfs to read the value from 0xa6 > >> which is the firmware version. > > > > > > If you switch to regmap I²C API you will get this for free for all defined > > registers. > > > > So, I highly recommend to consider above. > > > > I don't know all the devices and modes well enough if the current raw > read/write buffers always translate to (only) buf[0] being the register > address. > > If you tell me I can assume just that, I'm happy to do a larger change > that completely does aways with the current raw buffers.
I have only one device at hand, so, I can't tell for all neither. Maybe Dmitry has an opinion on this?
> Otherwise: > > Does is make sense to *add* just a few regmap calls but keep the current > i2c_transfer in place for the calls I'm not sure about? I could do that > in any case.
I think it's way to the opposite direction, i.e. increasing burden (for maintainability) and technical debt.
-- With Best Regards, Andy Shevchenko
| |