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
| ||
|
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/
Powered by blists - more mailing lists