Re: x86/genfb_machdep.c multiplies width with stride

[Michael Lorenz <macallan%netbsd.org@localhost>]

Hello,

on Monday 01 July 2013 06:21:16 Martin Husemann wrote:
> On Fri, Jun 28, 2013 at 04:05:55PM +0200, imre%vdsz.com@localhost wrote:
> > In sys/arch/x86/x86/genfb_machdep.c at line 161, I found the following
> > code:
> >
> >     err = _x86_memio_map(t, (bus_addr_t)fbinfo->physaddr,
> >         fbinfo->width * fbinfo->stride,
> >         BUS_SPACE_MAP_LINEAR | BUS_SPACE_MAP_PREFETCHABLE, &h);
> >
> > which looks to me like a typo, fbinfo->width should probably be replaced
> > by fbinfo->height, to map exactly the visible portion of the framebuffer.

Indeed. Easy mistake to make and hard to catch since it rarely ever causes 
trouble ( unless you're running some weird video mode where width < height )

> Good catch!
>
> Michael, could you please reveiw & commit?

Sorry for the delay, been AFK for a while - give me a minute or so.

have fun
Michael