Hi Sakari, Thank you for the patch. On Fri, Aug 04, 2023 at 01:52:26PM +0300, Sakari Ailus wrote: > Improve the Kconfig help text to convey the ACE is for changing camera > sensor ownership and CSI for configuring the CSI-2 rx and tx. > > Suggested-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > drivers/media/pci/intel/ivsc/Kconfig | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/pci/intel/ivsc/Kconfig b/drivers/media/pci/intel/ivsc/Kconfig > index 9535ac10f4f7..e9d94dbb4305 100644 > --- a/drivers/media/pci/intel/ivsc/Kconfig > +++ b/drivers/media/pci/intel/ivsc/Kconfig > @@ -8,5 +8,9 @@ config INTEL_VSC > This adds support for Intel Visual Sensing Controller (IVSC). > > Enables the IVSC firmware services required for controlling > - camera sensor ownership and CSI-2 link through Image Processing > - Unit(IPU) driver of Intel. > + camera sensor ownership between IVSC and host CPU (ACE) as well What does ACE stand for ? > + as configuring the CSI-2 link (CSI) towards the Image Processing > + Unit (IPU) driver of Intel. If I understand things correctly, this is about routing the data from the sensor to the IVSC output towards the host. Is that right ? If so, let's make that explicit. > + > + The modules will be called ivsc-csi and ivsc-ace. > + -- Regards, Laurent Pinchart
