Re: [PATCH v2 1/2] media: ipu-bridge: add IVSC support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andy,

On Fri, Jun 30, 2023 at 08:45:25PM +0300, Andy Shevchenko wrote:
> On Fri, Jun 30, 2023 at 04:45:05PM +0300, Sakari Ailus wrote:
> > From: Wentong Wu <wentong.wu@xxxxxxxxx>
> > 
> > Previously on ACPI platforms, sensors that are intended to be connected
> > to a IPU device for use with the ipu3-cio2 driver lacking the necessary
> > connection information in firmware. IPU bridge driver is to connect
> > sensors to IPU device via software nodes.
> > 
> > Currently IVSC located between IPU device and sensors is available in
> > existing commercial platforms from multiple OEMs. But the connection
> > information between them in firmware is also not enough to build V4L2
> > connection graph. This patch parses the connection properties from the
> > SSDB buffer in DSDT and build the connection using software nodes.
> > 
> > IVSC driver is based on MEI framework (previously known as HECI), it
> > has two MEI clients, MEI CSI and MEI ACE. Both clients are used to
> > communicate messages with IVSC firmware. Linux abstracts MEI client
> > as a device, whose bus type is MEI. And the device is addressed by a
> > GUID/UUID which is part of the device name of MEI client. After figured
> > out MEI CSI via the UUID composed device name, this patch setup the
> > connection between MEI CSI and IPU, and the connection between MEI CSI
> > and sensor via software nodes.
> > 
> > Signed-off-by: Wentong Wu <wentong.wu@xxxxxxxxx>
> > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > [Sakari Ailus: rebase on Hans de Goede's cleanups and fixes]
> 
> I believe we use past tense in these comment lines...

No objections, I'll add the missing "d" while applying if there are no
other comments.

> 
> > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> 

-- 
Regards,

Sakari Ailus



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux