* Re: [RFC PATCH 0/2] libcamera: Power down the lens while the camera is idle
[not found] <20261028102323.GB2587630@killaraus.ideasonboard.com>
@ 2026-10-28 11:31 ` Marco Ziliani
0 siblings, 0 replies; only message in thread
From: Marco Ziliani @ 2026-10-28 11:31 UTC (permalink / raw)
To: Laurent Pinchart
Cc: Jacopo Mondi, libcamera-devel, linux-media, Sakari Ailus,
Hans de Goede, Kieran Bingham, Harvey Yang, Daniel Scally
Hi Laurent,
On Wed, Oct 28, 2026 at 12:23:23PM +0200, Laurent Pinchart wrote:
> I want this discussion to involve the kernel side.
Agreed, adding linux-media and Sakari.
For the kernel side, the problem in short: lens (VCM) drivers such as
dw9714, dw9719 and ak7375 power the lens up on subdev open() and down
on close(). On ACPI platforms ipu-bridge also adds a DL_FLAG_PM_RUNTIME
device link from the VCM to its sensor, which Hans confirmed earlier in
this thread is needed because of shared power rails and I2C gating.
libcamera opens the lens subdev when it enumerates the cameras and
keeps it open, and the WirePlumber camera monitor keeps the
CameraManager alive for the whole session. On IPU3 and IPU6 Surface
devices the rear sensor therefore stays powered for as long as a user
is logged in.
The options I can see:
1. userspace opens the lens subdev only while the camera runs (this
RFC, and Harvey's patch [1]), which works with current kernels;
2. the kernel powers the VCM together with its sensor's stream (Hans'
series [2], or calling .enable_streams() on the VCM as Jacopo
suggested), with VCM drivers no longer powering up on open().
They are not exclusive: 1 fixes current kernels and stays correct once
2 is in place.
I can test either on a Surface Pro 5 (IPU3, ov8865 + dw9719).
[1] https://patchwork.libcamera.org/patch/21649/
[2] https://lore.kernel.org/linux-media/20240901211834.145186-1-hdegoede@redhat.com/
Regards,
Marco
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2026-10-28 11:31 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20261028102323.GB2587630@killaraus.ideasonboard.com>
2026-10-28 11:31 ` [RFC PATCH 0/2] libcamera: Power down the lens while the camera is idle Marco Ziliani
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.