[alsa-devel] [PATCH 1/1] usb: gadget: f_uac2: Fix pcm sample size selection

[Felipe Balbi]

On Tue, Jul 22, 2014 at 07:09:51PM +0200, Daniel Mack wrote:
> On 07/04/2014 07:43 AM, Takashi Iwai wrote:
> > At Thu,  3 Jul 2014 20:15:28 +0200,
> > Sebastian Reimers wrote:
> >>
> >> The pcm playback and capture sample size format was fixed
> >> SNDRV_PCM_FMTBIT_S16_LE.
> >>
> >> This patch respects also 16, 24 and 32 bit p_ssize and c_ssize values.
> >>
> >> Signed-off-by: Sebastian Reimers <sebastian.reimers at gmail.com>
> > 
> > In general, this driver should check more on p_ssize.  This can be any
> > integer value passed as a module parameter.  It should be cut to a
> > sane value or give an error in the probe function when an invalid
> > value is given.  Ditto for p_srate and other parameters.
> > 
> > Also, I don't understand the logic of the code
> > 	runtime->hw.period_bytes_min = 2 * uac2->p_prm.max_psize
> > 					/ runtime->hw.periods_min;
> > 
> > But this is mostly irrelevant with this patch, so for this patch:
> >    Reviewed-by: Takashi Iwai <tiwai at suse.de>
> 
> Felipe, are you fine with this one?

seems like I missed this one, do you mind if we take it during v3.17-rc
and add Cc: stable ?

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20140722/464cbe46/attachment.sig>