| 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
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
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.
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 +}