Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] package/openfpgaloader: bump to version 0.6.1
@ 2021-12-04  5:59 Gwenhael Goavec-Merou
  2021-12-04  5:59 ` [Buildroot] [PATCH 2/2] package/openfpgaloader: add cmsis-dap support option Gwenhael Goavec-Merou
  2021-12-04 20:07 ` [Buildroot] [PATCH 1/2] package/openfpgaloader: bump to version 0.6.1 Thomas Petazzoni
  0 siblings, 2 replies; 4+ messages in thread
From: Gwenhael Goavec-Merou @ 2021-12-04  5:59 UTC (permalink / raw)
  To: buildroot; +Cc: Gwenhael Goavec-Merou, Jean Burgat

From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>

- Update version to latest release
- change license type and hash
- add zlib dependency

Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
---
 package/openfpgaloader/Config.in           | 1 +
 package/openfpgaloader/openfpgaloader.hash | 4 ++--
 package/openfpgaloader/openfpgaloader.mk   | 8 ++++----
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/package/openfpgaloader/Config.in b/package/openfpgaloader/Config.in
index f1b34ef5dd..c998369bed 100644
--- a/package/openfpgaloader/Config.in
+++ b/package/openfpgaloader/Config.in
@@ -4,6 +4,7 @@ config BR2_PACKAGE_OPENFPGALOADER
 	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # libftdi1 -> libusb
 	depends on BR2_INSTALL_LIBSTDCPP
 	select BR2_PACKAGE_LIBFTDI1
+	select BR2_PACKAGE_ZLIB
 	help
 	  Universal utility for programming FPGA
 
diff --git a/package/openfpgaloader/openfpgaloader.hash b/package/openfpgaloader/openfpgaloader.hash
index 990149cded..6c9d8a2b91 100644
--- a/package/openfpgaloader/openfpgaloader.hash
+++ b/package/openfpgaloader/openfpgaloader.hash
@@ -1,3 +1,3 @@
 # Locally computed
-sha256  ef3ed60297ab560f15cd4362d61b7da86c44b84e18937e0411023f8d1cf6a606  openfpgaloader-381c67de00a3102cf6e9bb20ca84030a71c7a0f2.tar.gz
-sha256  8486a10c4393cee1c25392769ddd3b2d6c242d6ec7928e1414efff7dfb2f07ef  LICENSE
+sha256  a862a209d696becff915a77512e6a8c22f92d73480a45cc12273d9ad1db60d23  openfpgaloader-0.6.1.tar.gz
+sha256  c71d239df91726fc519c6eb72d318ec65820627232b2f796219e87dcf35d0ab4  LICENSE
diff --git a/package/openfpgaloader/openfpgaloader.mk b/package/openfpgaloader/openfpgaloader.mk
index 1bb3de37fb..73117cb994 100644
--- a/package/openfpgaloader/openfpgaloader.mk
+++ b/package/openfpgaloader/openfpgaloader.mk
@@ -4,11 +4,11 @@
 #
 ################################################################################
 
-OPENFPGALOADER_VERSION = 381c67de00a3102cf6e9bb20ca84030a71c7a0f2
-OPENFPGALOADER_SITE = $(call github,trabucayre,openFPGALoader,$(OPENFPGALOADER_VERSION))
-OPENFPGALOADER_LICENSE = AGPL-3.0
+OPENFPGALOADER_VERSION = 0.6.1
+OPENFPGALOADER_SITE = $(call github,trabucayre,openFPGALoader,v$(OPENFPGALOADER_VERSION))
+OPENFPGALOADER_LICENSE = Apache-2.0
 OPENFPGALOADER_LICENSE_FILES = LICENSE
-OPENFPGALOADER_DEPENDENCIES = libftdi1
+OPENFPGALOADER_DEPENDENCIES = libftdi1 zlib
 
 ifeq ($(BR2_PACKAGE_HAS_UDEV),y)
 OPENFPGALOADER_DEPENDENCIES += udev
-- 
2.30.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH 2/2] package/openfpgaloader: add cmsis-dap support option
  2021-12-04  5:59 [Buildroot] [PATCH 1/2] package/openfpgaloader: bump to version 0.6.1 Gwenhael Goavec-Merou
@ 2021-12-04  5:59 ` Gwenhael Goavec-Merou
  2021-12-04 20:07   ` Thomas Petazzoni
  2021-12-04 20:07 ` [Buildroot] [PATCH 1/2] package/openfpgaloader: bump to version 0.6.1 Thomas Petazzoni
  1 sibling, 1 reply; 4+ messages in thread
From: Gwenhael Goavec-Merou @ 2021-12-04  5:59 UTC (permalink / raw)
  To: buildroot; +Cc: Gwenhael Goavec-Merou, Jean Burgat

From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>

CMSIS-DAP support depends on hidapi, but this package has depends on
udev so instead of adding this option by default, this let user decide.

Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
---
 package/openfpgaloader/Config.in         | 13 +++++++++++++
 package/openfpgaloader/openfpgaloader.mk |  7 +++++++
 2 files changed, 20 insertions(+)

diff --git a/package/openfpgaloader/Config.in b/package/openfpgaloader/Config.in
index c998369bed..c3712a8ea0 100644
--- a/package/openfpgaloader/Config.in
+++ b/package/openfpgaloader/Config.in
@@ -10,6 +10,19 @@ config BR2_PACKAGE_OPENFPGALOADER
 
 	  https://github.com/trabucayre/openFPGALoader/
 
+if BR2_PACKAGE_OPENFPGALOADER
+
+config BR2_PACAKGE_OPENFPGALOADER_CMSIS
+	bool "CMSIS-DAP support"
+	depends on BR2_PACKAGE_HIDAPI
+	help
+	  openfpgaloader CMSIS-DAP support
+
+comment "openfpgaloader CMSIS-DAP needs hidapi"
+	depends on !BR2_PACKAGE_HIDAPI
+
+endif
+
 comment "openfpgaloader needs a toolchain w/ threads, C++, gcc >= 4.9"
 	depends on !BR2_TOOLCHAIN_HAS_THREADS || \
 		!BR2_INSTALL_LIBSTDCPP || \
diff --git a/package/openfpgaloader/openfpgaloader.mk b/package/openfpgaloader/openfpgaloader.mk
index 73117cb994..0c39aad687 100644
--- a/package/openfpgaloader/openfpgaloader.mk
+++ b/package/openfpgaloader/openfpgaloader.mk
@@ -17,4 +17,11 @@ else
 OPENFPGALOADER_CONF_OPTS += -DENABLE_UDEV=OFF
 endif
 
+ifeq ($(BR2_PACAKGE_OPENFPGALOADER_CMSIS),y)
+OPENFPGALOADER_DEPENDENCIES += hidapi
+OPENFPGALOADER_CONF_OPTS += -DENABLE_CMSISDAP=ON
+else
+OPENFPGALOADER_CONF_OPTS += -DENABLE_CMSISDAP=OFF
+endif
+
 $(eval $(cmake-package))
-- 
2.30.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Buildroot] [PATCH 1/2] package/openfpgaloader: bump to version 0.6.1
  2021-12-04  5:59 [Buildroot] [PATCH 1/2] package/openfpgaloader: bump to version 0.6.1 Gwenhael Goavec-Merou
  2021-12-04  5:59 ` [Buildroot] [PATCH 2/2] package/openfpgaloader: add cmsis-dap support option Gwenhael Goavec-Merou
@ 2021-12-04 20:07 ` Thomas Petazzoni
  1 sibling, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2021-12-04 20:07 UTC (permalink / raw)
  To: Gwenhael Goavec-Merou; +Cc: Jean Burgat, Gwenhael Goavec-Merou, buildroot

On Sat,  4 Dec 2021 06:59:43 +0100
Gwenhael Goavec-Merou <gwenj@trabucayre.com> wrote:

> From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
> 
> - Update version to latest release
> - change license type and hash
> - add zlib dependency
> 
> Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
> ---
>  package/openfpgaloader/Config.in           | 1 +
>  package/openfpgaloader/openfpgaloader.hash | 4 ++--
>  package/openfpgaloader/openfpgaloader.mk   | 8 ++++----
>  3 files changed, 7 insertions(+), 6 deletions(-)

Applied to next, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Buildroot] [PATCH 2/2] package/openfpgaloader: add cmsis-dap support option
  2021-12-04  5:59 ` [Buildroot] [PATCH 2/2] package/openfpgaloader: add cmsis-dap support option Gwenhael Goavec-Merou
@ 2021-12-04 20:07   ` Thomas Petazzoni
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2021-12-04 20:07 UTC (permalink / raw)
  To: Gwenhael Goavec-Merou; +Cc: Jean Burgat, Gwenhael Goavec-Merou, buildroot

On Sat,  4 Dec 2021 06:59:44 +0100
Gwenhael Goavec-Merou <gwenj@trabucayre.com> wrote:

> From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
> 
> CMSIS-DAP support depends on hidapi, but this package has depends on
> udev so instead of adding this option by default, this let user decide.

I think that despite this we really want a "select BR2_PACKAGE_HIDAPI",
as this is what we normally do for such dependencies. Of course, it
needs propagating the "depends on" of hidapi to this new openfpgaloader
option.

Thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-12-04 20:07 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-12-04  5:59 [Buildroot] [PATCH 1/2] package/openfpgaloader: bump to version 0.6.1 Gwenhael Goavec-Merou
2021-12-04  5:59 ` [Buildroot] [PATCH 2/2] package/openfpgaloader: add cmsis-dap support option Gwenhael Goavec-Merou
2021-12-04 20:07   ` Thomas Petazzoni
2021-12-04 20:07 ` [Buildroot] [PATCH 1/2] package/openfpgaloader: bump to version 0.6.1 Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox