[alsa-devel] Getting pcm_usb_stream plugin to know its limits. [Kind of SOLVED]

[Takashi Iwai]

At Wed, 10 Feb 2010 17:56:26 -0500,
Michaël Cadilhac wrote:
> 
> michael at cadilhac.name (Michaël Cadilhac) writes:
> 
> > Takashi Iwai <tiwai at suse.de> writes:
> >
> >> At Mon, 11 Jan 2010 13:35:19 -0500,
> >> Michaël Cadilhac wrote:
> >>> 
> >>> michael at cadilhac.name (Michaël Cadilhac) writes:
> >>> 
> >>> > michael at cadilhac.name (Michaël Cadilhac) writes:
> >>> >
> >>> >> So, my final saying is the following patch.  It fixes the segfault on
> >>> >> stopping a non-started usb_stream, plus it adds the ability to set a
> >>> >> period size and sound rate.  The latter offers a workaround for the
> >>> >> second issue I came with (that the default values for period bytes, and
> >>> >> hence period size, were too high for us122l to work).
> >>> >
> >>> > Any comment on that patch would be greatly appreciated.  I'm just
> >>> > wondering if it's safe for me to give it to some people.
> >>> 
> >>> Anyone?  Note that this also fixes a real bug, so it might be of
> >>> interest to include it in the trunk.
> >>
> >> Through a quick glance, the patch looks OK to me.
> >> Could you give a proper subject and changelog to merge your patch
> >> to git tree?
> >
> > Hi there Takashi, and sorry for the delay.  Here are the patch,
> > subdivised in two distinct logical changes.  I hope the format is what
> > you asked for.
> 
> Pinging.  IIUC, I forgot to include "subject" lines.  The first one
> should be "usb_stream: Check for NULL-ness before dereferencing", and the
> second one "usb_stream: Allow user-set period-size and rate.".

Thanks, applied now.

But, at the next time, please make each patch as directly applicable.
That is, put subject and from lines to the patch, together with the
patch description (and preferably with your sign-off) embedded to
the patch itself.  That is, the same style for linux kernel patches.

See $LINUX/Documentation/SubmittingPatches for details.


thanks,

Takashi