On Fri, 23 Jan 2009 08:42:39 +0100 Takashi Iwai <tiwai@xxxxxxx> wrote: > At Thu, 22 Jan 2009 21:52:37 +0100, > Krzysztof Helt wrote: > > + .rate_min = 8000, > > + .rate_max = 48000, > > + .channels_min = 1, > > + .channels_max = 2, > > + .buffer_bytes_max = 0x3000, > > + .period_bytes_min = 0x40, > > + .period_bytes_max = 0x1800, > > + .periods_min = 2, > > + .periods_max = 3, > > Wow, I didn't notice this restriction... Really up to 3 periods? > I have to check but probably it is. > > --- linux-rc5/sound/isa/msnd/msnd.h 1970-01-01 01:00:00.000000000 +0100 > > +++ linux-git/sound/isa/msnd/msnd.h 2009-01-22 21:17:09.125632934 +0100 > ... > > +#ifdef SLOWIO > > +# undef outb > > +# undef inb > > +# define outb outb_p > > +# define inb inb_p > > +#endif > > This is too hackish. If SLOWIO is only for alpha, and the driver is > configured to be X86-only, I'd suggest to get rid of this hack. > I would leave it as it allows Alpha guys to fix it later (if ever). > > --- linux-rc5/sound/isa/msnd/msnd_midi.c 1970-01-01 01:00:00.000000000 +0100 > > +++ linux-git/sound/isa/msnd/msnd_midi.c 2009-01-22 18:35:00.942072608 +0100 > > @@ -0,0 +1,179 @@ > > +/* > > + * Copyright (c) by Jaroslav Kysela <perex@xxxxxxxx> > > + * Routines for control of MPU-401 in UART mode > > Better to put your name here and there, too... > I can put my name along as the file is almost a strict copy of the mpu401 code. It was put this way in the Kirsten's driver and I did only cut some code from there. > > +static int index[SNDRV_CARDS] = SNDRV_DEFAULT_IDX; /* Index 0-MAX */ > > +static char *id[SNDRV_CARDS] = SNDRV_DEFAULT_STR; /* ID for this card */ > > + > > +module_param_array(index, int, NULL, S_IRUGO); > > +MODULE_PARM_DESC(index, "Index value for msnd_pinnacle soundcard."); > > +module_param_array(id, charp, NULL, S_IRUGO); > > +MODULE_PARM_DESC(id, "ID string for msnd_pinnacle soundcard."); > > + > > +static long io[SNDRV_CARDS] __devinitdata = SNDRV_DEFAULT_PORT; > > +static int irq[SNDRV_CARDS] __devinitdata = SNDRV_DEFAULT_IRQ; > > +static long mem[SNDRV_CARDS] __devinitdata = SNDRV_DEFAULT_PORT; > > + > > +static long cfg[SNDRV_CARDS] __devinitdata = SNDRV_DEFAULT_PORT; > > The modules parameters aren't __devinitdata. Remove these > annotations. > I haven't known that. Could you explain? Thanks, Krzysztof ---------------------------------------------------------------------- Speak Up. Angielski szybko i skutecznie. 3 miesiace nauki gratis. Sprawdz. >> http://link.interia.pl/f2019 _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
