lkml.org 
[lkml]   [2023]   [Feb]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 02/10] staging: rtl8192e: Calculate definition of MSR_LINK_MASK
From
Date
On Wed, 2023-02-08 at 13:31 +0100, Greg Kroah-Hartman wrote:
> On Tue, Feb 07, 2023 at 07:16:22PM +0100, Philipp Hortmann wrote:
> > Calculate definition of MSR_LINK_MASK ((1<<0)|(1<<1)) = 3. Equation is not
> > accepted by checkpatch because of missing spaces.
[]
> > diff --git a/drivers/staging/rtl8192e/rtl8192e/r8192E_hw.h b/drivers/staging/rtl8192e/rtl8192e/r8192E_hw.h
[]
> > @@ -170,7 +170,7 @@ enum _RTL8192PCI_HW {
> > #define BW_OPMODE_20MHZ BIT2
> > IC_VERRSION = 0x301,
> > MSR = 0x303,
> > -#define MSR_LINK_MASK ((1<<0)|(1<<1))
> > +#define MSR_LINK_MASK 3
>
> This is a bit field, it should be:
> #define MSR_LINK_MASK (BIT(1) | BIT(0))
> right?

Perhaps better as GENMASK(1, 0)

\
 
 \ /
  Last update: 2023-03-27 00:15    [W:0.423 / U:0.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site