Re: [PATCH] ACPI: scan: Honor _DEP for Intel CVS devices

[Rafael J. Wysocki]

On Wed, Jun 3, 2026 at 9:43 AM Sakari Ailus
<sakari.ailus@xxxxxxxxxxxxxxx> wrote:
>
> Hi Miguel, Rafael,
>
> Thanks for the patch.
>
> On Mon, Jun 01, 2026 at 12:40:40PM -0700, Miguel Vadillo wrote:
> > CVS (Computer Vision Sensing) is an ACPI-enumerated device that sits
> > inline in the CSI-2 path between the camera sensor and Intel IPU.
> > On platforms where CVS is present, the camera sensor's ACPI node
> > declares a _DEP dependency on the CVS device.
> >
> > The CVS driver must be fully initialized before camera sensor drivers
> > probe, because CVS controls the CSI-2 link ownership handshake (via
> > GPIO REQ/RESP), the MIPI/CSI-2 lane configuration, and the camera
> > power domain. Without CVS ready, the sensor driver can bind but the
> > CSI-2 stream will not function correctly.
> >
> > The CVS driver calls acpi_dev_clear_dependencies() at the end of its
> > probe() to unblock waiting consumers once it is ready.
> >
> > Move the CVS HIDs from acpi_ignore_dep_ids[] to acpi_honor_dep_ids[]
> > so that camera sensor enumeration is deferred until the CVS driver has
> > finished probing, matching the behavior already in place for IVSC.
> >
> > Signed-off-by: Miguel Vadillo <miguel.vadillo@xxxxxxxxx>
>
> Reviewed-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
>
> Rafael: do you think this could still make it to 7.2? Otherwise cameras
> might not work on a some laptops where they did use to.
>
> The default configuration is appropriate in some laptops so camera used to
> work without the CVS driver but now that we have the CVS driver merged and
> the CVS device is part of the software node / ACPI graph, probing the
> camera sensor driver will in fact not take place at all (and obviously the
> camera won't work in this case).

Applied as 7.2 material, thanks!