Re: [PATCH 7/7] media: ov2740: Return -EPROBE_DEFER if no endpoint is found

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

 



Hi Laurent,

On Fri, Sep 15, 2023 at 12:50:27PM +0300, Laurent Pinchart wrote:
> Hi Sakari,
> 
> Thank you for the patch.
> 
> On Fri, Sep 15, 2023 at 10:28:09AM +0300, Sakari Ailus wrote:
> > With ipu bridge, endpoints may only be created when ipu bridge has
> > initialised. This may happen after the sensor driver has first probed.
> 
> That's hard to understand for someone not familiar with the ipu-bridge
> driver. Could you please expand the commit message ?
> 
> Also, is there a way to avoid the ov2740 probing before the required
> initialization is complete ?

As of now, nothing else than, well, not having any endpoints as they are
created by the ipu bridge.

The ACPI device node does have a couple of custom objects but I'd rather
not add checks for those in the ACPI framework itself.

The proper solution (from device driver point of view at least) would be to
squash the ipu bridge to the ACPI framework itself so all this could be
initialised before any drivers get probed. I'm not sure how that would be
received, and in any case, as I wrote in a reply to another patch in this
series, I'd like to have DisCo for Imaging support merged first.

-- 
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