Re: [PATCH net-next v3 2/3] net: Add dev_getbyhwaddr_rtnl() helper
From: Kuniyuki Iwashima
Date: Thu Feb 13 2025 - 02:31:55 EST
> Subject: [PATCH net-next v3 2/3] net: Add dev_getbyhwaddr_rtnl() helper
s/_rtnl//
looks like Uday's comment was missed due to the lore issue.
From: Breno Leitao <leitao@xxxxxxxxxx>
Date: Wed, 12 Feb 2025 09:47:25 -0800
> +/**
> + * dev_getbyhwaddr - find a device by its hardware address
While at it, could you replace '\t' after '*' to a single '\s'
for all kernel-doc comment lines below ?
> + * @net: the applicable net namespace
> + * @type: media type of device
> + * @ha: hardware address
> + *
> + * Similar to dev_getbyhwaddr_rcu(), but the owner needs to hold
> + * rtnl_lock.
Otherwise the text here is mis-aligned.
$ ./scripts/kernel-doc -man net/core/dev.c | \
scripts/split-man.pl /tmp/man && \
man /tmp/man/dev_getbyhwaddr.9
Also, the latter part should be in Context:
Context: rtnl_lock() must be held.
See https://docs.kernel.org/doc-guide/kernel-doc.html
> + *
> + * Return: pointer to the net_device, or NULL if not found
> + */
> +struct net_device *dev_getbyhwaddr(struct net *net, unsigned short type,
> + const char *ha)