Hi Paul, On Wednesday 13 July 2011 10:24:24 Paul Mundt wrote: > On Fri, Jul 08, 2011 at 01:14:14PM +0200, Laurent Pinchart wrote: > > Hi everybody, > > > > Here's the third version of the FBIOPAN_DISPLAY fixes patch series. > > > > As a reminder, while playing with the FBIOPAN_DISPLAY ioctl I noticed > > that many drivers use information from the ioctl argument such as the > > display resolution when they should use the current settings from the > > fb_info structure. > > > > These 29 patches fix most of those drivers. Missing drivers are ivtv (for > > which a patch has Andy Walls has already pushed a patch), > > sh_mobile_lcdcfb (for which the fix is a bit more complex, I will work > > on it later) and viafb (for which Florian Tobias Schandinat has already > > sent a patch). > > > > The patches have been compile-tested on architectures for which I have a > > working compiler (x86 and ARM) with allyesconfig. I don't own any > > hardware supported by these drivers so I haven't been able to perform > > additional tests. > > > > I've split the fixes in one patch per driver for easier review and > > handling of potential problems. The patches can be squashed together if > > needed. I've tried to CC individual driver maintainers when possible, > > and Acked-by lines received in reply to the first version have been > > included. > > > > Compared to v2, I've fixed a compile issue in the atmel_lcdfb driver and > > added a couple of SoB lines. > > > > Paul, is it time to push this for 3.1 ? > > I have the v2 series queued up already, so it would be nice to simply > have the v2->v3 differences as incremental changes that can be piled on. Ah. Next time please notify me when you queue the patches up. That was too soon. This will end up potentially breaking bisections, which is never good. > The signed-off-by lines are not that critical in this case since it's a > largely mechanical change all over the place, so it's primarily the build > fix(es?) that would be nice to have. I would say they are more than "nice to have". Where have you queued v2 ? -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
