Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1304775 > unrolled thread

[PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block comment coding style

Back to article view | Back to linux.kernel

Contents

  [PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block comment coding style Pranjal Bhor <bhor.pranjal@gmail.com> - 2016-01-08 18:50 +0100
    Re: [PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block  comment coding style Dan Carpenter <dan.carpenter@oracle.com> - 2016-01-08 19:20 +0100
      Re: [PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block  comment coding style Pranjal <bhor.pranjal@gmail.com> - 2016-01-08 20:00 +0100
        Re: [PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block  comment coding style Dan Carpenter <dan.carpenter@oracle.com> - 2016-01-08 20:00 +0100

#1304775 — [PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block comment coding style

Block comments now end with '*/' on a newline and all lines
of block comments now begin with '*'

Signed-off-by: Pranjal Bhor <bhor.pranjal@gmail.com>
---
 drivers/staging/wlan-ng/p80211conv.c | 37 ++++++++++++++++++++++--------------
 1 file changed, 23 insertions(+), 14 deletions(-)

diff --git a/drivers/staging/wlan-ng/p80211conv.c b/drivers/staging/wlan-ng/p80211conv.c
index 1b02cdf..8175f17 100644
--- a/drivers/staging/wlan-ng/p80211conv.c
+++ b/drivers/staging/wlan-ng/p80211conv.c
@@ -49,7 +49,8 @@
 *
 * --------------------------------------------------------------------
 *
-*================================================================ */
+*================================================================
+*/

 #include <linux/module.h>
 #include <linux/kernel.h>
@@ -101,7 +102,8 @@ static u8 oui_8021h[] = { 0x00, 0x00, 0xf8 };
 *
 * Call context:
 *	May be called in interrupt or non-interrupt context
-----------------------------------------------------------------*/
+*----------------------------------------------------------------
+*/
 int skb_ether_to_p80211(wlandevice_t *wlandev, u32 ethconv,
			struct sk_buff *skb, union p80211_hdr *p80211_hdr,
			struct p80211_metawep *p80211_wep)
@@ -238,7 +240,8 @@ static void orinoco_spy_gather(wlandevice_t *wlandev, char *mac,
	int i;

	/* Gather wireless spy statistics: for each packet, compare the
-	 * source address with out list, and if match, get the stats... */
+	 * source address with out list, and if match, get the stats...
+	 */

	for (i = 0; i < wlandev->spy_number; i++) {

@@ -273,7 +276,8 @@ static void orinoco_spy_gather(wlandevice_t *wlandev, char *mac,
 *
 * Call context:
 *	May be called in interrupt or non-interrupt context
-----------------------------------------------------------------*/
+*----------------------------------------------------------------
+*/
 int skb_p80211_to_ether(wlandevice_t *wlandev, u32 ethconv,
			struct sk_buff *skb)
 {
@@ -421,7 +425,8 @@ int skb_p80211_to_ether(wlandevice_t *wlandev, u32 ethconv,
		&& (e_llc->ctl == 0x03)) {
		pr_debug("802.1h/RFC1042 len: %d\n", payload_length);
		/* it's an 802.1h frame || (an RFC1042 && protocol not in STT)
-		   build a DIXII + RFC894 */
+		 * build a DIXII + RFC894
+		 */

		/* Test for an overlength frame */
		if ((payload_length - sizeof(struct wlan_llc) -
@@ -521,14 +526,15 @@ int skb_p80211_to_ether(wlandevice_t *wlandev, u32 ethconv,
 *
 * Call context:
 *	May be called in interrupt or non-interrupt context
-----------------------------------------------------------------*/
+*----------------------------------------------------------------
+*/
 int p80211_stt_findproto(u16 proto)
 {
-	/* Always return found for now.  This is the behavior used by the */
-	/*  Zoom Win95 driver when 802.1h mode is selected */
-	/* TODO: If necessary, add an actual search we'll probably
-	   need this to match the CMAC's way of doing things.
-	   Need to do some testing to confirm.
+	/* Always return found for now.  This is the behavior used by the
+	 * Zoom Win95 driver when 802.1h mode is selected
+	 * TODO: If necessary, add an actual search we'll probably
+	 * need this to match the CMAC's way of doing things.
+	 * Need to do some testing to confirm.
	 */

	if (proto == ETH_P_AARP)	/* APPLETALK */
@@ -551,7 +557,8 @@ int p80211_stt_findproto(u16 proto)
 *
 * Call context:
 *	May be called in interrupt or non-interrupt context
-----------------------------------------------------------------*/
+*----------------------------------------------------------------
+*/
 void p80211skb_rxmeta_detach(struct sk_buff *skb)
 {
	struct p80211_rxmeta *rxmeta;
@@ -595,7 +602,8 @@ void p80211skb_rxmeta_detach(struct sk_buff *skb)
 *
 * Call context:
 *	May be called in interrupt or non-interrupt context
-----------------------------------------------------------------*/
+*----------------------------------------------------------------
+*/
 int p80211skb_rxmeta_attach(struct wlandevice *wlandev, struct sk_buff *skb)
 {
	int result = 0;
@@ -648,7 +656,8 @@ exit:
 *
 * Call context:
 *	May be called in interrupt or non-interrupt context
-----------------------------------------------------------------*/
+*----------------------------------------------------------------
+*/
 void p80211skb_free(struct wlandevice *wlandev, struct sk_buff *skb)
 {
	struct p80211_frmmeta *meta;
--
1.9.1

[toc] | [next] | [standalone]

#1304821 — Re: [PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block comment coding style

You're almost there...  Just resend with the correct patch prefix.

[PATCH 1/6] Staging: wlan-ng: Fixed block comment coding style in p80211conv.c

regards,
dan carpenter

[toc] | [prev] | [next] | [standalone]

#1304929 — Re: [PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block comment coding style

On Friday 08 January 2016 11:42 PM, Dan Carpenter wrote:
> You're almost there...  Just resend with the correct patch prefix.
>
> [PATCH 1/6] Staging: wlan-ng: Fixed block comment coding style in p80211conv.c
>
> regards,
> dan carpenter
>
Do you mean to say that instead of drivers/staging/wlan-ng/p80211conv.c, 
I should write only wlan-ng ?

[toc] | [prev] | [next] | [standalone]

#1304932 — Re: [PATCH 1/6] drivers/staging/wlan-ng/p80211conv.c: Fixed block comment coding style

On Sat, Jan 09, 2016 at 12:20:48AM +0530, Pranjal wrote:
> 
> 
> On Friday 08 January 2016 11:42 PM, Dan Carpenter wrote:
> >You're almost there...  Just resend with the correct patch prefix.
> >
> >[PATCH 1/6] Staging: wlan-ng: Fixed block comment coding style in p80211conv.c
> >
> >regards,
> >dan carpenter
> >
> Do you mean to say that instead of
> drivers/staging/wlan-ng/p80211conv.c, I should write only wlan-ng ?

Staging: wlan-ng:

yes.

regards,
dan carpetner

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web