jmassot_: uajain: Thanks for the pings. I'll try to check these later today. jmassot: Done. I later noticed you had cc'd me but I had missed the driver nonetheless. :-( How does this work power management-wise btw.? There's a single GPIO for powering everything on and off, in the demultiplexer driver? uajain: A few small comments only. thanks for the review, I'll take a look sailus: sent new version, simplified the ctrl error handling as you asked. I found simialr reference in ov9282.c driver uajain: It seems to be bug-compliant as well. ;) yes indeed. sailus: v4l2_fwnode_device_parse() is actually checked quite early in init_controls in many drivers. Should I go that route ? I think it'd be easier to handle errors properly that way, as it requires no further error handling. Ok, I'll align with that and send out a new version later today Sounds good.