DistroKit Mailinglist
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: distrokit@pengutronix.de
Cc: Ahmad Fatoum <a.fatoum@pengutronix.de>
Subject: [DistroKit] [PATCH] v7a: images: stm32mp: remove cargo-culted partition-type in GPT config
Date: Wed, 10 Jun 2020 13:32:08 +0200	[thread overview]
Message-ID: <20200610113207.31084-1-a.fatoum@pengutronix.de> (raw)

partition-type is specific to DOS partitions. The counterpart for GPT
partitions would be partition-type-uuid, which already defaults to "L",
which is ok for our purposes.

So far, this didn't have an impact, but starting with genimage v13,
commit 013b22f26b73 ("Add support for Hybrid MBR partitions"), support
for a partition-type key in a GPT partition was introduced:

  Using this option with a GPT partition table will create a hybrid
  MBR partition  table with a maximum of 3 partition entries(this
  limit does not effect the maximum number of GPT partition entries
  in the same image).

The ARM TF-A can't cope with this new format and fails to boot barebox
with a "ERROR:   Partition ssbl not found" message. Drop partition-type
to prevent the ptxdist 2020.06.0 upgrade that includes host-genimage v13
from breaking the stm32mp157c-dk2 and lxa-mc1 images.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
Hello Robert,

please apply before the v2020.06.0 ptxdist upgrade sitting in next
to avoid intermittent breakage.

Thanks,
---
 configs/platform-v7a/config/images/stm32mp.config | 2 --
 1 file changed, 2 deletions(-)

diff --git a/configs/platform-v7a/config/images/stm32mp.config b/configs/platform-v7a/config/images/stm32mp.config
index 0b9cfe888791..c10632013127 100644
--- a/configs/platform-v7a/config/images/stm32mp.config
+++ b/configs/platform-v7a/config/images/stm32mp.config
@@ -20,11 +20,9 @@ image @IMAGE@ {
 	}
 	partition root {
 		image = root.ext2
-		partition-type = 0x83
 		size = 512M
 	}
 	partition data {
-		partition-type = 0x83
 		size = 100M
 	}
 }
-- 
2.27.0


_______________________________________________
DistroKit mailing list
DistroKit@pengutronix.de

             reply	other threads:[~2020-06-10 11:32 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-10 11:32 Ahmad Fatoum [this message]
2020-06-12 13:24 ` Robert Schwebel

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=20200610113207.31084-1-a.fatoum@pengutronix.de \
    --to=a.fatoum@pengutronix.de \
    --cc=distrokit@pengutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox