linux-kernel - RE: [PATCH] fbdev: Minor cleanup in WM8505-related code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <EAF47CD23C76F840A9E7FCE10091EFAB0322F86AE6@dbde02.ent.ti.com>
Date:	Mon, 20 Dec 2010 10:36:24 +0530
From:	"Janorkar, Mayuresh" <mayur@...com>
To:	Alexey Charkov <alchark@...il.com>,
	Paul Mundt <lethal@...ux-sh.org>
CC:	"linux-fbdev@...r.kernel.org" <linux-fbdev@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"vt8500-wm8505-linux-kernel@...glegroups.com" 
	<vt8500-wm8505-linux-kernel@...glegroups.com>
Subject: RE: [PATCH] fbdev: Minor cleanup in WM8505-related code

Alexey,

How about sending this as two different patches?
1) Correcting vsync
2) code cleanup

-Thanks,
Mayuresh
> -----Original Message-----
> From: linux-fbdev-owner@...r.kernel.org [mailto:linux-fbdev-
> owner@...r.kernel.org] On Behalf Of Alexey Charkov
> Sent: Sunday, December 19, 2010 11:43 PM
> To: Paul Mundt
> Cc: linux-fbdev@...r.kernel.org; linux-kernel@...r.kernel.org; vt8500-
> wm8505-linux-kernel@...glegroups.com
> Subject: [PATCH] fbdev: Minor cleanup in WM8505-related code
> 
> This fixes vsync timing calculation in wm8505fb.c (was off by one,
> which caused some flicker with larger panels) and replaces the last
> remaining driver name reference with the macro for uniformity.
> 
> In addition, the error return path in wmt_ge_rops.c is made more
> uniform by getting rid of goto's in remaining places.
> 
> Signed-off-by: Alexey Charkov <alchark@...il.com>
> ---
> 
> Paul, could you please apply this to your fbdev tree? The patch seems
> to be trivial, and it would bring the mainline code in line with what
> people use when pulling from our Gitorious development repo.
> 
> Best regards,
> Alexey
> 
> diff --git a/drivers/video/wm8505fb.c b/drivers/video/wm8505fb.c
> index e37251b..96e34a5 100644
> --- a/drivers/video/wm8505fb.c
> +++ b/drivers/video/wm8505fb.c
> @@ -88,7 +88,7 @@ static int wm8505fb_set_timing(struct fb_info *info)
>  	int v_start = info->var.upper_margin;
>  	int v_end = v_start + info->var.yres;
>  	int v_all = v_end + info->var.lower_margin;
> -	int v_sync = info->var.vsync_len + 1;
> +	int v_sync = info->var.vsync_len;
> 
>  	writel(0, fbi->regbase + WMT_GOVR_TG);
> 
> @@ -291,7 +291,7 @@ static int __devinit wm8505fb_probe(struct
> platform_device *pdev)
>  		goto failed_fbi;
>  	}
> 
> -	res = request_mem_region(res->start, resource_size(res),
> "wm8505fb");
> +	res = request_mem_region(res->start, resource_size(res),
> DRIVER_NAME);
>  	if (res == NULL) {
>  		dev_err(&pdev->dev, "failed to request I/O memory\n");
>  		ret = -EBUSY;
> diff --git a/drivers/video/wmt_ge_rops.c b/drivers/video/wmt_ge_rops.c
> index f31883f..45832b7 100644
> --- a/drivers/video/wmt_ge_rops.c
> +++ b/drivers/video/wmt_ge_rops.c
> @@ -127,13 +127,11 @@ EXPORT_SYMBOL_GPL(wmt_ge_sync);
>  static int __devinit wmt_ge_rops_probe(struct platform_device *pdev)
>  {
>  	struct resource *res;
> -	int ret;
> 
>  	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
>  	if (res == NULL) {
>  		dev_err(&pdev->dev, "no I/O memory resource defined\n");
> -		ret = -ENODEV;
> -		goto error;
> +		return -ENODEV;
>  	}
> 
>  	/* Only one ROP engine is presently supported. */
> @@ -145,17 +143,13 @@ static int __devinit wmt_ge_rops_probe(struct
> platform_device *pdev)
>  	regbase = ioremap(res->start, resource_size(res));
>  	if (regbase == NULL) {
>  		dev_err(&pdev->dev, "failed to map I/O memory\n");
> -		ret = -EBUSY;
> -		goto error;
> +		return -EBUSY;
>  	}
> 
>  	writel(1, regbase + GE_ENABLE_OFF);
>  	printk(KERN_INFO "Enabled support for WMT GE raster
> acceleration\n");
> 
>  	return 0;
> -
> -error:
> -	return ret;
>  }
> 
>  static int __devexit wmt_ge_rops_remove(struct platform_device *pdev)
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/