linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH] OMAP2:OMAPFB: Fix error path
       [not found] <AANLkTilje0umiNM2Tn7p2S5xhfEbEhtj6pIuSBMKvShi@mail.gmail.com>
@ 2010-06-30  8:50 ` Tomi Valkeinen
  0 siblings, 0 replies; only message in thread
From: Tomi Valkeinen @ 2010-06-30  8:50 UTC (permalink / raw)
  To: ext Afzal Mohammed
  Cc: Syrjala Ville (Nokia-D/Helsinki),
	tj, linux-omap, linux-fbdev, linux-kernel

Hi,

Please send the patch as plaintext, without any HTML content.

Also, use the same patch subject as other patches for omapfb, in this
case "OMAP: DSS2: OMAPFB: ...", and include a bit more verbose
description.

 Tomi


On Tue, 2010-06-29 at 22:29 +0200, ext Afzal Mohammed wrote:
> 
> 
> sysfs file removed in error path of omapfb_create_framebuffers()
> 
> 
> Signed-off-by: Afzal Mohammed <lazfamam@gmail.com>
> ---
>  drivers/video/omap2/omapfb/omapfb-main.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> 
> diff --git a/drivers/video/omap2/omapfb/omapfb-main.c
> b/drivers/video/omap2/omapfb/omapfb-main.c
> index 4b4506d..2319b81 100644
> --- a/drivers/video/omap2/omapfb/omapfb-main.c
> +++ b/drivers/video/omap2/omapfb/omapfb-main.c
> @@ -1945,6 +1945,7 @@ static int omapfb_create_framebuffers(struct
> omapfb2_device *fbdev)
>   r = omapfb_overlay_enable(ovl, 1);
>  
>   if (r) {
> + omapfb_remove_sysfs(fbdev);
>   dev_err(fbdev->dev,
>   "failed to enable overlay\n");
>   return r;
> -- 
> 1.7.0.4
> 
> 



^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2010-06-30  9:10 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <AANLkTilje0umiNM2Tn7p2S5xhfEbEhtj6pIuSBMKvShi@mail.gmail.com>
2010-06-30  8:50 ` [PATCH] OMAP2:OMAPFB: Fix error path Tomi Valkeinen

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).