From ecfab603f9dce38fc63d0ba54ce66c2309050b25 Mon Sep 17 00:00:00 2001 From: Dhaval Patel Date: Mon, 23 Mar 2015 22:29:02 -0700 Subject: [PATCH] msm: mdss: fix ret value during error case Fix ret value for error case in framebuffer allocation api and use integer instead of unsigned int for sscanf api. CRs-fixed: 790640 Change-Id: Ib7fa217334365b8194571e220740a769597636e0 Signed-off-by: Dhaval Patel --- drivers/video/fbdev/msm/mdss_fb.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/video/fbdev/msm/mdss_fb.c b/drivers/video/fbdev/msm/mdss_fb.c index aec70e2cd70c..2a5845ddc0bb 100644 --- a/drivers/video/fbdev/msm/mdss_fb.c +++ b/drivers/video/fbdev/msm/mdss_fb.c @@ -365,7 +365,7 @@ static void mdss_fb_parse_dt_split(struct msm_fb_data_type *mfd) static ssize_t mdss_fb_store_split(struct device *dev, struct device_attribute *attr, const char *buf, size_t len) { - u32 data[2] = {0}; + int data[2] = {0}; struct fb_info *fbi = dev_get_drvdata(dev); struct msm_fb_data_type *mfd = (struct msm_fb_data_type *)fbi->par; @@ -1573,6 +1573,7 @@ int mdss_fb_alloc_fb_ion_memory(struct msm_fb_data_type *mfd, size_t fb_size) } } else { pr_err("No IOMMU Domain\n"); + rc = -EINVAL; goto fb_mmap_failed; }