Re: [PATCH][next] netdevsim: fix spelling mistake "forbidded" -> "forbid"

From: Maciej W. Rozycki
Date: Tue Oct 08 2019 - 04:30:04 EST


On Tue, 8 Oct 2019, Colin King wrote:

> diff --git a/drivers/net/netdevsim/dev.c b/drivers/net/netdevsim/dev.c
> index a3d7d39f231a..ff6ced5487b6 100644
> --- a/drivers/net/netdevsim/dev.c
> +++ b/drivers/net/netdevsim/dev.c
> @@ -486,7 +486,7 @@ static int nsim_dev_reload_down(struct devlink *devlink, bool netns_change,
> /* For testing purposes, user set debugfs dont_allow_reload
> * value to true. So forbid it.
> */
> - NL_SET_ERR_MSG_MOD(extack, "User forbidded reload for testing purposes");
> + NL_SET_ERR_MSG_MOD(extack, "User forbid the reload for testing purposes");

If nitpicking about grammar, then FWIW I believe it should actually be:

NL_SET_ERR_MSG_MOD(extack, "User forbade the reload for testing purposes");

(and then:

NL_SET_ERR_MSG_MOD(extack, "User set up the reload to fail for testing purposes");

elsewhere).

Maciej