[PATCH] net: appletalk: fix some mistakes in grammar

From: gushengxian <redacted>

Fix some mistakes in grammar.

Signed-off-by: gushengxian <redacted>
---
 net/appletalk/ddp.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/net/appletalk/ddp.c b/net/appletalk/ddp.c
index ebda397fa95a..bc76b2fa3dfb 100644
--- a/net/appletalk/ddp.c
+++ b/net/appletalk/ddp.c

@@ -707,7 +707,7 @@ static int atif_ioctl(int cmd, void __user *arg)
 
 		/*
 		 * Phase 1 is fine on LocalTalk but we don't do
-		 * EtherTalk phase 1. Anyone wanting to add it go ahead.
+		 * EtherTalk phase 1. Anyone wanting to add it goes ahead.
 		 */
 		if (dev->type == ARPHRD_ETHER && nr->nr_phase != 2)
 			return -EPROTONOSUPPORT;

@@ -828,7 +828,7 @@ static int atif_ioctl(int cmd, void __user *arg)
 		nr = (struct atalk_netrange *)&(atif->nets);
 		/*
 		 * Phase 1 is fine on Localtalk but we don't do
-		 * Ethertalk phase 1. Anyone wanting to add it go ahead.
+		 * Ethertalk phase 1. Anyone wanting to add it goes ahead.
 		 */
 		if (dev->type == ARPHRD_ETHER && nr->nr_phase != 2)
 			return -EPROTONOSUPPORT;

@@ -2018,7 +2018,7 @@ module_init(atalk_init);
  * by the network device layer.
  *
  * Ergo, before the AppleTalk module can be removed, all AppleTalk
- * sockets be closed from user space.
+ * sockets should be closed from user space.
  */
 static void __exit atalk_exit(void)
 {

-- 
2.25.1

On Mon, 2021-06-07 at 19:25 -0700, 13145886936@163.com wrote:

From: gushengxian <redacted>

Fix some mistakes in grammar.

[]

[]

This is really not better grammar.
It's describing how anyone should feel free to create an implementation.

It's also really old code that no one will use much anymore, so any
change to this module isn't particularly useful.

On 6/7/21 7:25 PM, 13145886936@163.com wrote:

Nak on these 2 changes.
If anything, it could be something like:
		                      Anyone wanting to add it, go ahead.
or
		                      If anyone wants to add it, go ahead.

ok.

  */
 static void __exit atalk_exit(void)
 {

-- 
~Randy

13145886936@163.com writes:

  * Ergo, before the AppleTalk module can be removed, all AppleTalk
- * sockets be closed from user space.
+ * sockets should be closed from user space.
  */

This is a good correction, but in the other case

 		/*
 		 * Phase 1 is fine on LocalTalk but we don't do
-		 * EtherTalk phase 1. Anyone wanting to add it go ahead.
+		 * EtherTalk phase 1. Anyone wanting to add it goes ahead.

"go ahead" is meant as an imperative (like "If you want to add EtherTalk
phase 1 support, please go ahead [and do so]"), not an indicative (like
"If you want to add EtherTalk phase 1 support, you are adding it").  It
is an invitation addressed to future developers.

Addressing unspecified people directly in the second person with "anyone"
is a little unusual, but is grammatically acceptable (more usually with a
comma).  Anyone reading this, you now understand this point.