[alsa-devel] [PATCH v2 3/6] ASoC: Intel: Reorganize the common dsp structure

Vinod Koul vinod.koul at intel.com
Thu Jul 9 06:32:39 CEST 2015


On Wed, Jul 08, 2015 at 07:38:33PM +0100, Mark Brown wrote:
> On Fri, Jul 03, 2015 at 04:04:04PM +0530, Vinod Koul wrote:
> > From: "Subhransu S. Prusty" <subhransu.s.prusty at intel.com>
> > 
> > We will add SKL platform data. So organizing common and platform
> > specific data helps.
> 
> >  struct sst_dsp {
> >  
> > +	/* Shared for all platforms */
> > +
> >  	/* runtime */
> >  	struct sst_dsp_device *sst_dev;
> >  	spinlock_t spinlock;	/* IPC locking */
> > @@ -268,10 +270,6 @@ struct sst_dsp {
> >  	int irq;
> >  	u32 id;
> >  
> > -	/* list of free and used ADSP memory blocks */
> > -	struct list_head used_block_list;
> > -	struct list_head free_block_list;
> > -
> 
> Perhaps a union might start to make sense at some point?
Initially we did start with a union but Liam didn't like another indirection.
Anyway since not a bunch a members are getting added here, this should be
okay

Liam you want to add here...

-- 
~Vinod
-------------- 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/20150709/8c6eed52/attachment.sig>


More information about the Alsa-devel mailing list