Re: [PATCH] pinctrl: Use of_node_name_eq for node name comparisons
On Wed, Dec 5, 2018 at 8:51 PM Rob Herring wrote: > Convert string compares of DT node names to use of_node_name_eq helper > instead. This removes direct access to the node name pointer. > > Cc: Tomasz Figa > Cc: Krzysztof Kozlowski > Cc: Sylwester Nawrocki > Cc: Linus Walleij > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-samsung-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Signed-off-by: Rob Herring Patch applied! Yours, Linus Walleij
Re: [PATCH] pinctrl: Use of_node_name_eq for node name comparisons
On Wed, 5 Dec 2018 at 20:51, Rob Herring wrote: > > Convert string compares of DT node names to use of_node_name_eq helper > instead. This removes direct access to the node name pointer. > > Cc: Tomasz Figa > Cc: Krzysztof Kozlowski > Cc: Sylwester Nawrocki > Cc: Linus Walleij > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-samsung-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Signed-off-by: Rob Herring > --- > drivers/pinctrl/samsung/pinctrl-samsung.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Krzysztof Kozlowski Linus, I do not have anything queued in pinctrl samsung for next release, so maybe you could apply this directly? Best regards, Krzysztof
Re: [PATCH] pinctrl: Use of_node_name_eq for node name comparisons
2018年12月5日(水) 11:51 Rob Herring : > > Convert string compares of DT node names to use of_node_name_eq helper > instead. This removes direct access to the node name pointer. > > Cc: Tomasz Figa > Cc: Krzysztof Kozlowski > Cc: Sylwester Nawrocki > Cc: Linus Walleij > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-samsung-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Signed-off-by: Rob Herring > --- > drivers/pinctrl/samsung/pinctrl-samsung.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pinctrl/samsung/pinctrl-samsung.c > b/drivers/pinctrl/samsung/pinctrl-samsung.c > index ee6ee2338606..de0477bb469d 100644 > --- a/drivers/pinctrl/samsung/pinctrl-samsung.c > +++ b/drivers/pinctrl/samsung/pinctrl-samsung.c > @@ -1071,7 +1071,7 @@ samsung_pinctrl_get_soc_data(struct > samsung_pinctrl_drv_data *d, > continue; > bank = d->pin_banks; > for (i = 0; i < d->nr_banks; ++i, ++bank) { > - if (!strcmp(bank->name, np->name)) { > + if (of_node_name_eq(np, bank->name)) { > bank->of_node = np; > break; > } > -- > 2.19.1 > Reviewed-by: Tomasz Figa Best regards, Tomasz
[PATCH] pinctrl: Use of_node_name_eq for node name comparisons
Convert string compares of DT node names to use of_node_name_eq helper instead. This removes direct access to the node name pointer. Cc: Tomasz Figa Cc: Krzysztof Kozlowski Cc: Sylwester Nawrocki Cc: Linus Walleij Cc: linux-arm-ker...@lists.infradead.org Cc: linux-samsung-...@vger.kernel.org Cc: linux-g...@vger.kernel.org Signed-off-by: Rob Herring --- drivers/pinctrl/samsung/pinctrl-samsung.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/samsung/pinctrl-samsung.c b/drivers/pinctrl/samsung/pinctrl-samsung.c index ee6ee2338606..de0477bb469d 100644 --- a/drivers/pinctrl/samsung/pinctrl-samsung.c +++ b/drivers/pinctrl/samsung/pinctrl-samsung.c @@ -1071,7 +1071,7 @@ samsung_pinctrl_get_soc_data(struct samsung_pinctrl_drv_data *d, continue; bank = d->pin_banks; for (i = 0; i < d->nr_banks; ++i, ++bank) { - if (!strcmp(bank->name, np->name)) { + if (of_node_name_eq(np, bank->name)) { bank->of_node = np; break; } -- 2.19.1