[PATCH v3 20/24] echo-cancel: Add beamforming support in the webrtc canceller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2016-02-17 at 18:47 +0530, Arun Raghavan wrote:
> On Mon, 2016-02-15 at 13:12 +0200, Tanu Kaskinen wrote:
> > On Mon, 2016-01-18 at 13:06 +0530, arun at accosted.net wrote:
> > > +                pa_log("The beamformer currently only supports targeting along the azimuth");
> > > +                goto fail;
> > > +            }
> > > +
> > > +            direction.s[0] = f[0];
> > > +            direction.s[1] = f[1];
> > > +            direction.s[2] = f[2];
> > > +        }
> > > +
> > > +        /* The beamformer gives us a single channel */
> > > +        out_ss->channels = 1;
> > > +        pa_channel_map_init_mono(out_map);
> > 
> > Could this be moved to fixate_spec()? I don't like the audio format
> > parameters being calculated all over the place.
> 
> I wanted to do that, but we can't because we need the capture channel
> count for parsing the mic geometry (there is a comment explaining that
> where if (beamforming) { ... } is).

I meant moving only the out_ss and out_map assignments. Those don't
depend on the capture channel count. fixate_spec() will then need to
take the beamforming bool as a parameter, but I think that's fine.

-- 
Tanu


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux