[PATCH v3 0/2] 'part name' subcommand and some robustification

Rasmus Villemoes ravi at prevas.dk
Mon Nov 10 21:54:09 CET 2025


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/20251020121100.1742812-1-ravi@prevas.dk/

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



More information about the U-Boot mailing list