Thanks, applied as 40a39c7a94d1f8821fab5a09a827937e00fe62af. Michael
[sent from post-receive hook] On Fri, 24 Mar 2023 10:03:34 +0100, Roland Hieber <r...@pengutronix.de> wrote: > Currently, choosing IMAGE_HDIMG_GPT together with IMAGE_HDIMG_BAREBOX > leads to an error in genimage because the full GPT array is larger than > the hole in the barebox image. However, the hole in the barebox images > is large enough to fit the protective MBR and the primary GPT header. > The rest of the GPT array needs to moved after the barebox image, so set > the gpt-location setting to the size of the barebox image. > > For the other bootloaders, nothing needs to change, so set the > gpt-location to its default value of 1024. > > For MBR, this config setting is ignored. > > Link: https://www.barebox.org/doc/latest/boards/imx.html#using-gpt-on-i-mx > Signed-off-by: Roland Hieber <r...@pengutronix.de> > Message-Id: <20230316165132.423488-1-...@pengutronix.de> > Signed-off-by: Michael Olbrich <m.olbr...@pengutronix.de> > > diff --git a/config/images/barebox_partitions.config > b/config/images/barebox_partitions.config > index a5c55e69e952..114b461d9099 100644 > --- a/config/images/barebox_partitions.config > +++ b/config/images/barebox_partitions.config > @@ -1,6 +1,6 @@ > partition barebox { > in-partition-table = false > image = "barebox-image" > - holes = {"(440; 512)"} > + holes = {"(440; 1024)"} > size = 2M > } > diff --git a/config/images/hd.config b/config/images/hd.config > index d33ddc43490a..0ae3c27b46b7 100644 > --- a/config/images/hd.config > +++ b/config/images/hd.config > @@ -5,6 +5,7 @@ image @IMAGE@ { > hdimage { > align = 1M > partition-table-type = "@PARTITION_TABLE_TYPE@" > + gpt-location = "@GPT_LOCATION@" > } > @BOOTLOADER_PARTITIONS@ > partition root { > diff --git a/rules/image-hdimg.make b/rules/image-hdimg.make > index b470c0fb6b1a..02f7f84bfcad 100644 > --- a/rules/image-hdimg.make > +++ b/rules/image-hdimg.make > @@ -30,17 +30,20 @@ IMAGE_HDIMG_BOOTLOADER_ENV := \ > > ifdef PTXCONF_IMAGE_HDIMG_GRUB > IMAGE_HDIMG_BOOTLOADER_ENV = \ > + GPT_LOCATION='1024' \ > GRUB_STAGE_DIR=$(GRUB_STAGE_DIR) \ > BOOTLOADER_IMAGES='include("grub.config")' \ > BOOTLOADER_PARTITIONS='include("grub_partitions.config")' > endif > ifdef PTXCONF_IMAGE_HDIMG_BAREBOX > IMAGE_HDIMG_BOOTLOADER_ENV := \ > + GPT_LOCATION='2M' \ > BOOTLOADER_IMAGES='' \ > BOOTLOADER_PARTITIONS='include("barebox_partitions.config")' > endif > ifdef PTXCONF_IMAGE_HDIMG_VFAT > IMAGE_HDIMG_BOOTLOADER_ENV := \ > + GPT_LOCATION='1024' \ > BOOTLOADER_IMAGES='' \ > BOOTLOADER_PARTITIONS='include("vfat_partitions.config")' >