From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Peter Seiderer <ps.report@gmx.net>
Cc: Marcus Folkesson <marcus.folkesson@gmail.com>,
Kieran Bingham <kieran.bingham@ideasonboard.com>,
buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v1 1/4] package/libcamera: pipeline option 'raspberrypi' changed to 'rpi/vc4'
Date: Wed, 27 Dec 2023 23:20:30 +0100 [thread overview]
Message-ID: <ZYyjLr25mkMFJqU8@landeda> (raw)
In-Reply-To: <ZYyiJOUePibRgeQq@landeda>
Peter, All,
On 2023-12-27 23:16 +0100, Yann E. MORIN spake thusly:
> On 2023-11-09 22:28 +0100, Peter Seiderer spake thusly:
> > With upstram commit [1] (since version v0.1.0) the pipeline option 'raspberrypi'
> > was renamed to 'rpi/vc4'.
> >
> > Change the buildroot option name from BR2_PACKAGE_LIBCAMERA_PIPELINE_RASPBERRYPI
> > to BR2_PACKAGE_LIBCAMERA_PIPELINE_RPI_VC4 (and add Config.in.legacy entry
> > accordingly) and move handling in Config.in/libcamer.mk to follow alphabetic
> > ordering.
> >
> > Fixes:
> >
> > .../build/libcamera-v0.1.0/meson.build:3:0: ERROR: Options "raspberrypi" are not in allowed choices: "all, auto, imx8-isi, ipu3, rkisp1, rpi/vc4, simple, uvcvideo, vimc"
> >
> > [1] https://git.libcamera.org/libcamera/libcamera.git/commit/?id=726e9274ea95fa46352556d340c5793a8da51fcd
> >
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> Applied to master, thanks.
[--SNIP--]
> > diff --git a/Config.in.legacy b/Config.in.legacy
> > index 6fef2318b0..06a40f0b38 100644
> > --- a/Config.in.legacy
> > +++ b/Config.in.legacy
> > @@ -146,6 +146,16 @@ endif
> >
> > comment "Legacy options removed in 2023.11"
> >
> > +config BR2_PACKAGE_LIBCAMERA_PIPELINE_RASPBERRYPI
> > + bool "libcamera pipeline 'raspberrypi' was renamed to 'rpi/vc4'"
> > + depends on BR2_arm || BR2_aarch64
> > + depends on BR2_USE_WCHAR
> > + select BR2_LEGACY
> > + select BR2_PACKAGE_LIBCAMERA_PIPELINE_RPI_VC4
> > + help
> > + Since version ... the libcamera pipeline option 'raspberrypi' was
$ ./utils/docker-run make check-pacakge
Config.in.legacy:156: help text: <tab><2 spaces><62 chars> (http://nightly.buildroot.org/#writing-rules-config-in)
I had it fixed locally, but forgot to commit before pushing, so I pushed
a follow-up fix.
I also replaced the '...' with the actual version you talked about ion
the commit log: 0.1.0.
Regards,
Yann E. MORIN.
> > + renamed to 'rpi/vc4'.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2023-12-27 22:20 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-09 21:28 [Buildroot] [PATCH v1 1/4] package/libcamera: pipeline option 'raspberrypi' changed to 'rpi/vc4' Peter Seiderer
2023-11-09 21:28 ` [Buildroot] [PATCH v1 2/4] package/libcamera: disable python bindings by default Peter Seiderer
2023-12-27 22:25 ` Yann E. MORIN
2024-01-10 15:39 ` Peter Korsgaard
2023-11-09 21:29 ` [Buildroot] [PATCH v1 3/4] package/libcamera: use udev configure option Peter Seiderer
2023-12-27 22:28 ` Yann E. MORIN
2024-01-10 15:40 ` Peter Korsgaard
2023-11-09 21:29 ` [Buildroot] [PATCH v1 4/4] package/libcamera: add i.MX8 ISI pipeline option Peter Seiderer
2023-12-27 22:31 ` Yann E. MORIN
2024-01-10 15:41 ` Peter Korsgaard
2023-12-27 22:16 ` [Buildroot] [PATCH v1 1/4] package/libcamera: pipeline option 'raspberrypi' changed to 'rpi/vc4' Yann E. MORIN
2023-12-27 22:20 ` Yann E. MORIN [this message]
2024-01-10 15:37 ` Peter Korsgaard
2024-01-10 15:37 ` Peter Korsgaard
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=ZYyjLr25mkMFJqU8@landeda \
--to=yann.morin.1998@free.fr \
--cc=buildroot@buildroot.org \
--cc=kieran.bingham@ideasonboard.com \
--cc=marcus.folkesson@gmail.com \
--cc=ps.report@gmx.net \
/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