On Mon, 18 Mar 2019 18:37:27 -0500 Aditya Pakki wrote: > > > of_match_device in as370_pinctrl_probe can return a NULL value > when the matching device is not found. The patch avoids a potential
Similar as the comment to your previous patch, this could not happen. If the probe is called, it means OF registered a device with a valid compatible string, so match cannot be NULL. > dereference in such scenario. > > Signed-off-by: Aditya Pakki <pakki...@umn.edu> > --- > drivers/pinctrl/berlin/pinctrl-as370.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/pinctrl/berlin/pinctrl-as370.c > b/drivers/pinctrl/berlin/pinctrl-as370.c > index 44f8ccdbeeff..78bb5b866c99 100644 > --- a/drivers/pinctrl/berlin/pinctrl-as370.c > +++ b/drivers/pinctrl/berlin/pinctrl-as370.c > @@ -337,6 +337,8 @@ static int as370_pinctrl_probe(struct platform_device > *pdev) > struct resource *res; > void __iomem *base; > > + if (!match) > + return -ENODEV; > rmconfig = devm_kzalloc(&pdev->dev, sizeof(*rmconfig), GFP_KERNEL); > if (!rmconfig) > return -ENOMEM; > -- > 2.17.1 >
