Hi Simon, On Wed, Apr 29, 2015 at 10:25 AM, Simon Glass <s...@chromium.org> wrote: > Since we do these sorts of operations a lot, it is useful to have a simpler > API, similar to clrsetbits_le32(). > > Signed-off-by: Simon Glass <s...@chromium.org> > --- > > Changes in v2: None > > arch/x86/include/asm/msr.h | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/arch/x86/include/asm/msr.h b/arch/x86/include/asm/msr.h > index 1955a75..5349519 100644 > --- a/arch/x86/include/asm/msr.h > +++ b/arch/x86/include/asm/msr.h > @@ -128,6 +128,25 @@ static inline void wrmsr(unsigned msr, unsigned low, > unsigned high) > #define wrmsrl(msr, val) \ > native_write_msr((msr), (u32)((u64)(val)), (u32)((u64)(val) >> 32)) > > +static inline void msr_clrsetbits_64(unsigned msr, u64 clear, u64 set) > +{ > + u64 val; > + > + val = native_read_msr(msr); > + val &= ~clear; > + val |= set; > + wrmsrl(msr, val); > +} > + > +static inline void msr_setbits_64(unsigned msr, u64 set) > +{ > + u64 val; > + > + val = native_read_msr(msr); > + val |= set; > + wrmsrl(msr, val); > +} > + > /* rdmsr with exception handling */ > #define rdmsr_safe(msr, p1, p2) \ > ({ \ > --
For completeness, should we add msr_clrbits_64() as well? Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot