all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH v2 0/2] Add u-boot package and its dependent packages
@ 2016-08-31 21:34 Danny Milosavljevic
  2016-08-31 21:34 ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
  2016-08-31 21:34 ` [PATCH v2 2/2] gnu: Add u-boot Danny Milosavljevic
  0 siblings, 2 replies; 9+ messages in thread
From: Danny Milosavljevic @ 2016-08-31 21:34 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 753 bytes --]

Danny Milosavljevic (2):
  gnu: Add dtc.
  gnu: Add u-boot.

 gnu/local.mk                                       |   4 +
 gnu/packages/patches/dtc-01_build_doc.patch        |  22 +++
 ...t-Add-missing-functions-to-shared-library.patch |  30 +++++
 ...t-Add-some-missing-symbols-to-version.lds.patch |  33 +++++
 gnu/packages/u-boot.scm                            | 147 +++++++++++++++++++++
 5 files changed, 236 insertions(+)
 create mode 100644 gnu/packages/patches/dtc-01_build_doc.patch
 create mode 100644 gnu/packages/patches/dtc-23-libfdt-Add-missing-functions-to-shared-library.patch
 create mode 100644 gnu/packages/patches/dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch
 create mode 100644 gnu/packages/u-boot.scm


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

* [PATCH v2 1/2] gnu: Add dtc.
  2016-08-31 21:34 [PATCH v2 0/2] Add u-boot package and its dependent packages Danny Milosavljevic
@ 2016-08-31 21:34 ` Danny Milosavljevic
  2016-09-01 15:16   ` David Craven
  2016-08-31 21:34 ` [PATCH v2 2/2] gnu: Add u-boot Danny Milosavljevic
  1 sibling, 1 reply; 9+ messages in thread
From: Danny Milosavljevic @ 2016-08-31 21:34 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 821 bytes --]

gnu: Add dtc.

* gnu/packages/u-boot.scm (dtc): New variable.

Co-authored-by: David Craven <david@craven.ch>
---
 gnu/local.mk                                       |  4 ++
 gnu/packages/patches/dtc-01_build_doc.patch        | 22 ++++++++
 ...t-Add-missing-functions-to-shared-library.patch | 30 ++++++++++
 ...t-Add-some-missing-symbols-to-version.lds.patch | 33 +++++++++++
 gnu/packages/u-boot.scm                            | 65 ++++++++++++++++++++++
 5 files changed, 154 insertions(+)
 create mode 100644 gnu/packages/patches/dtc-01_build_doc.patch
 create mode 100644 gnu/packages/patches/dtc-23-libfdt-Add-missing-functions-to-shared-library.patch
 create mode 100644 gnu/packages/patches/dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch
 create mode 100644 gnu/packages/u-boot.scm


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-dtc.patch --]
[-- Type: text/x-patch; name="0001-gnu-Add-dtc.patch", Size: 7511 bytes --]

diff --git a/gnu/local.mk b/gnu/local.mk
index 66ef07b..799b6f0 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -345,6 +345,7 @@ GNU_SYSTEM_MODULES =				\
   %D%/packages/tor.scm				\
   %D%/packages/tre.scm				\
   %D%/packages/tv.scm				\
+  %D%/packages/u-boot.scm			\
   %D%/packages/unrtf.scm			\
   %D%/packages/upnp.scm				\
   %D%/packages/uucp.scm				\
@@ -809,6 +810,9 @@ dist_patch_DATA =						\
   %D%/packages/patches/torsocks-dns-test.patch			\
   %D%/packages/patches/totem-debug-format-fix.patch		\
   %D%/packages/patches/tuxpaint-stamps-path.patch		\
+  %D%/packages/patches/u-boot-dtc-01_build_doc.patch		\
+  %D%/packages/patches/u-boot-dtc-23-libfdt-Add-missing-functions-to-shared-library.patch	\
+  %D%/packages/patches/u-boot-dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch	\
   %D%/packages/patches/unzip-CVE-2014-8139.patch		\
   %D%/packages/patches/unzip-CVE-2014-8140.patch		\
   %D%/packages/patches/unzip-CVE-2014-8141.patch		\
diff --git a/gnu/packages/patches/dtc-01_build_doc.patch b/gnu/packages/patches/dtc-01_build_doc.patch
new file mode 100644
index 0000000..0ecafa3
--- /dev/null
+++ b/gnu/packages/patches/dtc-01_build_doc.patch
@@ -0,0 +1,22 @@
+Note: the patch is from <https://launchpad.net/ubuntu/+archive/primary/+files/device-tree-compiler_1.4.0+dfsg-2ubuntu1.diff.gz>
+Index: device-tree-compiler-1.0.0/Documentation/Makefile
+===================================================================
+--- /dev/null	1970-01-01 00:00:00.000000000 +0000
++++ device-tree-compiler-1.0.0/Documentation/Makefile	2007-08-23 18:26:22.000000000 +0200
+@@ -0,0 +1,16 @@
++all: dtc-paper.ps dtc-paper.pdf dtc-manual.txt
++
++dtc-manual.txt: manual.txt
++	cp -f $< $@
++
++dtc-paper.dvi: dtc-paper.tex
++	latex $<
++
++dtc-paper.ps: dtc-paper.dvi
++	dvips $<
++
++dtc-paper.pdf: dtc-paper.tex
++	pdflatex $<
++
++clean:
++	rm -f *.aux *.log *.dvi *.ps *.pdf dtc-manual.txt
diff --git a/gnu/packages/patches/dtc-23-libfdt-Add-missing-functions-to-shared-library.patch b/gnu/packages/patches/dtc-23-libfdt-Add-missing-functions-to-shared-library.patch
new file mode 100644
index 0000000..3bfde67
--- /dev/null
+++ b/gnu/packages/patches/dtc-23-libfdt-Add-missing-functions-to-shared-library.patch
@@ -0,0 +1,30 @@
+From a4b093f7366fdb429ca1781144d3985fa50d0fbb Mon Sep 17 00:00:00 2001
+From: Julien Grall <julien.grall@linaro.org>
+Date: Tue, 17 Mar 2015 16:00:34 +0000
+Subject: [PATCH] libfdt: Add missing functions to shared library
+
+The commit 4e76ec7 "libfdt: Add fdt_next_subnode() to permit easy
+subnode iteration" adds new functions (fdt_{first,next}_subnode) but
+forgot to mark them as 'global' in the shared library.
+
+Signed-off-by: Julien Grall <julien.grall@linaro.org>
+---
+ libfdt/version.lds | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/libfdt/version.lds b/libfdt/version.lds
+index 80b322b..941208e 100644
+--- a/libfdt/version.lds
++++ b/libfdt/version.lds
+@@ -54,6 +54,8 @@ LIBFDT_1.2 {
+ 		fdt_get_property_by_offset;
+ 		fdt_getprop_by_offset;
+ 		fdt_next_property_offset;
++		fdt_first_subnode;
++		fdt_next_subnode;
+ 
+ 	local:
+ 		*;
+-- 
+2.8.1
+
diff --git a/gnu/packages/patches/dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch b/gnu/packages/patches/dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch
new file mode 100644
index 0000000..4f825c4
--- /dev/null
+++ b/gnu/packages/patches/dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch
@@ -0,0 +1,33 @@
+From f58799be130e27cc729cb2d45566daa0bb3b8605 Mon Sep 17 00:00:00 2001
+From: David Gibson <david@gibson.dropbear.id.au>
+Date: Tue, 1 Dec 2015 12:55:21 +1100
+Subject: [PATCH] libfdt: Add some missing symbols to version.lds
+
+Several functions in the header file were missing from the version.lds
+script, meaning that they couldn't be used from a libfdt shared library.
+
+Reported by Ken Aaker, via github issue tracker.
+
+Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
+---
+ libfdt/version.lds | 4 ++++
+ 1 file changed, 4 insertions(+)
+
+diff --git a/libfdt/version.lds b/libfdt/version.lds
+index f19f157..1f4e1ea 100644
+--- a/libfdt/version.lds
++++ b/libfdt/version.lds
+@@ -57,6 +57,10 @@ LIBFDT_1.2 {
+ 		fdt_next_property_offset;
+ 		fdt_first_subnode;
+ 		fdt_next_subnode;
++		fdt_address_cells;
++		fdt_size_cells;
++		fdt_stringlist_contains;
++		fdt_resize;
+ 
+ 	local:
+ 		*;
+-- 
+2.8.1
+
diff --git a/gnu/packages/u-boot.scm b/gnu/packages/u-boot.scm
new file mode 100644
index 0000000..5f3fd49
--- /dev/null
+++ b/gnu/packages/u-boot.scm
@@ -0,0 +1,65 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2016 Danny Milosavljevic <dannym@scratchpost.org>
+;;; Copyright © 2016 David Craven <david@craven.ch>
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
+
+(define-module (gnu packages u-boot)
+  #:use-module (guix build-system gnu)
+  #:use-module (guix download)
+  #:use-module (guix packages)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (gnu packages)
+  #:use-module (gnu packages bison)
+  #:use-module (gnu packages flex))
+
+(define-public dtc
+  (package
+    (name "dtc")
+    (version "1.4.1")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "https://www.kernel.org/pub/software/utils/dtc/"
+                    "dtc-" version ".tar.xz"))
+              (sha256
+               (base32
+                "155v52palf5fwfcnq696s41whjk0a5dqx98b7maqzdn7xbc2m6bp"))
+              (patches (search-patches "u-boot-dtc-01_build_doc.patch"
+                                       "u-boot-dtc-23-libfdt-Add-missing-functions-to-shared-library.patch"
+                                       "u-boot-dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch"))))
+    (build-system gnu-build-system)
+    (native-inputs
+     `(("bison" ,bison)
+       ("flex" ,flex)))
+    (arguments
+     `(#:make-flags
+       (list "CC=gcc" (string-append "PREFIX=" (assoc-ref %outputs "out")))
+       #:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'patch-paths
+           (lambda _
+             (substitute* "Makefile"
+               (("/usr/bin/install") "install"))
+             (substitute* "Makefile"
+               (("PREFIX = \\$\\(HOME\\)") ""))))
+         (delete 'configure))))
+    (home-page "https://www.devicetree.org")
+    (synopsis "Compiles device tree source files")
+    (description "@command{dtc} compiles device tree source (.dts) Files to
+device tree binary (.dtb) files. 
+These are hardware (board) description files (used by Linux and BSD).")
+    (license license:gpl2+)))

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

* [PATCH v2 2/2] gnu: Add u-boot.
  2016-08-31 21:34 [PATCH v2 0/2] Add u-boot package and its dependent packages Danny Milosavljevic
  2016-08-31 21:34 ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
@ 2016-08-31 21:34 ` Danny Milosavljevic
  1 sibling, 0 replies; 9+ messages in thread
From: Danny Milosavljevic @ 2016-08-31 21:34 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 326 bytes --]

gnu: Add u-boot.

* gnu/packages/u-boot.scm (u-boot, make-u-boot-package,
  armhf-linux-uboot, mips64el-linux-uboot): New variables.

Co-authored-by: David Craven <david@craven.ch>
---
 gnu/packages/u-boot.scm | 90 ++++++++++++++++++++++++++++++++++++++++++++++---
 1 file changed, 86 insertions(+), 4 deletions(-)


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0002-gnu-Add-u-boot.patch --]
[-- Type: text/x-patch; name="0002-gnu-Add-u-boot.patch", Size: 4879 bytes --]

diff --git a/gnu/packages/u-boot.scm b/gnu/packages/u-boot.scm
index 5f3fd49..4a10782 100644
--- a/gnu/packages/u-boot.scm
+++ b/gnu/packages/u-boot.scm
@@ -23,8 +23,11 @@
   #:use-module (guix packages)
   #:use-module ((guix licenses) #:prefix license:)
   #:use-module (gnu packages)
+  #:use-module ((gnu packages algebra) #:select (bc))
   #:use-module (gnu packages bison)
-  #:use-module (gnu packages flex))
+  #:use-module (gnu packages cross-base)
+  #:use-module (gnu packages flex)
+  #:use-module (gnu packages python))
 
 (define-public dtc
   (package
@@ -38,9 +41,9 @@
               (sha256
                (base32
                 "155v52palf5fwfcnq696s41whjk0a5dqx98b7maqzdn7xbc2m6bp"))
-              (patches (search-patches "u-boot-dtc-01_build_doc.patch"
-                                       "u-boot-dtc-23-libfdt-Add-missing-functions-to-shared-library.patch"
-                                       "u-boot-dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch"))))
+              (patches (search-patches "dtc-01_build_doc.patch"
+                                       "dtc-23-libfdt-Add-missing-functions-to-shared-library.patch"
+                                       "dtc-24_libfdt-Add-some-missing-symbols-to-version.lds.patch"))))
     (build-system gnu-build-system)
     (native-inputs
      `(("bison" ,bison)
@@ -63,3 +66,82 @@
 device tree binary (.dtb) files. 
 These are hardware (board) description files (used by Linux and BSD).")
     (license license:gpl2+)))
+
+(define u-boot
+  (package
+    (name "u-boot")
+    (version "2016.07")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "ftp://ftp.denx.de/pub/u-boot/"
+                    "u-boot-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "0lqj4ckmfqiap8mc6z2d5albs3g2h5mzccbn60hsgxhabhibfkwp"))))
+    (native-inputs
+     `(("bc" ,bc)
+       ("dtc" ,dtc)
+       ("python-2" ,python-2)))
+    (build-system gnu-build-system)
+    (home-page "http://www.denx.de/wiki/U-Boot/")
+    (synopsis "ARM Bootloader")
+    (description "U-Boot is a bootloader mostly used for ARM boards. 
+It also initializes the boards (RAM etc).")
+    (license license:gpl2+)))
+
+(define (make-u-boot-package board triplet)
+  (let ((xgcc (cross-gcc triplet)))
+   (package
+    (inherit u-boot)
+    (name (string-append "u-boot-" (string-downcase board)))
+    (native-inputs
+     `(("cross-gcc" ,xgcc)
+       ("cross-binutils" ,(cross-binutils triplet))
+       ,@(package-native-inputs u-boot)))
+    (arguments
+     `(#:test-target "test"
+       #:make-flags
+       (list "HOSTCC=gcc" (string-append "CROSS_COMPILE=" ,triplet "-"))
+       #:phases
+       (modify-phases %standard-phases
+         (replace
+          'configure
+           (lambda* (#:key outputs make-flags #:allow-other-keys)
+             (let ((config-name (string-append ,board "_defconfig")))
+               (if (zero? (system* "ls" (string-append "configs/" config-name)))
+                  (zero? (apply system* "make" `(,@make-flags ,config-name)))
+                  (begin
+                    (use-modules (ice-9 match) (ice-9 format))
+                    (display "Invalid board name. Valid board names would have been:")
+                    (newline)
+                    (let ((dir (opendir "configs")))
+                      (do ((file-name (readdir dir) (readdir dir)))
+                          ((eof-object? file-name))
+                        (if (string-suffix? "_defconfig" file-name)
+                          (format #t
+                                  "- ~A\n"
+                                  (string-drop-right file-name
+                                                     (string-length "_defconfig")))))
+                      (closedir dir))
+                    #f)))))
+         (replace 'install
+           (lambda* (#:key outputs make-flags #:allow-other-keys)
+             (let ((out (string-append (assoc-ref outputs "out") "/libexec")))
+               (mkdir-p out)
+               (for-each (lambda (name)
+                           (let ((outname (string-append out "/" name)))
+                             (mkdir-p (dirname outname))
+                             (copy-file name outname)))
+                         (find-files "." ".*\\.(bin|efi|spl)$")))))))))))
+
+(define-public armhf-linux-uboot
+  (make-u-boot-package "vexpress_ca9x4" "arm-linux-gnueabihf"))
+
+(define-public mips64el-linux-uboot
+  (make-u-boot-package "malta" "mips64el-linux-gnuabi64"))
+
+;;; Something should:
+;;; - create extlinux.conf and put it on the first bootable partition
+;;;   (the one with the Active flag)
+;;; - install the u-boot bootloader

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

* Re: [PATCH v2 1/2] gnu: Add dtc.
  2016-08-31 21:34 ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
@ 2016-09-01 15:16   ` David Craven
  2016-09-01 18:24     ` [PATCH] gnu: dtc: Add missing exports Danny Milosavljevic
  2016-09-01 18:29     ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
  0 siblings, 2 replies; 9+ messages in thread
From: David Craven @ 2016-09-01 15:16 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

So I'm ready to push dtc and u-boot. I'm still not sure about the
patches. If you want me to add them I think you should send a new
patch.

I see you dropped the uboot prefix from the patch names. The patch
names also need to be updated in local.mk.

I think that the 01 23 and 24 should be dropped, probably the libfdt
too. The patch names we have so far stick to lowercase patch names, so
we should too. And use - everywhere.

So that would be dtc-build-doc.patch,
dtc-add-missing-function-to-shared-library.patch and
dtc-add-some-missing-symbols-to-lds.patch.

If you make the changes I'll push your patch.

Thank you for your contribution!

David

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

* [PATCH] gnu: dtc: Add missing exports
  2016-09-01 15:16   ` David Craven
@ 2016-09-01 18:24     ` Danny Milosavljevic
  2016-09-01 18:29     ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
  1 sibling, 0 replies; 9+ messages in thread
From: Danny Milosavljevic @ 2016-09-01 18:24 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 506 bytes --]

gnu: dtc: Add missing exports:
  - fdt_first_subnode
  - fdt_next_subnode
  - fdt_address_cells
  - fdt_size_cells
  - fdt_stringlist_contains
  - fdt_resize

---
 gnu/local.mk                                       |  1 +
 .../patches/dtc-add-missing-symbols-to-lds.patch   | 59 ++++++++++++++++++++++
 gnu/packages/u-boot.scm                            |  4 +-
 3 files changed, 63 insertions(+), 1 deletion(-)
 create mode 100644 gnu/packages/patches/dtc-add-missing-symbols-to-lds.patch


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-dtc-Add-missing-exports.patch --]
[-- Type: text/x-patch; name="0001-gnu-dtc-Add-missing-exports.patch", Size: 3303 bytes --]

diff --git a/gnu/local.mk b/gnu/local.mk
index 8179920..efb00b9 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -483,6 +483,7 @@ dist_patch_DATA =						\
   %D%/packages/patches/doc++-include-directives.patch		\
   %D%/packages/patches/doc++-segfault-fix.patch			\
   %D%/packages/patches/doxygen-test.patch			\
+  %D%/packages/patches/dtc-add-missing-symbols-to-lds.patch     \
   %D%/packages/patches/duplicity-piped-password.patch		\
   %D%/packages/patches/duplicity-test_selection-tmp.patch	\
   %D%/packages/patches/elfutils-tests-ptrace.patch		\
diff --git a/gnu/packages/patches/dtc-add-missing-symbols-to-lds.patch b/gnu/packages/patches/dtc-add-missing-symbols-to-lds.patch
new file mode 100644
index 0000000..ccece0f
--- /dev/null
+++ b/gnu/packages/patches/dtc-add-missing-symbols-to-lds.patch
@@ -0,0 +1,59 @@
+From a4b093f7366fdb429ca1781144d3985fa50d0fbb Mon Sep 17 00:00:00 2001
+From: Julien Grall <julien.grall@linaro.org>
+Date: Tue, 17 Mar 2015 16:00:34 +0000
+Subject: [PATCH] libfdt: Add missing functions to shared library
+
+The commit 4e76ec7 "libfdt: Add fdt_next_subnode() to permit easy
+subnode iteration" adds new functions (fdt_{first,next}_subnode) but
+forgot to mark them as 'global' in the shared library.
+
+Signed-off-by: Julien Grall <julien.grall@linaro.org>
+---
+ libfdt/version.lds | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/libfdt/version.lds b/libfdt/version.lds
+index 80b322b..941208e 100644
+--- a/libfdt/version.lds
++++ b/libfdt/version.lds
+@@ -54,6 +54,8 @@ LIBFDT_1.2 {
+ 		fdt_get_property_by_offset;
+ 		fdt_getprop_by_offset;
+ 		fdt_next_property_offset;
++		fdt_first_subnode;
++		fdt_next_subnode;
+ 
+ 	local:
+ 		*;
+From f58799be130e27cc729cb2d45566daa0bb3b8605 Mon Sep 17 00:00:00 2001
+From: David Gibson <david@gibson.dropbear.id.au>
+Date: Tue, 1 Dec 2015 12:55:21 +1100
+Subject: [PATCH] libfdt: Add some missing symbols to version.lds
+
+Several functions in the header file were missing from the version.lds
+script, meaning that they couldn't be used from a libfdt shared library.
+
+Reported by Ken Aaker, via github issue tracker.
+
+Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
+---
+ libfdt/version.lds | 4 ++++
+ 1 file changed, 4 insertions(+)
+
+diff --git a/libfdt/version.lds b/libfdt/version.lds
+index f19f157..1f4e1ea 100644
+--- a/libfdt/version.lds
++++ b/libfdt/version.lds
+@@ -57,6 +57,10 @@ LIBFDT_1.2 {
+ 		fdt_next_property_offset;
+ 		fdt_first_subnode;
+ 		fdt_next_subnode;
++		fdt_address_cells;
++		fdt_size_cells;
++		fdt_stringlist_contains;
++		fdt_resize;
+ 
+ 	local:
+ 		*;
+-- 
+2.8.1
diff --git a/gnu/packages/u-boot.scm b/gnu/packages/u-boot.scm
index 77e7447..15b24a6 100644
--- a/gnu/packages/u-boot.scm
+++ b/gnu/packages/u-boot.scm
@@ -40,7 +40,9 @@
                     "dtc-" version ".tar.xz"))
               (sha256
                (base32
-                "155v52palf5fwfcnq696s41whjk0a5dqx98b7maqzdn7xbc2m6bp"))))
+                "155v52palf5fwfcnq696s41whjk0a5dqx98b7maqzdn7xbc2m6bp"))
+              (patches
+                (search-patches "dtc-add-missing-symbols-to-lds.patch"))))
     (build-system gnu-build-system)
     (native-inputs
      `(("bison" ,bison)

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

* Re: [PATCH v2 1/2] gnu: Add dtc.
  2016-09-01 15:16   ` David Craven
  2016-09-01 18:24     ` [PATCH] gnu: dtc: Add missing exports Danny Milosavljevic
@ 2016-09-01 18:29     ` Danny Milosavljevic
  2016-09-01 22:17       ` David Craven
  2016-09-03 18:45       ` Danny Milosavljevic
  1 sibling, 2 replies; 9+ messages in thread
From: Danny Milosavljevic @ 2016-09-01 18:29 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

Hi David,

I've posted just the one patch that is adding the exports now.
Leaving those exports out was certainly an oversight by the dtc developers.

Not adding them would mean that libfdt.h contained prototypes for functions which don't exist. This is something that is really annoying to have as a developer and I'd prefer if we didn't subject our devs to something like that - and on embedded where everything worth doing takes ages to do to begin with.

That said, long term it would be nice if the dtc developers made a new release. So I've also posted to their mailing list asking them to do that.

In the mean time, let's carry this one patch.

Thanks for your contribution too :)

Cheers,
  Danny

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

* Re: [PATCH v2 1/2] gnu: Add dtc.
  2016-09-01 18:29     ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
@ 2016-09-01 22:17       ` David Craven
  2016-09-03 18:45       ` Danny Milosavljevic
  1 sibling, 0 replies; 9+ messages in thread
From: David Craven @ 2016-09-01 22:17 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

Pushed as f63e5eaaeb2671ec550b0c21f4afe5b277de6a5d.

Thanks!

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

* Re: [PATCH v2 1/2] gnu: Add dtc.
  2016-09-01 18:29     ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
  2016-09-01 22:17       ` David Craven
@ 2016-09-03 18:45       ` Danny Milosavljevic
  2016-09-03 18:46         ` David Craven
  1 sibling, 1 reply; 9+ messages in thread
From: Danny Milosavljevic @ 2016-09-03 18:45 UTC (permalink / raw)
  To: guix-devel

Hi,

they've integrated these patches into their new dtc release 1.4.2 :)

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

* Re: [PATCH v2 1/2] gnu: Add dtc.
  2016-09-03 18:45       ` Danny Milosavljevic
@ 2016-09-03 18:46         ` David Craven
  0 siblings, 0 replies; 9+ messages in thread
From: David Craven @ 2016-09-03 18:46 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

> Hi,
>
> they've integrated these patches into their new dtc release 1.4.2 :)

Awesome! I think it's a problem if we patch every package with 10
patches that's all. But this is great! Can you send a patch to update
dtc to 1.4.2?

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

end of thread, other threads:[~2016-09-03 18:54 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-31 21:34 [PATCH v2 0/2] Add u-boot package and its dependent packages Danny Milosavljevic
2016-08-31 21:34 ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
2016-09-01 15:16   ` David Craven
2016-09-01 18:24     ` [PATCH] gnu: dtc: Add missing exports Danny Milosavljevic
2016-09-01 18:29     ` [PATCH v2 1/2] gnu: Add dtc Danny Milosavljevic
2016-09-01 22:17       ` David Craven
2016-09-03 18:45       ` Danny Milosavljevic
2016-09-03 18:46         ` David Craven
2016-08-31 21:34 ` [PATCH v2 2/2] gnu: Add u-boot Danny Milosavljevic

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

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.