Re: [PATCH] devfreq: Convert to using %pOF instead of full_name

From: Rob Herring
Date: Thu Aug 24 2017 - 10:23:49 EST


On Wed, Aug 23, 2017 at 6:18 PM, Chanwoo Choi <cw00.choi@xxxxxxxxxxx> wrote:
> Hi Rob,
>
> On 2017ë 08ì 24ì 03:09, Rob Herring wrote:
>> On Thu, Jul 20, 2017 at 12:25 AM, MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx> wrote:
>>>> On 2017ë 07ì 19ì 06:42, Rob Herring wrote:
>>>>> Now that we have a custom printf format specifier, convert users of
>>>>> full_name to use %pOF instead. This is preparation to remove storing
>>>>> of the full path string for each node.
>>>>>
>>>>> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
>>>>> Cc: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>
>>>>> Cc: MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx>
>>>>> Cc: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
>>>>> Cc: linux-pm@xxxxxxxxxxxxxxx
>>>>> ---
>>>>> drivers/devfreq/devfreq-event.c | 4 ++--
>>>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>>>
>>>>> diff --git a/drivers/devfreq/devfreq-event.c b/drivers/devfreq/devfreq-event.c
>>>>> index 8648b32ebc89..d67242d87744 100644
>>>>> --- a/drivers/devfreq/devfreq-event.c
>>>>> +++ b/drivers/devfreq/devfreq-event.c
>>>>> @@ -277,8 +277,8 @@ int devfreq_event_get_edev_count(struct device *dev)
>>>>> sizeof(u32));
>>>>> if (count < 0) {
>>>>> dev_err(dev,
>>>>> - "failed to get the count of devfreq-event in %s node\n",
>>>>> - dev->of_node->full_name);
>>>>> + "failed to get the count of devfreq-event in %pOF node\n",
>>>>> + dev->of_node);
>>>>> return count;
>>>>> }
>>>>>
>>>>
>>>> Reviewed-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>
>>>>
>>>
>>> Acked-by: MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx>
>>
>> Please take this thru the devfreq tree.
>
> This patch was merged on devfreq tree.
> https://git.kernel.org/pub/scm/linux/kernel/git/mzx/devfreq.git/commit/?h=for-next&id=7f46535e8a2e3c3e415e55eaadcb9d9a153a5300

Okay, it's not showing up in linux-next which is why I asked.

Rob