Patchwork u-boot-fw-utils: Initial version

login
register
mail settings
Submitter Tom Rini
Date Feb. 3, 2012, 10:53 p.m.
Message ID <1328309610-12397-1-git-send-email-tom.rini@gmail.com>
Download mbox | patch
Permalink /patch/20679/
State Accepted
Commit 4a2543c0641f1df6d74c18cfe0c3ac3cfd50e60c
Headers show

Comments

Tom Rini - Feb. 3, 2012, 10:53 p.m.
From: Tom Rini <trini@ti.com>

This provides the 'fw_setenv' and 'fw_getenv' programs for the target
so that the U-Boot environment can be modified from Linux.  These
programs are system-agnostic and rely on a config file that a given BSP
will provide (and RRECOMMEND this be installed).

Signed-off-by: Tom Rini <trini@ti.com>
---
 meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb |   26 ++++++++++++++++++++
 1 files changed, 26 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
Paul Menzel - Feb. 3, 2012, 11:06 p.m.
Dear Tom,


Am Freitag, den 03.02.2012, 15:53 -0700 schrieb Tom Rini:
> From: Tom Rini <trini@ti.com>

please add the version in the commit summary.

> This provides the 'fw_setenv' and 'fw_getenv' programs for the target
> so that the U-Boot environment can be modified from Linux.  These
> programs are system-agnostic and rely on a config file that a given BSP
> will provide (and RRECOMMEND this be installed).

I do not understand the addition in the parentheses.

> Signed-off-by: Tom Rini <trini@ti.com>
> ---
>  meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb |   26 ++++++++++++++++++++
>  1 files changed, 26 insertions(+), 0 deletions(-)
>  create mode 100644 meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
> 
> diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
> new file mode 100644
> index 0000000..3185cbd
> --- /dev/null
> +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
> @@ -0,0 +1,26 @@
> +DESCRIPTION = "U-boot bootloader fw_printenv/setenv utils"
> +LICENSE = "GPLv2+"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
> +                    file://README;beginline=1;endline=22;md5=5ba4218ac89af7846802d0348df3fb90"
> +SECTION = "bootloader"

Is this the standard ordering? Is there also a homepage for theses
utils?

> +
> +PR = "r0"
> +

Was a decision made if `PR = "r0"` should be set?

> +SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2"
> +
> +SRC_URI[md5sum] = "0cc5026aad02f218a9b9ac56b301c97a"
> +SRC_URI[sha256sum] = "362ddb935885da98cf461eba08f31e3e59d0c4ada6cb2fa15596f43af310ba8b"
> +
> +S = "${WORKDIR}/u-boot-${PV}"

Is that the default? I always forget.

> +
> +EXTRA_OEMAKE = 'HOSTCC="${CC}"'
> +
> +do_compile () {
> +  oe_runmake env
> +}
> +
> +do_install () {
> +  install -d ${D}${base_sbindir}
> +  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_printenv
> +  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_setenv
> +}


Thanks,

Paul
Koen Kooi - Feb. 4, 2012, 12:34 a.m.
Op 4 feb. 2012, om 00:06 heeft Paul Menzel het volgende geschreven:

> Dear Tom,
> 
> 
> Am Freitag, den 03.02.2012, 15:53 -0700 schrieb Tom Rini:
>> From: Tom Rini <trini@ti.com>
> 
> please add the version in the commit summary.
> 
>> This provides the 'fw_setenv' and 'fw_getenv' programs for the target
>> so that the U-Boot environment can be modified from Linux.  These
>> programs are system-agnostic and rely on a config file that a given BSP
>> will provide (and RRECOMMEND this be installed).
> 
> I do not understand the addition in the parentheses.
> 
>> Signed-off-by: Tom Rini <trini@ti.com>
>> ---
>> meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb |   26 ++++++++++++++++++++
>> 1 files changed, 26 insertions(+), 0 deletions(-)
>> create mode 100644 meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
>> 
>> diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
>> new file mode 100644
>> index 0000000..3185cbd
>> --- /dev/null
>> +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
>> @@ -0,0 +1,26 @@
>> +DESCRIPTION = "U-boot bootloader fw_printenv/setenv utils"
>> +LICENSE = "GPLv2+"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>> +                    file://README;beginline=1;endline=22;md5=5ba4218ac89af7846802d0348df3fb90"
>> +SECTION = "bootloader"
> 
> Is this the standard ordering? Is there also a homepage for theses
> utils?
> 
>> +
>> +PR = "r0"
>> +
> 
> Was a decision made if `PR = "r0"` should be set?
> 
>> +SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2"
>> +
>> +SRC_URI[md5sum] = "0cc5026aad02f218a9b9ac56b301c97a"
>> +SRC_URI[sha256sum] = "362ddb935885da98cf461eba08f31e3e59d0c4ada6cb2fa15596f43af310ba8b"
>> +
>> +S = "${WORKDIR}/u-boot-${PV}"

Notice that $PN} is not 'u-boot' in this case.
Tom Rini - Feb. 8, 2012, 5:57 p.m.
On Fri, Feb 3, 2012 at 4:06 PM, Paul Menzel
<paulepanter@users.sourceforge.net> wrote:
> Dear Tom,
>
>
> Am Freitag, den 03.02.2012, 15:53 -0700 schrieb Tom Rini:
>> From: Tom Rini <trini@ti.com>
>
> please add the version in the commit summary.

OK.

>> This provides the 'fw_setenv' and 'fw_getenv' programs for the target
>> so that the U-Boot environment can be modified from Linux.  These
>> programs are system-agnostic and rely on a config file that a given BSP
>> will provide (and RRECOMMEND this be installed).
>
> I do not understand the addition in the parentheses.

OK.

>> Signed-off-by: Tom Rini <trini@ti.com>
>> ---
>>  meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb |   26 ++++++++++++++++++++
>>  1 files changed, 26 insertions(+), 0 deletions(-)
>>  create mode 100644 meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
>>
>> diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
>> new file mode 100644
>> index 0000000..3185cbd
>> --- /dev/null
>> +++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
>> @@ -0,0 +1,26 @@
>> +DESCRIPTION = "U-boot bootloader fw_printenv/setenv utils"
>> +LICENSE = "GPLv2+"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>> +                    file://README;beginline=1;endline=22;md5=5ba4218ac89af7846802d0348df3fb90"
>> +SECTION = "bootloader"
>
> Is this the standard ordering? Is there also a homepage for theses
> utils?

This is just the u-boot recipe stripped out of building U-Boot and
only building the bundled utilities.

Patch

diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
new file mode 100644
index 0000000..3185cbd
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2011.06.bb
@@ -0,0 +1,26 @@ 
+DESCRIPTION = "U-boot bootloader fw_printenv/setenv utils"
+LICENSE = "GPLv2+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
+                    file://README;beginline=1;endline=22;md5=5ba4218ac89af7846802d0348df3fb90"
+SECTION = "bootloader"
+
+PR = "r0"
+
+SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2"
+
+SRC_URI[md5sum] = "0cc5026aad02f218a9b9ac56b301c97a"
+SRC_URI[sha256sum] = "362ddb935885da98cf461eba08f31e3e59d0c4ada6cb2fa15596f43af310ba8b"
+
+S = "${WORKDIR}/u-boot-${PV}"
+
+EXTRA_OEMAKE = 'HOSTCC="${CC}"'
+
+do_compile () {
+  oe_runmake env
+}
+
+do_install () {
+  install -d ${D}${base_sbindir}
+  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_printenv
+  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_setenv
+}