RE: [PATCH] pinctrl: Fix pinconf_pins_show iteration

[Chanho Park]

> -----Original Message-----
> From: Stephen Warren [mailto:swarren@xxxxxxxxxx]
> Sent: Saturday, January 07, 2012 5:39 AM
> To: Linus Walleij
> Cc: linux-kernel@xxxxxxxxxxxxxxx; Chanho Park; Stephen Warren
> Subject: [PATCH] pinctrl: Fix pinconf_pins_show iteration
> 
> Commit 706e852 "pinctrl: correct a offset while enumerating pins"
> modified the variable used by pinconf_pin_show()'s for loop, but didn't
> update the for loop test expression.
> 
> Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx>
> ---
>  drivers/pinctrl/pinconf.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/pinctrl/pinconf.c b/drivers/pinctrl/pinconf.c
> index 1259872..1892a37 100644
> --- a/drivers/pinctrl/pinconf.c
> +++ b/drivers/pinctrl/pinconf.c
> @@ -236,7 +236,7 @@ static int pinconf_pins_show(struct seq_file *s,
> void *what)
>  	seq_puts(s, "Format: pin (name): pinmux setting array\n");
> 
>  	/* The pin number can be retrived from the pin controller
> descriptor */
> -	for (i = 0; pin < pctldev->desc->npins; i++) {
> +	for (i = 0; i < pctldev->desc->npins; i++) {
>  		struct pin_desc *desc;
> 
>  		pin = pctldev->desc->pins[i].number;
> --
> 1.7.0.4

Oh, I missed it.
Thank you for refining it.

Best Regards,
Chanho Park

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/