From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: Sergey Matyukevich <geomatsi@gmail.com>
Cc: Alistair Francis <alistair@alistair23.me>,
Gerome Burlats <gerome.burlats@smile.fr>,
Michael Walle <michael@walle.cc>,
Jagan Teki <jagan@amarulasolutions.com>,
Suniel Mahesh <sunil@amarulasolutions.com>,
Romain Naour <romain.naour@gmail.com>,
buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 2/2] board/*/genimage.cfg: use partition-table-type = "gpt" for genimage-15
Date: Mon, 27 Dec 2021 15:43:08 +0100 [thread overview]
Message-ID: <20211227154308.1bcc4a4d@windsurf> (raw)
In-Reply-To: <YcchfJG1EDtyepcy@curiosity>
Hello Sergey,
On Sat, 25 Dec 2021 16:49:48 +0300
Sergey Matyukevich <geomatsi@gmail.com> wrote:
> Build still fails for orangepi-zero-plus2 with this patch. I get the
> following subsequent error messages from genimage:
>
> ERROR: hdimage(sdcard.img): part boot: 'partition-type' is only valid for mbr and hybrid partition-table-type
>
> After changing 'partition-type' entries to the appropriate 'partition-type-uuid',
> genimage complains about overlaps:
>
> ERROR: hdimage(sdcard.img): partition [GPT array] (offset 0x100000, size 0x4000) overlaps previous partition u-boot (offset 0x2000, size 0x102000)
> ERROR: hdimage(sdcard.img): bootloaders, etc. that overlap with the partition table must declare the overlapping area as a hole.
>
> It looks like new genimage is able to keep track of partition table.
> IIUC this can be fixed either using new 'holes' feature or just updating
> size of the bootloader file. For instance, with the following patch
> applied orangepi-pc-plus builds and boots fine:
Have you looked at
https://patchwork.ozlabs.org/project/buildroot/patch/20211219142035.6220-1-peter@korsgaard.com/
which is PATCH 1/2 of this series? It fixes exactly the problem you're
pointing to, just for a different board.
Do not hesitate to send patches to fix other boards.
Thanks a lot!
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
next prev parent reply other threads:[~2021-12-27 14:43 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-19 14:20 [Buildroot] [PATCH 1/2] board/orangepi/orangepi-r1/genimage.cfg: fix build after move to genimage 15 Peter Korsgaard
2021-12-19 14:20 ` [Buildroot] [PATCH 2/2] board/*/genimage.cfg: use partition-table-type = "gpt" for genimage-15 Peter Korsgaard
2021-12-25 13:49 ` Sergey Matyukevich
2021-12-25 13:58 ` Sergey Matyukevich
2021-12-27 14:43 ` Thomas Petazzoni [this message]
2021-12-27 14:43 ` Thomas Petazzoni
2021-12-27 14:43 ` [Buildroot] [PATCH 1/2] board/orangepi/orangepi-r1/genimage.cfg: fix build after move to genimage 15 Thomas Petazzoni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211227154308.1bcc4a4d@windsurf \
--to=thomas.petazzoni@bootlin.com \
--cc=alistair@alistair23.me \
--cc=buildroot@buildroot.org \
--cc=geomatsi@gmail.com \
--cc=gerome.burlats@smile.fr \
--cc=jagan@amarulasolutions.com \
--cc=michael@walle.cc \
--cc=romain.naour@gmail.com \
--cc=sunil@amarulasolutions.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.