netdev - Re: [PATCH 1/3] net: wireless: rt2x00: Space issue

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAGXE3d-XH5Efi_9cypT=HmJYzYdzyGoGJxcnrXZ86EVTcDM4Fg@mail.gmail.com>
Date:	Thu, 21 Jan 2016 18:04:07 +0100
From:	Helmut Schaa <helmut.schaa@...glemail.com>
To:	Paul McQuade <paulmcquad@...il.com>
Cc:	Stanislaw Gruszka <sgruszka@...hat.com>, kvalo@...eaurora.org,
	linux-wireless <linux-wireless@...r.kernel.org>,
	netdev <netdev@...r.kernel.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/3] net: wireless: rt2x00: Space issue

On Sat, Oct 17, 2015 at 11:06 PM, Paul McQuade <paulmcquad@...il.com> wrote:
> Removed empty spaces before/after parenthesis
>
> Signed-off-by: Paul McQuade <paulmcquad@...il.com>

Looks valid to me as well.

Acked-by: Helmut Schaa <helmut.schaa@...glemail.com>

> ---
>  drivers/net/wireless/rt2x00/rt2x00.h | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/net/wireless/rt2x00/rt2x00.h b/drivers/net/wireless/rt2x00/rt2x00.h
> index 3282ddb..6f8310a 100644
> --- a/drivers/net/wireless/rt2x00/rt2x00.h
> +++ b/drivers/net/wireless/rt2x00/rt2x00.h
> @@ -107,7 +107,7 @@
>   * amount of bytes needed to move the data.
>   */
>  #define ALIGN_SIZE(__skb, __header) \
> -       (  ((unsigned long)((__skb)->data + (__header))) & 3 )
> +       (((unsigned long)((__skb)->data + (__header))) & 3)
>
>  /*
>   * Constants for extra TX headroom for alignment purposes.
> @@ -128,14 +128,14 @@
>  #define SLOT_TIME              20
>  #define SHORT_SLOT_TIME                9
>  #define SIFS                   10
> -#define PIFS                   ( SIFS + SLOT_TIME )
> -#define SHORT_PIFS             ( SIFS + SHORT_SLOT_TIME )
> -#define DIFS                   ( PIFS + SLOT_TIME )
> -#define SHORT_DIFS             ( SHORT_PIFS + SHORT_SLOT_TIME )
> -#define EIFS                   ( SIFS + DIFS + \
> -                                 GET_DURATION(IEEE80211_HEADER + ACK_SIZE, 10) )
> -#define SHORT_EIFS             ( SIFS + SHORT_DIFS + \
> -                                 GET_DURATION(IEEE80211_HEADER + ACK_SIZE, 10) )
> +#define PIFS                   (SIFS + SLOT_TIME)
> +#define SHORT_PIFS             (SIFS + SHORT_SLOT_TIME)
> +#define DIFS                   (PIFS + SLOT_TIME)
> +#define SHORT_DIFS             (SHORT_PIFS + SHORT_SLOT_TIME)
> +#define EIFS                   (SIFS + DIFS + \
> +                                 GET_DURATION(IEEE80211_HEADER + ACK_SIZE, 10))
> +#define SHORT_EIFS             (SIFS + SHORT_DIFS + \
> +                                 GET_DURATION(IEEE80211_HEADER + ACK_SIZE, 10))
>
>  enum rt2x00_chip_intf {
>         RT2X00_CHIP_INTF_PCI,
>
> --
> 2.6.1
>