On Fri, Oct 09, 2020 at 07:22:38PM +0300, Andy Shevchenko wrote: > On Fri, Oct 9, 2020 at 6:10 PM Sakari Ailus > <sakari.ailus@xxxxxxxxxxxxxxx> wrote: > > > > Validate media bus code, width and height when setting the subdev format. > > > > This effectively reworks how setting subdev format is implemented in the > > driver. > > Does it make it dependent on patch 3/5? Good question. They're in the same set but these probably should be backported. If fact, I think I should add Cc: stable to most of these patches. > Or maybe you can use min_t() and update to min() in a separate patch? I would have said that's overkill but considering the stable tree it's not. I'll do that for v3. Thanks for the review! -- Sakari Ailus