All of 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.