diff mbox

[v3] sound: aoa: Convert to using %pOF instead of full_name

Message ID 20170807232919.30247-2-robh@kernel.org (mailing list archive)
State New, archived
Headers show

Commit Message

Rob Herring (Arm) Aug. 7, 2017, 11:29 p.m. UTC
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@kernel.org>
Cc: Johannes Berg <johannes@sipsolutions.net>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Takashi Iwai <tiwai@suse.com>
Cc: linuxppc-dev@lists.ozlabs.org
Cc: alsa-devel@alsa-project.org
---
v3: Split aoa to separate patch

 sound/aoa/codecs/tas.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Rob Herring (Arm) Aug. 29, 2017, 12:35 p.m. UTC | #1
On Mon, Aug 7, 2017 at 6:29 PM, Rob Herring <robh@kernel.org> 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@kernel.org>
> Cc: Johannes Berg <johannes@sipsolutions.net>
> Cc: Jaroslav Kysela <perex@perex.cz>
> Cc: Takashi Iwai <tiwai@suse.com>
> Cc: linuxppc-dev@lists.ozlabs.org
> Cc: alsa-devel@alsa-project.org
> ---
> v3: Split aoa to separate patch

Ping.

>
>  sound/aoa/codecs/tas.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/sound/aoa/codecs/tas.c b/sound/aoa/codecs/tas.c
> index 733b6365dad6..15c05755d270 100644
> --- a/sound/aoa/codecs/tas.c
> +++ b/sound/aoa/codecs/tas.c
> @@ -905,8 +905,8 @@ static int tas_i2c_probe(struct i2c_client *client,
>                 goto fail;
>         }
>         printk(KERN_DEBUG
> -              "snd-aoa-codec-tas: tas found, addr 0x%02x on %s\n",
> -              (unsigned int)client->addr, node->full_name);
> +              "snd-aoa-codec-tas: tas found, addr 0x%02x on %pOF\n",
> +              (unsigned int)client->addr, node);
>         return 0;
>   fail:
>         mutex_destroy(&tas->mtx);
> --
> 2.11.0
>
Takashi Iwai Aug. 29, 2017, 12:52 p.m. UTC | #2
On Tue, 29 Aug 2017 14:35:03 +0200,
Rob Herring wrote:
> 
> On Mon, Aug 7, 2017 at 6:29 PM, Rob Herring <robh@kernel.org> 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@kernel.org>
> > Cc: Johannes Berg <johannes@sipsolutions.net>
> > Cc: Jaroslav Kysela <perex@perex.cz>
> > Cc: Takashi Iwai <tiwai@suse.com>
> > Cc: linuxppc-dev@lists.ozlabs.org
> > Cc: alsa-devel@alsa-project.org
> > ---
> > v3: Split aoa to separate patch
> 
> Ping.

Sorry, overlooked this one.  Now applied.

Thanks!


Takashi
diff mbox

Patch

diff --git a/sound/aoa/codecs/tas.c b/sound/aoa/codecs/tas.c
index 733b6365dad6..15c05755d270 100644
--- a/sound/aoa/codecs/tas.c
+++ b/sound/aoa/codecs/tas.c
@@ -905,8 +905,8 @@  static int tas_i2c_probe(struct i2c_client *client,
 		goto fail;
 	}
 	printk(KERN_DEBUG
-	       "snd-aoa-codec-tas: tas found, addr 0x%02x on %s\n",
-	       (unsigned int)client->addr, node->full_name);
+	       "snd-aoa-codec-tas: tas found, addr 0x%02x on %pOF\n",
+	       (unsigned int)client->addr, node);
 	return 0;
  fail:
 	mutex_destroy(&tas->mtx);