Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755028AbdGXIfk (ORCPT ); Mon, 24 Jul 2017 04:35:40 -0400 Received: from mail-wm0-f51.google.com ([74.125.82.51]:37024 "EHLO mail-wm0-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753286AbdGXIfZ (ORCPT ); Mon, 24 Jul 2017 04:35:25 -0400 Subject: Re: [PATCH] pwm: Convert to using %pOF instead of full_name To: Rob Herring , Thierry Reding Cc: devicetree@vger.kernel.org, linux-pwm@vger.kernel.org, Kevin Hilman , linux-kernel@vger.kernel.org, Carlo Caione , linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org References: <20170718214339.7774-45-robh@kernel.org> From: Neil Armstrong Organization: Baylibre Message-ID: <5e9dfdc9-4c98-8e4a-f95d-7632bad21d2f@baylibre.com> Date: Mon, 24 Jul 2017 10:34:48 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <20170718214339.7774-45-robh@kernel.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1393 Lines: 40 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 > Cc: Thierry Reding > Cc: Carlo Caione > Cc: Kevin Hilman > Cc: linux-pwm@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-amlogic@lists.infradead.org > --- > 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 > linux-amlogic@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-amlogic > Acked-by: Neil Armstrong