[PATCH] addpart: size and offset not needed for already defined partitions

Carlo Caione carlo.caione at gmail.com
Fri Feb 15 13:09:01 EST 2013


You can use the name of an already defined partition
(i.e. hardcoded in board specific files) without specifying
size and offset that are automatically obtained

i.e.

$ addpart /dev/nor0 (self)
$ addpart /dev/nor0 (env0)

Signed-off-by: Carlo Caione <carlo.caione at gmail.com>
---
 commands/partition.c | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/commands/partition.c b/commands/partition.c
index fdd6041..8cb50ca 100644
--- a/commands/partition.c
+++ b/commands/partition.c
@@ -53,6 +53,7 @@ static int mtd_part_do_parse_one(char *devname, const char *partstr,
 	char buf[PATH_MAX] = {};
 	unsigned long flags = 0;
 	int ret;
+	struct cdev *cdev;
 
 	memset(buf, 0, PATH_MAX);
 
@@ -63,7 +64,7 @@ static int mtd_part_do_parse_one(char *devname, const char *partstr,
 		size = strtoul_suffix(partstr, &end, 0);
 	}
 
-	if (*end == '@')
+	if ((*end == '@') && (size != 0))
 		*offset = strtoul_suffix(end+1, &end, 0);
 
 	if (size == SIZE_REMAINING)
@@ -78,6 +79,18 @@ static int mtd_part_do_parse_one(char *devname, const char *partstr,
 			printf("could not find matching ')'\n");
 			return -EINVAL;
 		}
+		memcpy(buf, partstr, end - partstr);
+
+		if (size == 0) {
+			cdev = cdev_by_name(buf);
+			if (!cdev) {
+				printf("%s: partition not found\n", buf);
+				return -EINVAL;
+			}
+
+			size = cdev->size;
+			*offset = cdev->offset;
+		}
 
 		if (pflags & PART_ADD_DEVNAME)
 			sprintf(buf, "%s.", devname);
-- 
1.8.1.3




More information about the barebox mailing list