staging: unisys: fix strict checks in create_device()
authorBenjamin Romer <benjamin.romer@unisys.com>
Fri, 5 Dec 2014 22:08:45 +0000 (17:08 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 10 Jan 2015 01:30:20 +0000 (17:30 -0800)
Use the variable name rather than the type, and add a set of missing brackets
to the if statement in create_device().

Signed-off-by: Ken Depro <kenneth.depro@unisys.com>
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/uislib/uislib.c

index 1ddbe78999ae621e779887adfd6b8b43d85e3247..8a48091f993e57b1ac39f93153b34029be5b5bb1 100644 (file)
@@ -361,7 +361,7 @@ create_device(struct controlvm_message *msg, char *buf)
        POSTCODE_LINUX_4(DEVICE_CREATE_ENTRY_PC, devNo, busNo,
                         POSTCODE_SEVERITY_INFO);
 
-       dev = kzalloc(sizeof(struct device_info), GFP_ATOMIC);
+       dev = kzalloc(sizeof(*dev), GFP_ATOMIC);
        if (!dev) {
                LOGERR("CONTROLVM_DEVICE_CREATE Failed: kmalloc for dev failed.\n");
                POSTCODE_LINUX_4(DEVICE_CREATE_FAILURE_PC, devNo, busNo,
@@ -377,9 +377,9 @@ create_device(struct controlvm_message *msg, char *buf)
        sema_init(&dev->interrupt_callback_lock, 1);    /* unlocked */
        sprintf(dev->devid, "vbus%u:dev%u", (unsigned)busNo, (unsigned)devNo);
        /* map the channel memory for the device. */
-       if (msg->hdr.flags.test_message)
+       if (msg->hdr.flags.test_message) {
                dev->chanptr = (void __iomem *)__va(dev->channel_addr);
-       else {
+       else {
                pReqHandler = req_handler_find(dev->channel_uuid);
                if (pReqHandler)
                        /* generic service handler registered for this