On Thu, Sep 25, 2025 at 09:20:53AM +0300, Ilias Apalodimas wrote: > Hi Javier, > > On Thu, 25 Sept 2025 at 02:31, Javier Tia <[email protected]> wrote: > > > > Make part_driver_lookup_type non-static so it can be used outside > > part.c. This allows external callers to determine the appropriate > > partition driver for a block device, enabling more flexible handling of > > partition types. > > > > Add a prototype and kernel-doc comment in part.h to document the > > function contract. Provide a stub inline implementation returning NULL > > when partition support is disabled, ensuring build consistency across > > configurations. > > > > Signed-off-by: Javier Tia <[email protected]> > > --- > > disk/part.c | 2 +- > > include/part.h | 21 +++++++++++++++++++++ > > 2 files changed, 22 insertions(+), 1 deletion(-) > > > > diff --git a/disk/part.c b/disk/part.c > > index 66e2b3a7219..f0c6866e210 100644 > > --- a/disk/part.c > > +++ b/disk/part.c > > @@ -63,7 +63,7 @@ static struct part_driver *part_driver_get_type(int > > part_type) > > * @dev_desc: Device descriptor > > * Return: Driver found, or NULL if none > > */ > > -static struct part_driver *part_driver_lookup_type(struct blk_desc *desc) > > +struct part_driver *part_driver_lookup_type(struct blk_desc *desc)
Since Ilias has a comment, I'll also note that with the kerneldoc moving to the header, the version in the C file should be removed (this means they won't get out of sync). -- Tom
signature.asc
Description: PGP signature
