public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* PS3: replace bus_id usage
@ 2009-01-26 17:30 Greg KH
  2009-01-30  0:00 ` Geoff Levand
  0 siblings, 1 reply; 3+ messages in thread
From: Greg KH @ 2009-01-26 17:30 UTC (permalink / raw)
  To: Geoff Levand; +Cc: Kay Sievers, linux-kernel


From: Greg Kroah-Hartman <gregkh@suse.de>

These simple debug statments should be using dev_dbg() instead of
accessing bus_id directly (or they should use device_name).

As bus_id is going away, this patch is necessary.

Cc: Geoff Levand <geoffrey.levand@am.sony.com>
Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

---

This fixes a built-breakage in linux-next.  I'll be glad to keep this in
my tree if you don't want to take it in yours, just let me know.

 arch/powerpc/platforms/ps3/system-bus.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

--- a/arch/powerpc/platforms/ps3/system-bus.c
+++ b/arch/powerpc/platforms/ps3/system-bus.c
@@ -376,7 +376,7 @@ static int ps3_system_bus_probe(struct d
 	struct ps3_system_bus_driver *drv;
 
 	BUG_ON(!dev);
-	pr_debug(" -> %s:%d: %s\n", __func__, __LINE__, _dev->bus_id);
+	dev_dbg(_dev, "%s:%d\n", __func__, __LINE__);
 
 	drv = ps3_system_bus_dev_to_system_bus_drv(dev);
 	BUG_ON(!drv);
@@ -398,7 +398,7 @@ static int ps3_system_bus_remove(struct 
 	struct ps3_system_bus_driver *drv;
 
 	BUG_ON(!dev);
-	pr_debug(" -> %s:%d: %s\n", __func__, __LINE__, _dev->bus_id);
+	dev_dbg(_dev, "%s:%d\n", __func__, __LINE__);
 
 	drv = ps3_system_bus_dev_to_system_bus_drv(dev);
 	BUG_ON(!drv);

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: PS3: replace bus_id usage
  2009-01-26 17:30 PS3: replace bus_id usage Greg KH
@ 2009-01-30  0:00 ` Geoff Levand
  2009-02-04  1:03   ` Greg KH
  0 siblings, 1 reply; 3+ messages in thread
From: Geoff Levand @ 2009-01-30  0:00 UTC (permalink / raw)
  To: Greg KH; +Cc: Kay Sievers, linux-kernel

Greg KH wrote:
> From: Greg Kroah-Hartman <gregkh@suse.de>
> 
> These simple debug statments should be using dev_dbg() instead of
> accessing bus_id directly (or they should use device_name).
> 
> As bus_id is going away, this patch is necessary.
> 
> Cc: Geoff Levand <geoffrey.levand@am.sony.com>
> Cc: Kay Sievers <kay.sievers@vrfy.org>
> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
> 
> ---
> 
> This fixes a built-breakage in linux-next.  I'll be glad to keep this in
> my tree if you don't want to take it in yours, just let me know.
> 
>  arch/powerpc/platforms/ps3/system-bus.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

This looks OK.  Greg, please merge it through your tree.

Acked-by: Geoff Levand <geoffrey.levand@am.sony.com>



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: PS3: replace bus_id usage
  2009-01-30  0:00 ` Geoff Levand
@ 2009-02-04  1:03   ` Greg KH
  0 siblings, 0 replies; 3+ messages in thread
From: Greg KH @ 2009-02-04  1:03 UTC (permalink / raw)
  To: Geoff Levand; +Cc: Kay Sievers, linux-kernel

On Thu, Jan 29, 2009 at 04:00:08PM -0800, Geoff Levand wrote:
> Greg KH wrote:
> > From: Greg Kroah-Hartman <gregkh@suse.de>
> > 
> > These simple debug statments should be using dev_dbg() instead of
> > accessing bus_id directly (or they should use device_name).
> > 
> > As bus_id is going away, this patch is necessary.
> > 
> > Cc: Geoff Levand <geoffrey.levand@am.sony.com>
> > Cc: Kay Sievers <kay.sievers@vrfy.org>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
> > 
> > ---
> > 
> > This fixes a built-breakage in linux-next.  I'll be glad to keep this in
> > my tree if you don't want to take it in yours, just let me know.
> > 
> >  arch/powerpc/platforms/ps3/system-bus.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> This looks OK.  Greg, please merge it through your tree.
> 
> Acked-by: Geoff Levand <geoffrey.levand@am.sony.com>

Thanks, I will do that.

greg k-h

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-02-04  1:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-01-26 17:30 PS3: replace bus_id usage Greg KH
2009-01-30  0:00 ` Geoff Levand
2009-02-04  1:03   ` Greg KH

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox