On Tue, Jun 14, 2016 at 12:07 PM, Masahiro Yamada <yamada.masah...@socionext.com> wrote:
> This function is doing more complicated than needed. The caller of > this function, of_gpiochip_scan_gpios() already knows the pointer to > the gpio_chip. It can pass it to of_parse_own_gpio() instead of > looking up the gpio_chip by gpiochip_find(). > > Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com> Patch applied! First I was worried about this semantic change: > - chip_np = np->parent; > + chip_np = chip->of_node; But then I read up on the code and saw that of_gpiochip_add() sets it to the parent if not set before. Yours, Linus Walleij
