staging: r8188eu: remove comments about endpoint mapping
authorMartin Kaiser <martin@kaiser.cx>
Sat, 6 Aug 2022 19:55:34 +0000 (21:55 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 16 Aug 2022 13:17:57 +0000 (15:17 +0200)
Remove the comments in two_out_pipe and three_out_pipe that show the
mappings. They simply repeat the settings in the code and provide no
additional information. Keep the info which RtOutPipe is high, normal
or low.

Without the removed comments, it'll be easier to summarize and reorganize
the code.

Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20220806195540.777390-8-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/hal/usb_halinit.c

index 664028c14141475cebe3fec9546e479daf26cd03..e561c92f1dc95e47142ead7c752d533eba2489a7 100644 (file)
@@ -32,11 +32,9 @@ static void two_out_pipe(struct adapter *adapter, bool wifi_cfg)
 {
        struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(adapter);
 
-       if (wifi_cfg) { /* WMM */
-               /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
-               /*  0,  1,  0,  1,   0,   0,   0,    0,    0}; */
-               /* 0:H, 1:L */
+       /* 0:H, 1:L */
 
+       if (wifi_cfg) {
                pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[1];/* VO */
                pdvobjpriv->Queue2Pipe[1] = pdvobjpriv->RtOutPipe[0];/* VI */
                pdvobjpriv->Queue2Pipe[2] = pdvobjpriv->RtOutPipe[1];/* BE */
@@ -47,11 +45,7 @@ static void two_out_pipe(struct adapter *adapter, bool wifi_cfg)
                pdvobjpriv->Queue2Pipe[6] = pdvobjpriv->RtOutPipe[0];/* HIGH */
                pdvobjpriv->Queue2Pipe[7] = pdvobjpriv->RtOutPipe[0];/* TXCMD */
 
-       } else {/* typical setting */
-               /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
-               /*  1,  1,  0,  0,   0,   0,   0,    0,    0}; */
-               /* 0:H, 1:L */
-
+       } else {
                pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[0];/* VO */
                pdvobjpriv->Queue2Pipe[1] = pdvobjpriv->RtOutPipe[0];/* VI */
                pdvobjpriv->Queue2Pipe[2] = pdvobjpriv->RtOutPipe[1];/* BE */
@@ -68,11 +62,9 @@ static void three_out_pipe(struct adapter *adapter, bool wifi_cfg)
 {
        struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(adapter);
 
-       if (wifi_cfg) {/* for WMM */
-               /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
-               /*  1,  2,  1,  0,   0,   0,   0,    0,    0}; */
-               /* 0:H, 1:N, 2:L */
+       /* 0:H, 1:N, 2:L */
 
+       if (wifi_cfg) {
                pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[0];/* VO */
                pdvobjpriv->Queue2Pipe[1] = pdvobjpriv->RtOutPipe[1];/* VI */
                pdvobjpriv->Queue2Pipe[2] = pdvobjpriv->RtOutPipe[2];/* BE */
@@ -83,11 +75,7 @@ static void three_out_pipe(struct adapter *adapter, bool wifi_cfg)
                pdvobjpriv->Queue2Pipe[6] = pdvobjpriv->RtOutPipe[0];/* HIGH */
                pdvobjpriv->Queue2Pipe[7] = pdvobjpriv->RtOutPipe[0];/* TXCMD */
 
-       } else {/* typical setting */
-               /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
-               /*  2,  2,  1,  0,   0,   0,   0,    0,    0}; */
-               /* 0:H, 1:N, 2:L */
-
+       } else {
                pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[0];/* VO */
                pdvobjpriv->Queue2Pipe[1] = pdvobjpriv->RtOutPipe[1];/* VI */
                pdvobjpriv->Queue2Pipe[2] = pdvobjpriv->RtOutPipe[2];/* BE */