Hi Laurent, On Thu, Sep 28, 2023 at 01:23:05AM +0300, Laurent Pinchart wrote: > > I'll use this in v2: > > > > * - Driver name > > - File(s) > > - Driver type > > - Example topic > > * - CCS > > - ``drivers/media/i2c/ccs/`` > > - Freely configurable > > - Power management (ACPI and DT), UAPI > > * - imx219 > > - ``drivers/media/i2c/imx219.c`` > > - Register list based > > - Power management (DT), UAPI, mode selection > > * - imx319 > > - ``drivers/media/i2c/imx319.c`` > > - Register list based > > - Power management (ACPI and DT) > > * - ov8865 > > - ``drivers/media/i2c/ov8865.c`` > > - Register list based > > - Power management (ACPI and DT) > > It's not nice for developers to have to look at different drivers and > takes bits and pieces from different places, especially if it's not > clear which bits they should take on one side and which pieces on the > other side. Ideally we should be able to collapse all thre register > list based drivers into a single entry, but I understand we can't do > that yet. Could we however list only one of the imx319 and ov8865 ? I'm fine dropping one of them. The ov8865 is rather ugly otherwise so I'll drop that. -- Sakari Ailus