Date: Wed, 29 Oct 2014 10:50:52 +0530
From: Vinod Koul <vinod.koul@intel.com>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Qais Yousef <qais.yousef@imgtec.com>, devicetree@vger.kernel.org,
        Neil Jones <neil.jones@imgtec.com>, Arnd Bergmann <arnd@arndb.de>,
        Clemens Ladisch <clemens@ladisch.de>, alsa-devel@alsa-project.org,
        linux-kernel@vger.kernel.org
Subject: Re: [alsa-devel] [PATCH 00/11] Add AXD Audio Processing IP driver
Message-ID: <20141029052052.GS28745@intel.com>
References: <1414495589-8579-1-git-send-email-qais.yousef@imgtec.com>
 <544F8439.4080402@ladisch.de>
 <544F97A4.7080209@imgtec.com>
 <20141028141348.GD18384@kroah.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20141028141348.GD18384@kroah.com>
User-Agent: Mutt/1.5.20 (2009-06-14)
Sender: linux-kernel-owner@vger.kernel.org

On Tue, Oct 28, 2014 at 10:13:48PM +0800, Greg Kroah-Hartman wrote:
> On Tue, Oct 28, 2014 at 01:18:28PM +0000, Qais Yousef wrote:
> > On 10/28/2014 11:55 AM, Clemens Ladisch wrote:
> > >Qais Yousef wrote:
> > >>AXD Audio Processing IP performs audio decoding, encoding, mixing, equalisation,
> > >>synchronisation and playback.
> > >What exactly do you mean with "synchronisation" and "playback"?
> > 
> > Synchronisation refers to accurate audio playout relative to a master
> > clock source including compensation of drift between the master clock
> > source and the playout clock of the audio hardware. Hence allowing
> > synchronised audio playout across multiple independent devices.
> > 
> > Playback simple refers to the fact that AXD is capable of managing audio
> > playout hardware like I2S and SPDIF interfaces.
> > 
> > 
> > >>It doesn't fit in alsa subsystem but I Cced them to confirm.
> > >... because those two words sound like something that a sound card could do.
> > 
> > The problem mainly stems from the fact that we take a variety of
> > compressed audio as input and we could perform audio encoding. The
> > problem with the compressed audio is that the range of decoders and
> > configuration supported in alsa is limited and there's no support for
> > taking raw pcm and producing compressed output. I'm not an expert on
> > alsa but when I looked it looked like there's more infra structure
> > required.
> > 
> > The following not supported points from Documentation/sound/alsa/compress_offload.txt affect us:
> > 
> > - Volume control/routing is not handled by this API. Devices exposing a
> >   compressed data interface will be considered as regular ALSA devices;
> >   volume changes and routing information will be provided with regular
> >   ALSA kcontrols.
> > 
> > - Embedded audio effects. Such effects should be enabled in the same
> >   manner, no matter if the input was PCM or compressed.
> > 
> > - Encoding/decoding acceleration is not supported as mentioned
> >   above. It is possible to route the output of a decoder to a capture
> >   stream, or even implement transcoding capabilities. This routing
> >   would be enabled with ALSA kcontrols.
> 
> So instead you created a one-off api just for this hardware?  Ick, no,
> please work with the audio developers to incorporate it into the
> standard Linux audio apis so that everyone can benifit and not require
> special userspace programs to drive this hardware.
I think it more a case of I want it do this by method A, ...naaaaah thats
not what is availble in ALSA so let me redo the whole stack rather than
model my driver to use ALSA

-- 
~Vinod
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/