Skip to content

Commit

Permalink
xen/blkback: Fix checkpatch warnings of xenbus.c
Browse files Browse the repository at this point in the history
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
  • Loading branch information
Konrad Rzeszutek Wilk committed Apr 14, 2011
1 parent e5f4b3c commit d6091b2
Showing 1 changed file with 12 additions and 9 deletions.
21 changes: 12 additions & 9 deletions drivers/xen/blkback/xenbus.c
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,9 @@
#undef DPRINTK
#define DPRINTK(fmt, args...) \
pr_debug("blkback/xenbus (%s:%d) " fmt ".\n", \
__FUNCTION__, __LINE__, ##args)
__func__, __LINE__, ##args)

struct backend_info
{
struct backend_info {
struct xenbus_device *dev;
struct blkif_st *blkif;
struct xenbus_watch backend_watch;
Expand Down Expand Up @@ -56,7 +55,8 @@ static int blkback_name(struct blkif_st *blkif, char *buf)
if (IS_ERR(devpath))
return PTR_ERR(devpath);

if ((devname = strstr(devpath, "/dev/")) != NULL)
devname = strstr(devpath, "/dev/");
if (devname != NULL)
devname += strlen("/dev/");
else
devname = devpath;
Expand Down Expand Up @@ -153,7 +153,7 @@ int xenvbd_sysfs_addif(struct xenbus_device *dev)
int error;

error = device_create_file(&dev->dev, &dev_attr_physical_device);
if (error)
if (error)
goto fail1;

error = device_create_file(&dev->dev, &dev_attr_mode);
Expand Down Expand Up @@ -327,7 +327,10 @@ static void backend_changed(struct xenbus_watch *watch,
/* Front end dir is a number, which is used as the handle. */

char *p = strrchr(dev->otherend, '/') + 1;
long handle = simple_strtoul(p, NULL, 0);
long handle;
err = strict_strtoul(p, 0, &handle);
if (err)
return;

be->major = major;
be->minor = minor;
Expand Down Expand Up @@ -369,7 +372,7 @@ static void frontend_changed(struct xenbus_device *dev,
case XenbusStateInitialising:
if (dev->state == XenbusStateClosed) {
printk(KERN_INFO "%s: %s: prepare for reconnect\n",
__FUNCTION__, dev->nodename);
__func__, dev->nodename);
xenbus_switch_state(dev, XenbusStateInitWait);
}
break;
Expand Down Expand Up @@ -494,8 +497,8 @@ static int connect_ring(struct backend_info *be)

DPRINTK("%s", dev->otherend);

err = xenbus_gather(XBT_NIL, dev->otherend, "ring-ref", "%lu", &ring_ref,
"event-channel", "%u", &evtchn, NULL);
err = xenbus_gather(XBT_NIL, dev->otherend, "ring-ref", "%lu",
&ring_ref, "event-channel", "%u", &evtchn, NULL);
if (err) {
xenbus_dev_fatal(dev, err,
"reading %s/ring-ref and event-channel",
Expand Down

0 comments on commit d6091b2

Please sign in to comment.