* [PATCH ipmi/kcs_bmc v2] ipmi: kcs_bmc: make the code be more clean
@ 2018-02-19 15:55 Haiyue Wang
2018-02-20 13:29 ` Corey Minyard
0 siblings, 1 reply; 3+ messages in thread
From: Haiyue Wang @ 2018-02-19 15:55 UTC (permalink / raw)
To: minyard, openipmi-developer, linux-kernel; +Cc: Haiyue Wang
---
v1 -> v2:
Add 'SPDX-License-Identifier' style for header files modification.
---
1. Add the missed key word '__user' for read / write.
2. Remove the prefix 'file' of 'file_to_kcs_bmc', no need this
duplicated word as its parameter has 'struct file *filp'.
3. Change the 'unsigned int' to '__poll_t' to meet the new 'poll'
definition.
4. Correct the 'SPDX-License-Identifier' style for header files.
Signed-off-by: Haiyue Wang <haiyue.wang@linux.intel.com>
---
drivers/char/ipmi/kcs_bmc.c | 32 +++++++++++++++++---------------
drivers/char/ipmi/kcs_bmc.h | 6 ++++--
drivers/char/ipmi/kcs_bmc_aspeed.c | 4 +++-
include/uapi/linux/ipmi_bmc.h | 6 ++++--
4 files changed, 28 insertions(+), 20 deletions(-)
diff --git a/drivers/char/ipmi/kcs_bmc.c b/drivers/char/ipmi/kcs_bmc.c
index 6476bfb..fbfc05e 100644
--- a/drivers/char/ipmi/kcs_bmc.c
+++ b/drivers/char/ipmi/kcs_bmc.c
@@ -1,5 +1,7 @@
// SPDX-License-Identifier: GPL-2.0
-// Copyright (c) 2015-2018, Intel Corporation.
+/*
+ * Copyright (c) 2015-2018, Intel Corporation.
+ */
#define pr_fmt(fmt) "kcs-bmc: " fmt
@@ -242,14 +244,14 @@ int kcs_bmc_handle_event(struct kcs_bmc *kcs_bmc)
}
EXPORT_SYMBOL(kcs_bmc_handle_event);
-static inline struct kcs_bmc *file_to_kcs_bmc(struct file *filp)
+static inline struct kcs_bmc *to_kcs_bmc(struct file *filp)
{
return container_of(filp->private_data, struct kcs_bmc, miscdev);
}
static int kcs_bmc_open(struct inode *inode, struct file *filp)
{
- struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
+ struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
int ret = 0;
spin_lock_irq(&kcs_bmc->lock);
@@ -262,25 +264,25 @@ static int kcs_bmc_open(struct inode *inode, struct file *filp)
return ret;
}
-static unsigned int kcs_bmc_poll(struct file *filp, poll_table *wait)
+static __poll_t kcs_bmc_poll(struct file *filp, poll_table *wait)
{
- struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
- unsigned int mask = 0;
+ struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
+ __poll_t mask = 0;
poll_wait(filp, &kcs_bmc->queue, wait);
spin_lock_irq(&kcs_bmc->lock);
if (kcs_bmc->data_in_avail)
- mask |= POLLIN;
+ mask |= EPOLLIN;
spin_unlock_irq(&kcs_bmc->lock);
return mask;
}
-static ssize_t kcs_bmc_read(struct file *filp, char *buf,
- size_t count, loff_t *offset)
+static ssize_t kcs_bmc_read(struct file *filp, char __user *buf,
+ size_t count, loff_t *ppos)
{
- struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
+ struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
bool data_avail;
size_t data_len;
ssize_t ret;
@@ -339,10 +341,10 @@ static ssize_t kcs_bmc_read(struct file *filp, char *buf,
return ret;
}
-static ssize_t kcs_bmc_write(struct file *filp, const char *buf,
- size_t count, loff_t *offset)
+static ssize_t kcs_bmc_write(struct file *filp, const char __user *buf,
+ size_t count, loff_t *ppos)
{
- struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
+ struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
ssize_t ret;
/* a minimum response size '3' : netfn + cmd + ccode */
@@ -378,7 +380,7 @@ static ssize_t kcs_bmc_write(struct file *filp, const char *buf,
static long kcs_bmc_ioctl(struct file *filp, unsigned int cmd,
unsigned long arg)
{
- struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
+ struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
long ret = 0;
spin_lock_irq(&kcs_bmc->lock);
@@ -410,7 +412,7 @@ static long kcs_bmc_ioctl(struct file *filp, unsigned int cmd,
static int kcs_bmc_release(struct inode *inode, struct file *filp)
{
- struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
+ struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
spin_lock_irq(&kcs_bmc->lock);
kcs_bmc->running = 0;
diff --git a/drivers/char/ipmi/kcs_bmc.h b/drivers/char/ipmi/kcs_bmc.h
index c19501d..69d9a70 100644
--- a/drivers/char/ipmi/kcs_bmc.h
+++ b/drivers/char/ipmi/kcs_bmc.h
@@ -1,5 +1,7 @@
-// SPDX-License-Identifier: GPL-2.0
-// Copyright (c) 2015-2018, Intel Corporation.
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright (c) 2015-2018, Intel Corporation.
+ */
#ifndef __KCS_BMC_H__
#define __KCS_BMC_H__
diff --git a/drivers/char/ipmi/kcs_bmc_aspeed.c b/drivers/char/ipmi/kcs_bmc_aspeed.c
index 0c4d1a3..dba6075 100644
--- a/drivers/char/ipmi/kcs_bmc_aspeed.c
+++ b/drivers/char/ipmi/kcs_bmc_aspeed.c
@@ -1,5 +1,7 @@
// SPDX-License-Identifier: GPL-2.0
-// Copyright (c) 2015-2018, Intel Corporation.
+/*
+ * Copyright (c) 2015-2018, Intel Corporation.
+ */
#define pr_fmt(fmt) "aspeed-kcs-bmc: " fmt
diff --git a/include/uapi/linux/ipmi_bmc.h b/include/uapi/linux/ipmi_bmc.h
index 2f9f97e..d3efacd 100644
--- a/include/uapi/linux/ipmi_bmc.h
+++ b/include/uapi/linux/ipmi_bmc.h
@@ -1,5 +1,7 @@
-// SPDX-License-Identifier: GPL-2.0
-// Copyright (c) 2015-2018, Intel Corporation.
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright (c) 2015-2018, Intel Corporation.
+ */
#ifndef _UAPI_LINUX_IPMI_BMC_H
#define _UAPI_LINUX_IPMI_BMC_H
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH ipmi/kcs_bmc v2] ipmi: kcs_bmc: make the code be more clean
2018-02-19 15:55 [PATCH ipmi/kcs_bmc v2] ipmi: kcs_bmc: make the code be more clean Haiyue Wang
@ 2018-02-20 13:29 ` Corey Minyard
2018-02-21 5:21 ` Wang, Haiyue
0 siblings, 1 reply; 3+ messages in thread
From: Corey Minyard @ 2018-02-20 13:29 UTC (permalink / raw)
To: Haiyue Wang, openipmi-developer, linux-kernel
On 02/19/2018 09:55 AM, Haiyue Wang wrote:
> ---
When you use ---, it means everything following is not in the commit text,
including your signature.
> v1 -> v2:
Do you want me to fold this into the previous patch? That's generally
not how things work, a new patch is fine for this, with a list of things
done like below.
One comment inline below...
>
> Add 'SPDX-License-Identifier' style for header files modification.
> ---
>
> 1. Add the missed key word '__user' for read / write.
> 2. Remove the prefix 'file' of 'file_to_kcs_bmc', no need this
> duplicated word as its parameter has 'struct file *filp'.
> 3. Change the 'unsigned int' to '__poll_t' to meet the new 'poll'
> definition.
> 4. Correct the 'SPDX-License-Identifier' style for header files.
>
> Signed-off-by: Haiyue Wang <haiyue.wang@linux.intel.com>
> ---
> drivers/char/ipmi/kcs_bmc.c | 32 +++++++++++++++++---------------
> drivers/char/ipmi/kcs_bmc.h | 6 ++++--
> drivers/char/ipmi/kcs_bmc_aspeed.c | 4 +++-
> include/uapi/linux/ipmi_bmc.h | 6 ++++--
> 4 files changed, 28 insertions(+), 20 deletions(-)
>
> diff --git a/drivers/char/ipmi/kcs_bmc.c b/drivers/char/ipmi/kcs_bmc.c
> index 6476bfb..fbfc05e 100644
> --- a/drivers/char/ipmi/kcs_bmc.c
> +++ b/drivers/char/ipmi/kcs_bmc.c
> @@ -1,5 +1,7 @@
> // SPDX-License-Identifier: GPL-2.0
> -// Copyright (c) 2015-2018, Intel Corporation.
> +/*
> + * Copyright (c) 2015-2018, Intel Corporation.
> + */
>
> #define pr_fmt(fmt) "kcs-bmc: " fmt
>
> @@ -242,14 +244,14 @@ int kcs_bmc_handle_event(struct kcs_bmc *kcs_bmc)
> }
> EXPORT_SYMBOL(kcs_bmc_handle_event);
>
> -static inline struct kcs_bmc *file_to_kcs_bmc(struct file *filp)
> +static inline struct kcs_bmc *to_kcs_bmc(struct file *filp)
> {
> return container_of(filp->private_data, struct kcs_bmc, miscdev);
> }
>
> static int kcs_bmc_open(struct inode *inode, struct file *filp)
> {
> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
> int ret = 0;
>
> spin_lock_irq(&kcs_bmc->lock);
> @@ -262,25 +264,25 @@ static int kcs_bmc_open(struct inode *inode, struct file *filp)
> return ret;
> }
>
> -static unsigned int kcs_bmc_poll(struct file *filp, poll_table *wait)
> +static __poll_t kcs_bmc_poll(struct file *filp, poll_table *wait)
> {
> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
> - unsigned int mask = 0;
> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
> + __poll_t mask = 0;
>
> poll_wait(filp, &kcs_bmc->queue, wait);
>
> spin_lock_irq(&kcs_bmc->lock);
> if (kcs_bmc->data_in_avail)
> - mask |= POLLIN;
> + mask |= EPOLLIN;
I get this:
CC [M] drivers/char/ipmi/kcs_bmc.o
../drivers/char/ipmi/kcs_bmc.c: In function ‘kcs_bmc_poll’:
../drivers/char/ipmi/kcs_bmc.c:276:11: error: ‘EPOLLIN’ undeclared
(first use in this function)
mask |= EPOLLIN;
^
../drivers/char/ipmi/kcs_bmc.c:276:11: note: each undeclared identifier
is reported only once for each function it appears in
probably need to include linux/eventpoll.h
-corey
> spin_unlock_irq(&kcs_bmc->lock);
>
> return mask;
> }
>
> -static ssize_t kcs_bmc_read(struct file *filp, char *buf,
> - size_t count, loff_t *offset)
> +static ssize_t kcs_bmc_read(struct file *filp, char __user *buf,
> + size_t count, loff_t *ppos)
> {
> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
> bool data_avail;
> size_t data_len;
> ssize_t ret;
> @@ -339,10 +341,10 @@ static ssize_t kcs_bmc_read(struct file *filp, char *buf,
> return ret;
> }
>
> -static ssize_t kcs_bmc_write(struct file *filp, const char *buf,
> - size_t count, loff_t *offset)
> +static ssize_t kcs_bmc_write(struct file *filp, const char __user *buf,
> + size_t count, loff_t *ppos)
> {
> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
> ssize_t ret;
>
> /* a minimum response size '3' : netfn + cmd + ccode */
> @@ -378,7 +380,7 @@ static ssize_t kcs_bmc_write(struct file *filp, const char *buf,
> static long kcs_bmc_ioctl(struct file *filp, unsigned int cmd,
> unsigned long arg)
> {
> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
> long ret = 0;
>
> spin_lock_irq(&kcs_bmc->lock);
> @@ -410,7 +412,7 @@ static long kcs_bmc_ioctl(struct file *filp, unsigned int cmd,
>
> static int kcs_bmc_release(struct inode *inode, struct file *filp)
> {
> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
>
> spin_lock_irq(&kcs_bmc->lock);
> kcs_bmc->running = 0;
> diff --git a/drivers/char/ipmi/kcs_bmc.h b/drivers/char/ipmi/kcs_bmc.h
> index c19501d..69d9a70 100644
> --- a/drivers/char/ipmi/kcs_bmc.h
> +++ b/drivers/char/ipmi/kcs_bmc.h
> @@ -1,5 +1,7 @@
> -// SPDX-License-Identifier: GPL-2.0
> -// Copyright (c) 2015-2018, Intel Corporation.
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2015-2018, Intel Corporation.
> + */
>
> #ifndef __KCS_BMC_H__
> #define __KCS_BMC_H__
> diff --git a/drivers/char/ipmi/kcs_bmc_aspeed.c b/drivers/char/ipmi/kcs_bmc_aspeed.c
> index 0c4d1a3..dba6075 100644
> --- a/drivers/char/ipmi/kcs_bmc_aspeed.c
> +++ b/drivers/char/ipmi/kcs_bmc_aspeed.c
> @@ -1,5 +1,7 @@
> // SPDX-License-Identifier: GPL-2.0
> -// Copyright (c) 2015-2018, Intel Corporation.
> +/*
> + * Copyright (c) 2015-2018, Intel Corporation.
> + */
>
> #define pr_fmt(fmt) "aspeed-kcs-bmc: " fmt
>
> diff --git a/include/uapi/linux/ipmi_bmc.h b/include/uapi/linux/ipmi_bmc.h
> index 2f9f97e..d3efacd 100644
> --- a/include/uapi/linux/ipmi_bmc.h
> +++ b/include/uapi/linux/ipmi_bmc.h
> @@ -1,5 +1,7 @@
> -// SPDX-License-Identifier: GPL-2.0
> -// Copyright (c) 2015-2018, Intel Corporation.
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2015-2018, Intel Corporation.
> + */
>
> #ifndef _UAPI_LINUX_IPMI_BMC_H
> #define _UAPI_LINUX_IPMI_BMC_H
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH ipmi/kcs_bmc v2] ipmi: kcs_bmc: make the code be more clean
2018-02-20 13:29 ` Corey Minyard
@ 2018-02-21 5:21 ` Wang, Haiyue
0 siblings, 0 replies; 3+ messages in thread
From: Wang, Haiyue @ 2018-02-21 5:21 UTC (permalink / raw)
To: minyard, openipmi-developer, linux-kernel
On 2018-02-20 21:29, Corey Minyard wrote:
> On 02/19/2018 09:55 AM, Haiyue Wang wrote:
>> ---
> When you use ---, it means everything following is not in the commit
> text,
> including your signature.
>
Got it.
>> v1 -> v2:
>
> Do you want me to fold this into the previous patch? That's generally
> not how things work, a new patch is fine for this, with a list of things
> done like below.
>
I will submit a new patch.
> One comment inline below...
>
>>
>> Add 'SPDX-License-Identifier' style for header files modification.
>> ---
>>
>> 1. Add the missed key word '__user' for read / write.
>> 2. Remove the prefix 'file' of 'file_to_kcs_bmc', no need this
>> duplicated word as its parameter has 'struct file *filp'.
>> 3. Change the 'unsigned int' to '__poll_t' to meet the new 'poll'
>> definition.
>> 4. Correct the 'SPDX-License-Identifier' style for header files.
>>
>> Signed-off-by: Haiyue Wang <haiyue.wang@linux.intel.com>
>> ---
>> drivers/char/ipmi/kcs_bmc.c | 32
>> +++++++++++++++++---------------
>> drivers/char/ipmi/kcs_bmc.h | 6 ++++--
>> drivers/char/ipmi/kcs_bmc_aspeed.c | 4 +++-
>> include/uapi/linux/ipmi_bmc.h | 6 ++++--
>> 4 files changed, 28 insertions(+), 20 deletions(-)
>>
>> diff --git a/drivers/char/ipmi/kcs_bmc.c b/drivers/char/ipmi/kcs_bmc.c
>> index 6476bfb..fbfc05e 100644
>> --- a/drivers/char/ipmi/kcs_bmc.c
>> +++ b/drivers/char/ipmi/kcs_bmc.c
>> @@ -1,5 +1,7 @@
>> // SPDX-License-Identifier: GPL-2.0
>> -// Copyright (c) 2015-2018, Intel Corporation.
>> +/*
>> + * Copyright (c) 2015-2018, Intel Corporation.
>> + */
>> #define pr_fmt(fmt) "kcs-bmc: " fmt
>> @@ -242,14 +244,14 @@ int kcs_bmc_handle_event(struct kcs_bmc
>> *kcs_bmc)
>> }
>> EXPORT_SYMBOL(kcs_bmc_handle_event);
>> -static inline struct kcs_bmc *file_to_kcs_bmc(struct file *filp)
>> +static inline struct kcs_bmc *to_kcs_bmc(struct file *filp)
>> {
>> return container_of(filp->private_data, struct kcs_bmc, miscdev);
>> }
>> static int kcs_bmc_open(struct inode *inode, struct file *filp)
>> {
>> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
>> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
>> int ret = 0;
>> spin_lock_irq(&kcs_bmc->lock);
>> @@ -262,25 +264,25 @@ static int kcs_bmc_open(struct inode *inode,
>> struct file *filp)
>> return ret;
>> }
>> -static unsigned int kcs_bmc_poll(struct file *filp, poll_table *wait)
>> +static __poll_t kcs_bmc_poll(struct file *filp, poll_table *wait)
>> {
>> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
>> - unsigned int mask = 0;
>> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
>> + __poll_t mask = 0;
>> poll_wait(filp, &kcs_bmc->queue, wait);
>> spin_lock_irq(&kcs_bmc->lock);
>> if (kcs_bmc->data_in_avail)
>> - mask |= POLLIN;
>> + mask |= EPOLLIN;
>
> I get this:
>
> CC [M] drivers/char/ipmi/kcs_bmc.o
> ../drivers/char/ipmi/kcs_bmc.c: In function ‘kcs_bmc_poll’:
> ../drivers/char/ipmi/kcs_bmc.c:276:11: error: ‘EPOLLIN’ undeclared
> (first use in this function)
> mask |= EPOLLIN;
> ^
> ../drivers/char/ipmi/kcs_bmc.c:276:11: note: each undeclared
> identifier is reported only once for each function it appears in
>
> probably need to include linux/eventpoll.h
>
I forgot to tell you that you need update the git tree, it is merged in
from 'Linux 4.16-rc1'. Like bt-bmc.c. :)
git log -p drivers/char/ipmi/bt-bmc.c
commit a9a08845e9acbd224e4ee466f5c1275ed50054e8
Author: Linus Torvalds <torvalds@linux-foundation.org>
Date: Sun Feb 11 14:34:03 2018 -0800
vfs: do bulk POLL* -> EPOLL* replacement
This is the mindless scripted replacement of kernel use of POLL*
variables as described by Al, done by this script:
for V in IN OUT PRI ERR RDNORM RDBAND WRNORM WRBAND HUP RDHUP
NVAL MSG; do
L=`git grep -l -w POLL$V | grep -v '^t' | grep -v /um/ |
grep -v '^sa' | grep -v '/poll.h$'|grep -v '^D'`
for f in $L; do sed -i
"-es/^\([^\"]*\)\(\<POLL$V\>\)/\\1E\\2/" $f; done
done
with de-mangling cleanups yet to come.
NOTE! On almost all architectures, the EPOLL* constants have the same
values as the POLL* constants do. But they keyword here is "almost".
For various bad reasons they aren't the same, and epoll() doesn't
actually work quite correctly in some cases due to this on Sparc et al.
The next patch from Al will sort out the final differences, and we
should be all done.
Scripted-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
diff --git a/drivers/char/ipmi/bt-bmc.c b/drivers/char/ipmi/bt-bmc.c
index 7992c87..c95b93b 100644
--- a/drivers/char/ipmi/bt-bmc.c
+++ b/drivers/char/ipmi/bt-bmc.c
@@ -349,10 +349,10 @@ static __poll_t bt_bmc_poll(struct file *file,
poll_table *wait)
ctrl = bt_inb(bt_bmc, BT_CTRL);
if (ctrl & BT_CTRL_H2B_ATN)
- mask |= POLLIN;
+ mask |= EPOLLIN;
if (!(ctrl & (BT_CTRL_H_BUSY | BT_CTRL_B2H_ATN)))
- mask |= POLLOUT;
+ mask |= EPOLLOUT;
return mask;
> -corey
>
>> spin_unlock_irq(&kcs_bmc->lock);
>> return mask;
>> }
>> -static ssize_t kcs_bmc_read(struct file *filp, char *buf,
>> - size_t count, loff_t *offset)
>> +static ssize_t kcs_bmc_read(struct file *filp, char __user *buf,
>> + size_t count, loff_t *ppos)
>> {
>> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
>> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
>> bool data_avail;
>> size_t data_len;
>> ssize_t ret;
>> @@ -339,10 +341,10 @@ static ssize_t kcs_bmc_read(struct file *filp,
>> char *buf,
>> return ret;
>> }
>> -static ssize_t kcs_bmc_write(struct file *filp, const char *buf,
>> - size_t count, loff_t *offset)
>> +static ssize_t kcs_bmc_write(struct file *filp, const char __user *buf,
>> + size_t count, loff_t *ppos)
>> {
>> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
>> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
>> ssize_t ret;
>> /* a minimum response size '3' : netfn + cmd + ccode */
>> @@ -378,7 +380,7 @@ static ssize_t kcs_bmc_write(struct file *filp,
>> const char *buf,
>> static long kcs_bmc_ioctl(struct file *filp, unsigned int cmd,
>> unsigned long arg)
>> {
>> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
>> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
>> long ret = 0;
>> spin_lock_irq(&kcs_bmc->lock);
>> @@ -410,7 +412,7 @@ static long kcs_bmc_ioctl(struct file *filp,
>> unsigned int cmd,
>> static int kcs_bmc_release(struct inode *inode, struct file *filp)
>> {
>> - struct kcs_bmc *kcs_bmc = file_to_kcs_bmc(filp);
>> + struct kcs_bmc *kcs_bmc = to_kcs_bmc(filp);
>> spin_lock_irq(&kcs_bmc->lock);
>> kcs_bmc->running = 0;
>> diff --git a/drivers/char/ipmi/kcs_bmc.h b/drivers/char/ipmi/kcs_bmc.h
>> index c19501d..69d9a70 100644
>> --- a/drivers/char/ipmi/kcs_bmc.h
>> +++ b/drivers/char/ipmi/kcs_bmc.h
>> @@ -1,5 +1,7 @@
>> -// SPDX-License-Identifier: GPL-2.0
>> -// Copyright (c) 2015-2018, Intel Corporation.
>> +/* SPDX-License-Identifier: GPL-2.0 */
>> +/*
>> + * Copyright (c) 2015-2018, Intel Corporation.
>> + */
>> #ifndef __KCS_BMC_H__
>> #define __KCS_BMC_H__
>> diff --git a/drivers/char/ipmi/kcs_bmc_aspeed.c
>> b/drivers/char/ipmi/kcs_bmc_aspeed.c
>> index 0c4d1a3..dba6075 100644
>> --- a/drivers/char/ipmi/kcs_bmc_aspeed.c
>> +++ b/drivers/char/ipmi/kcs_bmc_aspeed.c
>> @@ -1,5 +1,7 @@
>> // SPDX-License-Identifier: GPL-2.0
>> -// Copyright (c) 2015-2018, Intel Corporation.
>> +/*
>> + * Copyright (c) 2015-2018, Intel Corporation.
>> + */
>> #define pr_fmt(fmt) "aspeed-kcs-bmc: " fmt
>> diff --git a/include/uapi/linux/ipmi_bmc.h
>> b/include/uapi/linux/ipmi_bmc.h
>> index 2f9f97e..d3efacd 100644
>> --- a/include/uapi/linux/ipmi_bmc.h
>> +++ b/include/uapi/linux/ipmi_bmc.h
>> @@ -1,5 +1,7 @@
>> -// SPDX-License-Identifier: GPL-2.0
>> -// Copyright (c) 2015-2018, Intel Corporation.
>> +/* SPDX-License-Identifier: GPL-2.0 */
>> +/*
>> + * Copyright (c) 2015-2018, Intel Corporation.
>> + */
>> #ifndef _UAPI_LINUX_IPMI_BMC_H
>> #define _UAPI_LINUX_IPMI_BMC_H
>
>
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-02-21 5:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-19 15:55 [PATCH ipmi/kcs_bmc v2] ipmi: kcs_bmc: make the code be more clean Haiyue Wang
2018-02-20 13:29 ` Corey Minyard
2018-02-21 5:21 ` Wang, Haiyue
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).