On Sat, Aug 12, 2023 at 3:34 AM Waiman Long <longman@xxxxxxxxxx> wrote: > > > On 8/2/23 12:46, guoren@xxxxxxxxxx wrote: > > \ > > diff --git a/arch/riscv/include/asm/spinlock.h b/arch/riscv/include/asm/spinlock.h > > new file mode 100644 > > index 000000000000..c644a92d4548 > > --- /dev/null > > +++ b/arch/riscv/include/asm/spinlock.h > > @@ -0,0 +1,17 @@ > > +/* SPDX-License-Identifier: GPL-2.0 */ > > + > > +#ifndef __ASM_RISCV_SPINLOCK_H > > +#define __ASM_RISCV_SPINLOCK_H > > + > > +#ifdef CONFIG_QUEUED_SPINLOCKS > > +#define _Q_PENDING_LOOPS (1 << 9) > > +#endif > > + > > +#ifdef CONFIG_QUEUED_SPINLOCKS > > You can merge the two "#ifdef CONFIG_QUEUED_SPINLOCKS" into single one > to avoid the duplication. Okay. > > Cheers, > Longman > > > +#include <asm/qspinlock.h> > > +#include <asm/qrwlock.h> > > +#else > > +#include <asm-generic/spinlock.h> > > +#endif > > + > > +#endif /* __ASM_RISCV_SPINLOCK_H */ > -- Best Regards Guo Ren
