Hi Chen, Chen Feng <puck.c...@hisilicon.com> 於 2019年6月27日 週四 上午10:03寫道: > > Thanks I assume this is an Ack. If you can add your Acked-by in the reply, it's easier for maintainer to accept the patch.
> > On 2019/6/26 21:30, Axel Lin wrote: > > Since devm_regmap_init_mmio_clk can fail, add return value checking. > > > > Signed-off-by: Axel Lin <axel....@ingics.com> > > Acked-by: Chen Feng <puck.c...@hisilicon.com> > > --- > > This was sent on https://lkml.org/lkml/2019/3/6/904 > > > > drivers/mfd/hi655x-pmic.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/mfd/hi655x-pmic.c b/drivers/mfd/hi655x-pmic.c > > index 96c07fa1802a..6693f74aa6ab 100644 > > --- a/drivers/mfd/hi655x-pmic.c > > +++ b/drivers/mfd/hi655x-pmic.c > > @@ -112,6 +112,8 @@ static int hi655x_pmic_probe(struct platform_device > > *pdev) > > > > pmic->regmap = devm_regmap_init_mmio_clk(dev, NULL, base, > > &hi655x_regmap_config); > > + if (IS_ERR(pmic->regmap)) > > + return PTR_ERR(pmic->regmap); > > > > regmap_read(pmic->regmap, HI655X_BUS_ADDR(HI655X_VER_REG), > > &pmic->ver); > > if ((pmic->ver < PMU_VER_START) || (pmic->ver > PMU_VER_END)) { > > >