* [PATCH] pinctrl: rzn1: Fix check for used MDIO bus
@ 2018-11-23 10:54 Phil Edworthy
2018-11-23 10:54 ` [PATCH] pinctrl: rzn1: Fix of_get_child_count() error check Phil Edworthy
2018-11-23 12:04 ` [PATCH] pinctrl: rzn1: Fix check for used MDIO bus Geert Uytterhoeven
0 siblings, 2 replies; 4+ messages in thread
From: Phil Edworthy @ 2018-11-23 10:54 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Jacopo Mondi, Linus Walleij, linux-gpio, linux-kernel,
linux-renesas-soc, Phil Edworthy
This fixes the check for unused mdio bus setting and the following static
checker warning:
drivers/pinctrl/pinctrl-rzn1.c:198 rzn1_pinctrl_mdio_select()
warn: always true condition '(ipctl->mdio_func[mdio] >= 0) => (0-u32max >= 0)'
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
---
v3:
- Split fixing the return var when calling of_get_child_count() into
a separate patch.
v2:
- Don't rely on rely on the implicit typecast from -1 to uint
---
drivers/pinctrl/pinctrl-rzn1.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-rzn1.c b/drivers/pinctrl/pinctrl-rzn1.c
index 57886dcff53d..a04235e3bec4 100644
--- a/drivers/pinctrl/pinctrl-rzn1.c
+++ b/drivers/pinctrl/pinctrl-rzn1.c
@@ -112,7 +112,7 @@ struct rzn1_pinctrl {
struct rzn1_pinctrl_regs __iomem *lev2;
u32 lev1_protect_phys;
u32 lev2_protect_phys;
- u32 mdio_func[2];
+ int mdio_func[2];
struct rzn1_pin_group *groups;
unsigned int ngroups;
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] pinctrl: rzn1: Fix of_get_child_count() error check
2018-11-23 10:54 [PATCH] pinctrl: rzn1: Fix check for used MDIO bus Phil Edworthy
@ 2018-11-23 10:54 ` Phil Edworthy
2018-11-23 12:06 ` Geert Uytterhoeven
2018-11-23 12:04 ` [PATCH] pinctrl: rzn1: Fix check for used MDIO bus Geert Uytterhoeven
1 sibling, 1 reply; 4+ messages in thread
From: Phil Edworthy @ 2018-11-23 10:54 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Jacopo Mondi, Linus Walleij, linux-gpio, linux-kernel,
linux-renesas-soc, Phil Edworthy
If we assign the result of of_get_child_count() to an unsigned int,
the code will not detect any errors. Therefore assign it to an int
instead.
Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
---
drivers/pinctrl/pinctrl-rzn1.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-rzn1.c b/drivers/pinctrl/pinctrl-rzn1.c
index a04235e3bec4..cc0e5aa9128a 100644
--- a/drivers/pinctrl/pinctrl-rzn1.c
+++ b/drivers/pinctrl/pinctrl-rzn1.c
@@ -810,8 +810,8 @@ static int rzn1_pinctrl_probe_dt(struct platform_device *pdev,
struct device_node *np = pdev->dev.of_node;
struct device_node *child;
unsigned int maxgroups = 0;
- unsigned int nfuncs = 0;
unsigned int i = 0;
+ int nfuncs = 0;
int ret;
nfuncs = of_get_child_count(np);
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] pinctrl: rzn1: Fix check for used MDIO bus
2018-11-23 10:54 [PATCH] pinctrl: rzn1: Fix check for used MDIO bus Phil Edworthy
2018-11-23 10:54 ` [PATCH] pinctrl: rzn1: Fix of_get_child_count() error check Phil Edworthy
@ 2018-11-23 12:04 ` Geert Uytterhoeven
1 sibling, 0 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2018-11-23 12:04 UTC (permalink / raw)
To: Phil Edworthy
Cc: Jacopo Mondi, Linus Walleij, open list:GPIO SUBSYSTEM,
Linux Kernel Mailing List, Linux-Renesas
On Fri, Nov 23, 2018 at 11:54 AM Phil Edworthy
<phil.edworthy@renesas.com> wrote:
> This fixes the check for unused mdio bus setting and the following static
> checker warning:
> drivers/pinctrl/pinctrl-rzn1.c:198 rzn1_pinctrl_mdio_select()
> warn: always true condition '(ipctl->mdio_func[mdio] >= 0) => (0-u32max >= 0)'
>
> Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
> Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> ---
> v3:
> - Split fixing the return var when calling of_get_child_count() into
> a separate patch.
> v2:
> - Don't rely on rely on the implicit typecast from -1 to uint
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in sh-pfc-for-v4.21.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] pinctrl: rzn1: Fix of_get_child_count() error check
2018-11-23 10:54 ` [PATCH] pinctrl: rzn1: Fix of_get_child_count() error check Phil Edworthy
@ 2018-11-23 12:06 ` Geert Uytterhoeven
0 siblings, 0 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2018-11-23 12:06 UTC (permalink / raw)
To: Phil Edworthy
Cc: Jacopo Mondi, Linus Walleij, open list:GPIO SUBSYSTEM,
Linux Kernel Mailing List, Linux-Renesas
On Fri, Nov 23, 2018 at 11:54 AM Phil Edworthy
<phil.edworthy@renesas.com> wrote:
> If we assign the result of of_get_child_count() to an unsigned int,
> the code will not detect any errors. Therefore assign it to an int
> instead.
Note that currently of_get_child_count() never returns a negative
error code. But it does return int, not unsigned int.
> Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in sh-pfc-for-v4.21.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-11-23 12:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-23 10:54 [PATCH] pinctrl: rzn1: Fix check for used MDIO bus Phil Edworthy
2018-11-23 10:54 ` [PATCH] pinctrl: rzn1: Fix of_get_child_count() error check Phil Edworthy
2018-11-23 12:06 ` Geert Uytterhoeven
2018-11-23 12:04 ` [PATCH] pinctrl: rzn1: Fix check for used MDIO bus Geert Uytterhoeven
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.