Re: [PATCH] devfreq: Use of_node_name_eq for node name comparisons
On Wed, Feb 13, 2019 at 9:49 PM Chanwoo Choi wrote: > > Hi Rob, > > On 19. 2. 14. 오전 1:09, Rob Herring wrote: > > On Wed, Dec 5, 2018 at 1:50 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. > >> > >> For instances using of_node_cmp, this has the side effect of now using > >> case sensitive comparisons. This should not matter for any FDT based > >> system which all of these are. > >> > >> Cc: Chanwoo Choi > >> Cc: MyungJoo Ham > >> Cc: Kyungmin Park > >> Cc: Kukjin Kim > >> Cc: Krzysztof Kozlowski > >> Cc: linux...@vger.kernel.org > >> Cc: linux-arm-ker...@lists.infradead.org > >> Cc: linux-samsung-...@vger.kernel.org > >> Signed-off-by: Rob Herring > >> --- > >> drivers/devfreq/devfreq-event.c | 2 +- > >> drivers/devfreq/event/exynos-ppmu.c | 2 +- > >> 2 files changed, 2 insertions(+), 2 deletions(-) > > > > Can someone apply this please. > > Myungjoo picked up this patch[1] on devfreq.git > but has not yet sent the pull request. Okay, thanks. Please add that tree to linux-next or reply to patches when they've been applied, so submitters have some way to know when their patch is accepted. Rob
Re: [PATCH] devfreq: Use of_node_name_eq for node name comparisons
Hi Rob, On 19. 2. 14. 오전 1:09, Rob Herring wrote: > On Wed, Dec 5, 2018 at 1:50 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. >> >> For instances using of_node_cmp, this has the side effect of now using >> case sensitive comparisons. This should not matter for any FDT based >> system which all of these are. >> >> Cc: Chanwoo Choi >> Cc: MyungJoo Ham >> Cc: Kyungmin Park >> Cc: Kukjin Kim >> Cc: Krzysztof Kozlowski >> Cc: linux...@vger.kernel.org >> Cc: linux-arm-ker...@lists.infradead.org >> Cc: linux-samsung-...@vger.kernel.org >> Signed-off-by: Rob Herring >> --- >> drivers/devfreq/devfreq-event.c | 2 +- >> drivers/devfreq/event/exynos-ppmu.c | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) > > Can someone apply this please. Myungjoo picked up this patch[1] on devfreq.git but has not yet sent the pull request. [1] https://git.kernel.org/pub/scm/linux/kernel/git/mzx/devfreq.git/commit/?h=for-next&id=0c1e8ce295acc3becc95e393e88a9a6da233074f > >> >> diff --git a/drivers/devfreq/devfreq-event.c >> b/drivers/devfreq/devfreq-event.c >> index d67242d87744..87e93406d7cd 100644 >> --- a/drivers/devfreq/devfreq-event.c >> +++ b/drivers/devfreq/devfreq-event.c >> @@ -240,7 +240,7 @@ struct devfreq_event_dev >> *devfreq_event_get_edev_by_phandle(struct device *dev, >> } >> >> list_for_each_entry(edev, &devfreq_event_list, node) { >> - if (!strcmp(edev->desc->name, node->name)) >> + if (of_node_name_eq(node, edev->desc->name)) >> goto out; >> } >> edev = NULL; >> diff --git a/drivers/devfreq/event/exynos-ppmu.c >> b/drivers/devfreq/event/exynos-ppmu.c >> index c61de0bdf053..c2ea94957501 100644 >> --- a/drivers/devfreq/event/exynos-ppmu.c >> +++ b/drivers/devfreq/event/exynos-ppmu.c >> @@ -529,7 +529,7 @@ static int of_get_devfreq_events(struct device_node *np, >> if (!ppmu_events[i].name) >> continue; >> >> - if (!of_node_cmp(node->name, ppmu_events[i].name)) >> + if (of_node_name_eq(node, ppmu_events[i].name)) >> break; >> } >> >> -- >> 2.19.1 >> > > -- Best Regards, Chanwoo Choi Samsung Electronics
Re: [PATCH] devfreq: Use of_node_name_eq for node name comparisons
On Wed, Dec 5, 2018 at 1:50 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. > > For instances using of_node_cmp, this has the side effect of now using > case sensitive comparisons. This should not matter for any FDT based > system which all of these are. > > Cc: Chanwoo Choi > Cc: MyungJoo Ham > Cc: Kyungmin Park > Cc: Kukjin Kim > Cc: Krzysztof Kozlowski > Cc: linux...@vger.kernel.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-samsung-...@vger.kernel.org > Signed-off-by: Rob Herring > --- > drivers/devfreq/devfreq-event.c | 2 +- > drivers/devfreq/event/exynos-ppmu.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) Can someone apply this please. > > diff --git a/drivers/devfreq/devfreq-event.c b/drivers/devfreq/devfreq-event.c > index d67242d87744..87e93406d7cd 100644 > --- a/drivers/devfreq/devfreq-event.c > +++ b/drivers/devfreq/devfreq-event.c > @@ -240,7 +240,7 @@ struct devfreq_event_dev > *devfreq_event_get_edev_by_phandle(struct device *dev, > } > > list_for_each_entry(edev, &devfreq_event_list, node) { > - if (!strcmp(edev->desc->name, node->name)) > + if (of_node_name_eq(node, edev->desc->name)) > goto out; > } > edev = NULL; > diff --git a/drivers/devfreq/event/exynos-ppmu.c > b/drivers/devfreq/event/exynos-ppmu.c > index c61de0bdf053..c2ea94957501 100644 > --- a/drivers/devfreq/event/exynos-ppmu.c > +++ b/drivers/devfreq/event/exynos-ppmu.c > @@ -529,7 +529,7 @@ static int of_get_devfreq_events(struct device_node *np, > if (!ppmu_events[i].name) > continue; > > - if (!of_node_cmp(node->name, ppmu_events[i].name)) > + if (of_node_name_eq(node, ppmu_events[i].name)) > break; > } > > -- > 2.19.1 >
Re: [PATCH] devfreq: Use of_node_name_eq for node name comparisons
On Thu, Dec 6, 2018 at 4:54 AM 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. > > For instances using of_node_cmp, this has the side effect of now using > case sensitive comparisons. This should not matter for any FDT based > system which all of these are. > > Cc: Chanwoo Choi > Cc: MyungJoo Ham > Cc: Kyungmin Park > Cc: Kukjin Kim > Cc: Krzysztof Kozlowski > Cc: linux...@vger.kernel.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-samsung-...@vger.kernel.org > Signed-off-by: Rob Herring Acked-by: MyungJoo Ham > --- > drivers/devfreq/devfreq-event.c | 2 +- > drivers/devfreq/event/exynos-ppmu.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-)
[PATCH] devfreq: 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. For instances using of_node_cmp, this has the side effect of now using case sensitive comparisons. This should not matter for any FDT based system which all of these are. Cc: Chanwoo Choi Cc: MyungJoo Ham Cc: Kyungmin Park Cc: Kukjin Kim Cc: Krzysztof Kozlowski Cc: linux...@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-samsung-...@vger.kernel.org Signed-off-by: Rob Herring --- drivers/devfreq/devfreq-event.c | 2 +- drivers/devfreq/event/exynos-ppmu.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/devfreq/devfreq-event.c b/drivers/devfreq/devfreq-event.c index d67242d87744..87e93406d7cd 100644 --- a/drivers/devfreq/devfreq-event.c +++ b/drivers/devfreq/devfreq-event.c @@ -240,7 +240,7 @@ struct devfreq_event_dev *devfreq_event_get_edev_by_phandle(struct device *dev, } list_for_each_entry(edev, &devfreq_event_list, node) { - if (!strcmp(edev->desc->name, node->name)) + if (of_node_name_eq(node, edev->desc->name)) goto out; } edev = NULL; diff --git a/drivers/devfreq/event/exynos-ppmu.c b/drivers/devfreq/event/exynos-ppmu.c index c61de0bdf053..c2ea94957501 100644 --- a/drivers/devfreq/event/exynos-ppmu.c +++ b/drivers/devfreq/event/exynos-ppmu.c @@ -529,7 +529,7 @@ static int of_get_devfreq_events(struct device_node *np, if (!ppmu_events[i].name) continue; - if (!of_node_cmp(node->name, ppmu_events[i].name)) + if (of_node_name_eq(node, ppmu_events[i].name)) break; } -- 2.19.1