Re: [PATCHSET 22/25] add support for PC-9800 architecture (sound alsa)

From: Takashi Iwai (tiwai@suse.de)
Date: Fri Nov 01 2002 - 08:44:25 EST


At Fri, 1 Nov 2002 22:02:18 +0900,
Osamu Tomita wrote:
>
> 2002.10.31 02:52, Takashi Iwai wrote:
> > At Wed, 30 Oct 2002 02:43:22 +0900,
> (snip)
> > do you mean that the interrupts are generated too slowly?
> > you can measure how many UART interrupts are generated per second by
> > checking /proc/interrupts.
> > if it's too little, the problem is likely in the hardware
> > initialization part, which i simply copied from your patch.
> > but it could be a wrong copy & paste.
> > please check pc98_mpu401_init() in pc98.c.
> >
> > btw, which MPU mode are you using, PC98II or normal MPU401?
> Thanks! Problem solved. MPU401 works fine by trivial fix as follows.
>
> #--- linux/sound/isa/cs423x/pc98.c.old Wed Oct 30 00:20:35 2002
> #+++ linux/sound/isa/cs423x/pc98.c Fri Nov 1 20:45:31 2002
> #@@ -379,7 +379,7 @@
> # err = pc98_mpu401_init(mpu_irq[dev]);
> # if (! err) {
> # err = snd_mpu401_uart_new(card, 0,
> #- pc98ii ?
> MPU401_HW_PC98II : MPU401_HW_MPU401,
> #+ pc98ii[dev] ?
> MPU401_HW_PC98II : MPU401_HW_MPU401,
> # mpu_port[dev], 0,
> # mpu_irq[dev],
> SA_INTERRUPT, NULL);
> # if (err < 0)

d'oh. thanks.

>
> Now, PCM OPL3 and MPU401 all works fine.
> Would you please add option "pc98ii" into snd-mpu401?
> This is for Box that has MPU-PC98II only, with no other sound card.

well, then i misunderstood the hardware configuration on PC98.

doesn't MPU-PC98II need any specific hardware init code like in
pc98_mpu401_init()?

if yes, then we should create another module for PC98II rather than
including a magical init code into a generic mpu401 module.

ciao,

Takashi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Thu Nov 07 2002 - 22:00:20 EST