Implement a "part name" subcommand, mirroring the existing "part number" subcommand.
In the discussion for v1 of that, it came up that there's a bit of inconsistency in how much and what one can assume to be initialized in 'struct disk_partition' after a successful call of one of the get_info* family of functions. Patch 1/2 tries to consolidate that by making sure all ->get_info invocations go through a common helper that at least always initializes the string members. Quentin, I've taken the liberty of including your Acks, as the incremental diff in patch 1 is quite minor, but do speak up if I should not have done that. v3: Make sure part_get_type_by_name() can only ever return 0, -ENOSYS (in case ->get_info is not implemented) and -ENOENT, rather than anything that ->get_info might return. v2: https://lore.kernel.org/u-boot/[email protected]/ Rasmus Villemoes (2): disk/part.c: ensure strings in struct disk_partition are valid after successful get_info cmd/part.c: implement "part name" subcommand cmd/gpt.c | 4 +-- cmd/part.c | 16 ++++++++++- disk/part.c | 63 +++++++++++++++++++++++++----------------- doc/usage/cmd/part.rst | 13 +++++++++ include/part.h | 16 +++++++++++ 5 files changed, 83 insertions(+), 29 deletions(-) -- 2.51.0
