Re: [PATCH] rtl8150: switch to BIT macro

[Petko Manolov <petkan@nucleusys.com>]

On 19-05-09 11:01:06, Oliver Neukum wrote:
> A bit of housekeeping switching the driver to the BIT()
> macro.

Looks good.  I hope you've at least compiled the driver? :)

Acked-by: Petko Manolov <petkan@nucleusys.com>


cheers,
Petko


> Signed-off-by: Oliver Neukum <oneukum@suse.com>
> ---
>  drivers/net/usb/rtl8150.c | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/net/usb/rtl8150.c b/drivers/net/usb/rtl8150.c
> index 59dbdbb5feff..1ed85fba1a7c 100644
> --- a/drivers/net/usb/rtl8150.c
> +++ b/drivers/net/usb/rtl8150.c
> @@ -41,7 +41,7 @@
>  #define	ANLP			0x0146
>  #define	AER			0x0148
>  #define CSCR			0x014C  /* This one has the link status */
> -#define CSCR_LINK_STATUS	(1 << 3)
> +#define CSCR_LINK_STATUS	BIT(3)
>  
>  #define	IDR_EEPROM		0x1202
>  
> @@ -59,20 +59,20 @@
>  
>  
>  /* Transmit status register errors */
> -#define TSR_ECOL		(1<<5)
> -#define TSR_LCOL		(1<<4)
> -#define TSR_LOSS_CRS		(1<<3)
> -#define TSR_JBR			(1<<2)
> +#define TSR_ECOL		BIT(5)
> +#define TSR_LCOL		BIT(4)
> +#define TSR_LOSS_CRS		BIT(3)
> +#define TSR_JBR			BIT(2)
>  #define TSR_ERRORS		(TSR_ECOL | TSR_LCOL | TSR_LOSS_CRS | TSR_JBR)
>  /* Receive status register errors */
> -#define RSR_CRC			(1<<2)
> -#define RSR_FAE			(1<<1)
> +#define RSR_CRC			BIT(2)
> +#define RSR_FAE			BIT(1)
>  #define RSR_ERRORS		(RSR_CRC | RSR_FAE)
>  
>  /* Media status register definitions */
> -#define MSR_DUPLEX		(1<<4)
> -#define MSR_SPEED		(1<<3)
> -#define MSR_LINK		(1<<2)
> +#define MSR_DUPLEX		BIT(4)
> +#define MSR_SPEED		BIT(3)
> +#define MSR_LINK		BIT(2)
>  
>  /* Interrupt pipe data */
>  #define INT_TSR			0x00
> -- 
> 2.16.4
> 
>