Hi Laurent, On Wed, 5 Apr 2023 05:30:48 +0300 Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > Hi Luca, > > On Tue, Apr 04, 2023 at 04:12:51PM +0200, Luca Ceresoli wrote: > > On Wed, 29 Mar 2023 13:16:22 +0200 Hans Verkuil wrote: > > > > > Hi Luca, > > > > > > I finally found the time to test this series. It looks OK, except for this patch. > > > > Thank you very much for taking the time! > > > > > The list of supported formats really has to be the intersection of what the tegra > > > supports and what the sensor supports. > > > > > > Otherwise you would advertise pixelformats that cannot be used, and the application > > > would have no way of knowing that. > > > > As far as I understand, I think we should rather make this driver fully > > behave as an MC-centric device. It is already using MC quite > > successfully after all. > > > > Do you think this is correct? > > Given the use cases for this driver, I agree. Ok, thanks for the feedback. I will send a v5 with this change. Best regards, Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com