Re: [PATCH v3] arm64: dts: qcom: x1e80100-dell-xps13-9345: add the front camera

[Aleksandrs Vinarskis]

On Sunday, October 11th, 2026 at 20:56, Liviu Nicoara <lnicoara@xxxxxxxxxxxx> wrote:

> On Sun, 11 Oct 2026 18:28:30 +0000, Aleksandrs Vinarskis wrote:
> > The initial patch for camera from Laurentiu Tudor that Abel was referring to
> > was never submitted upstream, as it depends on CAMSS PHY and DT camera
> > series which are trying to find their way upstream for over a year now.
>
> Thanks for the link; I had not seen it.
>
> > Just an FYI: CAMF_RES_QRD.bin is an AeoB file. Some time ago I made a
> > repo to keep a list of already de-compiled files for other developers to use,
> > might come handy.
> >
> > IMO this comment can be dropped.
>
> Will drop it in v4. The dump agrees with my read of the file.
>
> > Windows may only be switching one rail, however there in fact a 2v8 rail
> > provided by LDO 7B.
> >
> > I would recommend to add it.
>
> Will add vreg_l7b_2p8 and avdd-supply in v4.
>
> One question for dvdd. The sensor's DVDD pin is 1.2 V, so a regulator
> on the module or the board must step it down. On the schematic, which
> rail feeds that regulator: L7B, as Tudor's patch has it, or the 1.8 V
> behind GPIO 91? That decides whether dvdd-supply goes on vreg_l7b_2p8
> or on vreg_cam_1p8.

Unfortunately, I do not know. The step down is not on the motherboard,
but on the camera module itself.

Alex

>
> Thanks,
> Liviu
>