From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Simmons Date: Fri, 20 Nov 2009 20:48:12 +0000 Subject: Re: [Linux-fbdev-devel] drm_fb_helper: Impossible to change video Message-Id: List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-fbdev@vger.kernel.org On Fri, 20 Nov 2009, Paulius Zaleckas wrote: > On 11/20/2009 10:01 PM, James Simmons wrote: > > > > > > > > Paulius Zaleckas wrote: > > > > > > > On drivers using drm_fb_helper's in fb_ops it is not possible to > > > > > > > change > > > > > > > video mode, because of different var->pixclock evaluation: ... > > > > > > > > > > > > patch: > > > > > > http://www.mail-archive.com/dri-devel@lists.sourceforge.net/msg44369.html > > > > > > > > > > Those patches will enable fbdev apps to run properly. More patches are > > > > > needed if you want to support mode switching using the fbdev emulation > > > > > layer. I noticed my patches and yours where lost. Who do you send > > > > > patches > > > > > too that can merge them ? > > > > > > > > y:/usr/src/git26> perl scripts/get_maintainer.pl -f > > > > drivers/gpu/drm/drm_fb_helper.c > > > > David Airlie > > > > Dave Airlie > > > > Jesse Barnes > > > > Mikael Pettersson > > > > dri-devel@lists.sourceforge.net > > > > linux-kernel@vger.kernel.org > > > > > > > > That's accurate enough. > > > > > > > > Generally, if nothing has happened in a week, the chances that it's > > > > lost are very high. Resend. If you like, cc me and I'll maintain the > > > > patches > > > > and resend them for you. > > > > > > You can add Tested-by: Paulius Zaleckas > > > for > > > http://www.mail-archive.com/dri-devel@lists.sourceforge.net/msg44369.html > > > as this was preventing DirectFB from running on my Asus Eee PC 701. > > > > I tested it as well with the both my 3Dfx driver that I wrote with KMS and > > the nouveau driver. We just need to make sure that the patches end up in > > the drm-next tree or these patches will be lost when drm-next gets merged > > to linus tree. > > IMHO this patch should end up in current (2.6.32) kernel and we should > send it to stable ML. Honestly I like to see it there as well.