* 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