diff mbox series

[v1] package/libcamera: fix BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS handling

Message ID 20201117212519.7559-1-ps.report@gmx.net
State Accepted
Headers show
Series [v1] package/libcamera: fix BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS handling | expand

Commit Message

Peter Seiderer Nov. 17, 2020, 9:25 p.m. UTC
Fix BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS handling, change from
'depends on BR2_m68k' to 'depends on !BR2_m68k'.

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
 package/libcamera/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Kieran Bingham Nov. 17, 2020, 10:36 p.m. UTC | #1
Hi Peter,

On 17/11/2020 21:25, Peter Seiderer wrote:
> Fix BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS handling, change from
> 'depends on BR2_m68k' to 'depends on !BR2_m68k'.
> 
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
>  package/libcamera/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/libcamera/Config.in b/package/libcamera/Config.in
> index aadea56d19..b9c243084e 100644
> --- a/package/libcamera/Config.in
> +++ b/package/libcamera/Config.in
> @@ -2,7 +2,7 @@ config BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS
>  	bool
>  	default y
>  	# Invalid packing size of ControlValue struct on m68k
> -	depends on BR2_m68k
> +	depends on !BR2_m68k

Oh dear, this was definitely negated in the patch I submitted [0], but
looks like it landed as this in [1], which was incorrect.

[0]
https://lists.libcamera.org/pipermail/libcamera-devel/2020-September/012519.html

[1]
https://git.buildroot.net/buildroot/commit/?id=66526e3518e01a3fe16dede165c7890d194b7816

Fix looks good to me.

Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>


>  
>  menuconfig BR2_PACKAGE_LIBCAMERA
>  	bool "libcamera"
>
Yann E. MORIN Nov. 19, 2020, 6:40 p.m. UTC | #2
Peter, All,

On 2020-11-17 22:25 +0100, Peter Seiderer spake thusly:
> Fix BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS handling, change from
> 'depends on BR2_m68k' to 'depends on !BR2_m68k'.

Uh-oh... My bad...

> Signed-off-by: Peter Seiderer <ps.report@gmx.net>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
>  package/libcamera/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/libcamera/Config.in b/package/libcamera/Config.in
> index aadea56d19..b9c243084e 100644
> --- a/package/libcamera/Config.in
> +++ b/package/libcamera/Config.in
> @@ -2,7 +2,7 @@ config BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS
>  	bool
>  	default y
>  	# Invalid packing size of ControlValue struct on m68k
> -	depends on BR2_m68k
> +	depends on !BR2_m68k
>  
>  menuconfig BR2_PACKAGE_LIBCAMERA
>  	bool "libcamera"
> -- 
> 2.29.2
>
Peter Korsgaard Dec. 6, 2020, 10:25 p.m. UTC | #3
>>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:

 > Fix BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS handling, change from
 > 'depends on BR2_m68k' to 'depends on !BR2_m68k'.

 > Signed-off-by: Peter Seiderer <ps.report@gmx.net>

Committed to 2020.08.x, thanks.
diff mbox series

Patch

diff --git a/package/libcamera/Config.in b/package/libcamera/Config.in
index aadea56d19..b9c243084e 100644
--- a/package/libcamera/Config.in
+++ b/package/libcamera/Config.in
@@ -2,7 +2,7 @@  config BR2_PACKAGE_LIBCAMERA_ARCH_SUPPORTS
 	bool
 	default y
 	# Invalid packing size of ControlValue struct on m68k
-	depends on BR2_m68k
+	depends on !BR2_m68k
 
 menuconfig BR2_PACKAGE_LIBCAMERA
 	bool "libcamera"