* [Buildroot] [PATCH v2 1/9] board/olimex/a20_olinuxino/genimage.cfg: fix option suffix
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 2/9] board/altera/socrates_cyclone5/genimage.cfg: fix offset suffix Giulio Benetti
` (8 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
With commit[1] I've introduced a code style bug:
offset = 8KB
so let's substitute 8KB with 8K.
[1]: https://gitlab.com/buildroot.org/buildroot/-/commit/43bfa1cf1046b0e44e87a02bc5a358bf1de5bfdd
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
V1->V2: reworded as suggested by Luca Ceresoli
---
board/olimex/a20_olinuxino/genimage.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/olimex/a20_olinuxino/genimage.cfg b/board/olimex/a20_olinuxino/genimage.cfg
index 5a735ed2de..b48e12a831 100644
--- a/board/olimex/a20_olinuxino/genimage.cfg
+++ b/board/olimex/a20_olinuxino/genimage.cfg
@@ -5,7 +5,7 @@ image sdcard.img {
partition u-boot {
in-partition-table = "no"
image = "u-boot-sunxi-with-spl.bin"
- offset = 8KB
+ offset = 8K
size = 1016K # 1MB - 8KB
}
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* [Buildroot] [PATCH v2 2/9] board/altera/socrates_cyclone5/genimage.cfg: fix offset suffix
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 1/9] board/olimex/a20_olinuxino/genimage.cfg: fix option suffix Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 3/9] board/friendlyarm/nanopi-neo2/genimage.cfg: fix comment Giulio Benetti
` (7 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
For consistency 'offset = 256k' must be 'offset = 256K'
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
board/altera/socrates_cyclone5/genimage.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/altera/socrates_cyclone5/genimage.cfg b/board/altera/socrates_cyclone5/genimage.cfg
index 0af3f2ec60..d31c3838be 100644
--- a/board/altera/socrates_cyclone5/genimage.cfg
+++ b/board/altera/socrates_cyclone5/genimage.cfg
@@ -24,7 +24,7 @@ image uboot.img {
partition uboot-full {
in-partition-table = "no"
image = "u-boot.img"
- offset = 256k
+ offset = 256K
}
size = 1M
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* [Buildroot] [PATCH v2 3/9] board/friendlyarm/nanopi-neo2/genimage.cfg: fix comment
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 1/9] board/olimex/a20_olinuxino/genimage.cfg: fix option suffix Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 2/9] board/altera/socrates_cyclone5/genimage.cfg: fix offset suffix Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 4/9] board/orangepi/orangepi-pc2/genimage.cfg: " Giulio Benetti
` (6 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
For consistency comment '# 1MB - 40K' must be '# 1MB - 40KB'.
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
board/friendlyarm/nanopi-neo2/genimage.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/friendlyarm/nanopi-neo2/genimage.cfg b/board/friendlyarm/nanopi-neo2/genimage.cfg
index 45d081e197..3163a06592 100644
--- a/board/friendlyarm/nanopi-neo2/genimage.cfg
+++ b/board/friendlyarm/nanopi-neo2/genimage.cfg
@@ -24,7 +24,7 @@ image sdcard.img {
in-partition-table = "no"
image = "u-boot.itb"
offset = 40K
- size = 1M # 1MB - 40K
+ size = 1M # 1MB - 40KB
}
partition boot {
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* [Buildroot] [PATCH v2 4/9] board/orangepi/orangepi-pc2/genimage.cfg: fix comment
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
` (2 preceding siblings ...)
2021-11-09 10:18 ` [Buildroot] [PATCH v2 3/9] board/friendlyarm/nanopi-neo2/genimage.cfg: fix comment Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 5/9] board/minnowboard/genimage.cfg: fix option assignment spaces Giulio Benetti
` (5 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
For consistency comment '# 1MB - 40K' must be '# 1MB - 40KB'.
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
board/orangepi/orangepi-pc2/genimage.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/orangepi/orangepi-pc2/genimage.cfg b/board/orangepi/orangepi-pc2/genimage.cfg
index 72adc47518..cc8695c9f1 100644
--- a/board/orangepi/orangepi-pc2/genimage.cfg
+++ b/board/orangepi/orangepi-pc2/genimage.cfg
@@ -24,7 +24,7 @@ image sdcard.img {
in-partition-table = "no"
image = "u-boot.itb"
offset = 40K
- size = 1M # 1MB - 40K
+ size = 1M # 1MB - 40KB
}
partition boot {
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* [Buildroot] [PATCH v2 5/9] board/minnowboard/genimage.cfg: fix option assignment spaces
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
` (3 preceding siblings ...)
2021-11-09 10:18 ` [Buildroot] [PATCH v2 4/9] board/orangepi/orangepi-pc2/genimage.cfg: " Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 6/9] board/kontron/pitx-imx8m/genimage.cfg: fix option suffix Giulio Benetti
` (4 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
Option 'size=10M' must be 'size = 10M'.
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
board/minnowboard/genimage.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/minnowboard/genimage.cfg b/board/minnowboard/genimage.cfg
index 34b3070901..2eccce2de4 100644
--- a/board/minnowboard/genimage.cfg
+++ b/board/minnowboard/genimage.cfg
@@ -9,7 +9,7 @@ image efi-part.vfat {
}
}
- size=10M
+ size = 10M
}
# Create the sdcard image, pulling in
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* [Buildroot] [PATCH v2 6/9] board/kontron/pitx-imx8m/genimage.cfg: fix option suffix
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
` (4 preceding siblings ...)
2021-11-09 10:18 ` [Buildroot] [PATCH v2 5/9] board/minnowboard/genimage.cfg: fix option assignment spaces Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-09 10:22 ` Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 7/9] board/kontron/smarc-sal28/genimage.cfg: " Giulio Benetti
` (3 subsequent siblings)
9 siblings, 1 reply; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
Option 'offset = 4k' must be 'offset = 4K'.
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
board/kontron/pitx-imx8m/genimage.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/kontron/pitx-imx8m/genimage.cfg b/board/kontron/pitx-imx8m/genimage.cfg
index c335b0c026..c78ef0f910 100644
--- a/board/kontron/pitx-imx8m/genimage.cfg
+++ b/board/kontron/pitx-imx8m/genimage.cfg
@@ -5,7 +5,7 @@ image sdcard.img {
partition imx-boot {
in-partition-table = "no"
image = "imx8-boot-sd.bin"
- offset = 33k
+ offset = 33K
}
partition rootfs {
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [Buildroot] [PATCH v2 6/9] board/kontron/pitx-imx8m/genimage.cfg: fix option suffix
2021-11-09 10:18 ` [Buildroot] [PATCH v2 6/9] board/kontron/pitx-imx8m/genimage.cfg: fix option suffix Giulio Benetti
@ 2021-11-09 10:22 ` Giulio Benetti
0 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:22 UTC (permalink / raw)
To: buildroot
Cc: Michael Walle, Jagan Teki, Michał Łyszczek,
Luca Ceresoli, Heiko Thiery, Yann E . MORIN,
Thomas De Schampheleire
Sorry, I've forgotten,
On 11/9/21 11:18 AM, Giulio Benetti wrote:
> Option 'offset = 4k' must be 'offset = 4K'.
>
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
Acked-by: Heiko Thiery <heiko.thiery@gmail.com>
---
V1->V2:
* reworded 'option = 4K' to 'offset = 4K' as suggested by Heiko Thiery
> ---
> board/kontron/pitx-imx8m/genimage.cfg | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/board/kontron/pitx-imx8m/genimage.cfg b/board/kontron/pitx-imx8m/genimage.cfg
> index c335b0c026..c78ef0f910 100644
> --- a/board/kontron/pitx-imx8m/genimage.cfg
> +++ b/board/kontron/pitx-imx8m/genimage.cfg
> @@ -5,7 +5,7 @@ image sdcard.img {
> partition imx-boot {
> in-partition-table = "no"
> image = "imx8-boot-sd.bin"
> - offset = 33k
> + offset = 33K
> }
>
> partition rootfs {
>
--
Giulio Benetti
Benetti Engineering sas
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH v2 7/9] board/kontron/smarc-sal28/genimage.cfg: fix option suffix
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
` (5 preceding siblings ...)
2021-11-09 10:18 ` [Buildroot] [PATCH v2 6/9] board/kontron/pitx-imx8m/genimage.cfg: fix option suffix Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 8/9] docs/manual: add allowed notations section for genimage.cfg files Giulio Benetti
` (2 subsequent siblings)
9 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
Option 'offset = 4k' must be 'offset = 4K'.
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
board/kontron/smarc-sal28/genimage.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/kontron/smarc-sal28/genimage.cfg b/board/kontron/smarc-sal28/genimage.cfg
index f3fb77e00a..8c8a7baaae 100644
--- a/board/kontron/smarc-sal28/genimage.cfg
+++ b/board/kontron/smarc-sal28/genimage.cfg
@@ -4,7 +4,7 @@ image sdcard-emmc.img {
}
partition rcw {
- offset = 4k
+ offset = 4K
in-partition-table = "no"
image = "rcw.bin"
}
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* [Buildroot] [PATCH v2 8/9] docs/manual: add allowed notations section for genimage.cfg files
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
` (6 preceding siblings ...)
2021-11-09 10:18 ` [Buildroot] [PATCH v2 7/9] board/kontron/smarc-sal28/genimage.cfg: " Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-09 10:18 ` [Buildroot] [PATCH v2 9/9] board/nezha/genimage.cfg: fix offset value notation Giulio Benetti
2021-11-14 14:42 ` [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Thomas Petazzoni
9 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
Add allowed notations section for genimage.cfg files by specifying the
accepted suffix or prefix. Note that "K" has been decided to specify
KiloBytes as 1024 bytes. This has been done because in Buildroot the 90%
of the cases were using "K" and not "k".
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
docs/manual/writing-rules.txt | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/docs/manual/writing-rules.txt b/docs/manual/writing-rules.txt
index 28cb0fe4f1..138c39db14 100644
--- a/docs/manual/writing-rules.txt
+++ b/docs/manual/writing-rules.txt
@@ -192,6 +192,11 @@ image sdimage.img {
* Filename must at least begin with genimage prefix and have the .cfg
extension to be easy to recognize.
+* Allowed notations for +offset+ and +size+ options are: +G+, +M+, +K+(not k).
+ If it's not possible to express a precise byte count with notations above
+ then use hexadecimal +0x+ prefix or, as last chance, the byte count.
+ In comments instead use +GB+, +MB+, +KB+(not kb) in place of +G+, +M+, +K+.
+
The +genimage.cfg+ files are the input for the genimage tool used in
Buildroot to generate the final image file(i.e. sdcard.img). For further
details about the _genimage_ language, refer to
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* [Buildroot] [PATCH v2 9/9] board/nezha/genimage.cfg: fix offset value notation
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
` (7 preceding siblings ...)
2021-11-09 10:18 ` [Buildroot] [PATCH v2 8/9] docs/manual: add allowed notations section for genimage.cfg files Giulio Benetti
@ 2021-11-09 10:18 ` Giulio Benetti
2021-11-14 14:42 ` [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Thomas Petazzoni
9 siblings, 0 replies; 12+ messages in thread
From: Giulio Benetti @ 2021-11-09 10:18 UTC (permalink / raw)
To: buildroot
Cc: Giulio Benetti, Michael Walle, Jagan Teki,
Michał Łyszczek, Luca Ceresoli, Heiko Thiery,
Yann E . MORIN, Thomas De Schampheleire
Option "offset = 12288K" must be "offset = 12M" according to the new
guidelines specified in Buildroot Manual with previous patch.
Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
board/nezha/genimage.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/nezha/genimage.cfg b/board/nezha/genimage.cfg
index 9860b83eca..783391fb99 100644
--- a/board/nezha/genimage.cfg
+++ b/board/nezha/genimage.cfg
@@ -19,7 +19,7 @@ image sdcard.img {
partition u-boot-1 {
in-partition-table = "no"
image = "u-boot.toc1"
- offset = 12288K
+ offset = 12M
}
partition u-boot-2 {
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify
2021-11-09 10:18 [Buildroot] [PATCH v2 0/9] Post genimage.cfg beautify Giulio Benetti
` (8 preceding siblings ...)
2021-11-09 10:18 ` [Buildroot] [PATCH v2 9/9] board/nezha/genimage.cfg: fix offset value notation Giulio Benetti
@ 2021-11-14 14:42 ` Thomas Petazzoni
9 siblings, 0 replies; 12+ messages in thread
From: Thomas Petazzoni @ 2021-11-14 14:42 UTC (permalink / raw)
To: Giulio Benetti
Cc: Michael Walle, Jagan Teki, Michał Łyszczek, buildroot,
Luca Ceresoli, Heiko Thiery, Yann E . MORIN,
Thomas De Schampheleire
On Tue, 9 Nov 2021 11:18:35 +0100
Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
> Giulio Benetti (9):
> board/olimex/a20_olinuxino/genimage.cfg: fix option suffix
> board/altera/socrates_cyclone5/genimage.cfg: fix offset suffix
> board/friendlyarm/nanopi-neo2/genimage.cfg: fix comment
> board/orangepi/orangepi-pc2/genimage.cfg: fix comment
> board/minnowboard/genimage.cfg: fix option assignment spaces
> board/kontron/pitx-imx8m/genimage.cfg: fix option suffix
> board/kontron/smarc-sal28/genimage.cfg: fix option suffix
> docs/manual: add allowed notations section for genimage.cfg files
> board/nezha/genimage.cfg: fix offset value notation
Series applied to next, with some really minor adjustments on two
patches.
Thanks!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 12+ messages in thread