Re: [PATCH v3 2/6] ASoC: hdac-hdmi: Add hdmi driver

[Vinod Koul <vinod.koul@intel.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1193 bytes --]

On Sun, Nov 01, 2015 at 12:02:31PM +0900, Mark Brown wrote:
> On Tue, Oct 27, 2015 at 04:42:13PM +0900, Vinod Koul wrote:
> 
> > +	if (err < 0)
> > +		dev_err(&hdac->dev, "Failed to query pcm params for nid: %d\n", cvt->nid);
> 
> That looks like the NID is being printed as an error code.

No this prints the NID for which query fails, but agree we should print the
error code too, that might be very helpful :)

> > +	/*
> > +	 * Currently on board only 1 pin and 1 converter enabled for
> > +	 * simplification, more will be added eventually
> > +	 * So using fixed map for dai_id:pin:cvt
> > +	 */
> > +	return hdac_hdmi_init_dai_map(edev, &hdmi->dai_map[0], hdmi->pin_nid[0],
> > +			hdmi->cvt_nid[0], 0);
> 
> I'm not entirely sure I understand what this is all doing.  It looks
> like it's trying to translate the HDA widget map into a DAPM map which
> seems sensible but it appears it's making some simplifying assumptions
> about the device it's dealing with?

The device is actually quite simple and yes we simplified even further by
ignoring multiple pins for now. We will keep adding more features and adding
stuff to map as we go along..

-- 
~Vinod

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]