From: Sascha Hauer <s.hauer@pengutronix.de>
To: Barebox List <barebox@lists.infradead.org>
Subject: [PATCH v2 2/2] partitions: dos: change first partition number from 1 to 0
Date: Thu, 29 Feb 2024 08:18:25 +0100 [thread overview]
Message-ID: <20240229071825.1624411-2-s.hauer@pengutronix.de> (raw)
In-Reply-To: <20240229071825.1624411-1-s.hauer@pengutronix.de>
Traditionally barebox starts partition numbers at 0. This was changed
in 8f48e6366c to start at 1. This was done to make the parted numbering
consistent to the Linux tool. This breaks boot scripts though which
assume 0 to be the first partition, so change it back.
Fixes: 8f48e6366c ("partitions: dos: implement partition manipulation support")
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
Notes:
Changes since v1:
- fix partuuid number passed to Linux
common/partitions/dos.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/common/partitions/dos.c b/common/partitions/dos.c
index 47b5764124..1b0051f978 100644
--- a/common/partitions/dos.c
+++ b/common/partitions/dos.c
@@ -127,7 +127,7 @@ static void dos_extended_partition(struct block_device *blk, struct dos_partitio
uint8_t *buf = malloc(SECTOR_SIZE);
uint32_t ebr_sector = partition->first_sec;
struct partition_entry *table = (struct partition_entry *)&buf[0x1be];
- unsigned partno = 5;
+ unsigned partno = 4;
struct dos_partition *dpart;
struct partition *pentry;
@@ -170,7 +170,7 @@ static void dos_extended_partition(struct block_device *blk, struct dos_partitio
pentry->size = get_unaligned_le32(&table[0].partition_size);
pentry->dos_partition_type = table[0].type;
pentry->num = partno;
- sprintf(pentry->partuuid, "%08x-%02u", signature, partno);
+ sprintf(pentry->partuuid, "%08x-%02u", signature, partno + 1);
list_add_tail(&pentry->list, &dpd->pd.partitions);
@@ -237,7 +237,7 @@ static struct partition_desc *dos_partition(void *buf, struct block_device *blk)
pentry->first_sec = first_sec;
pentry->size = get_unaligned_le32(&table[i].partition_size);
pentry->dos_partition_type = table[i].type;
- pentry->num = i + 1;
+ pentry->num = i;
sprintf(pentry->partuuid, "%08x-%02d", signature, i + 1);
dpd->signature = signature;
--
2.39.2
next prev parent reply other threads:[~2024-02-29 7:18 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-29 7:18 [PATCH v2 1/2] partitions: efi: " Sascha Hauer
2024-02-29 7:18 ` Sascha Hauer [this message]
2024-03-01 8:15 ` [PATCH v2 2/2] partitions: dos: " Ahmad Fatoum
2024-03-01 9:21 ` [PATCH v2 1/2] partitions: efi: " Sascha Hauer
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=20240229071825.1624411-2-s.hauer@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=barebox@lists.infradead.org \
/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