Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 06208656 authored by Christophe JAILLET's avatar Christophe JAILLET Committed by Bartlomiej Zolnierkiewicz
Browse files

video: fbdev: au1200fb: Propagate an error code 



We should propagate the error code returned by 'fb_alloc_cmap()' instead
of returning -EFAULT.

Signed-off-by: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Cc: Tejun Heo <tj@kernel.org>
Signed-off-by: default avatarBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
parent 58a81afc

drivers/video/fbdev/au1200fb.c

+4 −3
Original line number Diff line number Diff line
@@ -1518,7 +1518,7 @@ static irqreturn_t au1200fb_handle_irq(int irq, void* dev_id) 
static int au1200fb_init_fbinfo(struct au1200fb_device *fbdev)

{

	struct fb_info *fbi = fbdev->fb_info;

	int bpp;

	int bpp, ret;



	fbi->fbops = &au1200fb_fb_ops;
@@ -1550,10 +1550,11 @@ static int au1200fb_init_fbinfo(struct au1200fb_device *fbdev) 
		return -ENOMEM;

	}



	if (fb_alloc_cmap(&fbi->cmap, AU1200_LCD_NBR_PALETTE_ENTRIES, 0) < 0) {

	ret = fb_alloc_cmap(&fbi->cmap, AU1200_LCD_NBR_PALETTE_ENTRIES, 0);

	if (ret < 0) {

		print_err("Fail to allocate colormap (%d entries)",

			   AU1200_LCD_NBR_PALETTE_ENTRIES);

		return -EFAULT;

		return ret;

	}



	strncpy(fbi->fix.id, "AU1200", sizeof(fbi->fix.id));