[PATCH] staging: davinci_vpfe: fix space prohibited before semicolon warning

2015-01-14 Thread Aya Mahfouz
This patch fixes the following checkpatch.pl warning: space prohibited before semicolon Signed-off-by: Aya Mahfouz mahfouz.saif.elya...@gmail.com --- v1: This patch applies to Greg's tree. drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

Re: [PATCH] staging: media: lirc: replace dev_err by pr_err

2014-11-05 Thread Aya Mahfouz
On Wed, Nov 05, 2014 at 12:57:38PM +0530, Sudip Mukherjee wrote: On Tue, Nov 04, 2014 at 11:48:26PM +0200, Aya Mahfouz wrote: On Tue, Nov 04, 2014 at 03:06:53PM +0530, Sudip Mukherjee wrote: On Tue, Nov 04, 2014 at 02:13:19AM +0200, Aya Mahfouz wrote: This patch replaces dev_err

Re: [PATCH] staging: media: lirc: modify print calls

2014-11-05 Thread Aya Mahfouz
On Wed, Nov 05, 2014 at 08:17:11AM -0200, Mauro Carvalho Chehab wrote: Em Tue, 4 Nov 2014 23:43:07 +0200 Aya Mahfouz mahfouz.saif.elya...@gmail.com escreveu: This patches replaces one pr_debug call by dev_dbg and changes the device used by one of the dev_err calls. Also doesn't apply

Re: [PATCH] staging: media: lirc: modify print calls

2014-11-05 Thread Aya Mahfouz
On Wed, Nov 05, 2014 at 04:59:15PM +0530, Sudip Mukherjee wrote: On Tue, Nov 04, 2014 at 11:43:07PM +0200, Aya Mahfouz wrote: This patches replaces one pr_debug call by dev_dbg and changes the device used by one of the dev_err calls. i think you should mention in the commit message why you

[PATCH] staging: media: lirc: modify print calls

2014-11-04 Thread Aya Mahfouz
This patches replaces one pr_debug call by dev_dbg and changes the device used by one of the dev_err calls. Signed-off-by: Aya Mahfouz mahfouz.saif.elya...@gmail.com --- drivers/staging/media/lirc/lirc_zilog.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers

[no subject]

2014-11-03 Thread Aya Mahfouz
: replace dev_err by pr_err Reply-To: This patch replaces dev_err by pr_err since the value of ir is NULL when the message is displayed. Signed-off-by: Aya Mahfouz mahfouz.saif.elya...@gmail.com --- drivers/staging/media/lirc/lirc_zilog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH] staging: media: lirc: replace dev_err by pr_err

2014-11-03 Thread Aya Mahfouz
This patch replaces dev_err by pr_err since the value of ir is NULL when the message is displayed. Signed-off-by: Aya Mahfouz mahfouz.saif.elya...@gmail.com --- drivers/staging/media/lirc/lirc_zilog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/lirc

Re: [PATCH] staging: media: lirc: lirc_zilog.c: adjust debug messages

2014-11-03 Thread Aya Mahfouz
On Sun, Nov 02, 2014 at 12:40:13PM +0100, Konrad Zapalowicz wrote: On 11/01, Aya Mahfouz wrote: This patch removes one debug message and replaces a dev_err call by pr_err. Usually you would like to send this as two separate patches because replacing a debug message is way different than

Re: staging: media: lirc: lirc_zilog.c: replace custom print macros with dev_* and pr_*

2014-11-01 Thread Aya Mahfouz
On Fri, Oct 31, 2014 at 05:35:41PM +0300, Dan Carpenter wrote: On Fri, Oct 31, 2014 at 04:26:45PM +0200, Aya Mahfouz wrote: On Fri, Oct 31, 2014 at 04:06:00PM +0300, Dan Carpenter wrote: drivers/staging/media/lirc/lirc_zilog.c 1333 /* Close the IR device */ 1334 static int close

[PATCH] staging: media: lirc: lirc_zilog.c: adjust debug messages

2014-11-01 Thread Aya Mahfouz
This patch removes one debug message and replaces a dev_err call by pr_err. Signed-off-by: Aya Mahfouz mahfouz.saif.elya...@gmail.com --- drivers/staging/media/lirc/lirc_zilog.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/media/lirc/lirc_zilog.c b

Re: staging: media: lirc: lirc_zilog.c: replace custom print macros with dev_* and pr_*

2014-10-31 Thread Aya Mahfouz
On Fri, Oct 31, 2014 at 04:06:00PM +0300, Dan Carpenter wrote: Hello Aya Mahfouz, Hello Dan, The patch be4aa8157c98: staging: media: lirc: lirc_zilog.c: replace custom print macros with dev_* and pr_* from Oct 26, 2014, leads to the following static checker warning: drivers