On Wed, Jun 07, 2023 at 11:22:48PM +0300, Andy Shevchenko wrote: > On Wed, Jun 07, 2023 at 06:46:52PM +0200, Hans de Goede wrote: > > The CSI2 specification specifies a standard method to access camera sensor > > registers called "Camera Control Interface (CCI)". > > > > This uses either 8 or 16 bit (big-endian wire order) register addresses > > and supports 8, 16, 24 or 32 bit (big-endian wire order) register widths. > > > > Currently a lot of Linux camera sensor drivers all have their own custom > > helpers for this, often copy and pasted from other drivers. > > > > Add a set of generic helpers for this so that all sensor drivers can > > switch to a single common implementation. > > > > These helpers take an extra optional "int *err" function parameter, > > this can be used to chain a bunch of register accesses together with > > only a single error check at the end, rather then needing to error > > check each individual register access. The first failing call will > > set the contents of err to a non 0 value and all other calls will > > then become no-ops. > > So, this version doesn't have updates we discussed previously. Is it only for > reference here? Okay, I found the answer in the cover letter. -- With Best Regards, Andy Shevchenko