Patchwork beagleboard-test-scripts: Add a script flash-fs.sh for flashing NAND.

login
register
mail settings
Submitter Joel A Fernandes
Date July 6, 2011, 1:27 a.m.
Message ID <1309915643-16401-1-git-send-email-agnel.joel@gmail.com>
Download mbox | patch
Permalink /patch/7067/
State New, archived
Headers show

Comments

Joel A Fernandes - July 6, 2011, 1:27 a.m.
This script flashes the NAND of a beagleboard if it exists, and if there is a valid image to flash.

Signed-off-by: Joel A Fernandes <agnel.joel@gmail.com>
---
 .../beagleboard/beagleboard-test-scripts_git.bb    |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)
Paul Menzel - July 6, 2011, 9:40 a.m.
Dear Joel,


Am Dienstag, den 05.07.2011, 20:27 -0500 schrieb Joel A Fernandes:
                                                       ^

• Is in your Name a dot missing after the second name?
• In the commit summary no full stop at the end is used according to the
new commit guidelines.

> This script flashes the NAND of a beagleboard if it exists, and if there is a valid image to flash.

If you capitalize NAND you could also write BeagleBoard.

> Signed-off-by: Joel A Fernandes <agnel.joel@gmail.com>
> ---
>  .../beagleboard/beagleboard-test-scripts_git.bb    |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/recipes/beagleboard/beagleboard-test-scripts_git.bb b/recipes/beagleboard/beagleboard-test-scripts_git.bb
> index be0d059..237d8d6 100644
> --- a/recipes/beagleboard/beagleboard-test-scripts_git.bb
> +++ b/recipes/beagleboard/beagleboard-test-scripts_git.bb
> @@ -3,10 +3,10 @@ DESCRIPTION = "BeagleBoard test scripts"
>  HOMEPAGE = "http://beagleboad.org/support"
>  PR = "r2"

Since the package content changes you have to increase the number.

> -SRC_URI = "git://gitorious.org/beagleboard-validation/scripts.git;protocol=git \
> +SRC_URI = "git://gitorious.org/~joelf/beagleboard-validation/validation-scripts.git;protocol=git \
>  "
>  
> -SRCREV = "473dd2ab20d866be6168c9f992c2c9e74e485c9d"
> +SRCREV = "${AUTOREV}"

In OE it is not wanted to use `AUTOREV`. I do not know though how you
can specify a revision for each repository.

>  S = "${WORKDIR}/git"
>  
>  do_install() {
> @@ -28,4 +28,7 @@ do_install() {
>    for i in ${TEST_FILES}; do
>      install -m 0755 ${S}/${i} ${D}/${bindir}
>    done
> +  # Add  script to flash the NAND if it exists and if there is a valid image to flash on bootup.

• After »Add a space can be removed.
• »on bootup« is not clear to me. Does the BB have to be rebooted and
then the image is flashed?

> +  install -d ${D}/${sysconfdir}/rc5.d/
> +  install -m 0755 ${S}/flashing/flash-fs.sh ${D}/${sysconfdir}/rc5.d/S99flashfs
>  }


Thanks,

Paul
Joel A Fernandes - July 6, 2011, 3:44 p.m.
Hi Paul,

Thanks for your suggestions. I will make the requested changes,

>>  do_install() {
>> @@ -28,4 +28,7 @@ do_install() {
>>    for i in ${TEST_FILES}; do
>>      install -m 0755 ${S}/${i} ${D}/${bindir}
>>    done
>> +  # Add  script to flash the NAND if it exists and if there is a valid image to flash on bootup.
>
> • After »Add a space can be removed.
> • »on bootup« is not clear to me. Does the BB have to be rebooted and
> then the image is flashed?

In boards with a NAND device, an image is written to the NAND if the
SD Card has an image. This is during startup.

Regards,
Joel

Patch

diff --git a/recipes/beagleboard/beagleboard-test-scripts_git.bb b/recipes/beagleboard/beagleboard-test-scripts_git.bb
index be0d059..237d8d6 100644
--- a/recipes/beagleboard/beagleboard-test-scripts_git.bb
+++ b/recipes/beagleboard/beagleboard-test-scripts_git.bb
@@ -3,10 +3,10 @@  DESCRIPTION = "BeagleBoard test scripts"
 HOMEPAGE = "http://beagleboad.org/support"
 PR = "r2"
 
-SRC_URI = "git://gitorious.org/beagleboard-validation/scripts.git;protocol=git \
+SRC_URI = "git://gitorious.org/~joelf/beagleboard-validation/validation-scripts.git;protocol=git \
 "
 
-SRCREV = "473dd2ab20d866be6168c9f992c2c9e74e485c9d"
+SRCREV = "${AUTOREV}"
 S = "${WORKDIR}/git"
 
 do_install() {
@@ -28,4 +28,7 @@  do_install() {
   for i in ${TEST_FILES}; do
     install -m 0755 ${S}/${i} ${D}/${bindir}
   done
+  # Add  script to flash the NAND if it exists and if there is a valid image to flash on bootup.
+  install -d ${D}/${sysconfdir}/rc5.d/
+  install -m 0755 ${S}/flashing/flash-fs.sh ${D}/${sysconfdir}/rc5.d/S99flashfs
 }