> -----Original Message----- > From: Ye Bin <yebin10@xxxxxxxxxx> > Sent: Friday, April 9, 2021 4:52 AM > To: yebin10@xxxxxxxxxx; Qiang Zhao <qiang.zhao@xxxxxxx>; Leo Li > <leoyang.li@xxxxxxx> > Cc: linuxppc-dev@xxxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx; Hulk Robot > <hulkci@xxxxxxxxxx> > Subject: [PATCH -next] soc: fsl: qe: use DEFINE_SPINLOCK() for spinlock > > spinlock can be initialized automatically with DEFINE_SPINLOCK() rather than > explicitly calling spin_lock_init(). The previous version has been applied. Thanks. > > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Ye Bin <yebin10@xxxxxxxxxx> > --- > drivers/soc/fsl/qe/qe_common.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/soc/fsl/qe/qe_common.c > b/drivers/soc/fsl/qe/qe_common.c index 654e9246ce6b..a0cb8e746879 > 100644 > --- a/drivers/soc/fsl/qe/qe_common.c > +++ b/drivers/soc/fsl/qe/qe_common.c > @@ -26,7 +26,7 @@ > #include <soc/fsl/qe/qe.h> > > static struct gen_pool *muram_pool; > -static spinlock_t cpm_muram_lock; > +static DEFINE_SPINLOCK(cpm_muram_lock); > static void __iomem *muram_vbase; > static phys_addr_t muram_pbase; > > @@ -54,7 +54,6 @@ int cpm_muram_init(void) > if (muram_pbase) > return 0; > > - spin_lock_init(&cpm_muram_lock); > np = of_find_compatible_node(NULL, NULL, "fsl,cpm-muram-data"); > if (!np) { > /* try legacy bindings */