Takashi Iwai wrote:
>
> Hi,
>
> At Thu, 24 Oct 2002 00:35:25 +0900,
> Osamu Tomita wrote:
> >
> > Thanks for comments.
> >
> > Takashi Iwai wrote:
> > > At Sat, 19 Oct 2002 01:56:53 +0900,
> > > Osamu Tomita wrote:
> > > >
> > > > This is part 23/26 of patchset for add support NEC PC-9800 architecture,
> > > > against 2.5.43.
> > > >
> > > > Summary:
> > > > ALSA sound driver related modules.
> > > > - add feature to support CS4231+OPL3 (not PNP)
> > >
> > > Are you sure that it's really CS4231?
> > > If it's a higher model, such as cs4232, cs4235 or cs4236, FM OPL3 is
> > > already supported (although additional codes to opl3 module are
> > > necessary for PC9800).
> > Your comment is reasonable. Some card has CS4232. But CS4232 is used
> > as CS4231. I guess there is some hardwired circuit. And some card has
> > CS4231. CS4231 driver works fine for ether chip on PC-9800.
> > So I choose CS4231 driver for PC-9800.
>
> well, in fact, cs4232 is backward compatible to cs4231.
>
> the question is, whether cs4232 module works on PC9800, or not.
> i guess the control-port is not used on this card. in such a case,
> you can deactivate the control-port via module option (or even add
> ifdef for the specific kernel config).
>
> if cs4232 doesn't work, we'll apply your patch to cs4231.
>
> > > > - add hardware specific initialization.
> > >
> > > The MPU401 hack looks odd.
> > > I'd propose to split a PC9800 specific driver up, rather than
> > > including bunch of ifdefs...
> > I see. I'll split MPU401 driver.
>
> thanks.
>
> please send me the new patch to me, too, so that i can merge it also
> to ALSA tree.
Thanks! I'll do everythig.
Regard
Osamu Tomita
-
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 : Wed Oct 23 2002 - 22:01:04 EST