On Sat, Aug 15, 2015 at 09:33:58PM -0400, Raphaël Beamonte wrote:
> Fix multiple occurences of the need consistent spacing code style error
> 
> Signed-off-by: Raphaël Beamonte <raphael.beamo...@gmail.com>
> ---
>  drivers/staging/rtl8192u/r8192U_core.c | 28 ++++++++++++++--------------
>  1 file changed, 14 insertions(+), 14 deletions(-)
> 
> diff --git a/drivers/staging/rtl8192u/r8192U_core.c 
> b/drivers/staging/rtl8192u/r8192U_core.c
> index 915493d..c4ab2a8 100644
> --- a/drivers/staging/rtl8192u/r8192U_core.c
> +++ b/drivers/staging/rtl8192u/r8192U_core.c
> @@ -2182,8 +2182,8 @@ static void rtl8192_init_priv_variable(struct 
> net_device *dev)
>       priv->EarlyRxThreshold = 7;
>       priv->enable_gpio0 = 0;
>       priv->TransmitConfig =
> -             (TCR_MXDMA_2048<<TCR_MXDMA_OFFSET)|  // Max DMA Burst Size per 
> Tx DMA Burst, 7: reserved.
> -             (priv->ShortRetryLimit<<TCR_SRL_OFFSET)|        // Short retry 
> limit
> +             (TCR_MXDMA_2048<<TCR_MXDMA_OFFSET) |  // Max DMA Burst Size per 
> Tx DMA Burst, 7: reserved.
> +             (priv->ShortRetryLimit<<TCR_SRL_OFFSET) |       // Short retry 
> limit
since you are modifying these lines for consistent spacing, it will be
better to have:
(priv->ShortRetryLimit << TCR_SRL_OFFSET) |       // Short retry limit
>  
<snip>
> @@ -2304,7 +2304,7 @@ static void rtl8192_read_eeprom_info(struct net_device 
> *dev)
>               tmpValue = eprom_read(dev, EEPROM_ChannelPlan>>1);
>               priv->eeprom_ChannelPlan = (tmpValue & 0xff00)>>8;
>               priv->btxpowerdata_readfromEEPORM = true;
> -             priv->eeprom_CustomerID = eprom_read(dev, 
> (EEPROM_Customer_ID>>1)) >>8;
> +             priv->eeprom_CustomerID = eprom_read(dev, 
> (EEPROM_Customer_ID>>1))>>8;
This should be:
priv->eeprom_CustomerID = eprom_read(dev, (EEPROM_Customer_ID>>1)) >> 8;

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

Reply via email to