فهرست منبع

greybus: core: add device prefix to error messages

Use dev_err and dev_warn where appropriate and remove now unused pr_fmt
defines.

Testing Done:
Tested on DB3.5 with the generic bridge firmware on APB2.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Johan Hovold 9 سال پیش
والد
کامیت
b427572ebf
3فایلهای تغییر یافته به همراه2 افزوده شده و 6 حذف شده
  1. 0 2
      drivers/staging/greybus/debugfs.c
  2. 1 3
      drivers/staging/greybus/hd.c
  3. 1 1
      drivers/staging/greybus/operation.c

+ 0 - 2
drivers/staging/greybus/debugfs.c

@@ -7,8 +7,6 @@
  * Released under the GPLv2 only.
  */
 
-#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
-
 #include <linux/debugfs.h>
 
 #include "greybus.h"

+ 1 - 3
drivers/staging/greybus/hd.c

@@ -7,8 +7,6 @@
  * Released under the GPLv2 only.
  */
 
-#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
-
 #include <linux/kernel.h>
 #include <linux/slab.h>
 
@@ -55,7 +53,7 @@ struct gb_host_device *gb_hd_create(struct gb_hd_driver *driver,
 	 * so that we don't have to every time we make them.
 	 */
 	if ((!driver->message_send) || (!driver->message_cancel)) {
-		pr_err("Must implement all gb_hd_driver callbacks!\n");
+		dev_err(parent, "mandatory hd-callbacks missing\n");
 		return ERR_PTR(-EINVAL);
 	}
 

+ 1 - 1
drivers/staging/greybus/operation.c

@@ -327,7 +327,7 @@ gb_operation_message_alloc(struct gb_host_device *hd, u8 type,
 	size_t message_size = payload_size + sizeof(*header);
 
 	if (message_size > hd->buffer_size_max) {
-		pr_warn("requested message size too big (%zu > %zu)\n",
+		dev_warn(&hd->dev, "requested message size too big (%zu > %zu)\n",
 				message_size, hd->buffer_size_max);
 		return NULL;
 	}