On Wed, May 21, 2014 at 09:37:28AM +0200, jes.soren...@redhat.com wrote:
> From: Jes Sorensen <jes.soren...@redhat.com>
> 
> Signed-off-by: Jes Sorensen <jes.soren...@redhat.com>
> ---
>  drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c 
> b/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
> index e2d426a..0afc742 100644
> --- a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
> +++ b/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
> @@ -1720,9 +1720,9 @@ static void Hal_EEValueCheck(u8 EEType, void *pInValue, 
> void *pOutValue)
>               u8 *pIn, *pOut;
>               pIn = (u8 *) pInValue;
>               pOut = (u8 *) pOutValue;
> -             if (*pIn >= 0 && *pIn <= 63) {
> +             if (*pIn <= 63)
>                       *pOut = *pIn;
> -             } else {
> +             else {

Don't resend but the curly braces were the correct kernel style in the
original.  If one side gets braces then both do.

regards,
dan carpenter

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to