On 3 August 2017 at 18:53, Dan Carpenter <dan.carpen...@oracle.com> wrote:
> On Thu, Aug 03, 2017 at 06:23:54PM +0530, hari prasath wrote:
>> On 3 August 2017 at 11:52, kbuild test robot <l...@intel.com> wrote:
>> > Hi Hari,
>> >
>> > [auto build tes
rop us a note to
> help improve the system]
>
> url:
> https://github.com/0day-ci/linux/commits/Hari-Prasath/Remove-explicit-return-type-cast/20170803-080312
> config: blackfin-allyesconfig (attached as .config)
> compiler: bfin-uclinux-gcc (GCC) 6.2.0
> reproduce:
>
Remove explicit typecasting of return value in the interrupt handlers.
Signed-off-by: Hari Prasath <geharipras...@gmail.com>
---
drivers/staging/pi433/pi433_if.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging
Fix checkpatch warning to add space around the substraction operation
Signed-off-by: Hari Prasath <geharipras...@gmail.com>
---
drivers/staging/iio/adc/ad7280a.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/iio/adc/ad7280a.c
b/drivers/staging/i
dio_dev->dev.of_node = indio_dev->dev.parent->of_node;
>
> - Lars
>
> On 07/17/2017 02:34 PM, Hari Prasath wrote:
>> Adding missing indio_dev->dev.of_node references to allow iio consumers
>> to access the device channels.
>>
>> Signed-off-by: Ha
Adding missing indio_dev->dev.of_node references to allow iio consumers
to access the device channels.
Signed-off-by: Hari Prasath <geharipras...@gmail.com>
---
v2: Wrong from mail ID in first version of patch
---
drivers/staging/iio/adc/ad7192.c | 1 -
drivers/staging/iio/adc
Kindly discard this patch versions. I will send a new one.
On 17 July 2017 at 17:49, Hari Prasath <geharipras...@gmail.com> wrote:
> Adding missing indio_dev->dev.of_node references to allow iio consumers
> to access the device channels.
>
> Signed-off-by: Hari Prasath <
Adding missing indio_dev->dev.of_node references to allow iio consumers
to access the device channels.
Signed-off-by: Hari Prasath <geharipras...@gmail.com>
---
v2: Wrong from email ID in first version of patch
---
drivers/staging/iio/adc/ad7192.c | 1 +
drivers/staging/iio/adc
From: Hari Prasath <hguju...@visteon.com>
Adding missing indio_dev->dev.of_node references to allow iio consumers
to access the device channels.
Signed-off-by: Hari Prasath <geharipras...@gmail.com>
---
drivers/staging/iio/adc/ad7192.c | 1 +
drivers/staging/iio/adc/ad7780.c
On 10 July 2017 at 01:22, Sakari Ailus <sakari.ai...@iki.fi> wrote:
> On Sun, Jul 09, 2017 at 05:56:15PM +0530, hari prasath wrote:
>> On 8 July 2017 at 16:31, Sakari Ailus <sakari.ai...@iki.fi> wrote:
>> > Hi Hari,
>> >
>> > On Fri, Jul
kstrdup kernel primitive can be used to replace kmalloc followed by
string copy. This was reported by coccinelle tool.
Signed-off-by: Hari Prasath <geharipras...@gmail.com>
---
v1: Replace kmalloc followed by memcpy with kmemdup. Based on
review comments from Al
On 8 July 2017 at 16:31, Sakari Ailus <sakari.ai...@iki.fi> wrote:
> Hi Hari,
>
> On Fri, Jul 07, 2017 at 08:15:21PM +0530, Hari Prasath wrote:
>> kstrdup kernel primitive can be used to replace kmalloc followed by
>> string copy. This was reported by coccinelle too
kstrdup kernel primitive can be used to replace kmalloc followed by
string copy. This was reported by coccinelle tool
Signed-off-by: Hari Prasath <geharipras...@gmail.com>
---
.../media/atomisp/pci/atomisp2/css2400/sh_css_firmware.c | 10 +++---
1 file changed, 3 insertions
On 07-Jul-2017 5:25 PM, "Alan Cox" <a...@linux.intel.com> wrote:
On Fri, 2017-07-07 at 17:20 +0530, Hari Prasath wrote:
> kmemdup can be used to replace kmalloc followed by a memcpy.This was
> pointed out by the coccinelle tool.
And kstrdup could do the job even better I
kmemdup can be used to replace kmalloc followed by a memcpy.This was
pointed out by the coccinelle tool.
Signed-off-by: Hari Prasath <geharipras...@gmail.com>
---
drivers/staging/media/atomisp/pci/atomisp2/css2400/sh_css_firmware.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Prefer using the eth_broadcast_addr() instead of memset to broadcast
address 0xFF to the array.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
v2: From email address was missing as poi
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
This patch makes use of wait_event_interruptible_timeout to achieve
timeout functionality.This is a TODO mentiond in the comment which is also
removed.It also aligns with what the function is supposed to do as in the
comments.
Sign
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Replace the timer API's to initialize a timer & then assign the callback
function by the setup_timer() API.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
v2: Unwanted text present in the
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Date: Thu, 4 Feb 2016 18:46:31 +0530
Replace the timer API's to initialize a timer & then assign the callback
function by the setup_timer() API.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
driv
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Use the managed version of the dma_alloc_coherent() i.e. the
dmam_alloc_coherent() & accordingly cleanup the error handling
part.Also,remove the references to dma_free_coherent.
Signed-off-by: Hari Prasath Gujulan Ela
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Use the managed version of the dma_alloc_coherent() i.e. the
dmam_alloc_coherent() & accordingly cleanup the error handling
part.Also,remove the references to dma_free_coherent
Signed-off-by: Hari Prasath Gujulan Ela
This patch replaces the clk_get() with devm_clk_get().Accordingly,modified
the error paths,rename error labels and removed clk_put() in probe() &
remove functions.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
v3: Rename error path labels to suite mod
On Thu, Jan 21, 2016 at 12:45:59PM +0300, Dan Carpenter wrote:
> On Thu, Jan 21, 2016 at 09:11:38AM +, Gujulan Elango, Hari Prasath (H.)
> wrote:
> > From: Hari Prasath Gujulan Elango
> >
>
> Your email address is missing here.
Hello Dan,
Many thanks for not
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Replace the led_classdev_register() with the platform managed version
devm_led_classdev_register() & henceforth remove the redundant
nvec_paz00_remove() function.
Signed-off-by: Hari Prasath Gujulan Elango <hguju..
From: Hari Prasath Gujulan Elango
This patch replaces the clk_get() with devm_clk_get().
Accordingly,modified the error paths and removed clk_put() as well.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
drivers/staging/iio/adc/spear_adc.c | 6 ++
1 file chan
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
This patch replaces the clk_get() with devm_clk_get().
Accordingly,modified the error paths and removed clk_put() as well.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
v2: email address
On Thu, Jan 21, 2016 at 11:21:53AM -0800, Matt Ranostay wrote:
> On Thu, Jan 21, 2016 at 1:11 AM, Gujulan Elango, Hari Prasath (H.)
> <hguju...@visteon.com> wrote:
> > From: Hari Prasath Gujulan Elango
> >
> > This patch replaces the clk_get() with devm_clk_g
From: Hari Prasath Gujulan Elango
Replace the memset of array to broadcast address 0xFF by using the
eth_broadcast_addr() API
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
drivers/staging/rtl8188eu/os_dep/ioctl_linux.c | 2 +-
1 file changed, 1 insertion(+), 1 de
From: Hari Prasath Gujulan Elango
Prefer using the eth_broadcast_addr() instead of memset to broadcast
address 0xFF to the array.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
drivers/staging/rtl8192e/rtllib_softmac.c | 4 ++--
1 file changed, 2 insertions
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
This patch move's a macro defined in the middle of a structure
definition to make it more readable.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
drivers/staging/wilc1000/wilc_sdio.c | 2 +-
1 fil
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Replace multiple if..else if..statements with simple table lookup in two
functions.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
drivers/staging/most/mostcore/core.c | 41 +--
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Replace multiple if..else if..statements with simple table lookup in two
functions.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
drivers/staging/most/mostcore/core.c | 39 ---
This patch makes use of wait_event_interruptible_timeout to achieve timeout
functionality.This is a TODO mentiond in the comment which is also removed.
It also aligns with what the function is supposed to do as in the
comments.
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.
From: Hari Prasath Gujulan Elango <hguju...@visteon.com>
Replace the pre-defined macro signifying the ethernet protocol type
defined in the kernel headers instead of the numerical constant
Signed-off-by: Hari Prasath Gujulan Elango <hguju...@visteon.com>
---
drivers/staging/wilc1000
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch simplifies the 'memset' done on a static 2D array.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/host_interface.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch uses the BIT macro for bit shift operation.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/xgifb/XGI_main_26.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes commented code.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/lustre/lustre/osc/osc_cache.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/lustre/lustre/osc
From: Hari Prasath Gujulan Elango hguju...@visteon.com
Prefer using the BIT macro for shifting operation
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/comedi/comedi.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes the redundant __func__ from dynamic debug prints as
the pr_xxx set of functions can be dynamically controlled to include
function name.Also fix a typo in debug statement.
Signed-off-by: Hari Prasath Gujulan Elango hguju
On Mon, Jun 22, 2015 at 06:15:06PM +0300, Dan Carpenter wrote:
On Thu, Jun 18, 2015 at 12:56:54PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch modifies few debug prints from pr_err() to pr_info() as they
fall under
On Tue, Jun 23, 2015 at 11:49:24AM +0300, Dan Carpenter wrote:
On Mon, Jun 22, 2015 at 01:13:58PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes ununsed variable 'len'
Signed-off-by: Hari Prasath Gujulan
added this patch.I will ensure not to miss him next time.
On Thu, Jun 18, 2015 at 01:27:28PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
The memory allocated in dgnc_tty_register() for two objects is not freed
anywhere.This patch
On Monday 22 June 2015 08:46 PM, Dan Carpenter wrote:
On Thu, Jun 18, 2015 at 12:56:54PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch modifies few debug prints from pr_err() to pr_info() as they
fall under that category
On Tuesday 23 June 2015 02:07 PM, Dan Carpenter wrote:
On Mon, Jun 22, 2015 at 04:48:01AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes a typo in the debug statement
Signed-off-by: Hari Prasath Gujulan Elango hguju
On Tuesday 23 June 2015 02:16 PM, Dan Carpenter wrote:
On Mon, Jun 22, 2015 at 07:26:25AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
@@ -122,7 +121,7 @@ void setChipClock(unsigned int frequency)
* Sometime, the chip cannot set up the exact clock required by
User
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes a couple of ununsed variable.The lines in which these
variables are assigned are also removed as they are not necessary.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes ununsed variable 'ret'.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/wilc_wlan.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch the removes the redundant assignement of the variable ret as
its being overwritren before being used anywhere.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/wilc_wlan.c | 2 +-
1
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes a unused variable timeout and the associated code.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/linux_wlan.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes ununsed if..else... code blocks. Its actually some
dead code.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/coreconfigurator.c | 22 --
1 file changed, 22
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes a unused variable 'u16RespLen' that is assigned a
value that is never used. The line that does the assignment is also
removed.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes a ununsed variable msg_len and its associated code.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/wilc_wlan_cfg.c | 6 --
1 file changed, 6 deletions(-)
diff --git
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes the ununsed variable 'priv' at multiple instances and
all its associated code where its assigned a value.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch addresses the checkpatch warning advising the usage of the
BIT macro for Bit shift operation.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/wilc_wlan_if.h | 4 ++--
1 file changed, 2
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes unused variables as reported by cppcheck
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/sm750fb/ddk750_chip.c | 3 +--
drivers/staging/sm750fb/ddk750_mode.c | 3 +--
drivers
On Mon, Jun 22, 2015 at 06:12:11PM +0530, Sudip Mukherjee wrote:
On Mon, Jun 22, 2015 at 07:05:04AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes ununsed variable 'ret'.
ret or len? :)
regards
sudip
yes
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes ununsed variable 'len'
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/wilc_wlan.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes a typo in the debug statement
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/sm750fb/sm750_accel.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes the redundant __func__ from dynamic debug prints as
the pr_xxx set of functions can be dynamically controlled to include
function name as well
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch silences the checkpatch warning 'min() should
probably be min_t'
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/lustre/lnet/selftest/rpc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch modifies few debug prints from pr_err() to pr_info() as they
fall under that category.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/sm750fb/sm750.c | 6 +++---
1 file changed, 3 insertions
From: Hari Prasath Gujulan Elango hguju...@visteon.com
The memory allocated in dgnc_tty_register() for two objects is not freed
anywhere.This patch addresses this by freeing the memory in
dgnc_tty_uninit.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/dgnc
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes a typo in the comments section as warned by
checkpatch.pl
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/dgnc/dgnc_driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes the redundant __func__ from dynamic debug prints as
the pr_xxx set of functions can be dynamically controlled to include
function name as well.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers
On Wed, Jun 17, 2015 at 09:40:42PM -0700, gre...@linuxfoundation.org wrote:
On Tue, Jun 16, 2015 at 05:09:35AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
On Mon, Jun 15, 2015 at 09:55:15PM -0700, gre...@linuxfoundation.org wrote:
On Mon, Jun 15, 2015 at 11:44:24AM +, Gujulan
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removed a undefined macro and debug prints associated with
it.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wlan-ng/p80211wep.c | 14 --
1 file changed, 14 deletions(-)
diff
On Wed, Jun 17, 2015 at 09:40:42PM -0700, gre...@linuxfoundation.org wrote:
On Tue, Jun 16, 2015 at 05:09:35AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
On Mon, Jun 15, 2015 at 09:55:15PM -0700, gre...@linuxfoundation.org wrote:
On Mon, Jun 15, 2015 at 11:44:24AM +, Gujulan
On Tue, Jun 16, 2015 at 05:07:22PM +0100, Ian Abbott wrote:
On 16/06/15 15:22, Gujulan Elango, Hari Prasath (H.) wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes the checkpatch warning that suggests to use the BIT
macro for the shifting operation.
Signed-off
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes a couple of ununsed MACRO's in this header file
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/slicoss/slic.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes the checkpatch warning that suggests to use the BIT
macro for the shifting operation.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/comedi/comedi.h | 10 +-
1 file changed, 5
On Tue, Jun 16, 2015 at 02:04:21PM +0100, Ian Abbott wrote:
On 16/06/15 06:34, Gujulan Elango, Hari Prasath (H.) wrote:
On Mon, Jun 15, 2015 at 03:42:07PM +0100, Ian Abbott wrote:
On 15/06/15 15:01, Gujulan Elango, Hari Prasath (H.) wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
On Tue, Jun 16, 2015 at 02:04:21PM +0100, Ian Abbott wrote:
On 16/06/15 06:34, Gujulan Elango, Hari Prasath (H.) wrote:
On Mon, Jun 15, 2015 at 03:42:07PM +0100, Ian Abbott wrote:
On 15/06/15 15:01, Gujulan Elango, Hari Prasath (H.) wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch renames the very generic variable name 'todo' to
nsamples.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
Subject line modified and variable name kept short as per
suggestion from Ian Abbott
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch silences the Checkpatch.pl warning 'Prefer using the
BIT macro'
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/comedi/comedidev.h | 12 ++--
1 file changed, 6 insertions(+), 6
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes SIOCDEVPRIVATE + 1 ioctl. It currently is just a
stub which does some useless printks and returns. In the original code,
if the user passes priv_cmd.total_len == 0 then it will Oops. Also it
leaks memory every time it's
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes a typo in the comment section.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/rtl8188eu/os_dep/os_intfs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch replaces hard coded values with global definitions for the
Ethernet IEEE 802.3 interface defined in standard header file.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wlan-ng/p80211conv.c
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes commented code warned by checkpatch.pl
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/emxx_udc/emxx_udc.h | 10 --
1 file changed, 10 deletions(-)
diff --git a/drivers/staging
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes commented code.This was a checkpatch warning.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/comedi/comedi_fops.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers
From: Hari Prasath Gujulan Elango hguju...@visteon.com
Thi patch removes ununsed debug macro and its references in this
file.The macro is used around pr_debug which itself can be dynamically
enabled/disabled.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch renames the very generic variable name 'todo' to
num_samples_to_read.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/comedi/drivers/amplc_pci230.c | 9 +
1 file changed, 5 insertions
On Mon, Jun 15, 2015 at 03:42:07PM +0100, Ian Abbott wrote:
On 15/06/15 15:01, Gujulan Elango, Hari Prasath (H.) wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch renames the very generic variable name 'todo' to
num_samples_to_read.
Signed-off-by: Hari Prasath
On Mon, Jun 15, 2015 at 03:42:07PM +0100, Ian Abbott wrote:
On 15/06/15 15:01, Gujulan Elango, Hari Prasath (H.) wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch renames the very generic variable name 'todo' to
num_samples_to_read.
Signed-off-by: Hari Prasath
On Mon, Jun 15, 2015 at 09:55:15PM -0700, gre...@linuxfoundation.org wrote:
On Mon, Jun 15, 2015 at 11:44:24AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
Thi patch removes ununsed debug macro and its references
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes the timer related wrapper functions
WILC_TimerCreate(),WILC_TimerStart(),WILC_TimerStop() and
WILC_TimerDestroy() and uses kernel API's directly.In the process,the
unused variables,files are also deleted.
Signed-off
On Thu, Jun 11, 2015 at 12:35:48PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes the timer related wrapper functions
WILC_TimerCreate(),WILC_TimerStart(),WILC_TimerStop() and
WILC_TimerDestroy() and uses kernel
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes the timer related wrapper functions
WILC_TimerCreate(),WILC_TimerStart(),WILC_TimerStop() and
WILC_TimerDestroy() and uses kernel API's directly.In the process,the
unused variables,files are also deleted.
Signed-off
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch replaces the kmalloc followed by copy_from_user by the
wrapper routine memdup_user.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wilc1000/linux_wlan.c | 12 +++-
1 file changed, 3
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removes code that is unused for any functionality in the
ioctl SIOCDEVPRIVATE + 1.But since it is mentioned in the TODO
list that it needs to be replaced with generic API functions,the
debug print alone is left as it is.
Signed
patches to the wilc1000
driver and thats how atmel guys were CC'd by mistake.Thanks Dan for
pointing this out and thanks sudip for copying the correct maintainers.
I will take some time to go through the code/review comments and send
out another version.
Thanks Regards,
Hari Prasath
From: Hari Prasath Gujulan Elango hguju...@visteon.com
Cleanup the device entry,device class unregister the character device
in the module exit.All this cleanup is done already in the dgap_stop()
function.We need to call this in the cleanup module.
Signed-off-by: Hari Prasath Gujulan Elango
On 9 June 2015 at 13:00, Gujulan Elango, Hari Prasath (H.)
hguju...@visteon.com wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
Cleanup the device entry,device class unregister the character device
in the module exit.All this cleanup is done already in the dgap_stop
From: Hari Prasath Gujulan Elango hguju...@visteon.com
Cleanup the device entry,device class unregister the character device
in the module exit.All this cleanup is done already in the dgap_stop()
function.We need to call this in the cleanup module.
Signed-off-by: Hari Prasath Gujulan Elango
On Mon, Jun 08, 2015 at 01:29:30PM -0700, gre...@linuxfoundation.org wrote:
On Mon, Jun 01, 2015 at 01:06:15PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch deletes all references to 'flipbuf'.Memory is allocated
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes a build warning related to the use of the time_after_eq
macro.Adding a typecast to the second argument suppresses the
warning.This warning was created by one my previous patch.
Signed-off-by: Hari Prasath Gujulan Elango
way of doing things.I read your other email and it sounds better
of way cleaning up.Thanks for your comments.I would have to send a new
patch anyways.
Regards
Hari Prasath
___
devel mailing list
de...@linuxdriverproject.org
http
On Tue, Jun 09, 2015 at 04:41:33PM +0300, Dan Carpenter wrote:
On Tue, Jun 09, 2015 at 10:54:02AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
On Tue, Jun 09, 2015 at 03:32:20PM +0530, Sudip Mukherjee wrote:
On Tue, Jun 09, 2015 at 09:27:22AM +, Gujulan Elango, Hari Prasath
(H
From: Hari Prasath Gujulan Elango hguju...@visteon.com
Deleting obvious comments
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/dgap/dgap.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c
index
On Tue, Jun 09, 2015 at 10:41:18AM -0700, gre...@linuxfoundation.org wrote:
On Tue, Jun 09, 2015 at 06:43:59AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes a build warning related to the use of the time_after_eq
From: Hari Prasath Gujulan Elango hguju...@visteon.com
The driver remove function is not the right place to cleanup resources
allocated in the driver module init.The cleanup is moved from remove
function to module exit as appropriate.
Signed-off-by: Hari Prasath Gujulan Elango hguju
1 - 100 of 222 matches
Mail list logo
