On Thu, 20 Mar 2008 06:30:37 +0800 "Jaya Kumar" <jayakumar.lkml@xxxxxxxxx> wrote: > On Thu, Mar 20, 2008 at 2:37 AM, Andrew Morton > <akpm@xxxxxxxxxxxxxxxxxxxx> wrote: > > > > Do we need some of all of that patch in 2.6.25? I wasn't aware of such a > > need. > > > > If the drivers/video/fb_defio.c hunk of that patch fixes something then I'd > > be inclined to merge the whole thing - adding a new driver won't hurt > > anyone. > > > > Yes, the defio hunk is a bugfix. I agree with pushing the whole thing > to 2.6.25 if there are people who need it in 2.6.25. I believe there > is no risk of impact on anyone else. > Well all we have in the changelog is Implement support for the E-Ink Metronome controller. It provides an mmapable interface to the controller using defio support. It was tested with a gumstix pxa255 with Vizplex media using Xfbdev and various X clients such as xeyes, xpdf, xloadimage. grump. This say nothing at all about the change to drivers/video/fb_defio.c. a) the changelog _should_ document this change. Please send a paragraph and I'll add it. b) If this discussion hadn't happened, that bug (whatever it is) would have remained unfixed in 2.6.25. As a direct consequence of poor changelogging. c) This bugfix (whatever it is) should have been in a separate patch. Thanks. _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/virtualization
