Hi Laurent, On Fri, Jan 28, 2022 at 04:58:40PM +0200, Laurent Pinchart wrote: > Hi Sakari, > > Thank you for the patch. And thanks for the review! > > On Fri, Jan 28, 2022 at 10:33:09AM +0200, Sakari Ailus wrote: > > The bus_info field is set by most drivers based on the type of the device > > bus as well as the name of the device. Do this in v4l_querycap() so > > drivers don't need to. This keeps compatibility with non-default and silly > > bus_info. > > Do we have many PCI or platform drivers with non-default bus_info ? Quite a few actually. Most have something static in bus_info and there are also PCI Express devices that use "PCIe:" prefix. These are mostly old drivers though. I've left them as-is. New drivers shouldn't bother changing the default unless there's a good reason to do that. > > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> Thanks! I'll address the other comments and send v3. -- Kind regards, Sakari Ailus
