[alsa-devel] [PATCH 5/9] ASoC: Intel: Skylake: Add topology core init and handlers

[Vinod Koul]

On Sat, Aug 15, 2015 at 10:00:58AM -0700, Mark Brown wrote:
> On Sat, Aug 15, 2015 at 07:46:58PM +0530, Vinod Koul wrote:
> > On Fri, Aug 14, 2015 at 11:03:13PM +0100, Mark Brown wrote:
> 
> > > > +	/* Index is for each config load */
> > > > +	ret = snd_soc_tplg_component_load(&platform->component, &skl_tplg_ops, fw, 0);
> 
> > > Which index?
> 
> > The last arg of snd_soc_tplg_component_load() is id which is set as
> > tplg.req_index = id;
> 
> > So the comment tries to explain how last 0 index is added. We have only one
> > load so we will be always 0 index
> 
> Your comment isn't explaining that at all, sorry - it's making things
> less clear.  It's peering into the implementation to translate ID to
> index and even with s/id/index/ it's begging the question "ID/index is
> what for each component load?".  If you're adding a comment here I'd
> expect to see something like "Use the same ID each time because..."
> which explains something that isn't in the code.

Fair point, will add this detail now

-- 
~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/20150815/b0ed41e8/attachment.sig>