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)