Re: [PATCH] usb: gadget: f_uac2: modulate playback data rate

2014-08-29 Thread Jassi Brar
On Fri, Aug 29, 2014 at 1:21 PM, Jassi Brar wrote: >>> + >>> + rate = opts->p_srate; >>> + if (rate == 5512) { /* which implies 5512.5 practically */ >>> + rate = 55125; >>> + intvl *= 10; >>> + } >> Well, I'd say that Al

Re: [PATCH] usb: gadget: f_uac2: modulate playback data rate

2014-08-29 Thread Jassi Brar
Hi Daniel, On 29 August 2014 12:52, Daniel Mack wrote: > Hi, > > On 08/29/2014 08:13 AM, Jassi Brar wrote: >> +/* >> + * 5512.5Hz is going to need the maximum number of elements (80), >> + * in the length-pattern loop, among standard ALSA supported rates. >> + */ >> +#define MAX_LOOP_LEN 80 >> +

Re: [PATCH] usb: gadget: f_uac2: modulate playback data rate

2014-08-29 Thread Daniel Mack
Hi, On 08/29/2014 08:13 AM, Jassi Brar wrote: > +/* > + * 5512.5Hz is going to need the maximum number of elements (80), > + * in the length-pattern loop, among standard ALSA supported rates. > + */ > +#define MAX_LOOP_LEN 80 > + > struct uac2_rtd_params { > struct snd_uac2_chip *uac2; /* p

Re: [PATCH] usb: gadget: f_uac2: modulate playback data rate

2014-08-28 Thread Jassi Brar
On Fri, Aug 29, 2014 at 11:43 AM, Jassi Brar wrote: > The UAC2 function driver currently responds to all packets at all times > with wMaxPacketSize packets. That results in way too fast audio > playback as the function driver (which is in fact supposed to define > the audio stream pace) delivers a

[PATCH] usb: gadget: f_uac2: modulate playback data rate

2014-08-28 Thread Jassi Brar
The UAC2 function driver currently responds to all packets at all times with wMaxPacketSize packets. That results in way too fast audio playback as the function driver (which is in fact supposed to define the audio stream pace) delivers as fast as it can. We need data rate to match, as accurately