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

[Takashi Iwai <tiwai@suse.de>]

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