Re: [alsa-devel] [RFC PATCH 2/3] ucm: Parse sequence of component devices

-----Original Message----- From: Liam Girdwood [mailto:liam.r.girdwood@linux.intel.com] Sent: Tuesday, November 15, 2016 4:46 PM

...

@@ -306,6 +381,30 @@ static int parse_sequence(snd_use_case_mgr_t

*uc_mgr ATTRIBUTE_UNUSED,

...

	continue;
}

• if (strcmp(cmd, "enadev") == 0) {
  

• 	/* need to enable a component device */
  

• 	curr->type = SEQUENCE_ELEMENT_TYPE_CMPT_SEQ;
  

• 	err = parse_component_seq(uc_mgr, n, 1,
  

• 				&curr->data.cmpt_seq);
  

• 	if (err < 0) {
  

• 		uc_error("error: enadev requires a valid
  

device!");

...

• 		return err;
  

• 	}
  

• 	continue;
  

• }
  

• if (strcmp(cmd, "disdev") == 0) {
  

Do we need to document enadev and disdev anywhere ?

Liam

Maybe we could document this in the description of function parse_sequence() and parse_verb()? parse_verb() has an example of EnableSequence and DisableSequence.

Thanks Mengdong