2017-07-18 21:48:52

by Rob Herring (Arm)

[permalink] [raw]
Subject: [PATCH] pwm: Convert to using %pOF instead of full_name

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 <[email protected]>
Cc: Thierry Reding <[email protected]>
Cc: Carlo Caione <[email protected]>
Cc: Kevin Hilman <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
---
drivers/pwm/pwm-meson.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pwm/pwm-meson.c b/drivers/pwm/pwm-meson.c
index cb845edfe2b4..d589331d1884 100644
--- a/drivers/pwm/pwm-meson.c
+++ b/drivers/pwm/pwm-meson.c
@@ -441,7 +441,7 @@ static int meson_pwm_init_channels(struct meson_pwm *meson,
for (i = 0; i < meson->chip.npwm; i++) {
struct meson_pwm_channel *channel = &channels[i];

- snprintf(name, sizeof(name), "%s#mux%u", np->full_name, i);
+ snprintf(name, sizeof(name), "%pOF#mux%u", np, i);

init.name = name;
init.ops = &clk_mux_ops;
--
2.11.0


2017-07-24 08:35:40

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH] pwm: Convert to using %pOF instead of full_name

On 07/18/2017 11:43 PM, 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 <[email protected]>
> Cc: Thierry Reding <[email protected]>
> Cc: Carlo Caione <[email protected]>
> Cc: Kevin Hilman <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> ---
> drivers/pwm/pwm-meson.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pwm/pwm-meson.c b/drivers/pwm/pwm-meson.c
> index cb845edfe2b4..d589331d1884 100644
> --- a/drivers/pwm/pwm-meson.c
> +++ b/drivers/pwm/pwm-meson.c
> @@ -441,7 +441,7 @@ static int meson_pwm_init_channels(struct meson_pwm *meson,
> for (i = 0; i < meson->chip.npwm; i++) {
> struct meson_pwm_channel *channel = &channels[i];
>
> - snprintf(name, sizeof(name), "%s#mux%u", np->full_name, i);
> + snprintf(name, sizeof(name), "%pOF#mux%u", np, i);
>
> init.name = name;
> init.ops = &clk_mux_ops;
> --
> 2.11.0
>
>
> _______________________________________________
> linux-amlogic mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-amlogic
>

Acked-by: Neil Armstrong <[email protected]>

2017-07-25 11:39:03

by Thierry Reding

[permalink] [raw]
Subject: Re: [PATCH] pwm: Convert to using %pOF instead of full_name

On Tue, Jul 18, 2017 at 04:43:25PM -0500, 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 <[email protected]>
> Cc: Thierry Reding <[email protected]>
> Cc: Carlo Caione <[email protected]>
> Cc: Kevin Hilman <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> ---
> drivers/pwm/pwm-meson.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.

Thierry


Attachments:
(No filename) (652.00 B)
signature.asc (833.00 B)
Download all attachments

2017-08-09 22:13:19

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] pwm: Convert to using %pOF instead of full_name

On Tue, Jul 25, 2017 at 6:38 AM, Thierry Reding
<[email protected]> wrote:
> On Tue, Jul 18, 2017 at 04:43:25PM -0500, 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 <[email protected]>
>> Cc: Thierry Reding <[email protected]>
>> Cc: Carlo Caione <[email protected]>
>> Cc: Kevin Hilman <[email protected]>
>> Cc: [email protected]
>> Cc: [email protected]
>> Cc: [email protected]
>> ---
>> drivers/pwm/pwm-meson.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Applied, thanks.

This is not showing up in -next.

Rob

2017-08-21 06:57:38

by Thierry Reding

[permalink] [raw]
Subject: Re: [PATCH] pwm: Convert to using %pOF instead of full_name

On Wed, Aug 09, 2017 at 05:12:55PM -0500, Rob Herring wrote:
> On Tue, Jul 25, 2017 at 6:38 AM, Thierry Reding
> <[email protected]> wrote:
> > On Tue, Jul 18, 2017 at 04:43:25PM -0500, 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 <[email protected]>
> >> Cc: Thierry Reding <[email protected]>
> >> Cc: Carlo Caione <[email protected]>
> >> Cc: Kevin Hilman <[email protected]>
> >> Cc: [email protected]
> >> Cc: [email protected]
> >> Cc: [email protected]
> >> ---
> >> drivers/pwm/pwm-meson.c | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Applied, thanks.
>
> This is not showing up in -next.

Yes, I had forgotten to push this out. I've done that now, so it should
be in the next -next.

Thierry


Attachments:
(No filename) (984.00 B)
signature.asc (833.00 B)
Download all attachments