From: Aliaksandr Smirnou <asmirnou@pinefeat.co.uk>
To: alen.karnil@ideasonboard.com
Cc: asmirnou@pinefeat.co.uk, conor+dt@kernel.org,
devicetree@vger.kernel.org, hverkuil@xs4all.nl,
jacopo.mondi@ideasonboard.com, krzk+dt@kernel.org,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
mchehab@kernel.org, robh@kernel.org
Subject: Re: [PATCH v6 0/2] Pinefeat cef168 lens control board driver
Date: Wed, 27 May 2026 20:49:03 +0100 [thread overview]
Message-ID: <20260527194903.10296-1-asmirnou@pinefeat.co.uk> (raw)
In-Reply-To: <cb202484-3a53-4821-b1bd-5f910e0cd953@ideasonboard.com>
On Wed, 27 May 2026 12:45:26 +0100, Alen Karnil wrote:
> > Could you provide the build error logs and the overlay files generated
> > by the tool before you modified them?
> using the configure.sh tool with imx477
> imx477_378-overlay.dtsi
> ...
The files generated after running the configure.sh tool look correct and
match mine exactly.
> If put these into my kernel build I get:
>
> $ make dtbs
> DTCO arch/arm64/boot/dts/overlays/camera-mux-2port.dtbo
> arch/arm64/boot/dts/overlays/imx477_378.dtsi:26.20-31.3: ERROR
> (duplicate_label): /fragment@200/__overlay__/pca@70/i2c@1/cef168@d:
> Duplicate label 'vcm_node' on
> /fragment@200/__overlay__/pca@70/i2c@1/cef168@d and
> /fragment@200/__overlay__/pca@70/i2c@0/cef168@d
> ERROR: Input tree has errors, aborting (use -f to force output)
> make[3]: *** [scripts/Makefile.dtbs:142:
> arch/arm64/boot/dts/overlays/camera-mux-2port.dtbo] Error 2
> make[2]: *** [scripts/Makefile.build:544: arch/arm64/boot/dts/overlays]
> Error 2
> make[1]: *** [/home/admin/linux/Makefile:1498: dtbs] Error 2
> make: *** [Makefile:248: __sub-make] Error 2
It looks like a different Makefile is being used. In the cef168
repository's Makefile, there is no dtbs target. Also, our Makefile is
only 27 lines long, while the log output indicates the error occurs at
line 1498.
Could you please ensure you are using the Makefile provided in the cef168
repository?
> >> there needs to be some further discussions on how we could
> >> improve the user experience when it comes to enabling the vcm with
> >> camera sensors.
> >
> > I agree. I started the discussion on the Raspberry Pi forum, but the
> > driver needs to be merged first before moving forward with that.
> Sure that makes sense, lets focus on the driver for now.
> >
> > https://forums.raspberrypi.com/viewtopic.php?p=2318070#p2318070
>
> Also you will most likely need to update the patch since the line
> offsets have moved to work 6.18
Will do. Should I update the patch before or after your review?
next prev parent reply other threads:[~2026-05-27 19:49 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-07 19:57 [PATCH v6 0/2] Pinefeat cef168 lens control board driver Aliaksandr Smirnou
2025-10-07 19:57 ` [PATCH v6 1/2] dt-bindings: Pinefeat cef168 lens control board Aliaksandr Smirnou
2025-10-07 20:47 ` Conor Dooley
2025-10-08 17:50 ` Aliaksandr Smirnou
2025-10-08 18:31 ` Conor Dooley
2025-10-08 21:32 ` Aliaksandr Smirnou
2025-10-08 23:46 ` Krzysztof Kozlowski
2025-10-09 20:17 ` Aliaksandr Smirnou
2025-10-07 19:57 ` [PATCH v6 2/2] media: i2c: Pinefeat cef168 lens control board driver Aliaksandr Smirnou
2026-05-26 15:54 ` [PATCH v6 0/2] " Alen Karnil
2026-05-26 20:35 ` Aliaksandr Smirnou
2026-05-27 11:45 ` Alen Karnil
2026-05-27 19:49 ` Aliaksandr Smirnou [this message]
2026-05-28 8:36 ` Alen Karnil
2026-05-30 11:46 ` Aliaksandr Smirnou
2026-06-01 8:59 ` Alen Karnil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260527194903.10296-1-asmirnou@pinefeat.co.uk \
--to=asmirnou@pinefeat.co.uk \
--cc=alen.karnil@ideasonboard.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=hverkuil@xs4all.nl \
--cc=jacopo.mondi@ideasonboard.com \
--cc=krzk+dt@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox