On Mon, Jan 16, 2023 at 03:55:44PM +0000, Jon Hunter wrote:
> The print in in gether_setup_name_default() does not provide any useful
> information because a random MAC address will always be generated when
> calling this function. Rather than removing the print, update the print
> to show MAC address that is generated which is similar to other ethernet
> drivers.
>
> Finally, given that the strings 'self' and 'host' are static we do not
> need to pass these strings as an arguments.
>
> Signed-off-by: Jon Hunter <jonathanh@xxxxxxxxxx>
> ---
> V1 -> V2: Added this patch
>
> drivers/usb/gadget/function/u_ether.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/gadget/function/u_ether.c b/drivers/usb/gadget/function/u_ether.c
> index 8f12f3f8f6ee..be8e7b448933 100644
> --- a/drivers/usb/gadget/function/u_ether.c
> +++ b/drivers/usb/gadget/function/u_ether.c
> @@ -845,13 +845,13 @@ struct net_device *gether_setup_name_default(const char *netname)
> snprintf(net->name, sizeof(net->name), "%s%%d", netname);
>
> eth_random_addr(dev->dev_mac);
> - pr_warn("using random %s ethernet address\n", "self");
> + pr_warn("using random self ethernet address %pM\n", dev->dev_mac);
If you are going to fix these up, please use dev_warn() and friends,
don't use "raw" pr_* calls in a driver.
thanks,
greg k-h
