pinctrl: imx: Return NULL if no group is matched and found
[ Upstream commit e64c0ff0d5d85791fbcd126ee558100a06a24a97 ]
Currently if no group is matched and found, this function will return
the last grp to the caller, this is not expected, it is supposed to
return NULL in this case.
Fixes: e566fc11ea ("pinctrl: imx: use generic pinctrl helpers for managing groups")
Signed-off-by: Hui Wang <hui.wang@canonical.com>
Reviewed-by: Frank Li <Frank.Li@nxp.com>
Link: https://lore.kernel.org/20250327031600.99723-1-hui.wang@canonical.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
7f5476d80f
commit
fcf36f6ed4
@@ -37,16 +37,16 @@ static inline const struct group_desc *imx_pinctrl_find_group_by_name(
|
||||
struct pinctrl_dev *pctldev,
|
||||
const char *name)
|
||||
{
|
||||
const struct group_desc *grp = NULL;
|
||||
const struct group_desc *grp;
|
||||
int i;
|
||||
|
||||
for (i = 0; i < pctldev->num_groups; i++) {
|
||||
grp = pinctrl_generic_get_group(pctldev, i);
|
||||
if (grp && !strcmp(grp->grp.name, name))
|
||||
break;
|
||||
return grp;
|
||||
}
|
||||
|
||||
return grp;
|
||||
return NULL;
|
||||
}
|
||||
|
||||
static void imx_pin_dbg_show(struct pinctrl_dev *pctldev, struct seq_file *s,
|
||||
|
||||
Reference in New Issue
Block a user