Home » Mailing list archives » linux-kernel » 2007 » March » 12

Re: [PATCH] drivers/media/video/videocodec.c: check kmalloc() return value.

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Ahmed S. Darwish <darwish.07@...>
To: Amit Choudhary <amit2030@...>
Cc: Linux Kernel <linux-kernel@...>
Subject: Re: [PATCH] drivers/media/video/videocodec.c: check kmalloc() return value.
Date: Monday, March 12, 2007 - 6:45 am

Hi Amit,

On Thu, Mar 08, 2007 at 11:14:01PM -0800, Amit Choudhary wrote:

No need for `Description:'. This line is automatically put in the logs 
as a patch description if the patch got accepted. 

It's better not to use very large lines. 

Also there's no need to specify the exact function in the log, it's already
displayed in the patch.


As said in Documentation/SubmittingPatches put a "---" line after
Signed-off-by signature.

Thanks,

-- 
Ahmed S. Darwish
http://darwish.07.googlepages.com

-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH] drivers/media/video/videocodec.c: check kmalloc() re..., Amit Choudhary, (Fri Mar 9, 3:14 am)
Re: [PATCH] drivers/media/video/videocodec.c: check kmalloc(..., Ahmed S. Darwish, (Mon Mar 12, 6:45 am)
Re: [PATCH] drivers/media/video/videocodec.c: check kmalloc(..., Andrey Panin, (Fri Mar 9, 5:04 am)