On Thu, Sep 17, 2020 at 4:31 PM Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx> wrote: > On 17/09/2020 10:47, Dan Scally wrote: > > On 17/09/2020 08:53, Greg KH wrote: > >> On Wed, Sep 16, 2020 at 10:36:18PM +0100, Daniel Scally wrote: > >>> drivers/staging/media/ipu3/Kconfig | 15 + > >>> drivers/staging/media/ipu3/Makefile | 1 + > >>> drivers/staging/media/ipu3/cio2-bridge.c | 448 +++++++++++++++++++++++ > >> Why does this have to be in drivers/staging/ at all? Why not spend the > >> time to fix it up properly and get it merged correctly? It's a very > >> small driver, and should be smaller, so it should not be a lot of work > >> to do. And it would be faster to do that than to take it through > >> staging... > > I was just under the impression that that was the process to be honest, > > if that's not right I'll just move it directly to drivers/media/ipu3 > > The IPU3 driver is still in staging (unless I've missed something), so I > think this cio2-bridge should stay with it. You missed something. https://elixir.bootlin.com/linux/v5.9-rc5/source/drivers/media/pci/intel IPU3 from Freescale (IIRC) is a different story. > Hopefully with more users of the IPU3 brought in by this cio2-bridge, > that will help gather momentum to get the IPU3 developments required > completed and moved into drivers/media. -- With Best Regards, Andy Shevchenko
