On Thu, 20 May 2010 10:36:59 +0800 Wan ZongShun <mcuos.com@xxxxxxxxx> wrote: > Hi Dan, > > Qiang brother have a busy doing business, so I can maintain the > nuc900fb driver instead. Feel free to send an update to the MAINTAINERS file. > It looks good to me. > > Andrew , Could you help merge this patch? thanks a lot! > > Acked-by: Wan ZongShun<mcuos.com@xxxxxxxxx> > > > 2010/5/20 Dan Carpenter <error27@xxxxxxxxx>: > > Ping. __Any feedback on this patch? > > > > regards, > > dan carpenter > > > > On Thu, Mar 25, 2010 at 01:54:08PM +0300, Dan Carpenter wrote: > >> We should be releasing "res->start" here. > >> > >> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > >> --- > >> BTW. __Congratulations on being an official driver maintainer now. __:) __Can > >> you add yourself to the MAINTAINERS file? > >> > >> diff --git a/drivers/video/nuc900fb.c b/drivers/video/nuc900fb.c > >> index 6bf0d46..d4cde79 100644 > >> --- a/drivers/video/nuc900fb.c > >> +++ b/drivers/video/nuc900fb.c > >> @@ -667,7 +667,7 @@ release_irq: > >> __release_regs: > >> __ __ __ iounmap(fbi->io); > >> __release_mem_region: > >> - __ __ release_mem_region((unsigned long)fbi->mem, size); > >> + __ __ release_mem_region(res->start, size); > >> __free_fb: > >> __ __ __ framebuffer_release(fbinfo); > >> __ __ __ return ret; > > I don't think I ever received a copy of this patch. I reconstructed it. -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
