diff mbox series

cmd: ubi: add a command to rename volume

Message ID 1584644739-10258-1-git-send-email-philippe.reynes@softathome.com
State Superseded
Headers show
Series cmd: ubi: add a command to rename volume | expand

Commit Message

Philippe REYNES March 19, 2020, 7:05 p.m. UTC
This commit add the command ubi rename to rename an ubi volume.
The format of the command is: ubi rename <oldname> <newname>.

Signed-off-by: Philippe Reynes <philippe.reynes at softathome.com>
---
 cmd/ubi.c | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

Comments

Wolfgang Denk March 19, 2020, 8:40 p.m. UTC | #1
Dear Philippe,

In message <1584644739-10258-1-git-send-email-philippe.reynes at softathome.com> you wrote:
> This commit add the command ubi rename to rename an ubi volume.
> The format of the command is: ubi rename <oldname> <newname>.

Can we plase make this optional / configurable?

Thanks!

Wolfgang Denk
Philippe REYNES March 20, 2020, 9:44 a.m. UTC | #2
Hi Wolfgang,

> Dear Philippe,
> 
> In message <1584644739-10258-1-git-send-email-philippe.reynes at softathome.com>
> you wrote:
>> This commit add the command ubi rename to rename an ubi volume.
>> The format of the command is: ubi rename <oldname> <newname>.
> 
> Can we plase make this optional / configurable?

Yes, of course, I sent a v2 with an option CMD_UBI_RENAME.
 
> Thanks!
> 
> Wolfgang Denk

Regards,
Philippe
diff mbox series

Patch

diff --git a/cmd/ubi.c b/cmd/ubi.c
index cecf251..424f555 100644
--- a/cmd/ubi.c
+++ b/cmd/ubi.c
@@ -251,6 +251,39 @@  out_err:
 	return err;
 }
 
+static int ubi_rename_vol(char *oldname, char *newname)
+{
+	struct ubi_volume *vol;
+	struct ubi_rename_entry rename;
+	struct ubi_volume_desc desc;
+	struct list_head list;
+
+	vol = ubi_find_volume(oldname);
+	if (!vol) {
+		printf("%s: volume %s doesn't exist\n", __func__, oldname);
+		return ENODEV;
+	}
+
+	printf("Rename UBI volume %s to %s\n", oldname, newname);
+
+	if (ubi->ro_mode) {
+		printf("%s: ubi device is in read-only mode\n", __func__);
+		return EROFS;
+	}
+
+	rename.new_name_len = strlen(newname);
+	strcpy(rename.new_name, newname);
+	rename.remove = 0;
+	desc.vol = vol;
+	desc.mode = 0;
+	rename.desc = &desc;
+	INIT_LIST_HEAD(&rename.list);
+	INIT_LIST_HEAD(&list);
+	list_add(&rename.list, &list);
+
+	return ubi_rename_volumes(ubi, &list);
+}
+
 static int ubi_volume_continue_write(char *volume, void *buf, size_t size)
 {
 	int err = 1;
@@ -604,6 +637,9 @@  static int do_ubi(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 			return ubi_remove_vol(argv[2]);
 	}
 
+	if (strncmp(argv[1], "rename", 6) == 0)
+		return ubi_rename_vol(argv[2], argv[3]);
+
 	if (strncmp(argv[1], "skipcheck", 9) == 0) {
 		/* E.g., change skip_check flag */
 		if (argc == 4) {
@@ -693,6 +729,7 @@  U_BOOT_CMD(
 	"ubi remove[vol] volume"
 		" - Remove volume\n"
 	"ubi skipcheck volume on/off - Set or clear skip_check flag in volume header\n"
+	"ubi rename oldname newname\n"
 	"[Legends]\n"
 	" volume: character name\n"
 	" size: specified in bytes\n"