Re: [PATCH 7/8] fbdev: atyfb_base.c requires atyfb_cursor()

["Antonino A. Daplas" <adaplas@hotpop.com>]

On Wednesday 03 November 2004 06:26, Ville Syrjälä wrote:
> On Wed, Nov 03, 2004 at 05:31:36AM +0800, Antonino A. Daplas wrote:
> > On Tuesday 02 November 2004 21:58, Ville Syrjälä wrote:
> > > On Tue, Nov 02, 2004 at 07:46:26PM +0800, Antonino A. Daplas wrote:
> > > > atyfb_base.c requires atyfb_cursor, but it is only available for
> > > > linking if CONFIG_FB_ATY_CT=y.
> > > > This patch moves the .o file to the CONFIG_FB_ATY rule.
> > >
> > > Here's a better fix. It was a part of my swcursor patch which was
> > > partially included. Looks like Alex accidentally dropped this part.
> > >
> > >
> > > Fix atyfb cursor problems:
> > > - oops with noaccel option
> > > - CONFIG_FB_ATY_CT dependency problem
> >
> > If I read Alex's patch correctly, it looks like the aty has a working
> > hardware cursor implementation, is this correct?  Then a better patch may
> > be to use soft_cursor if noaccel is true, and atyfb_cursor otherwise.
>
> That is how things work with this patch. aty_init_cursor() sets fb_cursor
> to atyfb_cursor.

Ah, ok, did not see that.

Tony




-------------------------------------------------------
This SF.Net email is sponsored by:
Sybase ASE Linux Express Edition - download now for FREE
LinuxWorld Reader's Choice Award Winner for best database on Linux.
http://ads.osdn.com/?ad_idU88&alloc_id\x12065&op=click