* [Buildroot] Repair boot from SD card on i.MX6 Sabre boards
@ 2015-10-14 14:03 Vincent Stehlé
2015-10-14 14:03 ` [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh Vincent Stehlé
` (2 more replies)
0 siblings, 3 replies; 11+ messages in thread
From: Vincent Stehlé @ 2015-10-14 14:03 UTC (permalink / raw)
To: buildroot
Hi,
Here is a two-patches series, to repair the boot from SD card on
Freescale Sabre boards.
The first patch adapts the SD card creation script to the new versions
of sfdisk; a prerequisite to actually preparing an SD card and booting
from it.
The second patch switches the defconfigs to building a zImage instead of
an uImage, to reflect what u-boot in Freescale releases awaits nowadays.
This was build-tested for all the five touched defconfigs but I did only
verify the boot on i.MX6 quad Sabre AI.
Feedbacks and testers welcome!
Best regards,
V.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh
2015-10-14 14:03 [Buildroot] Repair boot from SD card on i.MX6 Sabre boards Vincent Stehlé
@ 2015-10-14 14:03 ` Vincent Stehlé
2015-10-14 16:04 ` Luca Ceresoli
2015-10-14 14:03 ` [Buildroot] [PATCH 2/2] imx6sabre: use zImage format for kernel Vincent Stehlé
2015-10-15 13:03 ` [Buildroot] Repair boot from SD card on i.MX6 Sabre boards Jeremy Rosen
2 siblings, 1 reply; 11+ messages in thread
From: Vincent Stehlé @ 2015-10-14 14:03 UTC (permalink / raw)
To: buildroot
Starting with version 2.26, sfdisk defaults to a unit of 512 B sectors.
With those recent versions of sfdisk, the create-boot-sd.sh script ends
up creating a boot partition of 240 KB, which is too small to contain a
Linux kernel.
Version 2.25 of sfdisk and older defaulted to cylinders; a
media-dependant unit, typically much larger than 512 B. With those
versions the create-boot-sd.sh creates paritions, large enough to hold a
kernel.
We update the create-boot-sd.sh script to use the more readable and more
stable named-fields sfdisk format for the partitioning command, and set
the boot partition size to 64 MB, which should be enough for everyone.
Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
Cc: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Gary Bisson <gary.bisson@boundarydevices.com>
---
board/freescale/create-boot-sd.sh | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/board/freescale/create-boot-sd.sh b/board/freescale/create-boot-sd.sh
index af45115..cfb1101 100755
--- a/board/freescale/create-boot-sd.sh
+++ b/board/freescale/create-boot-sd.sh
@@ -60,8 +60,8 @@ sync
# - FAT partition starting at 1MB offset, containing uImage and *.dtb
# - ext2/3 partition formatted as ext2 or ext3, containing the root filesystem.
sfdisk ${DEV} <<EOF
-32,480,b
-512,,L
+start=1MiB, size=64MiB, type=b
+type=83
EOF
sync
--
2.5.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH 2/2] imx6sabre: use zImage format for kernel
2015-10-14 14:03 [Buildroot] Repair boot from SD card on i.MX6 Sabre boards Vincent Stehlé
2015-10-14 14:03 ` [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh Vincent Stehlé
@ 2015-10-14 14:03 ` Vincent Stehlé
2015-10-15 13:03 ` [Buildroot] Repair boot from SD card on i.MX6 Sabre boards Jeremy Rosen
2 siblings, 0 replies; 11+ messages in thread
From: Vincent Stehlé @ 2015-10-14 14:03 UTC (permalink / raw)
To: buildroot
Starting with u-boot rel_imx_3.10.53_1.1.0_ga, the default is to boot a
zImage instead of an uImage. Update the i.MX6 defconfigs accordingly.
This repairs the boot from SD card.
While at it, align freescale_imx6sololiteevk_defconfig a bit more on the
other i.MX6 defconfigs, update documentation and SD card creation script
comments to reflect the change to zImage.
Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
Cc: Gary Bisson <gary.bisson@boundarydevices.com>
---
board/freescale/create-boot-sd.sh | 2 +-
board/freescale/imx6sabre/readme.txt | 2 +-
configs/freescale_imx6dlsabreauto_defconfig | 2 +-
configs/freescale_imx6dlsabresd_defconfig | 2 +-
configs/freescale_imx6qsabreauto_defconfig | 2 +-
configs/freescale_imx6qsabresd_defconfig | 2 +-
configs/freescale_imx6sololiteevk_defconfig | 10 +++++++++-
7 files changed, 15 insertions(+), 7 deletions(-)
diff --git a/board/freescale/create-boot-sd.sh b/board/freescale/create-boot-sd.sh
index cfb1101..e75d107 100755
--- a/board/freescale/create-boot-sd.sh
+++ b/board/freescale/create-boot-sd.sh
@@ -57,7 +57,7 @@ sync
# Partition the card.
# SD layout for i.MX6 boot:
# - Bootloader at offset 1024
-# - FAT partition starting at 1MB offset, containing uImage and *.dtb
+# - FAT partition starting at 1MB offset, containing uImage or zImage, and *.dtb
# - ext2/3 partition formatted as ext2 or ext3, containing the root filesystem.
sfdisk ${DEV} <<EOF
start=1MiB, size=64MiB, type=b
diff --git a/board/freescale/imx6sabre/readme.txt b/board/freescale/imx6sabre/readme.txt
index 6850885..6958d92 100644
--- a/board/freescale/imx6sabre/readme.txt
+++ b/board/freescale/imx6sabre/readme.txt
@@ -43,7 +43,7 @@ You will find in ./output/images/ the following files:
- rootfs.ext2
- rootfs.tar
- u-boot.imx
- - uImage
+ - zImage
Create a bootable SD card
=========================
diff --git a/configs/freescale_imx6dlsabreauto_defconfig b/configs/freescale_imx6dlsabreauto_defconfig
index 0de8eea..c175530 100644
--- a/configs/freescale_imx6dlsabreauto_defconfig
+++ b/configs/freescale_imx6dlsabreauto_defconfig
@@ -19,7 +19,7 @@ BR2_LINUX_KERNEL_CUSTOM_GIT=y
BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://git.freescale.com/imx/linux-2.6-imx.git"
BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rel_imx_3.14.28_1.0.0_ga"
BR2_LINUX_KERNEL_DEFCONFIG="imx_v7"
-BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
+BR2_LINUX_KERNEL_ZIMAGE=y
BR2_LINUX_KERNEL_DTS_SUPPORT=y
BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6dl-sabreauto"
diff --git a/configs/freescale_imx6dlsabresd_defconfig b/configs/freescale_imx6dlsabresd_defconfig
index 3010d7d..25bd6ce 100644
--- a/configs/freescale_imx6dlsabresd_defconfig
+++ b/configs/freescale_imx6dlsabresd_defconfig
@@ -19,7 +19,7 @@ BR2_LINUX_KERNEL_CUSTOM_GIT=y
BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://git.freescale.com/imx/linux-2.6-imx.git"
BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rel_imx_3.14.28_1.0.0_ga"
BR2_LINUX_KERNEL_DEFCONFIG="imx_v7"
-BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
+BR2_LINUX_KERNEL_ZIMAGE=y
BR2_LINUX_KERNEL_DTS_SUPPORT=y
BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6dl-sabresd"
diff --git a/configs/freescale_imx6qsabreauto_defconfig b/configs/freescale_imx6qsabreauto_defconfig
index 70367a7..b5ecc0e 100644
--- a/configs/freescale_imx6qsabreauto_defconfig
+++ b/configs/freescale_imx6qsabreauto_defconfig
@@ -19,7 +19,7 @@ BR2_LINUX_KERNEL_CUSTOM_GIT=y
BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://git.freescale.com/imx/linux-2.6-imx.git"
BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rel_imx_3.14.28_1.0.0_ga"
BR2_LINUX_KERNEL_DEFCONFIG="imx_v7"
-BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
+BR2_LINUX_KERNEL_ZIMAGE=y
BR2_LINUX_KERNEL_DTS_SUPPORT=y
BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6q-sabreauto"
diff --git a/configs/freescale_imx6qsabresd_defconfig b/configs/freescale_imx6qsabresd_defconfig
index 3fad77a..6dea756 100644
--- a/configs/freescale_imx6qsabresd_defconfig
+++ b/configs/freescale_imx6qsabresd_defconfig
@@ -19,7 +19,7 @@ BR2_LINUX_KERNEL_CUSTOM_GIT=y
BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://git.freescale.com/imx/linux-2.6-imx.git"
BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rel_imx_3.14.28_1.0.0_ga"
BR2_LINUX_KERNEL_DEFCONFIG="imx_v7"
-BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
+BR2_LINUX_KERNEL_ZIMAGE=y
BR2_LINUX_KERNEL_DTS_SUPPORT=y
BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6q-sabresd"
diff --git a/configs/freescale_imx6sololiteevk_defconfig b/configs/freescale_imx6sololiteevk_defconfig
index 5d98e9a..18ed588 100644
--- a/configs/freescale_imx6sololiteevk_defconfig
+++ b/configs/freescale_imx6sololiteevk_defconfig
@@ -2,6 +2,14 @@
BR2_arm=y
BR2_cortex_a9=y
+# patches
+BR2_GLOBAL_PATCH_DIR="board/freescale/imx6sabre/patches"
+
+# toolchain
+BR2_KERNEL_HEADERS_VERSION=y
+BR2_DEFAULT_KERNEL_VERSION="3.14.28"
+BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_3_14=y
+
# system
BR2_TARGET_GENERIC_GETTY_PORT="ttymxc0"
@@ -11,7 +19,7 @@ BR2_LINUX_KERNEL_CUSTOM_GIT=y
BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://git.freescale.com/imx/linux-2.6-imx.git"
BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rel_imx_3.14.28_1.0.0_ga"
BR2_LINUX_KERNEL_DEFCONFIG="imx_v7"
-BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x80008000"
+BR2_LINUX_KERNEL_ZIMAGE=y
BR2_LINUX_KERNEL_DTS_SUPPORT=y
BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6sl-evk"
--
2.5.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh
2015-10-14 14:03 ` [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh Vincent Stehlé
@ 2015-10-14 16:04 ` Luca Ceresoli
2015-10-14 21:10 ` Arnout Vandecappelle
0 siblings, 1 reply; 11+ messages in thread
From: Luca Ceresoli @ 2015-10-14 16:04 UTC (permalink / raw)
To: buildroot
Dear Vincent,
Vincent Stehl? wrote:
> Starting with version 2.26, sfdisk defaults to a unit of 512 B sectors.
> With those recent versions of sfdisk, the create-boot-sd.sh script ends
> up creating a boot partition of 240 KB, which is too small to contain a
> Linux kernel.
Ouch!
>
> Version 2.25 of sfdisk and older defaulted to cylinders; a
> media-dependant unit, typically much larger than 512 B. With those
> versions the create-boot-sd.sh creates paritions, large enough to hold a
paritions -> partitions, and remove the ',' after it.
> kernel.
>
> We update the create-boot-sd.sh script to use the more readable and more
> stable named-fields sfdisk format for the partitioning command, and set
> the boot partition size to 64 MB, which should be enough for everyone.
>
> Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
> Cc: Luca Ceresoli <luca@lucaceresoli.net>
> Cc: Gary Bisson <gary.bisson@boundarydevices.com>
> ---
> board/freescale/create-boot-sd.sh | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/board/freescale/create-boot-sd.sh b/board/freescale/create-boot-sd.sh
> index af45115..cfb1101 100755
> --- a/board/freescale/create-boot-sd.sh
> +++ b/board/freescale/create-boot-sd.sh
> @@ -60,8 +60,8 @@ sync
> # - FAT partition starting at 1MB offset, containing uImage and *.dtb
> # - ext2/3 partition formatted as ext2 or ext3, containing the root filesystem.
> sfdisk ${DEV} <<EOF
> -32,480,b
> -512,,L
> +start=1MiB, size=64MiB, type=b
Is the "MiB" suffix supported on older sfdisk releases? Ubuntu 14.04
ships sfdisk 2.20.1, and its manpage does not contain "MiB".
If that is not supported, then I guess "--unit M" should work. It's in
the manpage, but I haven't tested it yet.
I strongly think we should support the latest Ubuntu LTS, AFAIK it's
widely used by embedded Linux developers.
--
Luca
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh
2015-10-14 16:04 ` Luca Ceresoli
@ 2015-10-14 21:10 ` Arnout Vandecappelle
2015-10-15 8:30 ` Gary Bisson
0 siblings, 1 reply; 11+ messages in thread
From: Arnout Vandecappelle @ 2015-10-14 21:10 UTC (permalink / raw)
To: buildroot
On 14-10-15 18:04, Luca Ceresoli wrote:
> Dear Vincent,
>
> Vincent Stehl? wrote:
>> Starting with version 2.26, sfdisk defaults to a unit of 512 B sectors.
>> With those recent versions of sfdisk, the create-boot-sd.sh script ends
>> up creating a boot partition of 240 KB, which is too small to contain a
>> Linux kernel.
>
> Ouch!
>
>>
>> Version 2.25 of sfdisk and older defaulted to cylinders; a
>> media-dependant unit, typically much larger than 512 B. With those
>> versions the create-boot-sd.sh creates paritions, large enough to hold a
>
> paritions -> partitions, and remove the ',' after it.
>
>> kernel.
>>
>> We update the create-boot-sd.sh script to use the more readable and more
>> stable named-fields sfdisk format for the partitioning command, and set
>> the boot partition size to 64 MB, which should be enough for everyone.
>>
>> Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
>> Cc: Luca Ceresoli <luca@lucaceresoli.net>
>> Cc: Gary Bisson <gary.bisson@boundarydevices.com>
>> ---
>> board/freescale/create-boot-sd.sh | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/board/freescale/create-boot-sd.sh
>> b/board/freescale/create-boot-sd.sh
>> index af45115..cfb1101 100755
>> --- a/board/freescale/create-boot-sd.sh
>> +++ b/board/freescale/create-boot-sd.sh
>> @@ -60,8 +60,8 @@ sync
>> # - FAT partition starting at 1MB offset, containing uImage and *.dtb
>> # - ext2/3 partition formatted as ext2 or ext3, containing the root filesystem.
>> sfdisk ${DEV} <<EOF
>> -32,480,b
>> -512,,L
>> +start=1MiB, size=64MiB, type=b
>
> Is the "MiB" suffix supported on older sfdisk releases? Ubuntu 14.04
> ships sfdisk 2.20.1, and its manpage does not contain "MiB".
>
> If that is not supported, then I guess "--unit M" should work. It's in
> the manpage, but I haven't tested it yet.
>
> I strongly think we should support the latest Ubuntu LTS, AFAIK it's
> widely used by embedded Linux developers.
We should also still support Ubuntu 12.02 IMHO and even older RHEL.
So to avoid this kind of issue, it would be better to use genimage. But that's
a much bigger change, of course.
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh
2015-10-14 21:10 ` Arnout Vandecappelle
@ 2015-10-15 8:30 ` Gary Bisson
2015-10-21 8:57 ` [Buildroot] [PATCH v2] board/freescale: switch create-boot-sd.sh to parted Vincent Stehlé
0 siblings, 1 reply; 11+ messages in thread
From: Gary Bisson @ 2015-10-15 8:30 UTC (permalink / raw)
To: buildroot
Hi,
On Wed, Oct 14, 2015 at 11:10 PM, Arnout Vandecappelle <arnout@mind.be>
wrote:
> On 14-10-15 18:04, Luca Ceresoli wrote:
>> Dear Vincent,
>>
>> Vincent Stehl? wrote:
>>> Starting with version 2.26, sfdisk defaults to a unit of 512 B sectors.
>>> With those recent versions of sfdisk, the create-boot-sd.sh script ends
>>> up creating a boot partition of 240 KB, which is too small to contain a
>>> Linux kernel.
>>
>> Ouch!
>>
>>>
>>> Version 2.25 of sfdisk and older defaulted to cylinders; a
>>> media-dependant unit, typically much larger than 512 B. With those
>>> versions the create-boot-sd.sh creates paritions, large enough to hold a
>>
>> paritions -> partitions, and remove the ',' after it.
>>
>>> kernel.
>>>
>>> We update the create-boot-sd.sh script to use the more readable and more
>>> stable named-fields sfdisk format for the partitioning command, and set
>>> the boot partition size to 64 MB, which should be enough for everyone.
>>>
>>> Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
>>> Cc: Luca Ceresoli <luca@lucaceresoli.net>
>>> Cc: Gary Bisson <gary.bisson@boundarydevices.com>
>>> ---
>>> board/freescale/create-boot-sd.sh | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/board/freescale/create-boot-sd.sh
>>> b/board/freescale/create-boot-sd.sh
>>> index af45115..cfb1101 100755
>>> --- a/board/freescale/create-boot-sd.sh
>>> +++ b/board/freescale/create-boot-sd.sh
>>> @@ -60,8 +60,8 @@ sync
>>> # - FAT partition starting at 1MB offset, containing uImage and *.dtb
>>> # - ext2/3 partition formatted as ext2 or ext3, containing the root
filesystem.
>>> sfdisk ${DEV} <<EOF
>>> -32,480,b
>>> -512,,L
>>> +start=1MiB, size=64MiB, type=b
>>
>> Is the "MiB" suffix supported on older sfdisk releases? Ubuntu 14.04
>> ships sfdisk 2.20.1, and its manpage does not contain "MiB".
>>
>> If that is not supported, then I guess "--unit M" should work. It's in
>> the manpage, but I haven't tested it yet.
>>
>> I strongly think we should support the latest Ubuntu LTS, AFAIK it's
>> widely used by embedded Linux developers.
>
> We should also still support Ubuntu 12.02 IMHO and even older RHEL.
Agreed.
> So to avoid this kind of issue, it would be better to use genimage. But
that's
> a much bigger change, of course.
What about parted instead of sfdisk? Seems to be more consistent across the
different releases.
Regards,
Gary
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20151015/5f08f60a/attachment.html>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] Repair boot from SD card on i.MX6 Sabre boards
2015-10-14 14:03 [Buildroot] Repair boot from SD card on i.MX6 Sabre boards Vincent Stehlé
2015-10-14 14:03 ` [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh Vincent Stehlé
2015-10-14 14:03 ` [Buildroot] [PATCH 2/2] imx6sabre: use zImage format for kernel Vincent Stehlé
@ 2015-10-15 13:03 ` Jeremy Rosen
2 siblings, 0 replies; 11+ messages in thread
From: Jeremy Rosen @ 2015-10-15 13:03 UTC (permalink / raw)
To: buildroot
I know that this patch is still under review and might change, but in it's present form,
usind a imx6sabreq and util-linux 2.26.2-9 from debian
Tested-by: J?r?my Rosen <jeremy.rosen@openwide.fr>
Cordialement
J?r?my Rosen
+33 (0)1 42 68 28 13
fight key loggers : write some perl using vim
Open Wide Ingenierie
23, rue Daviel
75013 Paris - France
www.openwide.fr
----- Le 14 Oct 15, ? 16:03, Vincent Stehl? vincent.stehle at freescale.com a ?crit :
> Hi,
>
> Here is a two-patches series, to repair the boot from SD card on
> Freescale Sabre boards.
>
> The first patch adapts the SD card creation script to the new versions
> of sfdisk; a prerequisite to actually preparing an SD card and booting
> from it.
>
> The second patch switches the defconfigs to building a zImage instead of
> an uImage, to reflect what u-boot in Freescale releases awaits nowadays.
>
> This was build-tested for all the five touched defconfigs but I did only
> verify the boot on i.MX6 quad Sabre AI.
>
> Feedbacks and testers welcome!
>
> Best regards,
>
> V.
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2] board/freescale: switch create-boot-sd.sh to parted
2015-10-15 8:30 ` Gary Bisson
@ 2015-10-21 8:57 ` Vincent Stehlé
2015-10-21 9:14 ` Baruch Siach
2015-10-21 18:37 ` Arnout Vandecappelle
0 siblings, 2 replies; 11+ messages in thread
From: Vincent Stehlé @ 2015-10-21 8:57 UTC (permalink / raw)
To: buildroot
Starting with version 2.26, sfdisk defaults to a unit of 512 B sectors.
With those recent versions of sfdisk, the create-boot-sd.sh script ends
up creating a boot partition of 240 KB, which is too small to contain a
Linux kernel.
As it is difficult to address both pre and post v2.26 sfdisk versions in
a single command, we change the create-boot-sd.sh script to use parted
instead. While at it, we set the boot partition size to 64 MB, which
should be enough for everyone.
Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
Cc: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Gary Bisson <gary.bisson@boundarydevices.com>
---
Hi,
This is a second proposal, using parted this time. Even with sfdisk
`--unit' parameter, I could not find an eleguant solution to support all
versions of sfdisk, sorry. Please let me know if you find switching to
parted acceptable.
This is tested on Sabre AI with an i.MX6 Quad.
Best regards,
V.
Changes in v2
-------------
- Switch to parted, as advised by Gary.
- Fix spelling in commit message, spotted by Luca.
board/freescale/create-boot-sd.sh | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/board/freescale/create-boot-sd.sh b/board/freescale/create-boot-sd.sh
index af45115..d80bd52 100755
--- a/board/freescale/create-boot-sd.sh
+++ b/board/freescale/create-boot-sd.sh
@@ -59,10 +59,12 @@ sync
# - Bootloader at offset 1024
# - FAT partition starting at 1MB offset, containing uImage and *.dtb
# - ext2/3 partition formatted as ext2 or ext3, containing the root filesystem.
-sfdisk ${DEV} <<EOF
-32,480,b
-512,,L
-EOF
+parted --script "${DEV}" \
+ mklabel msdos \
+ unit MiB \
+ mkpart primary fat16 1 64 \
+ mkpart primary ext2 64 100% \
+ print
sync
--
2.6.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2] board/freescale: switch create-boot-sd.sh to parted
2015-10-21 8:57 ` [Buildroot] [PATCH v2] board/freescale: switch create-boot-sd.sh to parted Vincent Stehlé
@ 2015-10-21 9:14 ` Baruch Siach
2015-10-21 18:37 ` Arnout Vandecappelle
1 sibling, 0 replies; 11+ messages in thread
From: Baruch Siach @ 2015-10-21 9:14 UTC (permalink / raw)
To: buildroot
Hi Vincent,
On Wed, Oct 21, 2015 at 10:57:06AM +0200, Vincent Stehl? wrote:
> Starting with version 2.26, sfdisk defaults to a unit of 512 B sectors.
> With those recent versions of sfdisk, the create-boot-sd.sh script ends
> up creating a boot partition of 240 KB, which is too small to contain a
> Linux kernel.
>
> As it is difficult to address both pre and post v2.26 sfdisk versions in
> a single command, we change the create-boot-sd.sh script to use parted
> instead. While at it, we set the boot partition size to 64 MB, which
> should be enough for everyone.
You should mention in the relevant readme.txt files under board/freescale/
that parted needs to be installed on the host.
Alternatively, you can enable host-parted in the config files, and use that in
create-boot-sd.sh.
> Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
> Cc: Luca Ceresoli <luca@lucaceresoli.net>
> Cc: Gary Bisson <gary.bisson@boundarydevices.com>
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2] board/freescale: switch create-boot-sd.sh to parted
2015-10-21 8:57 ` [Buildroot] [PATCH v2] board/freescale: switch create-boot-sd.sh to parted Vincent Stehlé
2015-10-21 9:14 ` Baruch Siach
@ 2015-10-21 18:37 ` Arnout Vandecappelle
2015-10-22 10:08 ` Vincent Stehlé
1 sibling, 1 reply; 11+ messages in thread
From: Arnout Vandecappelle @ 2015-10-21 18:37 UTC (permalink / raw)
To: buildroot
On 21-10-15 10:57, Vincent Stehl? wrote:
> Starting with version 2.26, sfdisk defaults to a unit of 512 B sectors.
> With those recent versions of sfdisk, the create-boot-sd.sh script ends
> up creating a boot partition of 240 KB, which is too small to contain a
> Linux kernel.
>
> As it is difficult to address both pre and post v2.26 sfdisk versions in
> a single command, we change the create-boot-sd.sh script to use parted
> instead. While at it, we set the boot partition size to 64 MB, which
> should be enough for everyone.
>
> Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
> Cc: Luca Ceresoli <luca@lucaceresoli.net>
> Cc: Gary Bisson <gary.bisson@boundarydevices.com>
>
> ---
>
>
> Hi,
>
> This is a second proposal, using parted this time. Even with sfdisk
> `--unit' parameter, I could not find an eleguant solution to support all
> versions of sfdisk, sorry. Please let me know if you find switching to
> parted acceptable.
I really don't like that we would rely on a tool installed on the host.
Is there any reason why genimage (cfr. wandboard) wouldn't work, like I
suggested earlier?
Regards,
Arnout
>
> This is tested on Sabre AI with an i.MX6 Quad.
>
> Best regards,
>
> V.
>
>
> Changes in v2
> -------------
> - Switch to parted, as advised by Gary.
> - Fix spelling in commit message, spotted by Luca.
>
>
> board/freescale/create-boot-sd.sh | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/board/freescale/create-boot-sd.sh b/board/freescale/create-boot-sd.sh
> index af45115..d80bd52 100755
> --- a/board/freescale/create-boot-sd.sh
> +++ b/board/freescale/create-boot-sd.sh
> @@ -59,10 +59,12 @@ sync
> # - Bootloader at offset 1024
> # - FAT partition starting at 1MB offset, containing uImage and *.dtb
> # - ext2/3 partition formatted as ext2 or ext3, containing the root filesystem.
> -sfdisk ${DEV} <<EOF
> -32,480,b
> -512,,L
> -EOF
> +parted --script "${DEV}" \
> + mklabel msdos \
> + unit MiB \
> + mkpart primary fat16 1 64 \
> + mkpart primary ext2 64 100% \
> + print
>
> sync
>
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2] board/freescale: switch create-boot-sd.sh to parted
2015-10-21 18:37 ` Arnout Vandecappelle
@ 2015-10-22 10:08 ` Vincent Stehlé
0 siblings, 0 replies; 11+ messages in thread
From: Vincent Stehlé @ 2015-10-22 10:08 UTC (permalink / raw)
To: buildroot
On 10/21/2015 08:37 PM, Arnout Vandecappelle wrote:
..
> Is there any reason why genimage (cfr. wandboard) wouldn't work, like I
> suggested earlier?
Hi Arnout,
You are right, genimage seems like a nice solution. In fact I think I
prefer this SD card image generation approach over the direct SD card
modification method.
Thank you for this suggestion.
Best regards,
V.
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2015-10-22 10:08 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-14 14:03 [Buildroot] Repair boot from SD card on i.MX6 Sabre boards Vincent Stehlé
2015-10-14 14:03 ` [Buildroot] [PATCH 1/2] board/freescale: refresh create-boot-sd.sh Vincent Stehlé
2015-10-14 16:04 ` Luca Ceresoli
2015-10-14 21:10 ` Arnout Vandecappelle
2015-10-15 8:30 ` Gary Bisson
2015-10-21 8:57 ` [Buildroot] [PATCH v2] board/freescale: switch create-boot-sd.sh to parted Vincent Stehlé
2015-10-21 9:14 ` Baruch Siach
2015-10-21 18:37 ` Arnout Vandecappelle
2015-10-22 10:08 ` Vincent Stehlé
2015-10-14 14:03 ` [Buildroot] [PATCH 2/2] imx6sabre: use zImage format for kernel Vincent Stehlé
2015-10-15 13:03 ` [Buildroot] Repair boot from SD card on i.MX6 Sabre boards Jeremy Rosen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox