Re: [PATCH 00/15] Intel IPU6 and IPU6 input system drivers

[Claus Stovgaard <claus.stovgaard@gmail.com>]

Bingbu

On Mon, 2023-08-21 at 14:22 +0800, Bingbu Cao wrote:
> 
> Claus,
> 
> 
> On 8/21/23 11:14 AM, Bingbu Cao wrote:
> > Hi, Claus,
> > 
> > Thanks for your mail.
> > 
> > On 8/20/23 11:09 PM, Claus Stovgaard wrote:
> > > On Thu, 2023-07-27 at 15:15 +0800, bingbu.cao@intel.com wrote:
> > > > From: Bingbu Cao <bingbu.cao@intel.com>
> > > > 
> > > > This patch series adds a driver for Intel IPU6 input system.
> > > > IPU6 is the sixth generation of Imaging Processing Unit, it is
> > > > a PCI
> > > > device which can be found in some Intel Client Platforms. User
> > > > can
> > > > use
> > > > IPU6 to capture images from MIPI camera sensors.
> > > > 
> > > > 
> > > 
> > > Hello Bingbu.
> > > 
> > > ...	
> > > 
> > > Will continue investigating, but I would like any input in
> > > getting the
> > > driver up an running and testing on this Dell laptop. I think it
> > > should
> > > be very close to working.
> > 
> > Do you any failure log for ov01a10?
> > 
> > For Dell XPS 9320, the camera sensor module has a dependency on
> > Intel
> > IVSC driver, so please make sure you have the latest ivsc driver.
> > I remember they are already in media tree.
> > 
> > I will check again with latest IVSC driver, feel free to mail me or
> > Wentong Wu meanwhile if you have any problems for camera sensor and
> > IVSC.
> 
> I see that the ivsc driver has not been in master branch. Before
> that,
> could you try several hack to check whether camera can work on
> master?
> 
> https://github.com/bingbucao/linux/commits/ipu_dev
> 
> 7ebff51284d9 media: ov01a10: hack ivsc to make camera can work
> 01cc9f3d1b61 i2c: ljca: Call acpi_dev_clear_dependencies()
> 92e5d122e105 vsc: Defer firmware loading to avoid long probing time
> 5f5d5f0df06b driver: ivsc: add intel ivsc driver
> 0f4819dec533 Revert "gpio: Add support for Intel LJCA USB GPIO
> driver"

Thanks for your quick reply.

I was missing understanding of ivsc when I wrote the mail yesterday.
Got some basic understanding yesterday after I wrote, and big thanks
for confirming it, and also thanks for your ipu_dev branch. Has just
cloned it, and is building as I write.

Just fyi, I was trying to hack something together yesterday, and got
further, but not yet working.

My hack was to combine the out-of-tree ivsc drivers and firmware from

* https://github.com/intel/ivsc-firmware.git
* https://github.com/intel/ivsc-driver.git

Though noticed that I need some changes to the sensor driver so was
also building all the drivers from ipu6-drivers (with minor changes to
get_pages) as out-of-tree modules.

* https://github.com/intel/ipu6-drivers.git 

Here I used everything beside media/pci/*.ko files. I could see the
sensor and got further, but was missing the last.

Looking forward to try your branch. Looks much cleaner, and would be
nice to get working :)

/Claus
> >