[PATCH] drivers:staging: Fix alignment warnings reported by tool

2018-02-24 Thread umesh . freelance
From: Umesh3034 Fix alignment warnings reported by checkpatch.pl Signed-off-by: Umesh3034 --- drivers/staging/clocking-wizard/clk-xlnx-clock-wizard.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/staging/clocking-wizard/clk-xlnx-clock-wizard.c

Re: [PATCH v3] staging:iio:meter: Add name to function definition arguments

2018-02-24 Thread Rodrigo Siqueira
On 02/24, Jonathan Cameron wrote: > On Wed, 21 Feb 2018 16:54:35 -0300 > Rodrigo Siqueira wrote: > > > This patch fixes the checkpatch.pl warning: > > > > drivers/staging/iio/meter/ade7854.h:157: WARNING: function definition > > argument 'struct device *' should also have an identifier name... >

Re: [PATCH v3] staging:iio:meter: Add name to function definition arguments

2018-02-24 Thread Jonathan Cameron
On Wed, 21 Feb 2018 16:54:35 -0300 Rodrigo Siqueira wrote: > This patch fixes the checkpatch.pl warning: > > drivers/staging/iio/meter/ade7854.h:157: WARNING: function definition > argument 'struct device *' should also have an identifier name... > > This commit adds arguments names to the sign

[PATCH 1/4] staging: most: Fix coding style problems.

2018-02-24 Thread Quytelda Kahja
Makes two very minor changes indicated by checkpatch: 1) Add a newline after components_show() definition. 2) Fix a line over the 80 character limit. Signed-off-by: Quytelda Kahja --- drivers/staging/most/core.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/stag

[PATCH 2/4] staging: most: Replace calls to BUG_ON() with WARN_ONCE() and return.

2018-02-24 Thread Quytelda Kahja
Replace calls to BUG_ON() used to check for NULL pointers with WARN_ONCE() followed by a return. Signed-off-by: Quytelda Kahja --- drivers/staging/most/core.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/drivers/staging/most/core.c b/drivers/staging/most/cor

[PATCH 3/4] staging: most: Remove unnecessary OOM messages.

2018-02-24 Thread Quytelda Kahja
It isn't necessary for the driver to log out-of-memory errors, so these have been removed and the functions simply return -ENOMEM. Signed-off-by: Quytelda Kahja --- drivers/staging/most/core.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/most/core.c b/drivers/staging/most

[PATCH 4/4] staging: most: Fix missing identifier in function definition argument.

2018-02-24 Thread Quytelda Kahja
The function pointer 'complete' in 'struct mbo' should use an identifier for its argument. Signed-off-by: Quytelda Kahja --- drivers/staging/most/core.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/most/core.h b/drivers/staging/most/core.h index 74a29163b68