On Sun, 2022-01-09 at 16:34 +0100, Jacopo Mondi wrote: > Hi Krzysztof Hi Jacopo Can you please trim your replies to just the sections that you are commenting on? Finding the reply in the duplicated reply content isn't easy. > > + // access the sensor only if it's powered up > > + if (!pm_runtime_get_if_in_use(&sensor->i2c_client->dev)) > > As you correctly do not access the chip's registers if it's powered > off, you have to call __v4l2_ctrl_handler_setup() at power on time to > make sure controls are actually set. It was 25 pages down in the reply to find this.