Re: [PATCH v2 4/4] media: uvcvideo: eUSB2 double isochronous bandwidth support

[Sakari Ailus <sakari.ailus@linux.intel.com>]

Hi Laurent,

On Tue, Jul 15, 2025 at 12:10:32PM +0300, Laurent Pinchart wrote:
> Hi Sakari, Tao,
> 
> Thank you for the patch.
> 
> On Fri, Jul 11, 2025 at 11:34:13AM +0300, Sakari Ailus wrote:
> > From: Tao Q Tao <tao.q.tao@intel.com>
> > 
> > Use usb_endpoint_max_isoc_bpi() from the USB framework to find the maximum
> > bytes per interval for the endpoint. Consequently this adds eUSB2
> > isochronous mode and SuperSpeedPlus Isochronous Endpoint Compaion support
> > where larger bpi values are possible.
> > 
> > Co-developed-by: Amardeep Rai <amardeep.rai@intel.com>
> > Signed-off-by: Amardeep Rai <amardeep.rai@intel.com>
> > Signed-off-by: Tao Q Tao <tao.q.tao@intel.com>
> > Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
> > Co-developed-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> > Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> 
> Assuming usb_endpoint_max_isoc_bpi() works correctly :-),
> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Thank you!

> 
> I won't queue this patch for the time being as it depends on the rest of
> the series. Please let me know if I should queue it at a later point, or
> if you would like to merge it through the linux-usb tree. We would need
> to make sure there's no conflict with other scheduled patches for that
> kernel development cycle.

I think we'd prefer to go through the linux-usb tree, it seems like the
simplest way to get these to the same kernel release. The changes aren't
very big either.

Are you aware of UVC patches that might conflict with this one?

-- 
Kind regards,

Sakari Ailus