Message ID | 1615790877-55259-1-git-send-email-f.fangjian@huawei.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | soc: mediatek: pm-domains: fix missing error codes | expand |
Hi Zihao, On 15/03/2021 07:47, Jay Fang wrote: > From: Zihao Tang <tangzihao1@hisilicon.com> > > Fixes one smatch warnings: > > drivers/soc/mediatek/mtk-pm-domains.c:492 scpsys_add_subdomain() warn: missing error code 'ret' > > Reported-by: kernel test robot <lkp@intel.com> > Reported-by: Dan Carpenter <dan.carpenter@oracle.com> > Signed-off-by: Zihao Tang <tangzihao1@hisilicon.com> > Signed-off-by: Jay Fang <f.fangjian@huawei.com> That got already fixed by Enric [1]. Anway thanks for the patch. Matthias [1] https://patchwork.kernel.org/project/linux-mediatek/patch/20210303091054.796975-1-enric.balletbo@collabora.com/ > --- > drivers/soc/mediatek/mtk-pm-domains.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/soc/mediatek/mtk-pm-domains.c b/drivers/soc/mediatek/mtk-pm-domains.c > index b7f6976..5778dad 100644 > --- a/drivers/soc/mediatek/mtk-pm-domains.c > +++ b/drivers/soc/mediatek/mtk-pm-domains.c > @@ -487,6 +487,7 @@ static int scpsys_add_subdomain(struct scpsys *scpsys, struct device_node *paren > > child_pd = scpsys_add_one_domain(scpsys, child); > if (IS_ERR(child_pd)) { > + ret = PTR_ERR(child_pd); > dev_err_probe(scpsys->dev, PTR_ERR(child_pd), > "%pOF: failed to get child domain id\n", child); > goto err_put_node; >
diff --git a/drivers/soc/mediatek/mtk-pm-domains.c b/drivers/soc/mediatek/mtk-pm-domains.c index b7f6976..5778dad 100644 --- a/drivers/soc/mediatek/mtk-pm-domains.c +++ b/drivers/soc/mediatek/mtk-pm-domains.c @@ -487,6 +487,7 @@ static int scpsys_add_subdomain(struct scpsys *scpsys, struct device_node *paren child_pd = scpsys_add_one_domain(scpsys, child); if (IS_ERR(child_pd)) { + ret = PTR_ERR(child_pd); dev_err_probe(scpsys->dev, PTR_ERR(child_pd), "%pOF: failed to get child domain id\n", child); goto err_put_node;