Re: [Qemu-devel] [PATCH 1/3] fix: buffer_length is ignored

2019-03-15 Thread Martin Schrodt
On 3/15/19 9:01 AM, Gerd Hoffmann wrote: > On Fri, Mar 15, 2019 at 08:49:06AM +0100, Martin Schrodt wrote: >> Hi, >> >> On 3/15/19 8:43 AM, Gerd Hoffmann wrote: >>> Hi, >>> -qapi_AudiodevPaPerDirectionOptions_base(ppdo), &obt_as, 46440); +qapi_AudiodevPaPerDirectionOptio

Re: [Qemu-devel] [PATCH 1/3] fix: buffer_length is ignored

2019-03-15 Thread Martin Schrodt
Hi, On 3/15/19 8:43 AM, Gerd Hoffmann wrote: > Hi, > >> -qapi_AudiodevPaPerDirectionOptions_base(ppdo), &obt_as, 46440); >> +qapi_AudiodevPaPerDirectionOptions_base(ppdo), &obt_as, >> ppdo->buffer_length); > > I'd just use > > ppdo->has_buffer_length ? ppdo->buffer_leng

Re: [Qemu-devel] [PATCH 1/3] fix: buffer_length is ignored

2019-03-15 Thread Gerd Hoffmann
On Fri, Mar 15, 2019 at 08:49:06AM +0100, Martin Schrodt wrote: > Hi, > > On 3/15/19 8:43 AM, Gerd Hoffmann wrote: > > Hi, > > > >> -qapi_AudiodevPaPerDirectionOptions_base(ppdo), &obt_as, 46440); > >> +qapi_AudiodevPaPerDirectionOptions_base(ppdo), &obt_as, > >> ppdo->buffer_l

Re: [Qemu-devel] [PATCH 1/3] fix: buffer_length is ignored

2019-03-15 Thread Gerd Hoffmann
Hi, > -qapi_AudiodevPaPerDirectionOptions_base(ppdo), &obt_as, 46440); > +qapi_AudiodevPaPerDirectionOptions_base(ppdo), &obt_as, > ppdo->buffer_length); I'd just use ppdo->has_buffer_length ? ppdo->buffer_length : dev->timer_period * 4 here. cheers, Gerd

[Qemu-devel] [PATCH 1/3] fix: buffer_length is ignored

2019-03-14 Thread Martin Schrodt
Signed-off-by: Martin Schrodt --- audio/paaudio.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/audio/paaudio.c b/audio/paaudio.c index 5d410ed73f..1a799ca3e7 100644 --- a/audio/paaudio.c +++ b/audio/paaudio.c @@ -577,7 +577,7 @@ static int qpa_init_ou