linux-kernel - Re: [PATCH 1586/1586] Staging: RT2860: Fixed my previous patch.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20100520150644.GA3862@suse.de>
Date:	Thu, 20 May 2010 08:06:44 -0700
From:	Greg KH <gregkh@...e.de>
To:	Neil Munro <neilmunro@...il.com>
Cc:	wfp5p@...ginia.edu, devel@...verdev.osuosl.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1586/1586] Staging: RT2860: Fixed my previous patch.

On Thu, May 20, 2010 at 12:28:53PM +0100, Neil Munro wrote:
> From: Niadh <neilmunro@...il.com>
> 
> I have corrected the confusion between braces and parethesis. In
> addition I have removed a few warnings where the lines were over 80
> lines and finaly have implemented a few suggestions Joe Perches
> suggested (thanks Joe).
> 
> Signed-off-by: Neil Munro <neilmunro@...il.com>
> ---
>  drivers/staging/rt2860/mlme.h |   89 +++++++++++++++++++++++++----------------
>  1 files changed, 55 insertions(+), 34 deletions(-)
> 
> diff --git a/drivers/staging/rt2860/mlme.h b/drivers/staging/rt2860/mlme.h
> index 63cada3..02cbe9a 100644
> --- a/drivers/staging/rt2860/mlme.h
> +++ b/drivers/staging/rt2860/mlme.h
> @@ -31,7 +31,7 @@
> 
>  	Revision History:
>  	Who			When			What
> -	--------	----------		----------------------------------------------
> +	--------	----------		------------
>  	John Chang	2003-08-28		Created
>  	John Chang  2004-09-06      modified for RT2600
> 
> @@ -49,8 +49,9 @@
>  #define LFSR_MASK                   0x80000057
>  #define MLME_TASK_EXEC_INTV         100/*200*/	/* */
>  #define LEAD_TIME                   5
> -#define MLME_TASK_EXEC_MULTIPLE       10  /*5*/	/* MLME_TASK_EXEC_MULTIPLE * MLME_TASK_EXEC_INTV = 1 sec */
> -#define REORDER_EXEC_INTV         	100	/* 0.1 sec */
> +#define MLME_TASK_EXEC_MULTIPLE       10
> +/*5*/	/* MLME_TASK_EXEC_MULTIPLE * MLME_TASK_EXEC_INTV = 1 sec */

This doesn't make sense.  Put the comment on the line before.

And drop the /*5*/, it too doesn't make sense.

> +#define REORDER_EXEC_INTV		100	/* 0.1 sec */
> 
>  /* The definition of Radar detection duration region */
>  #define CE		0
> @@ -60,7 +61,7 @@
>  #define JAP_W56	4
>  #define MAX_RD_REGION 5
> 
> -#define BEACON_LOST_TIME            { 4 * OS_HZ }	/* 2048 msec = 2 sec */
> +#define BEACON_LOST_TIME            (4 * OS_HZ)	/* 2048 msec = 2 sec */

As I didn't take your original patch, this patch will not apply.  Please
generate a new patch that I can apply properly to the linux-next tree.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/