Hi Hans, On Fri, Feb 24, 2023 at 10:14:00AM +0200, Sakari Ailus wrote: > Hi Hans, > > On Thu, Feb 23, 2023 at 11:04:12PM +0100, Hans de Goede wrote: > > Hi All, > > > > While trying to test Kate's tps68470 patches for the privacy LED > > on the back of the Surface Go 1/2 I hit this oops when trying > > to run qcam: > > Thanks for reporting this. > > There have been recent changes in how link validation works, also cc Tomi. It seems link validation is broken in the ImgU driver --- it tries to use the default link validator that works between two sub-devices but not between a video node and a sub-device. I guess this somehow got ignored before Tomi's patches. It looks like it's calling v4l2_subdev_get_unlocked_active_state() with NULL argument. No idea why the 32nd bit is set though. I'll see how to address this. -- Kind regards, Sakari Ailus