Re: the needless loop?

[Linus Walleij <linus.walleij@xxxxxxxxxx>]

On Wed, Oct 21, 2015 at 6:45 PM, Shevchenko, Andriy
<andriy.shevchenko@xxxxxxxxx> wrote:

> Catch such code in pinconf.c
>
>          for (j = 0; j < map->data.configs.num_configs; j++) {
>                  if (!strcmp(map->data.configs.group_or_pin,
>                                  dbg->pin_name)) {
>                          /* We found the right pin / state */
>                          found = map;
>                          break;
>                  }
>          }
>
> I do not get how the result of strcmp() would be different during the
> loop. Can you elaborate?

As you can see from git blame Laurent Meunier wrote this code,
(commit f07512e615dd8) so let's
ask him. I suspect a simple copy/paste error? Laurent?

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html