2013-09-27 16:49:54

by Dutt, Sudeep

[permalink] [raw]
Subject: [PATCH char-misc-next 0/5] misc: mic: driver cleanups and a build fix.

These patches address code review feedback received on the
patch series @ https://lkml.org/lkml/2013/9/5/561 ,from:
a) Greg Kroah-Hartman on sysfs and header file cleanups.
b) Joe Perches on issues found by "--strict" checkpatch.

It should also fix the build failures on certain non X86
architectures, reported in linux-next and 0-DAY builds.

Ashutosh Dixit (1):
misc: mic: cleanups for "--strict" checkpatch.

Sudeep Dutt (4):
misc: mic: host driver sysfs cleanups.
misc: mic: header file cleanups.
misc: mic: fix a warning in the IOCTL header file.
misc: mic: depend on X86 for both host and card drivers.

Documentation/mic/mpssd/mpssd.c | 65 +++++++--------
Documentation/mic/mpssd/sysfs.c | 4 +-
drivers/misc/mic/Kconfig | 4 +-
drivers/misc/mic/card/mic_debugfs.c | 2 +-
drivers/misc/mic/card/mic_device.c | 3 +-
drivers/misc/mic/card/mic_virtio.c | 19 +++--
drivers/misc/mic/card/mic_x100.c | 6 +-
.../misc/mic/common/{mic_device.h => mic_dev.h} | 4 +-
drivers/misc/mic/host/mic_boot.c | 2 +-
drivers/misc/mic/host/mic_debugfs.c | 92 ++++++++++-----------
drivers/misc/mic/host/mic_fops.c | 8 +-
drivers/misc/mic/host/mic_intr.c | 36 +++++----
drivers/misc/mic/host/mic_main.c | 2 +-
drivers/misc/mic/host/mic_smpt.c | 16 ++--
drivers/misc/mic/host/mic_sysfs.c | 94 +++++++++-------------
drivers/misc/mic/host/mic_virtio.c | 39 +++++----
drivers/misc/mic/host/mic_x100.c | 35 ++++----
include/uapi/linux/mic_ioctl.h | 2 +
18 files changed, 206 insertions(+), 227 deletions(-)
rename drivers/misc/mic/common/{mic_device.h => mic_dev.h} (95%)

--
1.8.2.1


2013-09-27 16:50:04

by Dutt, Sudeep

[permalink] [raw]
Subject: [PATCH char-misc-next 1/5] misc: mic: host driver sysfs cleanups.

+ Use DEVICE_ATTR_RO/RW instead of DEVICE_ATTR
+ Use ATTRIBUTE_GROUPS

These changes were suggested by Greg Kroah-Hartman during the
code review @ https://lkml.org/lkml/2013/9/6/13

Reported-by: Greg Kroah-Hartman <[email protected]>
Signed-off-by: Ashutosh Dixit <[email protected]>
Signed-off-by: Dasaratharaman Chandramouli <[email protected]>
Signed-off-by: Nikhil Rao <[email protected]>
Signed-off-by: Harshavardhan R Kharche <[email protected]>
Signed-off-by: Sudeep Dutt <[email protected]>
---
drivers/misc/mic/host/mic_sysfs.c | 72 ++++++++++++++++-----------------------
1 file changed, 29 insertions(+), 43 deletions(-)

diff --git a/drivers/misc/mic/host/mic_sysfs.c b/drivers/misc/mic/host/mic_sysfs.c
index aaf8499..bebbae2 100644
--- a/drivers/misc/mic/host/mic_sysfs.c
+++ b/drivers/misc/mic/host/mic_sysfs.c
@@ -65,7 +65,7 @@ void mic_set_state(struct mic_device *mdev, u8 state)
}

static ssize_t
-mic_show_family(struct device *dev, struct device_attribute *attr, char *buf)
+family_show(struct device *dev, struct device_attribute *attr, char *buf)
{
static const char x100[] = "x100";
static const char unknown[] = "Unknown";
@@ -85,10 +85,10 @@ mic_show_family(struct device *dev, struct device_attribute *attr, char *buf)
}
return scnprintf(buf, PAGE_SIZE, "%s\n", card);
}
-static DEVICE_ATTR(family, S_IRUGO, mic_show_family, NULL);
+static DEVICE_ATTR_RO(family);

static ssize_t
-mic_show_stepping(struct device *dev, struct device_attribute *attr, char *buf)
+stepping_show(struct device *dev, struct device_attribute *attr, char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
char *string = "??";
@@ -114,10 +114,10 @@ mic_show_stepping(struct device *dev, struct device_attribute *attr, char *buf)
}
return scnprintf(buf, PAGE_SIZE, "%s\n", string);
}
-static DEVICE_ATTR(stepping, S_IRUGO, mic_show_stepping, NULL);
+static DEVICE_ATTR_RO(stepping);

static ssize_t
-mic_show_state(struct device *dev, struct device_attribute *attr, char *buf)
+state_show(struct device *dev, struct device_attribute *attr, char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);

@@ -129,7 +129,7 @@ mic_show_state(struct device *dev, struct device_attribute *attr, char *buf)
}

static ssize_t
-mic_store_state(struct device *dev, struct device_attribute *attr,
+state_store(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count)
{
int rc = 0;
@@ -160,9 +160,9 @@ mic_store_state(struct device *dev, struct device_attribute *attr,
done:
return count;
}
-static DEVICE_ATTR(state, S_IRUGO|S_IWUSR, mic_show_state, mic_store_state);
+static DEVICE_ATTR_RW(state);

-static ssize_t mic_show_shutdown_status(struct device *dev,
+static ssize_t shutdown_status_show(struct device *dev,
struct device_attribute *attr, char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -173,11 +173,10 @@ static ssize_t mic_show_shutdown_status(struct device *dev,
return scnprintf(buf, PAGE_SIZE, "%s\n",
mic_shutdown_status_string[mdev->shutdown_status]);
}
-static DEVICE_ATTR(shutdown_status, S_IRUGO|S_IWUSR,
- mic_show_shutdown_status, NULL);
+static DEVICE_ATTR_RO(shutdown_status);

static ssize_t
-mic_show_cmdline(struct device *dev, struct device_attribute *attr, char *buf)
+cmdline_show(struct device *dev, struct device_attribute *attr, char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
char *cmdline;
@@ -193,7 +192,7 @@ mic_show_cmdline(struct device *dev, struct device_attribute *attr, char *buf)
}

static ssize_t
-mic_store_cmdline(struct device *dev, struct device_attribute *attr,
+cmdline_store(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -220,11 +219,10 @@ unlock:
mutex_unlock(&mdev->mic_mutex);
return count;
}
-static DEVICE_ATTR(cmdline, S_IRUGO | S_IWUSR,
- mic_show_cmdline, mic_store_cmdline);
+static DEVICE_ATTR_RW(cmdline);

static ssize_t
-mic_show_firmware(struct device *dev, struct device_attribute *attr, char *buf)
+firmware_show(struct device *dev, struct device_attribute *attr, char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
char *firmware;
@@ -240,7 +238,7 @@ mic_show_firmware(struct device *dev, struct device_attribute *attr, char *buf)
}

static ssize_t
-mic_store_firmware(struct device *dev, struct device_attribute *attr,
+firmware_store(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -266,11 +264,10 @@ unlock:
mutex_unlock(&mdev->mic_mutex);
return count;
}
-static DEVICE_ATTR(firmware, S_IRUGO | S_IWUSR,
- mic_show_firmware, mic_store_firmware);
+static DEVICE_ATTR_RW(firmware);

static ssize_t
-mic_show_ramdisk(struct device *dev, struct device_attribute *attr, char *buf)
+ramdisk_show(struct device *dev, struct device_attribute *attr, char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
char *ramdisk;
@@ -286,7 +283,7 @@ mic_show_ramdisk(struct device *dev, struct device_attribute *attr, char *buf)
}

static ssize_t
-mic_store_ramdisk(struct device *dev, struct device_attribute *attr,
+ramdisk_store(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -313,11 +310,10 @@ unlock:
mutex_unlock(&mdev->mic_mutex);
return count;
}
-static DEVICE_ATTR(ramdisk, S_IRUGO | S_IWUSR,
- mic_show_ramdisk, mic_store_ramdisk);
+static DEVICE_ATTR_RW(ramdisk);

static ssize_t
-mic_show_bootmode(struct device *dev, struct device_attribute *attr, char *buf)
+bootmode_show(struct device *dev, struct device_attribute *attr, char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
char *bootmode;
@@ -333,7 +329,7 @@ mic_show_bootmode(struct device *dev, struct device_attribute *attr, char *buf)
}

static ssize_t
-mic_store_bootmode(struct device *dev, struct device_attribute *attr,
+bootmode_store(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -363,11 +359,10 @@ unlock:
mutex_unlock(&mdev->mic_mutex);
return count;
}
-static DEVICE_ATTR(bootmode, S_IRUGO | S_IWUSR,
- mic_show_bootmode, mic_store_bootmode);
+static DEVICE_ATTR_RW(bootmode);

static ssize_t
-mic_show_log_buf_addr(struct device *dev, struct device_attribute *attr,
+log_buf_addr_show(struct device *dev, struct device_attribute *attr,
char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -379,7 +374,7 @@ mic_show_log_buf_addr(struct device *dev, struct device_attribute *attr,
}

static ssize_t
-mic_store_log_buf_addr(struct device *dev, struct device_attribute *attr,
+log_buf_addr_store(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -398,11 +393,10 @@ mic_store_log_buf_addr(struct device *dev, struct device_attribute *attr,
exit:
return ret;
}
-static DEVICE_ATTR(log_buf_addr, S_IRUGO | S_IWUSR,
- mic_show_log_buf_addr, mic_store_log_buf_addr);
+static DEVICE_ATTR_RW(log_buf_addr);

static ssize_t
-mic_show_log_buf_len(struct device *dev, struct device_attribute *attr,
+log_buf_len_show(struct device *dev, struct device_attribute *attr,
char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -414,7 +408,7 @@ mic_show_log_buf_len(struct device *dev, struct device_attribute *attr,
}

static ssize_t
-mic_store_log_buf_len(struct device *dev, struct device_attribute *attr,
+log_buf_len_store(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -433,8 +427,7 @@ mic_store_log_buf_len(struct device *dev, struct device_attribute *attr,
exit:
return ret;
}
-static DEVICE_ATTR(log_buf_len, S_IRUGO | S_IWUSR,
- mic_show_log_buf_len, mic_store_log_buf_len);
+static DEVICE_ATTR_RW(log_buf_len);

static struct attribute *mic_default_attrs[] = {
&dev_attr_family.attr,
@@ -451,16 +444,9 @@ static struct attribute *mic_default_attrs[] = {
NULL
};

-static struct attribute_group mic_attr_group = {
- .attrs = mic_default_attrs,
-};
-
-static const struct attribute_group *__mic_attr_group[] = {
- &mic_attr_group,
- NULL
-};
+ATTRIBUTE_GROUPS(mic_default);

void mic_sysfs_init(struct mic_device *mdev)
{
- mdev->attr_group = __mic_attr_group;
+ mdev->attr_group = mic_default_groups;
}
--
1.8.2.1

2013-09-27 16:50:18

by Dutt, Sudeep

[permalink] [raw]
Subject: [PATCH char-misc-next 2/5] misc: mic: header file cleanups.

Dont use same name for header files in different folders.

These changes were suggested by Greg Kroah-Hartman during the
code review @ https://lkml.org/lkml/2013/9/6/18

Reported-by: Greg Kroah-Hartman <[email protected]>
Signed-off-by: Ashutosh Dixit <[email protected]>
Signed-off-by: Dasaratharaman Chandramouli <[email protected]>
Signed-off-by: Nikhil Rao <[email protected]>
Signed-off-by: Harshavardhan R Kharche <[email protected]>
Signed-off-by: Sudeep Dutt <[email protected]>
---
drivers/misc/mic/card/mic_debugfs.c | 2 +-
drivers/misc/mic/card/mic_device.c | 2 +-
drivers/misc/mic/card/mic_virtio.c | 2 +-
drivers/misc/mic/card/mic_x100.c | 2 +-
drivers/misc/mic/common/{mic_device.h => mic_dev.h} | 4 ++--
drivers/misc/mic/host/mic_boot.c | 2 +-
drivers/misc/mic/host/mic_debugfs.c | 2 +-
drivers/misc/mic/host/mic_fops.c | 2 +-
drivers/misc/mic/host/mic_intr.c | 2 +-
drivers/misc/mic/host/mic_main.c | 2 +-
drivers/misc/mic/host/mic_smpt.c | 2 +-
drivers/misc/mic/host/mic_sysfs.c | 2 +-
drivers/misc/mic/host/mic_virtio.c | 2 +-
drivers/misc/mic/host/mic_x100.c | 2 +-
14 files changed, 15 insertions(+), 15 deletions(-)
rename drivers/misc/mic/common/{mic_device.h => mic_dev.h} (95%)

diff --git a/drivers/misc/mic/card/mic_debugfs.c b/drivers/misc/mic/card/mic_debugfs.c
index 95cf186..421b3d7 100644
--- a/drivers/misc/mic/card/mic_debugfs.c
+++ b/drivers/misc/mic/card/mic_debugfs.c
@@ -30,7 +30,7 @@
#include <linux/interrupt.h>
#include <linux/device.h>

-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"

/* Debugfs parent dir */
diff --git a/drivers/misc/mic/card/mic_device.c b/drivers/misc/mic/card/mic_device.c
index 4125217..175d114 100644
--- a/drivers/misc/mic/card/mic_device.c
+++ b/drivers/misc/mic/card/mic_device.c
@@ -30,7 +30,7 @@
#include <linux/reboot.h>

#include <linux/mic_common.h>
-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_virtio.h"

diff --git a/drivers/misc/mic/card/mic_virtio.c b/drivers/misc/mic/card/mic_virtio.c
index 38275c1..7420c33 100644
--- a/drivers/misc/mic/card/mic_virtio.c
+++ b/drivers/misc/mic/card/mic_virtio.c
@@ -40,7 +40,7 @@
#include <linux/slab.h>
#include <linux/virtio_config.h>

-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_virtio.h"

#define VIRTIO_SUBCODE_64 0x0D00
diff --git a/drivers/misc/mic/card/mic_x100.c b/drivers/misc/mic/card/mic_x100.c
index 7cb3469..f491c91 100644
--- a/drivers/misc/mic/card/mic_x100.c
+++ b/drivers/misc/mic/card/mic_x100.c
@@ -28,7 +28,7 @@
#include <linux/pci.h>
#include <linux/platform_device.h>

-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_x100.h"

diff --git a/drivers/misc/mic/common/mic_device.h b/drivers/misc/mic/common/mic_dev.h
similarity index 95%
rename from drivers/misc/mic/common/mic_device.h
rename to drivers/misc/mic/common/mic_dev.h
index 01eb74f..92999c2 100644
--- a/drivers/misc/mic/common/mic_device.h
+++ b/drivers/misc/mic/common/mic_dev.h
@@ -18,8 +18,8 @@
* Intel MIC driver.
*
*/
-#ifndef __MIC_COMMON_DEVICE_H_
-#define __MIC_COMMON_DEVICE_H_
+#ifndef __MIC_DEV_H__
+#define __MIC_DEV_H__

/**
* struct mic_mw - MIC memory window
diff --git a/drivers/misc/mic/host/mic_boot.c b/drivers/misc/mic/host/mic_boot.c
index fd9ff6d..60c54d5 100644
--- a/drivers/misc/mic/host/mic_boot.c
+++ b/drivers/misc/mic/host/mic_boot.c
@@ -22,7 +22,7 @@
#include <linux/firmware.h>

#include <linux/mic_common.h>
-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_smpt.h"
#include "mic_virtio.h"
diff --git a/drivers/misc/mic/host/mic_debugfs.c b/drivers/misc/mic/host/mic_debugfs.c
index e22fb7b..98a61b4 100644
--- a/drivers/misc/mic/host/mic_debugfs.c
+++ b/drivers/misc/mic/host/mic_debugfs.c
@@ -23,7 +23,7 @@
#include <linux/seq_file.h>

#include <linux/mic_common.h>
-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_smpt.h"
#include "mic_virtio.h"
diff --git a/drivers/misc/mic/host/mic_fops.c b/drivers/misc/mic/host/mic_fops.c
index 661469a..e699c80 100644
--- a/drivers/misc/mic/host/mic_fops.c
+++ b/drivers/misc/mic/host/mic_fops.c
@@ -21,7 +21,7 @@
#include <linux/poll.h>

#include <linux/mic_common.h>
-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_fops.h"
#include "mic_virtio.h"
diff --git a/drivers/misc/mic/host/mic_intr.c b/drivers/misc/mic/host/mic_intr.c
index cdae314..71a7521 100644
--- a/drivers/misc/mic/host/mic_intr.c
+++ b/drivers/misc/mic/host/mic_intr.c
@@ -21,7 +21,7 @@
#include <linux/pci.h>
#include <linux/interrupt.h>

-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"

/*
diff --git a/drivers/misc/mic/host/mic_main.c b/drivers/misc/mic/host/mic_main.c
index a8965d4..ca06aa9 100644
--- a/drivers/misc/mic/host/mic_main.c
+++ b/drivers/misc/mic/host/mic_main.c
@@ -28,7 +28,7 @@
#include <linux/poll.h>

#include <linux/mic_common.h>
-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_x100.h"
#include "mic_smpt.h"
diff --git a/drivers/misc/mic/host/mic_smpt.c b/drivers/misc/mic/host/mic_smpt.c
index a346207..003d02b 100644
--- a/drivers/misc/mic/host/mic_smpt.c
+++ b/drivers/misc/mic/host/mic_smpt.c
@@ -20,7 +20,7 @@
*/
#include <linux/pci.h>

-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_smpt.h"

diff --git a/drivers/misc/mic/host/mic_sysfs.c b/drivers/misc/mic/host/mic_sysfs.c
index bebbae2..029a4f3 100644
--- a/drivers/misc/mic/host/mic_sysfs.c
+++ b/drivers/misc/mic/host/mic_sysfs.c
@@ -21,7 +21,7 @@
#include <linux/pci.h>

#include <linux/mic_common.h>
-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"

/*
diff --git a/drivers/misc/mic/host/mic_virtio.c b/drivers/misc/mic/host/mic_virtio.c
index be2a1f0..9e0456f 100644
--- a/drivers/misc/mic/host/mic_virtio.c
+++ b/drivers/misc/mic/host/mic_virtio.c
@@ -23,7 +23,7 @@
#include <linux/uaccess.h>

#include <linux/mic_common.h>
-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_smpt.h"
#include "mic_virtio.h"
diff --git a/drivers/misc/mic/host/mic_x100.c b/drivers/misc/mic/host/mic_x100.c
index a12ae5c..3a0d660 100644
--- a/drivers/misc/mic/host/mic_x100.c
+++ b/drivers/misc/mic/host/mic_x100.c
@@ -24,7 +24,7 @@
#include <linux/firmware.h>
#include <linux/delay.h>

-#include "../common/mic_device.h"
+#include "../common/mic_dev.h"
#include "mic_device.h"
#include "mic_x100.h"
#include "mic_smpt.h"
--
1.8.2.1

2013-09-27 16:50:55

by Dutt, Sudeep

[permalink] [raw]
Subject: [PATCH char-misc-next 3/5] misc: mic: cleanups for "--strict" checkpatch.

From: Ashutosh Dixit <[email protected]>

These changes were mostly authored by Joe Perches <[email protected]>
@ https://lkml.org/lkml/2013/9/5/602

Reported-by: Joe Perches <[email protected]>
Signed-off-by: Ashutosh Dixit <[email protected]>
Signed-off-by: Dasaratharaman Chandramouli <[email protected]>
Signed-off-by: Nikhil Rao <[email protected]>
Signed-off-by: Harshavardhan R Kharche <[email protected]>
Signed-off-by: Sudeep Dutt <[email protected]>
---
Documentation/mic/mpssd/mpssd.c | 65 ++++++++++++++-------------
Documentation/mic/mpssd/sysfs.c | 4 +-
drivers/misc/mic/card/mic_device.c | 1 -
drivers/misc/mic/card/mic_virtio.c | 17 ++++---
drivers/misc/mic/card/mic_x100.c | 4 +-
drivers/misc/mic/host/mic_debugfs.c | 90 ++++++++++++++++++-------------------
drivers/misc/mic/host/mic_fops.c | 6 +--
drivers/misc/mic/host/mic_intr.c | 34 +++++++-------
drivers/misc/mic/host/mic_smpt.c | 14 +++---
drivers/misc/mic/host/mic_sysfs.c | 20 ++++-----
drivers/misc/mic/host/mic_virtio.c | 37 +++++++--------
drivers/misc/mic/host/mic_x100.c | 33 +++++++-------
12 files changed, 158 insertions(+), 167 deletions(-)

diff --git a/Documentation/mic/mpssd/mpssd.c b/Documentation/mic/mpssd/mpssd.c
index 8064804..f9327a2 100644
--- a/Documentation/mic/mpssd/mpssd.c
+++ b/Documentation/mic/mpssd/mpssd.c
@@ -246,7 +246,7 @@ static int tun_alloc(struct mic_info *mic, char *dev)
if (*dev)
strncpy(ifr.ifr_name, dev, IFNAMSIZ);

- err = ioctl(fd, TUNSETIFF, (void *) &ifr);
+ err = ioctl(fd, TUNSETIFF, (void *)&ifr);
if (err < 0) {
mpsslog("%s %s %d TUNSETIFF failed %s\n",
mic->name, __func__, __LINE__, strerror(errno));
@@ -363,7 +363,7 @@ static inline void verify_out_len(struct mic_info *mic,
{
if (copy->out_len != sum_iovec_len(copy)) {
mpsslog("%s %s %d BUG copy->out_len 0x%x len 0x%x\n",
- mic->name, __func__, __LINE__,
+ mic->name, __func__, __LINE__,
copy->out_len, sum_iovec_len(copy));
assert(copy->out_len == sum_iovec_len(copy));
}
@@ -372,7 +372,7 @@ static inline void verify_out_len(struct mic_info *mic,
/* Display an iovec */
static void
disp_iovec(struct mic_info *mic, struct mic_copy_desc *copy,
- const char *s, int line)
+ const char *s, int line)
{
int i;

@@ -401,7 +401,7 @@ static inline void txrx_prepare(int type, bool tx, struct mic_vring *vr,
/* Central API which triggers the copies */
static int
mic_virtio_copy(struct mic_info *mic, int fd,
- struct mic_vring *vr, struct mic_copy_desc *copy)
+ struct mic_vring *vr, struct mic_copy_desc *copy)
{
int ret;

@@ -440,7 +440,7 @@ init_vr(struct mic_info *mic, int fd, int type,
vr0->info = vr0->va +
vring_size(MIC_VRING_ENTRIES, MIC_VIRTIO_RING_ALIGN);
vring_init(&vr0->vr,
- MIC_VRING_ENTRIES, vr0->va, MIC_VIRTIO_RING_ALIGN);
+ MIC_VRING_ENTRIES, vr0->va, MIC_VIRTIO_RING_ALIGN);
mpsslog("%s %s vr0 %p vr0->info %p vr_size 0x%x vring 0x%x ",
__func__, mic->name, vr0->va, vr0->info, vr_size,
vring_size(MIC_VRING_ENTRIES, MIC_VIRTIO_RING_ALIGN));
@@ -453,7 +453,7 @@ init_vr(struct mic_info *mic, int fd, int type,
vr1->info = vr1->va + vring_size(MIC_VRING_ENTRIES,
MIC_VIRTIO_RING_ALIGN);
vring_init(&vr1->vr,
- MIC_VRING_ENTRIES, vr1->va, MIC_VIRTIO_RING_ALIGN);
+ MIC_VRING_ENTRIES, vr1->va, MIC_VIRTIO_RING_ALIGN);
mpsslog("%s %s vr1 %p vr1->info %p vr_size 0x%x vring 0x%x ",
__func__, mic->name, vr1->va, vr1->info, vr_size,
vring_size(MIC_VRING_ENTRIES, MIC_VIRTIO_RING_ALIGN));
@@ -551,7 +551,7 @@ virtio_net(void *arg)
net_poll[NET_FD_TUN].events = POLLIN;

if (MAP_FAILED == init_vr(mic, mic->mic_net.virtio_net_fd,
- VIRTIO_ID_NET, &tx_vr, &rx_vr,
+ VIRTIO_ID_NET, &tx_vr, &rx_vr,
virtnet_dev_page.dd.num_vq)) {
mpsslog("%s init_vr failed %s\n",
mic->name, strerror(errno));
@@ -576,7 +576,7 @@ virtio_net(void *arg)
}
if (!(desc->status & VIRTIO_CONFIG_S_DRIVER_OK))
wait_for_card_driver(mic, mic->mic_net.virtio_net_fd,
- VIRTIO_ID_NET);
+ VIRTIO_ID_NET);
/*
* Check if there is data to be read from TUN and write to
* virtio net fd if there is.
@@ -587,7 +587,7 @@ virtio_net(void *arg)
copy.iov, copy.iovcnt);
if (len > 0) {
struct virtio_net_hdr *hdr
- = (struct virtio_net_hdr *) vnet_hdr[0];
+ = (struct virtio_net_hdr *)vnet_hdr[0];

/* Disable checksums on the card since we are on
a reliable PCIe link */
@@ -606,7 +606,7 @@ virtio_net(void *arg)
#endif
spin_for_descriptors(mic, &tx_vr);
txrx_prepare(VIRTIO_ID_NET, 1, &tx_vr, &copy,
- len);
+ len);

err = mic_virtio_copy(mic,
mic->mic_net.virtio_net_fd, &tx_vr,
@@ -644,7 +644,7 @@ virtio_net(void *arg)
le16toh(rx_vr.vr.avail->idx)) {
copy.iov = iov1;
txrx_prepare(VIRTIO_ID_NET, 0, &rx_vr, &copy,
- MAX_NET_PKT_SIZE
+ MAX_NET_PKT_SIZE
+ sizeof(struct virtio_net_hdr));

err = mic_virtio_copy(mic,
@@ -669,7 +669,7 @@ virtio_net(void *arg)
verify_out_len(mic, &copy);
#ifdef DEBUG
disp_iovec(mic, copy, __func__,
- __LINE__);
+ __LINE__);
mpsslog("%s %s %d ",
mic->name, __func__, __LINE__);
mpsslog("read from net 0x%lx\n",
@@ -686,7 +686,7 @@ virtio_net(void *arg)
} else {
#ifdef DEBUG
disp_iovec(mic, &copy, __func__,
- __LINE__);
+ __LINE__);
mpsslog("%s %s %d ",
mic->name, __func__,
__LINE__);
@@ -750,13 +750,13 @@ virtio_console(void *arg)
err = grantpt(pty_fd);
if (err < 0) {
mpsslog("can't grant access: %s %s\n",
- pts_name, strerror(errno));
+ pts_name, strerror(errno));
goto _close_pty;
}
err = unlockpt(pty_fd);
if (err < 0) {
mpsslog("can't unlock a pseudoterminal: %s %s\n",
- pts_name, strerror(errno));
+ pts_name, strerror(errno));
goto _close_pty;
}
console_poll[MONITOR_FD].fd = pty_fd;
@@ -766,7 +766,7 @@ virtio_console(void *arg)
console_poll[VIRTIO_CONSOLE_FD].events = POLLIN;

if (MAP_FAILED == init_vr(mic, mic->mic_console.virtio_console_fd,
- VIRTIO_ID_CONSOLE, &tx_vr, &rx_vr,
+ VIRTIO_ID_CONSOLE, &tx_vr, &rx_vr,
virtcons_dev_page.dd.num_vq)) {
mpsslog("%s init_vr failed %s\n",
mic->name, strerror(errno));
@@ -787,7 +787,7 @@ virtio_console(void *arg)
}
if (!(desc->status & VIRTIO_CONFIG_S_DRIVER_OK))
wait_for_card_driver(mic,
- mic->mic_console.virtio_console_fd,
+ mic->mic_console.virtio_console_fd,
VIRTIO_ID_CONSOLE);

if (console_poll[MONITOR_FD].revents & POLLIN) {
@@ -802,7 +802,7 @@ virtio_console(void *arg)
#endif
spin_for_descriptors(mic, &tx_vr);
txrx_prepare(VIRTIO_ID_CONSOLE, 1, &tx_vr,
- &copy, len);
+ &copy, len);

err = mic_virtio_copy(mic,
mic->mic_console.virtio_console_fd,
@@ -837,7 +837,7 @@ virtio_console(void *arg)
le16toh(rx_vr.vr.avail->idx)) {
copy.iov = iov1;
txrx_prepare(VIRTIO_ID_CONSOLE, 0, &rx_vr,
- &copy, PAGE_SIZE);
+ &copy, PAGE_SIZE);

err = mic_virtio_copy(mic,
mic->mic_console.virtio_console_fd,
@@ -848,7 +848,7 @@ virtio_console(void *arg)
verify_out_len(mic, &copy);
#ifdef DEBUG
disp_iovec(mic, copy, __func__,
- __LINE__);
+ __LINE__);
mpsslog("%s %s %d ",
mic->name, __func__, __LINE__);
mpsslog("read from net 0x%lx\n",
@@ -865,7 +865,7 @@ virtio_console(void *arg)
} else {
#ifdef DEBUG
disp_iovec(mic, copy, __func__,
- __LINE__);
+ __LINE__);
mpsslog("%s %s %d ",
mic->name, __func__,
__LINE__);
@@ -1033,7 +1033,8 @@ start_virtblk(struct mic_info *mic, struct mic_vring *vring)
}
add_virtio_device(mic, &virtblk_dev_page.dd);
if (MAP_FAILED == init_vr(mic, mic->mic_virtblk.virtio_block_fd,
- VIRTIO_ID_BLOCK, vring, NULL, virtblk_dev_page.dd.num_vq)) {
+ VIRTIO_ID_BLOCK, vring, NULL,
+ virtblk_dev_page.dd.num_vq)) {
mpsslog("%s init_vr failed %s\n",
mic->name, strerror(errno));
return false;
@@ -1060,7 +1061,7 @@ header_error_check(struct vring_desc *desc)
{
if (le32toh(desc->len) != sizeof(struct virtio_blk_outhdr)) {
mpsslog("%s() %d: length is not sizeof(virtio_blk_outhd)\n",
- __func__, __LINE__);
+ __func__, __LINE__);
return -EIO;
}
if (!(le16toh(desc->flags) & VRING_DESC_F_NEXT)) {
@@ -1132,7 +1133,7 @@ write_status(int fd, __u8 *status)
static void *
virtio_block(void *arg)
{
- struct mic_info *mic = (struct mic_info *) arg;
+ struct mic_info *mic = (struct mic_info *)arg;
int ret;
struct pollfd block_poll;
struct mic_vring vring;
@@ -1219,8 +1220,8 @@ virtio_block(void *arg)
status = 0;
fos = mic->mic_virtblk.backend_addr +
(hdr.sector * SECTOR_SIZE);
- buffer_desc_idx = desc_idx =
- next_desc(desc);
+ buffer_desc_idx = next_desc(desc);
+ desc_idx = buffer_desc_idx;
for (desc = &vring.vr.desc[buffer_desc_idx];
desc->flags & VRING_DESC_F_NEXT;
desc_idx = next_desc(desc),
@@ -1255,7 +1256,7 @@ virtio_block(void *arg)
iovec,
piov - iovec);
if (ret < 0 &&
- status != 0)
+ status != 0)
status = ret;
}
/* write status and update used pointer */
@@ -1391,7 +1392,7 @@ mic_config(void *arg)
char value[4096];

snprintf(pathname, PATH_MAX - 1, "%s/%s/%s",
- MICSYSFSDIR, mic->name, "state");
+ MICSYSFSDIR, mic->name, "state");

fd = open(pathname, O_RDONLY);
if (fd < 0) {
@@ -1560,22 +1561,22 @@ init_mic(struct mic_info *mic)
virtio_console, mic);
if (err)
mpsslog("%s virtcons pthread_create failed %s\n",
- mic->name, strerror(err));
+ mic->name, strerror(err));
err = pthread_create(&mic->mic_net.net_thread, NULL,
virtio_net, mic);
if (err)
mpsslog("%s virtnet pthread_create failed %s\n",
- mic->name, strerror(err));
+ mic->name, strerror(err));
err = pthread_create(&mic->mic_virtblk.block_thread, NULL,
virtio_block, mic);
if (err)
mpsslog("%s virtblk pthread_create failed %s\n",
- mic->name, strerror(err));
+ mic->name, strerror(err));
sigemptyset(&act.sa_mask);
err = sigaction(SIGUSR1, &act, NULL);
if (err)
mpsslog("%s sigaction SIGUSR1 failed %s\n",
- mic->name, strerror(errno));
+ mic->name, strerror(errno));
while (1)
sleep(60);
case -1:
diff --git a/Documentation/mic/mpssd/sysfs.c b/Documentation/mic/mpssd/sysfs.c
index 11de72b..8dd3269 100644
--- a/Documentation/mic/mpssd/sysfs.c
+++ b/Documentation/mic/mpssd/sysfs.c
@@ -35,7 +35,7 @@ readsysfs(char *dir, char *entry)
snprintf(filename, PATH_MAX, "%s/%s", MICSYSFSDIR, entry);
else
snprintf(filename, PATH_MAX,
- "%s/%s/%s", MICSYSFSDIR, dir, entry);
+ "%s/%s/%s", MICSYSFSDIR, dir, entry);

fd = open(filename, O_RDONLY);
if (fd < 0) {
@@ -75,7 +75,7 @@ setsysfs(char *dir, char *entry, char *value)
snprintf(filename, PATH_MAX, "%s/%s", MICSYSFSDIR, entry);
else
snprintf(filename, PATH_MAX, "%s/%s/%s",
- MICSYSFSDIR, dir, entry);
+ MICSYSFSDIR, dir, entry);

oldvalue = readsysfs(dir, entry);

diff --git a/drivers/misc/mic/card/mic_device.c b/drivers/misc/mic/card/mic_device.c
index 175d114..d0980ff 100644
--- a/drivers/misc/mic/card/mic_device.c
+++ b/drivers/misc/mic/card/mic_device.c
@@ -167,7 +167,6 @@ struct mic_irq *mic_request_card_irq(irqreturn_t (*func)(int irq, void *data),
return (struct mic_irq *)cookie;
err:
return ERR_PTR(rc);
-
}

/**
diff --git a/drivers/misc/mic/card/mic_virtio.c b/drivers/misc/mic/card/mic_virtio.c
index 7420c33..914cc9b 100644
--- a/drivers/misc/mic/card/mic_virtio.c
+++ b/drivers/misc/mic/card/mic_virtio.c
@@ -103,7 +103,7 @@ static void mic_finalize_features(struct virtio_device *vdev)
for (i = 0; i < bits; i++) {
if (test_bit(i, vdev->features))
iowrite8(ioread8(&out_features[i / 8]) | (1 << (i % 8)),
- &out_features[i / 8]);
+ &out_features[i / 8]);
}
}

@@ -197,10 +197,9 @@ static void mic_notify(struct virtqueue *vq)
static void mic_del_vq(struct virtqueue *vq, int n)
{
struct mic_vdev *mvdev = to_micvdev(vq->vdev);
- struct vring *vr = (struct vring *) (vq + 1);
+ struct vring *vr = (struct vring *)(vq + 1);

- free_pages((unsigned long) vr->used,
- get_order(mvdev->used_size[n]));
+ free_pages((unsigned long) vr->used, get_order(mvdev->used_size[n]));
vring_del_virtqueue(vq);
mic_card_unmap(mvdev->mdev, mvdev->vr[n]);
mvdev->vr[n] = NULL;
@@ -274,8 +273,8 @@ static struct virtqueue *mic_find_vq(struct virtio_device *vdev,
/* Allocate and reassign used ring now */
mvdev->used_size[index] = PAGE_ALIGN(sizeof(__u16) * 3 +
sizeof(struct vring_used_elem) * config.num);
- used = (void *) __get_free_pages(GFP_KERNEL | __GFP_ZERO,
- get_order(mvdev->used_size[index]));
+ used = (void *)__get_free_pages(GFP_KERNEL | __GFP_ZERO,
+ get_order(mvdev->used_size[index]));
if (!used) {
err = -ENOMEM;
dev_err(mic_dev(mvdev), "%s %d err %d\n",
@@ -291,7 +290,7 @@ static struct virtqueue *mic_find_vq(struct virtio_device *vdev,
* vring_new_virtqueue() would ensure that
* (&vq->vring == (struct vring *) (&vq->vq + 1));
*/
- vr = (struct vring *) (vq + 1);
+ vr = (struct vring *)(vq + 1);
vr->used = used;

vq->priv = mvdev;
@@ -544,7 +543,7 @@ static void mic_scan_devices(struct mic_driver *mdrv, bool remove)
if (dev) {
if (remove)
iowrite8(MIC_VIRTIO_PARAM_DEV_REMOVE,
- &dc->config_change);
+ &dc->config_change);
put_device(dev);
mic_handle_config_change(d, i, mdrv);
ret = mic_remove_device(d, i, mdrv);
@@ -559,7 +558,7 @@ static void mic_scan_devices(struct mic_driver *mdrv, bool remove)

/* new device */
dev_dbg(mdrv->dev, "%s %d Adding new virtio device %p\n",
- __func__, __LINE__, d);
+ __func__, __LINE__, d);
if (!remove)
mic_add_device(d, i, mdrv);
}
diff --git a/drivers/misc/mic/card/mic_x100.c b/drivers/misc/mic/card/mic_x100.c
index f491c91..2868945 100644
--- a/drivers/misc/mic/card/mic_x100.c
+++ b/drivers/misc/mic/card/mic_x100.c
@@ -66,8 +66,8 @@ void mic_send_intr(struct mic_device *mdev, int doorbell)
/* Ensure that the interrupt is ordered w.r.t previous stores. */
wmb();
mic_mmio_write(mw, MIC_X100_SBOX_SDBIC0_DBREQ_BIT,
- MIC_X100_SBOX_BASE_ADDRESS +
- (MIC_X100_SBOX_SDBIC0 + (4 * doorbell)));
+ MIC_X100_SBOX_BASE_ADDRESS +
+ (MIC_X100_SBOX_SDBIC0 + (4 * doorbell)));
}

/**
diff --git a/drivers/misc/mic/host/mic_debugfs.c b/drivers/misc/mic/host/mic_debugfs.c
index 98a61b4..028ba5d6 100644
--- a/drivers/misc/mic/host/mic_debugfs.c
+++ b/drivers/misc/mic/host/mic_debugfs.c
@@ -103,7 +103,7 @@ static int mic_smpt_show(struct seq_file *s, void *pos)
unsigned long flags;

seq_printf(s, "MIC %-2d |%-10s| %-14s %-10s\n",
- mdev->id, "SMPT entry", "SW DMA addr", "RefCount");
+ mdev->id, "SMPT entry", "SW DMA addr", "RefCount");
seq_puts(s, "====================================================\n");

if (mdev->smpt) {
@@ -111,8 +111,8 @@ static int mic_smpt_show(struct seq_file *s, void *pos)
spin_lock_irqsave(&smpt_info->smpt_lock, flags);
for (i = 0; i < smpt_info->info.num_reg; i++) {
seq_printf(s, "%9s|%-10d| %-#14llx %-10lld\n",
- " ", i, smpt_info->entry[i].dma_addr,
- smpt_info->entry[i].ref_count);
+ " ", i, smpt_info->entry[i].dma_addr,
+ smpt_info->entry[i].ref_count);
}
spin_unlock_irqrestore(&smpt_info->smpt_lock, flags);
}
@@ -203,17 +203,17 @@ static int mic_dp_show(struct seq_file *s, void *pos)
int i, j;

seq_printf(s, "Bootparam: magic 0x%x\n",
- bootparam->magic);
+ bootparam->magic);
seq_printf(s, "Bootparam: h2c_shutdown_db %d\n",
- bootparam->h2c_shutdown_db);
+ bootparam->h2c_shutdown_db);
seq_printf(s, "Bootparam: h2c_config_db %d\n",
- bootparam->h2c_config_db);
+ bootparam->h2c_config_db);
seq_printf(s, "Bootparam: c2h_shutdown_db %d\n",
- bootparam->c2h_shutdown_db);
+ bootparam->c2h_shutdown_db);
seq_printf(s, "Bootparam: shutdown_status %d\n",
- bootparam->shutdown_status);
+ bootparam->shutdown_status);
seq_printf(s, "Bootparam: shutdown_card %d\n",
- bootparam->shutdown_card);
+ bootparam->shutdown_card);

for (i = sizeof(*bootparam); i < MIC_DP_SIZE;
i += mic_total_desc_size(d)) {
@@ -239,10 +239,10 @@ static int mic_dp_show(struct seq_file *s, void *pos)
seq_printf(s, "address 0x%llx ", vqconfig->address);
seq_printf(s, "num %d ", vqconfig->num);
seq_printf(s, "used address 0x%llx\n",
- vqconfig->used_address);
+ vqconfig->used_address);
}

- features = (__u32 *) mic_vq_features(d);
+ features = (__u32 *)mic_vq_features(d);
seq_printf(s, "Features: Host 0x%x ", features[0]);
seq_printf(s, "Guest 0x%x\n", features[1]);

@@ -256,7 +256,7 @@ static int mic_dp_show(struct seq_file *s, void *pos)
seq_printf(s, "Guest Ack %d ", dc->guest_ack);
seq_printf(s, "Host ack %d\n", dc->host_ack);
seq_printf(s, "Used address updated %d ",
- dc->used_address_updated);
+ dc->used_address_updated);
seq_printf(s, "Vdev 0x%llx\n", dc->vdev);
seq_printf(s, "c2h doorbell %d ", dc->c2h_vdev_db);
seq_printf(s, "h2c doorbell %d\n", dc->h2c_vdev_db);
@@ -294,10 +294,10 @@ static int mic_vdev_info_show(struct seq_file *s, void *unused)
list_for_each_safe(pos, tmp, &mdev->vdev_list) {
mvdev = list_entry(pos, struct mic_vdev, list);
seq_printf(s, "VDEV type %d state %s in %ld out %ld\n",
- mvdev->virtio_id,
- mic_vdevup(mvdev) ? "UP" : "DOWN",
- mvdev->in_bytes,
- mvdev->out_bytes);
+ mvdev->virtio_id,
+ mic_vdevup(mvdev) ? "UP" : "DOWN",
+ mvdev->in_bytes,
+ mvdev->out_bytes);
for (i = 0; i < MIC_MAX_VRINGS; i++) {
struct vring_desc *desc;
struct vring_avail *avail;
@@ -309,38 +309,38 @@ static int mic_vdev_info_show(struct seq_file *s, void *unused)
continue;
desc = vrh->vring.desc;
seq_printf(s, "vring i %d avail_idx %d",
- i, mvr->vring.info->avail_idx & (num - 1));
+ i, mvr->vring.info->avail_idx & (num - 1));
seq_printf(s, " vring i %d avail_idx %d\n",
- i, mvr->vring.info->avail_idx);
+ i, mvr->vring.info->avail_idx);
seq_printf(s, "vrh i %d weak_barriers %d",
- i, vrh->weak_barriers);
+ i, vrh->weak_barriers);
seq_printf(s, " last_avail_idx %d last_used_idx %d",
- vrh->last_avail_idx, vrh->last_used_idx);
+ vrh->last_avail_idx, vrh->last_used_idx);
seq_printf(s, " completed %d\n", vrh->completed);
for (j = 0; j < num; j++) {
seq_printf(s, "desc[%d] addr 0x%llx len %d",
- j, desc->addr, desc->len);
+ j, desc->addr, desc->len);
seq_printf(s, " flags 0x%x next %d\n",
- desc->flags,
- desc->next);
+ desc->flags, desc->next);
desc++;
}
avail = vrh->vring.avail;
seq_printf(s, "avail flags 0x%x idx %d\n",
- avail->flags, avail->idx & (num - 1));
+ avail->flags, avail->idx & (num - 1));
seq_printf(s, "avail flags 0x%x idx %d\n",
- avail->flags, avail->idx);
+ avail->flags, avail->idx);
for (j = 0; j < num; j++)
seq_printf(s, "avail ring[%d] %d\n",
- j, avail->ring[j]);
+ j, avail->ring[j]);
used = vrh->vring.used;
seq_printf(s, "used flags 0x%x idx %d\n",
- used->flags, used->idx & (num - 1));
+ used->flags, used->idx & (num - 1));
seq_printf(s, "used flags 0x%x idx %d\n",
- used->flags, used->idx);
+ used->flags, used->idx);
for (j = 0; j < num; j++)
seq_printf(s, "used ring[%d] id %d len %d\n",
- j, used->ring[j].id, used->ring[j].len);
+ j, used->ring[j].id,
+ used->ring[j].len);
}
}
mutex_unlock(&mdev->mic_mutex);
@@ -389,7 +389,7 @@ static int mic_msi_irq_info_show(struct seq_file *s, void *pos)
reg = mdev->intr_ops->read_msi_to_src_map(mdev, entry);

seq_printf(s, "%s %-10d %s %-10d MXAR[%d]: %08X\n",
- "IRQ:", vector, "Entry:", entry, i, reg);
+ "IRQ:", vector, "Entry:", entry, i, reg);

seq_printf(s, "%-10s", "offset:");
for (j = (MIC_NUM_OFFSETS - 1); j >= 0; j--)
@@ -400,8 +400,8 @@ static int mic_msi_irq_info_show(struct seq_file *s, void *pos)
seq_printf(s, "%-10s", "count:");
for (j = (MIC_NUM_OFFSETS - 1); j >= 0; j--)
seq_printf(s, "%4d ",
- (mdev->irq_info.mic_msi_map[i] & BIT(j)) ?
- 1 : 0);
+ (mdev->irq_info.mic_msi_map[i] &
+ BIT(j)) ? 1 : 0);
seq_puts(s, "\n\n");
}
} else {
@@ -409,7 +409,6 @@ static int mic_msi_irq_info_show(struct seq_file *s, void *pos)
}

return 0;
-
}

static int mic_msi_irq_info_debug_open(struct inode *inode, struct file *file)
@@ -443,26 +442,23 @@ void mic_create_debug_dir(struct mic_device *mdev)
if (!mdev->dbg_dir)
return;

- debugfs_create_file("log_buf", 0444, mdev->dbg_dir,
- mdev, &log_buf_ops);
+ debugfs_create_file("log_buf", 0444, mdev->dbg_dir, mdev, &log_buf_ops);

- debugfs_create_file("smpt", 0444, mdev->dbg_dir,
- mdev, &smpt_file_ops);
+ debugfs_create_file("smpt", 0444, mdev->dbg_dir, mdev, &smpt_file_ops);

- debugfs_create_file("soft_reset", 0444, mdev->dbg_dir,
- mdev, &soft_reset_ops);
+ debugfs_create_file("soft_reset", 0444, mdev->dbg_dir, mdev,
+ &soft_reset_ops);

- debugfs_create_file("post_code", 0444, mdev->dbg_dir,
- mdev, &post_code_ops);
+ debugfs_create_file("post_code", 0444, mdev->dbg_dir, mdev,
+ &post_code_ops);

- debugfs_create_file("dp", 0444, mdev->dbg_dir,
- mdev, &dp_ops);
+ debugfs_create_file("dp", 0444, mdev->dbg_dir, mdev, &dp_ops);

- debugfs_create_file("vdev_info", 0444, mdev->dbg_dir,
- mdev, &vdev_info_ops);
+ debugfs_create_file("vdev_info", 0444, mdev->dbg_dir, mdev,
+ &vdev_info_ops);

- debugfs_create_file("msi_irq_info", 0444, mdev->dbg_dir,
- mdev, &msi_irq_info_ops);
+ debugfs_create_file("msi_irq_info", 0444, mdev->dbg_dir, mdev,
+ &msi_irq_info_ops);
}

/**
diff --git a/drivers/misc/mic/host/mic_fops.c b/drivers/misc/mic/host/mic_fops.c
index e699c80..8dc6ff1 100644
--- a/drivers/misc/mic/host/mic_fops.c
+++ b/drivers/misc/mic/host/mic_fops.c
@@ -140,9 +140,9 @@ unsigned int mic_poll(struct file *f, poll_table *wait)

poll_wait(f, &mvdev->waitq, wait);

- if (mic_vdev_inited(mvdev))
+ if (mic_vdev_inited(mvdev)) {
mask = POLLERR;
- else if (mvdev->poll_wake) {
+ } else if (mvdev->poll_wake) {
mvdev->poll_wake = 0;
mask = POLLIN | POLLOUT;
}
@@ -152,7 +152,7 @@ unsigned int mic_poll(struct file *f, poll_table *wait)

static inline int
mic_query_offset(struct mic_vdev *mvdev, unsigned long offset,
- unsigned long *size, unsigned long *pa)
+ unsigned long *size, unsigned long *pa)
{
struct mic_device *mdev = mvdev->mdev;
unsigned long start = MIC_DP_SIZE;
diff --git a/drivers/misc/mic/host/mic_intr.c b/drivers/misc/mic/host/mic_intr.c
index 71a7521..f9c29bc 100644
--- a/drivers/misc/mic/host/mic_intr.c
+++ b/drivers/misc/mic/host/mic_intr.c
@@ -71,8 +71,8 @@ static irqreturn_t mic_interrupt(int irq, void *dev)

/* Return the interrupt offset from the index. Index is 0 based. */
static u16 mic_map_src_to_offset(struct mic_device *mdev,
- int intr_src, enum mic_intr_type type) {
-
+ int intr_src, enum mic_intr_type type)
+{
if (type >= MIC_NUM_INTR_TYPES)
return MIC_NUM_OFFSETS;
if (intr_src >= mdev->intr_info->intr_len[type])
@@ -112,7 +112,7 @@ static struct mic_intr_cb *mic_register_intr_callback(struct mic_device *mdev,
struct mic_intr_cb *intr_cb;
unsigned long flags;
int rc;
- intr_cb = kmalloc(sizeof(struct mic_intr_cb), GFP_KERNEL);
+ intr_cb = kmalloc(sizeof(*intr_cb), GFP_KERNEL);

if (!intr_cb)
return ERR_PTR(-ENOMEM);
@@ -159,7 +159,7 @@ static u8 mic_unregister_intr_callback(struct mic_device *mdev, u32 idx)
if (intr_cb->cb_id == idx) {
list_del(pos);
ida_simple_remove(&mdev->irq_info.cb_ida,
- intr_cb->cb_id);
+ intr_cb->cb_id);
kfree(intr_cb);
spin_unlock_irqrestore(
&mdev->irq_info.mic_intr_lock, flags);
@@ -182,9 +182,10 @@ static u8 mic_unregister_intr_callback(struct mic_device *mdev, u32 idx)
static int mic_setup_msix(struct mic_device *mdev, struct pci_dev *pdev)
{
int rc, i;
+ int entry_size = sizeof(*mdev->irq_info.msix_entries);

- mdev->irq_info.msix_entries = kmalloc(sizeof(struct msix_entry) *
- MIC_MIN_MSIX, GFP_KERNEL);
+ mdev->irq_info.msix_entries = kmalloc_array(MIC_MIN_MSIX,
+ entry_size, GFP_KERNEL);
if (!mdev->irq_info.msix_entries) {
rc = -ENOMEM;
goto err_nomem1;
@@ -231,8 +232,9 @@ static int mic_setup_callbacks(struct mic_device *mdev)
{
int i;

- mdev->irq_info.cb_list = kmalloc(sizeof(struct list_head) *
- MIC_NUM_OFFSETS, GFP_KERNEL);
+ mdev->irq_info.cb_list = kmalloc_array(MIC_NUM_OFFSETS,
+ sizeof(*mdev->irq_info.cb_list),
+ GFP_KERNEL);
if (!mdev->irq_info.cb_list)
return -ENOMEM;

@@ -261,7 +263,7 @@ static void mic_release_callbacks(struct mic_device *mdev)

if (list_empty(&mdev->irq_info.cb_list[i])) {
spin_unlock_irqrestore(&mdev->irq_info.mic_intr_lock,
- flags);
+ flags);
break;
}

@@ -269,7 +271,7 @@ static void mic_release_callbacks(struct mic_device *mdev)
intr_cb = list_entry(pos, struct mic_intr_cb, list);
list_del(pos);
ida_simple_remove(&mdev->irq_info.cb_ida,
- intr_cb->cb_id);
+ intr_cb->cb_id);
kfree(intr_cb);
}
spin_unlock_irqrestore(&mdev->irq_info.mic_intr_lock, flags);
@@ -427,8 +429,8 @@ struct mic_irq *mic_request_irq(struct mic_device *mdev,
offset = mic_map_src_to_offset(mdev, intr_src, type);
if (offset >= MIC_NUM_OFFSETS) {
dev_err(mdev->sdev->parent,
- "Error mapping index %d to a valid source id.\n",
- intr_src);
+ "Error mapping index %d to a valid source id.\n",
+ intr_src);
rc = -EINVAL;
goto err;
}
@@ -437,7 +439,7 @@ struct mic_irq *mic_request_irq(struct mic_device *mdev,
msix = mic_get_available_vector(mdev);
if (!msix) {
dev_err(mdev->sdev->parent,
- "No MSIx vectors available for use.\n");
+ "No MSIx vectors available for use.\n");
rc = -ENOSPC;
goto err;
}
@@ -460,7 +462,7 @@ struct mic_irq *mic_request_irq(struct mic_device *mdev,
offset, func, data);
if (IS_ERR(intr_cb)) {
dev_err(mdev->sdev->parent,
- "No available callback entries for use\n");
+ "No available callback entries for use\n");
rc = PTR_ERR(intr_cb);
goto err;
}
@@ -506,7 +508,7 @@ void mic_free_irq(struct mic_device *mdev,
if (mdev->irq_info.num_vectors > 1) {
if (entry >= mdev->irq_info.num_vectors) {
dev_warn(mdev->sdev->parent,
- "entry %d should be < num_irq %d\n",
+ "entry %d should be < num_irq %d\n",
entry, mdev->irq_info.num_vectors);
return;
}
@@ -581,7 +583,7 @@ void mic_free_interrupts(struct mic_device *mdev, struct pci_dev *pdev)
for (i = 0; i < mdev->irq_info.num_vectors; i++) {
if (mdev->irq_info.mic_msi_map[i])
dev_warn(&pdev->dev, "irq %d may still be in use.\n",
- mdev->irq_info.msix_entries[i].vector);
+ mdev->irq_info.msix_entries[i].vector);
}
kfree(mdev->irq_info.mic_msi_map);
kfree(mdev->irq_info.msix_entries);
diff --git a/drivers/misc/mic/host/mic_smpt.c b/drivers/misc/mic/host/mic_smpt.c
index 003d02b..fae474c 100644
--- a/drivers/misc/mic/host/mic_smpt.c
+++ b/drivers/misc/mic/host/mic_smpt.c
@@ -84,7 +84,7 @@ static void mic_add_smpt_entry(int spt, s64 *ref, u64 addr,
for (i = spt; i < spt + entries; i++,
addr += smpt_info->info.page_size) {
if (!smpt_info->entry[i].ref_count &&
- (smpt_info->entry[i].dma_addr != addr)) {
+ (smpt_info->entry[i].dma_addr != addr)) {
mdev->smpt_ops->set(mdev, addr, i);
smpt_info->entry[i].dma_addr = addr;
}
@@ -183,7 +183,7 @@ mic_to_dma_addr(struct mic_device *mdev, dma_addr_t mic_addr)

if (!mic_is_system_addr(mdev, mic_addr)) {
dev_err(mdev->sdev->parent,
- "mic_addr is invalid. mic_addr = 0x%llx\n", mic_addr);
+ "mic_addr is invalid. mic_addr = 0x%llx\n", mic_addr);
return -EINVAL;
}
spt = mic_sys_addr_to_smpt(mdev, mic_addr);
@@ -286,7 +286,7 @@ void mic_unmap(struct mic_device *mdev, dma_addr_t mic_addr, size_t size)
smpt_info->entry[i].ref_count -= ref[i - spt];
if (smpt_info->entry[i].ref_count < 0)
dev_warn(mdev->sdev->parent,
- "ref count for entry %d is negative\n", i);
+ "ref count for entry %d is negative\n", i);
}
spin_unlock_irqrestore(&smpt_info->smpt_lock, flags);
kfree(ref);
@@ -320,7 +320,7 @@ dma_addr_t mic_map_single(struct mic_device *mdev, void *va, size_t size)
"mic_map failed dma_addr 0x%llx size 0x%lx\n",
dma_addr, size);
pci_unmap_single(pdev, dma_addr,
- size, PCI_DMA_BIDIRECTIONAL);
+ size, PCI_DMA_BIDIRECTIONAL);
}
}
return mic_addr;
@@ -366,8 +366,8 @@ int mic_smpt_init(struct mic_device *mdev)

smpt_info = mdev->smpt;
mdev->smpt_ops->init(mdev);
- smpt_info->entry = kmalloc(sizeof(struct mic_smpt)
- * smpt_info->info.num_reg, GFP_KERNEL);
+ smpt_info->entry = kmalloc_array(smpt_info->info.num_reg,
+ sizeof(*smpt_info->entry), GFP_KERNEL);
if (!smpt_info->entry) {
err = -ENOMEM;
goto free_smpt;
@@ -412,7 +412,7 @@ void mic_smpt_uninit(struct mic_device *mdev)
smpt_info->entry[i].ref_count);
if (smpt_info->entry[i].ref_count)
dev_warn(mdev->sdev->parent,
- "ref count for entry %d is not zero\n", i);
+ "ref count for entry %d is not zero\n", i);
}
kfree(smpt_info->entry);
kfree(smpt_info);
diff --git a/drivers/misc/mic/host/mic_sysfs.c b/drivers/misc/mic/host/mic_sysfs.c
index 029a4f3..75746ad 100644
--- a/drivers/misc/mic/host/mic_sysfs.c
+++ b/drivers/misc/mic/host/mic_sysfs.c
@@ -130,7 +130,7 @@ state_show(struct device *dev, struct device_attribute *attr, char *buf)

static ssize_t
state_store(struct device *dev, struct device_attribute *attr,
- const char *buf, size_t count)
+ const char *buf, size_t count)
{
int rc = 0;
struct mic_device *mdev = dev_get_drvdata(dev->parent);
@@ -163,7 +163,7 @@ done:
static DEVICE_ATTR_RW(state);

static ssize_t shutdown_status_show(struct device *dev,
- struct device_attribute *attr, char *buf)
+ struct device_attribute *attr, char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);

@@ -193,7 +193,7 @@ cmdline_show(struct device *dev, struct device_attribute *attr, char *buf)

static ssize_t
cmdline_store(struct device *dev, struct device_attribute *attr,
- const char *buf, size_t count)
+ const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);

@@ -239,7 +239,7 @@ firmware_show(struct device *dev, struct device_attribute *attr, char *buf)

static ssize_t
firmware_store(struct device *dev, struct device_attribute *attr,
- const char *buf, size_t count)
+ const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);

@@ -284,7 +284,7 @@ ramdisk_show(struct device *dev, struct device_attribute *attr, char *buf)

static ssize_t
ramdisk_store(struct device *dev, struct device_attribute *attr,
- const char *buf, size_t count)
+ const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);

@@ -330,7 +330,7 @@ bootmode_show(struct device *dev, struct device_attribute *attr, char *buf)

static ssize_t
bootmode_store(struct device *dev, struct device_attribute *attr,
- const char *buf, size_t count)
+ const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);

@@ -363,7 +363,7 @@ static DEVICE_ATTR_RW(bootmode);

static ssize_t
log_buf_addr_show(struct device *dev, struct device_attribute *attr,
- char *buf)
+ char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);

@@ -375,7 +375,7 @@ log_buf_addr_show(struct device *dev, struct device_attribute *attr,

static ssize_t
log_buf_addr_store(struct device *dev, struct device_attribute *attr,
- const char *buf, size_t count)
+ const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
int ret;
@@ -397,7 +397,7 @@ static DEVICE_ATTR_RW(log_buf_addr);

static ssize_t
log_buf_len_show(struct device *dev, struct device_attribute *attr,
- char *buf)
+ char *buf)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);

@@ -409,7 +409,7 @@ log_buf_len_show(struct device *dev, struct device_attribute *attr,

static ssize_t
log_buf_len_store(struct device *dev, struct device_attribute *attr,
- const char *buf, size_t count)
+ const char *buf, size_t count)
{
struct mic_device *mdev = dev_get_drvdata(dev->parent);
int ret;
diff --git a/drivers/misc/mic/host/mic_virtio.c b/drivers/misc/mic/host/mic_virtio.c
index 9e0456f..0c883cd 100644
--- a/drivers/misc/mic/host/mic_virtio.c
+++ b/drivers/misc/mic/host/mic_virtio.c
@@ -195,7 +195,7 @@ static int _mic_virtio_copy(struct mic_vdev *mvdev,
MIC_VRINGH_READ, &out_len);
if (ret) {
dev_err(mic_dev(mvdev), "%s %d err %d\n",
- __func__, __LINE__, ret);
+ __func__, __LINE__, ret);
break;
}
len -= out_len;
@@ -206,7 +206,7 @@ static int _mic_virtio_copy(struct mic_vdev *mvdev,
!MIC_VRINGH_READ, &out_len);
if (ret) {
dev_err(mic_dev(mvdev), "%s %d err %d\n",
- __func__, __LINE__, ret);
+ __func__, __LINE__, ret);
break;
}
len -= out_len;
@@ -225,8 +225,7 @@ static int _mic_virtio_copy(struct mic_vdev *mvdev,
* Update the used ring if a descriptor was available and some data was
* copied in/out and the user asked for a used ring update.
*/
- if (*head != USHRT_MAX && copy->out_len &&
- copy->update_used) {
+ if (*head != USHRT_MAX && copy->out_len && copy->update_used) {
u32 total = 0;

/* Determine the total data consumed */
@@ -367,7 +366,6 @@ void mic_bh_handler(struct work_struct *work)

static irqreturn_t mic_virtio_intr_handler(int irq, void *data)
{
-
struct mic_vdev *mvdev = data;
struct mic_device *mdev = mvdev->mdev;

@@ -394,7 +392,7 @@ int mic_virtio_config_change(struct mic_vdev *mvdev,
}

if (copy_from_user(mic_vq_configspace(mvdev->dd),
- argp, mvdev->dd->config_len)) {
+ argp, mvdev->dd->config_len)) {
dev_err(mic_dev(mvdev), "%s %d err %d\n",
__func__, __LINE__, -EFAULT);
ret = -EFAULT;
@@ -438,8 +436,8 @@ static int mic_copy_dp_entry(struct mic_vdev *mvdev,
return -EFAULT;
}

- if (mic_aligned_desc_size(&dd) > MIC_MAX_DESC_BLK_SIZE
- || dd.num_vq > MIC_MAX_VRINGS) {
+ if (mic_aligned_desc_size(&dd) > MIC_MAX_DESC_BLK_SIZE ||
+ dd.num_vq > MIC_MAX_VRINGS) {
dev_err(mic_dev(mvdev), "%s %d err %d\n",
__func__, __LINE__, -EINVAL);
return -EINVAL;
@@ -503,7 +501,7 @@ static void mic_init_device_ctrl(struct mic_vdev *mvdev,
{
struct mic_device_ctrl *dc;

- dc = mvdev->dc = (void *)devpage + mic_aligned_desc_size(devpage);
+ dc = (void *)devpage + mic_aligned_desc_size(devpage);

dc->config_change = 0;
dc->guest_ack = 0;
@@ -512,6 +510,7 @@ static void mic_init_device_ctrl(struct mic_vdev *mvdev,
dc->used_address_updated = 0;
dc->c2h_vdev_db = -1;
dc->h2c_vdev_db = -1;
+ mvdev->dc = dc;
}

int mic_virtio_add_device(struct mic_vdev *mvdev,
@@ -551,7 +550,7 @@ int mic_virtio_add_device(struct mic_vdev *mvdev,
sizeof(struct _mic_vring_info));
vr->va = (void *)
__get_free_pages(GFP_KERNEL | __GFP_ZERO,
- get_order(vr_size));
+ get_order(vr_size));
if (!vr->va) {
ret = -ENOMEM;
dev_err(mic_dev(mvdev), "%s %d err %d\n",
@@ -564,8 +563,7 @@ int mic_virtio_add_device(struct mic_vdev *mvdev,
vqconfig[i].address = mic_map_single(mdev,
vr->va, vr_size);
if (mic_map_error(vqconfig[i].address)) {
- free_pages((unsigned long)vr->va,
- get_order(vr_size));
+ free_pages((unsigned long)vr->va, get_order(vr_size));
ret = -ENOMEM;
dev_err(mic_dev(mvdev), "%s %d err %d\n",
__func__, __LINE__, ret);
@@ -573,8 +571,7 @@ int mic_virtio_add_device(struct mic_vdev *mvdev,
}
vqconfig[i].address = cpu_to_le64(vqconfig[i].address);

- vring_init(&vr->vr, num,
- vr->va, MIC_VIRTIO_RING_ALIGN);
+ vring_init(&vr->vr, num, vr->va, MIC_VIRTIO_RING_ALIGN);
ret = vringh_init_kern(&mvr->vrh,
*(u32 *)mic_vq_features(mvdev->dd), num, false,
vr->vr.desc, vr->vr.avail, vr->vr.used);
@@ -593,8 +590,8 @@ int mic_virtio_add_device(struct mic_vdev *mvdev,
__func__, __LINE__, i, vr->va, vr->info, vr_size);
}

- snprintf(irqname, sizeof(irqname),
- "mic%dvirtio%d", mdev->id, mvdev->virtio_id);
+ snprintf(irqname, sizeof(irqname), "mic%dvirtio%d", mdev->id,
+ mvdev->virtio_id);
mvdev->virtio_db = mic_next_db(mdev);
mvdev->virtio_cookie = mic_request_irq(mdev, mic_virtio_intr_handler,
irqname, mvdev, mvdev->virtio_db, MIC_INTR_DB);
@@ -628,9 +625,9 @@ err:
for (j = 0; j < i; j++) {
struct mic_vringh *mvr = &mvdev->mvr[j];
mic_unmap_single(mdev, le64_to_cpu(vqconfig[j].address),
- mvr->vring.len);
+ mvr->vring.len);
free_pages((unsigned long)mvr->vring.va,
- get_order(mvr->vring.len));
+ get_order(mvr->vring.len));
}
mutex_unlock(&mdev->mic_mutex);
return ret;
@@ -676,9 +673,9 @@ skip_hot_remove:
vringh_kiov_cleanup(&mvr->riov);
vringh_kiov_cleanup(&mvr->wiov);
mic_unmap_single(mdev, le64_to_cpu(vqconfig[i].address),
- mvr->vring.len);
+ mvr->vring.len);
free_pages((unsigned long)mvr->vring.va,
- get_order(mvr->vring.len));
+ get_order(mvr->vring.len));
}

list_for_each_safe(pos, tmp, &mdev->vdev_list) {
diff --git a/drivers/misc/mic/host/mic_x100.c b/drivers/misc/mic/host/mic_x100.c
index 3a0d660..81e9541 100644
--- a/drivers/misc/mic/host/mic_x100.c
+++ b/drivers/misc/mic/host/mic_x100.c
@@ -46,8 +46,8 @@ mic_x100_write_spad(struct mic_device *mdev, unsigned int idx, u32 val)
dev_dbg(mdev->sdev->parent, "Writing 0x%x to scratch pad index %d\n",
val, idx);
mic_mmio_write(&mdev->mmio, val,
- MIC_X100_SBOX_BASE_ADDRESS +
- MIC_X100_SBOX_SPAD0 + idx * 4);
+ MIC_X100_SBOX_BASE_ADDRESS +
+ MIC_X100_SBOX_SPAD0 + idx * 4);
}

/**
@@ -130,8 +130,8 @@ static void mic_x100_send_sbox_intr(struct mic_device *mdev,
{
struct mic_mw *mw = &mdev->mmio;
u64 apic_icr_offset = MIC_X100_SBOX_APICICR0 + doorbell * 8;
- u32 apicicr_low = mic_mmio_read(mw,
- MIC_X100_SBOX_BASE_ADDRESS + apic_icr_offset);
+ u32 apicicr_low = mic_mmio_read(mw, MIC_X100_SBOX_BASE_ADDRESS +
+ apic_icr_offset);

/* for MIC we need to make sure we "hit" the send_icr bit (13) */
apicicr_low = (apicicr_low | (1 << 13));
@@ -139,7 +139,7 @@ static void mic_x100_send_sbox_intr(struct mic_device *mdev,
/* Ensure that the interrupt is ordered w.r.t. previous stores. */
wmb();
mic_mmio_write(mw, apicicr_low,
- MIC_X100_SBOX_BASE_ADDRESS + apic_icr_offset);
+ MIC_X100_SBOX_BASE_ADDRESS + apic_icr_offset);
}

/**
@@ -153,7 +153,7 @@ static void mic_x100_send_rdmasr_intr(struct mic_device *mdev,
/* Ensure that the interrupt is ordered w.r.t. previous stores. */
wmb();
mic_mmio_write(&mdev->mmio, 0,
- MIC_X100_SBOX_BASE_ADDRESS + rdmasr_offset);
+ MIC_X100_SBOX_BASE_ADDRESS + rdmasr_offset);
}

/**
@@ -212,7 +212,7 @@ done:
*/
static void mic_x100_hw_intr_init(struct mic_device *mdev)
{
- mdev->intr_info = (struct mic_intr_info *) mic_x100_intr_init;
+ mdev->intr_info = (struct mic_intr_info *)mic_x100_intr_init;
}

/**
@@ -244,7 +244,7 @@ mic_x100_read_msi_to_src_map(struct mic_device *mdev, int idx)
*/
static void
mic_x100_program_msi_to_src_map(struct mic_device *mdev,
- int idx, int offset, bool set)
+ int idx, int offset, bool set)
{
unsigned long reg;
struct mic_mw *mw = &mdev->mmio;
@@ -308,12 +308,12 @@ static void mic_x100_send_firmware_intr(struct mic_device *mdev)
apicicr_low = (vector | (1 << 13));

mic_mmio_write(mw, mic_x100_get_apic_id(mdev),
- MIC_X100_SBOX_BASE_ADDRESS + apic_icr_offset + 4);
+ MIC_X100_SBOX_BASE_ADDRESS + apic_icr_offset + 4);

/* Ensure that the interrupt is ordered w.r.t. previous stores. */
wmb();
mic_mmio_write(mw, apicicr_low,
- MIC_X100_SBOX_BASE_ADDRESS + apic_icr_offset);
+ MIC_X100_SBOX_BASE_ADDRESS + apic_icr_offset);
}

/**
@@ -365,8 +365,7 @@ mic_x100_load_command_line(struct mic_device *mdev, const struct firmware *fw)
len += snprintf(buf, CMDLINE_SIZE - len,
" mem=%dM", boot_mem);
if (mdev->cmdline)
- snprintf(buf + len, CMDLINE_SIZE - len,
- " %s", mdev->cmdline);
+ snprintf(buf + len, CMDLINE_SIZE - len, " %s", mdev->cmdline);
memcpy_toio(cmd_line_va, buf, strlen(buf) + 1);
kfree(buf);
return 0;
@@ -397,8 +396,7 @@ mic_x100_load_ramdisk(struct mic_device *mdev)
* Typically the bootaddr for card OS is 64M
* so copy over the ramdisk @ 128M.
*/
- memcpy_toio(mdev->aper.va + (mdev->bootaddr << 1),
- fw->data, fw->size);
+ memcpy_toio(mdev->aper.va + (mdev->bootaddr << 1), fw->data, fw->size);
iowrite32(cpu_to_le32(mdev->bootaddr << 1), &bp->hdr.ramdisk_image);
iowrite32(cpu_to_le32(fw->size), &bp->hdr.ramdisk_size);
release_firmware(fw);
@@ -484,8 +482,7 @@ mic_x100_load_firmware(struct mic_device *mdev, const char *buf)
if (mdev->ramdisk)
rc = mic_x100_load_ramdisk(mdev);
error:
- dev_dbg(mdev->sdev->parent, "%s %d rc %d\n",
- __func__, __LINE__, rc);
+ dev_dbg(mdev->sdev->parent, "%s %d rc %d\n", __func__, __LINE__, rc);
done:
return rc;
}
@@ -524,8 +521,8 @@ mic_x100_smpt_set(struct mic_device *mdev, dma_addr_t dma_addr, u8 index)
uint32_t smpt_reg_val = BUILD_SMPT(SNOOP_ON,
dma_addr >> mdev->smpt->info.page_shift);
mic_mmio_write(&mdev->mmio, smpt_reg_val,
- MIC_X100_SBOX_BASE_ADDRESS +
- MIC_X100_SBOX_SMPT00 + (4 * index));
+ MIC_X100_SBOX_BASE_ADDRESS +
+ MIC_X100_SBOX_SMPT00 + (4 * index));
}

/**
--
1.8.2.1

2013-09-27 16:51:09

by Dutt, Sudeep

[permalink] [raw]
Subject: [PATCH char-misc-next 5/5] misc: mic: depend on X86 for both host and card drivers.

This fixes build failures seen on certain non X86 architectures.
The card driver should correctly always depend on X86. The host
driver can potentially work on non X86 architectures although
it has never been built or validated in such configurations.
The host driver dependency on X86 can be removed at some point
in the future but this workaround is required for now.

Reported-by: Fengguang Wu <[email protected]>
Reported-by: Stephen Rothwell <[email protected]>
Signed-off-by: Ashutosh Dixit <[email protected]>
Signed-off-by: Dasaratharaman Chandramouli <[email protected]>
Signed-off-by: Nikhil Rao <[email protected]>
Signed-off-by: Harshavardhan R Kharche <[email protected]>
Signed-off-by: Sudeep Dutt <[email protected]>
---
drivers/misc/mic/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/misc/mic/Kconfig b/drivers/misc/mic/Kconfig
index d453768..e42b331 100644
--- a/drivers/misc/mic/Kconfig
+++ b/drivers/misc/mic/Kconfig
@@ -2,7 +2,7 @@ comment "Intel MIC Host Driver"

config INTEL_MIC_HOST
tristate "Intel MIC Host Driver"
- depends on 64BIT && PCI
+ depends on 64BIT && PCI && X86
select VHOST_RING
default N
help
@@ -23,7 +23,7 @@ comment "Intel MIC Card Driver"

config INTEL_MIC_CARD
tristate "Intel MIC Card Driver"
- depends on 64BIT
+ depends on 64BIT && X86
select VIRTIO
default N
help
--
1.8.2.1

2013-09-27 16:51:06

by Dutt, Sudeep

[permalink] [raw]
Subject: [PATCH char-misc-next 4/5] misc: mic: fix a warning in the IOCTL header file.

The following warning from mic_ioctl.h is fixed via this patch:
found __[us]{8,16,32,64} type without #include <linux/types.h>

Reported-by: Greg Kroah-Hartman <[email protected]>
Signed-off-by: Ashutosh Dixit <[email protected]>
Signed-off-by: Dasaratharaman Chandramouli <[email protected]>
Signed-off-by: Nikhil Rao <[email protected]>
Signed-off-by: Harshavardhan R Kharche <[email protected]>
Signed-off-by: Sudeep Dutt <[email protected]>
---
include/uapi/linux/mic_ioctl.h | 2 ++
1 file changed, 2 insertions(+)

diff --git a/include/uapi/linux/mic_ioctl.h b/include/uapi/linux/mic_ioctl.h
index 0e6cbf3..7fabba5 100644
--- a/include/uapi/linux/mic_ioctl.h
+++ b/include/uapi/linux/mic_ioctl.h
@@ -21,6 +21,8 @@
#ifndef _MIC_IOCTL_H_
#define _MIC_IOCTL_H_

+#include <linux/types.h>
+
/*
* mic_copy - MIC virtio descriptor copy.
*
--
1.8.2.1

2013-09-27 17:29:30

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH char-misc-next 0/5] misc: mic: driver cleanups and a build fix.

On Fri, 2013-09-27 at 09:49 -0700, Sudeep Dutt wrote:
> These patches address code review feedback received on the
> patch series @ https://lkml.org/lkml/2013/9/5/561 ,from:
> a) Greg Kroah-Hartman on sysfs and header file cleanups.
> b) Joe Perches on issues found by "--strict" checkpatch.
>
> It should also fix the build failures on certain non X86
> architectures, reported in linux-next and 0-DAY builds.
>
> Ashutosh Dixit (1):
> misc: mic: cleanups for "--strict" checkpatch.
>
> Sudeep Dutt (4):
> misc: mic: host driver sysfs cleanups.
> misc: mic: header file cleanups.
> misc: mic: fix a warning in the IOCTL header file.
> misc: mic: depend on X86 for both host and card drivers.
>
> Documentation/mic/mpssd/mpssd.c | 65 +++++++--------
> Documentation/mic/mpssd/sysfs.c | 4 +-

Why are these in Documentation/ at all?
Shouldn't the directory be moved to tools/ ?

It doesn't build normally btw.

Also, there are some defects/shortcomings here:

Documentation/mic/mpssd/mpssd.c | 7 ++++---
Documentation/mic/mpssd/mpssd.h | 1 +
2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/Documentation/mic/mpssd/mpssd.c b/Documentation/mic/mpssd/mpssd.c
index 8064804..cb62663 100644
--- a/Documentation/mic/mpssd/mpssd.c
+++ b/Documentation/mic/mpssd/mpssd.c
@@ -379,7 +379,7 @@ disp_iovec(struct mic_info *mic, struct mic_copy_desc *copy,
for (i = 0; i < copy->iovcnt; i++)
mpsslog("%s %s %d copy->iov[%d] addr %p len 0x%lx\n",
mic->name, s, line, i,
- copy->iov[i].iov_base, copy->iov[i].iov_len);
+ copy->iov[i].iov_base, (long)copy->iov[i].iov_len);
}

static inline __u16 read_avail_idx(struct mic_vring *vr)
@@ -953,7 +953,8 @@ set_backend_file(struct mic_info *mic)
return false;
mic->mic_virtblk.backend_file = malloc(strlen(evv) + 1);
if (mic->mic_virtblk.backend_file == NULL) {
- mpsslog("can't allocate memory\n", mic->name, mic->id);
+ mpsslog("%s id %d - can't allocate memory\n",
+ mic->name, mic->id);
return false;
}
strcpy(mic->mic_virtblk.backend_file, evv + 1);
@@ -1026,7 +1027,7 @@ close_backend(struct mic_info *mic)
static bool
start_virtblk(struct mic_info *mic, struct mic_vring *vring)
{
- if (((__u64)&virtblk_dev_page.blk_config % 8) != 0) {
+ if (((__u64)(unsigned long)&virtblk_dev_page.blk_config % 8) != 0) {
mpsslog("%s: blk_config is not 8 byte aligned.\n",
mic->name);
return false;
diff --git a/Documentation/mic/mpssd/mpssd.h b/Documentation/mic/mpssd/mpssd.h
index b6dee38..ccd589f 100644
--- a/Documentation/mic/mpssd/mpssd.h
+++ b/Documentation/mic/mpssd/mpssd.h
@@ -94,6 +94,7 @@ struct mic_info {
struct mic_info *next;
};

+__attribute__((format(printf, 1, 2)))
void mpsslog(char *format, ...);
char *readsysfs(char *dir, char *entry);
int setsysfs(char *dir, char *entry, char *value);

2013-09-27 18:14:54

by Dutt, Sudeep

[permalink] [raw]
Subject: Re: [PATCH char-misc-next 0/5] misc: mic: driver cleanups and a build fix.

On Fri, 2013-09-27 at 10:29 -0700, Joe Perches wrote:
> On Fri, 2013-09-27 at 09:49 -0700, Sudeep Dutt wrote:
> > These patches address code review feedback received on the
> > patch series @ https://lkml.org/lkml/2013/9/5/561 ,from:
> > a) Greg Kroah-Hartman on sysfs and header file cleanups.
> > b) Joe Perches on issues found by "--strict" checkpatch.
> >
> > It should also fix the build failures on certain non X86
> > architectures, reported in linux-next and 0-DAY builds.
> >
> > Ashutosh Dixit (1):
> > misc: mic: cleanups for "--strict" checkpatch.
> >
> > Sudeep Dutt (4):
> > misc: mic: host driver sysfs cleanups.
> > misc: mic: header file cleanups.
> > misc: mic: fix a warning in the IOCTL header file.
> > misc: mic: depend on X86 for both host and card drivers.
> >
> > Documentation/mic/mpssd/mpssd.c | 65 +++++++--------
> > Documentation/mic/mpssd/sysfs.c | 4 +-
>
> Why are these in Documentation/ at all?
> Shouldn't the directory be moved to tools/ ?
>

Currently, this is just sample working code for configuring MIC devices.
The longer term plan is to move this code to tools/ but not with this
patch series.

> It doesn't build normally btw.
>
> Also, there are some defects/shortcomings here:
>

We will post a follow up patch incorporating the changes below next
week. It would be great if this cleanup series is applied to
char-misc-next as is since patch 5 should fix a linux-next build failure
with powerpc.

Thanks for the review!
Sudeep Dutt

> Documentation/mic/mpssd/mpssd.c | 7 ++++---
> Documentation/mic/mpssd/mpssd.h | 1 +
> 2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/mic/mpssd/mpssd.c b/Documentation/mic/mpssd/mpssd.c
> index 8064804..cb62663 100644
> --- a/Documentation/mic/mpssd/mpssd.c
> +++ b/Documentation/mic/mpssd/mpssd.c
> @@ -379,7 +379,7 @@ disp_iovec(struct mic_info *mic, struct mic_copy_desc *copy,
> for (i = 0; i < copy->iovcnt; i++)
> mpsslog("%s %s %d copy->iov[%d] addr %p len 0x%lx\n",
> mic->name, s, line, i,
> - copy->iov[i].iov_base, copy->iov[i].iov_len);
> + copy->iov[i].iov_base, (long)copy->iov[i].iov_len);
> }
>
> static inline __u16 read_avail_idx(struct mic_vring *vr)
> @@ -953,7 +953,8 @@ set_backend_file(struct mic_info *mic)
> return false;
> mic->mic_virtblk.backend_file = malloc(strlen(evv) + 1);
> if (mic->mic_virtblk.backend_file == NULL) {
> - mpsslog("can't allocate memory\n", mic->name, mic->id);
> + mpsslog("%s id %d - can't allocate memory\n",
> + mic->name, mic->id);
> return false;
> }
> strcpy(mic->mic_virtblk.backend_file, evv + 1);
> @@ -1026,7 +1027,7 @@ close_backend(struct mic_info *mic)
> static bool
> start_virtblk(struct mic_info *mic, struct mic_vring *vring)
> {
> - if (((__u64)&virtblk_dev_page.blk_config % 8) != 0) {
> + if (((__u64)(unsigned long)&virtblk_dev_page.blk_config % 8) != 0) {
> mpsslog("%s: blk_config is not 8 byte aligned.\n",
> mic->name);
> return false;
> diff --git a/Documentation/mic/mpssd/mpssd.h b/Documentation/mic/mpssd/mpssd.h
> index b6dee38..ccd589f 100644
> --- a/Documentation/mic/mpssd/mpssd.h
> +++ b/Documentation/mic/mpssd/mpssd.h
> @@ -94,6 +94,7 @@ struct mic_info {
> struct mic_info *next;
> };
>
> +__attribute__((format(printf, 1, 2)))
> void mpsslog(char *format, ...);
> char *readsysfs(char *dir, char *entry);
> int setsysfs(char *dir, char *entry, char *value);
>
>