linux-apus-devel

Re: Sound patch

[Geert U. <ge...@li...>]

	Hi Ken,

> A while ago you posted a patch to arch/m68k/amiga/amisound.c :
> 
>         if (amiga_chipset == CS_ECS || amiga_chipset == CS_AGA) {
>             /* program Denise/Lisa for a higher maximum play rate */
>             /* (may be overridden by amifb later) */
>             custom.htotal = 113;        /* 31 kHz */
>             custom.vtotal = 223;        /* 70 Hz */
>             custom.beamcon0 = 0x4390;   /* HARDDIS,VAR{BEAM,VSY,HSY,CSY}EN */
>             /* suspend the monitor */
>             custom.hsstrt = custom.hsstop = 116;
>             custom.vsstrt = custom.vsstop = 226;
>             amiga_audio_min_period = 57;
>         }
> 
> It works, no more stabbing at F keys before ABBA ;) but it makes the
> cursor flash annoyingly fast (on cv64-3d at least).
> 
> Why...

Because drivers/video/fbcon.c uses the Amiga vertical blank interrupt for
cursor control. Perhaps we should switch to a timer on all platforms?

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@li...

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

Re: Sound patch

[Richard Z. <Ric...@st...>]

> > It works, no more stabbing at F keys before ABBA ;) but it makes the
> > cursor flash annoyingly fast (on cv64-3d at least).
> > 
> > Why...
> 
> Because drivers/video/fbcon.c uses the Amiga vertical blank interrupt for
> cursor control. Perhaps we should switch to a timer on all platforms?

it sure enough works for the Q40, no need for special hacks. Or does the
Amiga solution offer special advantages like extra smooth cursor animation ;) ?

Bye
Richard

Re: Sound patch

[Ken T. <ke...@we...>]

On Fri, 2 Feb 2001, Geert Uytterhoeven wrote:


> Because drivers/video/fbcon.c uses the Amiga vertical blank interrupt for
> cursor control. Perhaps we should switch to a timer on all platforms?

OK, where's the interrupt derrived from if no CONFIG_FB_AMIGA ?.

Ken.  

Re: Sound patch

[Richard Z. <Ric...@st...>]

On Sat, Feb 03, 2001 at 05:03:55PM +1100, Ken Tyler wrote:
> 
> 
> On Fri, 2 Feb 2001, Geert Uytterhoeven wrote:
> 
> 
> > Because drivers/video/fbcon.c uses the Amiga vertical blank interrupt for
> > cursor control. Perhaps we should switch to a timer on all platforms?
> 
> OK, where's the interrupt derrived from if no CONFIG_FB_AMIGA ?.

did you have a look at fbcon.c:fbcon_startup ?

Bye
Richard

Re: Sound patch

[Ken T. <ke...@we...>]

On Sun, 4 Feb 2001, Richard Zidlicky wrote:

> > OK, where's the interrupt derrived from if no CONFIG_FB_AMIGA ?.
> 
> did you have a look at fbcon.c:fbcon_startup ?

Yes but no wiser. I see

        irqres = request_irq(IRQ_AMIGA_VERTB, fbcon_vbl_handler, 0,
                             "console/cursor", fbcon_vbl_handler);


but not what's generating the vertical rate interrupts.

Ken.

Re: Sound patch

[Richard Z. <Ric...@st...>]

On Sun, Feb 04, 2001 at 07:32:10PM +1100, Ken Tyler wrote:
> 
> 
> On Sun, 4 Feb 2001, Richard Zidlicky wrote:
> 
> > > OK, where's the interrupt derrived from if no CONFIG_FB_AMIGA ?.
> > 
> > did you have a look at fbcon.c:fbcon_startup ?
> 
> Yes but no wiser. I see
> 
>         irqres = request_irq(IRQ_AMIGA_VERTB, fbcon_vbl_handler, 0,
>                              "console/cursor", fbcon_vbl_handler);
> 
> 
> but not what's generating the vertical rate interrupts.

try simply deleting that request_irq() and fbcon should fall back
to simple timer based cursor.


Bye
Richard