Hi Andy, On Tue, Jul 26, 2022 at 03:05:50PM +0300, Andy Shevchenko wrote: > Besides not being understandable at the first glance, the code > might provoke a compiler or a static analyser tool to warn about > out-of-bound access (when len == 0). I've never seen one. However the same pattern is repeatedly used by many, many drivers and addressing just one doesn't make much sense. The proper way to fix this would be to have a set of common CCI (Camera Control Interface) functions that all drivers could use, and then switch the drivers to use them. This isn't currently a great fit for e.g. regmap but perhaps something light on top of regmap-i2c could do the trick? The rest of the set seems good to me. -- Kind regards, Sakari Ailus
