staging: unisys: visorbus: replaced use of vague 'x' variable
authorAlexander Curtin <alexander.curtin@unisys.com>
Thu, 24 Mar 2016 02:15:52 +0000 (22:15 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 28 Mar 2016 14:30:36 +0000 (07:30 -0700)
In client_bus_info_show, the variable 'x' is used to create keep track
of the offset that the current 'pos' in the output buffer needs to be
incremented by. Since 'off' is already taken 'shift' was used since it's
used to shift the pointer.

Signed-off-by: Alexander Curtin <alexander.curtin@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/visorbus/visorbus_main.c

index 33407a760ab9f0f03f93c6db8fe89526425aad9f..1fcb1775bb5940b96d2db3817b3b664eb82f482a 100644 (file)
@@ -420,7 +420,7 @@ static ssize_t client_bus_info_show(struct device *dev,
        struct visor_device *vdev = to_visor_device(dev);
        struct visorchannel *channel = vdev->visorchannel;
 
-       int i, x, remain = PAGE_SIZE;
+       int i, shift, remain = PAGE_SIZE;
        unsigned long off;
        char *pos = buf;
        u8 *partition_name;
@@ -430,44 +430,45 @@ static ssize_t client_bus_info_show(struct device *dev,
        if (channel) {
                if (vdev->name)
                        partition_name = vdev->name;
-               x = snprintf(pos, remain,
-                            "Client device / client driver info for %s partition (vbus #%d):\n",
-                            partition_name, vdev->chipset_dev_no);
-               pos += x;
-               remain -= x;
-               x = visorchannel_read(channel,
-                                     offsetof(struct
-                                              spar_vbus_channel_protocol,
-                                              chp_info),
-                                     &dev_info, sizeof(dev_info));
-               if (x >= 0) {
-                       x = vbuschannel_devinfo_to_string(&dev_info, pos,
-                                                         remain, -1);
-                       pos += x;
-                       remain -= x;
+               shift = snprintf(pos, remain,
+                                "Client device / client driver info for %s eartition (vbus #%d):\n",
+                                partition_name, vdev->chipset_dev_no);
+               pos += shift;
+               remain -= shift;
+               shift = visorchannel_read(channel,
+                                         offsetof(struct
+                                                  spar_vbus_channel_protocol,
+                                                  chp_info),
+                                         &dev_info, sizeof(dev_info));
+               if (shift >= 0) {
+                       shift = vbuschannel_devinfo_to_string(&dev_info, pos,
+                                                             remain, -1);
+                       pos += shift;
+                       remain -= shift;
                }
-               x = visorchannel_read(channel,
-                                     offsetof(struct
-                                              spar_vbus_channel_protocol,
-                                              bus_info),
-                                     &dev_info, sizeof(dev_info));
-               if (x >= 0) {
-                       x = vbuschannel_devinfo_to_string(&dev_info, pos,
-                                                         remain, -1);
-                       pos += x;
-                       remain -= x;
+               shift = visorchannel_read(channel,
+                                         offsetof(struct
+                                                  spar_vbus_channel_protocol,
+                                                  bus_info),
+                                         &dev_info, sizeof(dev_info));
+               if (shift >= 0) {
+                       shift = vbuschannel_devinfo_to_string(&dev_info, pos,
+                                                             remain, -1);
+                       pos += shift;
+                       remain -= shift;
                }
                off = offsetof(struct spar_vbus_channel_protocol, dev_info);
                i = 0;
                while (off + sizeof(dev_info) <=
                       visorchannel_get_nbytes(channel)) {
-                       x = visorchannel_read(channel,
-                                             off, &dev_info, sizeof(dev_info));
-                       if (x >= 0) {
-                               x = vbuschannel_devinfo_to_string
+                       shift = visorchannel_read(channel,
+                                                 off, &dev_info,
+                                                 sizeof(dev_info));
+                       if (shift >= 0) {
+                               shift = vbuschannel_devinfo_to_string
                                    (&dev_info, pos, remain, i);
-                               pos += x;
-                               remain -= x;
+                               pos += shift;
+                               remain -= shift;
                        }
                        off += sizeof(dev_info);
                        i++;