Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> writes: > I think Krzysztof meant caching the value in the ar0521_dev structure, > so it doesn't have to be read back. I2C is slow, let's avoid reads as > much as possible. Right, ATM there is no I2C read function at all in this driver. > This being said, if all gain controls are in the same cluster, you won't > need to read back or cache anything yourself, the control framework will > handle that for you. Yep. Then there could be a common gain for all 3 colors, or a set of 3 for R/G/B. I only don't know what should the former one return on read, if individual values are used. -- Krzysztof "Chris" Hałasa Sieć Badawcza Łukasiewicz Przemysłowy Instytut Automatyki i Pomiarów PIAP Al. Jerozolimskie 202, 02-486 Warszawa