diff mbox

[v2,1/1] avrdude: new package

Message ID 1387924680-16737-1-git-send-email-wzab01@gmail.com
State Accepted
Headers show

Commit Message

Wojciech M. Zabolotny Dec. 24, 2013, 10:38 p.m. UTC 
Signed-off-by: Wojciech M. Zabolotny <wzab01@gmail.com>
---
 package/Config.in          |  1 +
 package/avrdude/Config.in  | 16 ++++++++++++++++
 package/avrdude/avrdude.mk | 18 ++++++++++++++++++
 3 files changed, 35 insertions(+)
 create mode 100644 package/avrdude/Config.in
 create mode 100644 package/avrdude/avrdude.mk

Comments

Thomas Petazzoni Dec. 28, 2013, 4:09 p.m. UTC | #1 
Dear Wojciech M. Zabolotny,

On Tue, 24 Dec 2013 23:38:00 +0100, Wojciech M. Zabolotny wrote:
> Signed-off-by: Wojciech M. Zabolotny <wzab01@gmail.com>
> ---
>  package/Config.in          |  1 +
>  package/avrdude/Config.in  | 16 ++++++++++++++++
>  package/avrdude/avrdude.mk | 18 ++++++++++++++++++
>  3 files changed, 35 insertions(+)
>  create mode 100644 package/avrdude/Config.in
>  create mode 100644 package/avrdude/avrdude.mk

Thanks, applied. FYI, I've made the following small modifications:

     - remove empty newline at the end of Config.in
     - use lowercase for the package name in the comment header
     - reword the comment explaining why autoreconf is needed
     - add optional dependency on libftdi, which avrdude can use
     - remove avrdude.conf.bak file from /etc

Thanks!

Thomas
diff mbox

Patch

diff --git a/package/Config.in b/package/Config.in
index 3685807..2abffb8 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -258,6 +258,7 @@  source "package/zd1211-firmware/Config.in"
 endmenu
 source "package/a10disp/Config.in"
 source "package/acpid/Config.in"
+source "package/avrdude/Config.in"
 source "package/cdrkit/Config.in"
 source "package/cryptsetup/Config.in"
 source "package/dbus/Config.in"
diff --git a/package/avrdude/Config.in b/package/avrdude/Config.in
new file mode 100644
index 0000000..f3b075a
--- /dev/null
+++ b/package/avrdude/Config.in
@@ -0,0 +1,16 @@ 
+config BR2_PACKAGE_AVRDUDE
+	bool "avrdude"
+	select BR2_PACKAGE_LIBELF
+	select BR2_PACKAGE_LIBUSB
+	select BR2_PACKAGE_LIBUSB_COMPAT
+	select BR2_PACKAGE_NCURSES
+	depends on BR2_TOOLCHAIN_HAS_THREADS
+	help
+	  avrdude is a programmer for Atmel AVR microcontrollers
+	  this package provides a version with new linuxspi driver
+
+	  https://github.com/kcuzner/avrdude
+
+comment "avrdude needs a toolchain w/ threads"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
+
diff --git a/package/avrdude/avrdude.mk b/package/avrdude/avrdude.mk
new file mode 100644
index 0000000..b04e2e0
--- /dev/null
+++ b/package/avrdude/avrdude.mk
@@ -0,0 +1,18 @@ 
+################################################################################
+#
+# AVRDUDE
+#
+################################################################################
+
+AVRDUDE_VERSION = eabe067c4527bc2eedc5db9288ef5cf1818ec720
+AVRDUDE_SITE = $(call github,kcuzner,avrdude,$(AVRDUDE_VERSION))
+AVRDUDE_LICENSE = GPLv2+
+AVRDUDR_LICENSE_FILES = avrdude/COPYING
+AVRDUDE_SUBDIR = avrdude
+# Sources are distributed with ./bootstrap file
+# it is necessary to perform autoreconfiguration 
+AVRDUDE_AUTORECONF = YES
+AVRDUDE_DEPENDENCIES = libelf libusb libusb-compat ncurses
+
+$(eval $(autotools-package))
+