From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liam Girdwood Subject: Re: [PATCH v2] ucm: add another sequence 'msleep' Date: Mon, 22 Aug 2011 11:39:28 +0100 Message-ID: <4E5231E0.9020305@ti.com> References: <20110822012544.5379.75207.stgit@localhost6.localdomain6> <20110822053532.7740.34323.stgit@localhost6.localdomain6> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by alsa0.perex.cz (Postfix) with ESMTP id E7A142454E for ; Mon, 22 Aug 2011 12:39:31 +0200 (CEST) In-Reply-To: <20110822053532.7740.34323.stgit@localhost6.localdomain6> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Lu Guanqun Cc: Takashi Iwai , ALSA List-Id: alsa-devel@alsa-project.org On 22/08/11 06:35, Lu Guanqun wrote: > Thus, we have two sleep statements: > msleep > usleep > > Signed-off-by: Lu Guanqun > --- > src/ucm/parser.c | 11 +++++++++++ > src/ucm/ucm_local.h | 2 +- > 2 files changed, 12 insertions(+), 1 deletions(-) > > diff --git a/src/ucm/parser.c b/src/ucm/parser.c > index 23b67bc..b93d832 100644 > --- a/src/ucm/parser.c > +++ b/src/ucm/parser.c > @@ -316,6 +316,17 @@ static int parse_sequence(snd_use_case_mgr_t *uc_mgr ATTRIBUTE_UNUSED, > continue; > } > > + if (strcmp(cmd, "msleep") == 0) { > + curr->type = SEQUENCE_ELEMENT_TYPE_SLEEP; > + err = snd_config_get_integer(n, &curr->data.sleep); > + if (err < 0) { > + uc_error("error: msleep requires integer!"); > + return err; > + } > + curr->data.sleep *= 1000L; > + continue; > + } > + > if (strcmp(cmd, "exec") == 0) { > curr->type = SEQUENCE_ELEMENT_TYPE_EXEC; > err = parse_string(n, &curr->data.exec); > diff --git a/src/ucm/ucm_local.h b/src/ucm/ucm_local.h > index 0522bf5..03d3ace 100644 > --- a/src/ucm/ucm_local.h > +++ b/src/ucm/ucm_local.h > @@ -57,7 +57,7 @@ struct sequence_element { > struct list_head list; > unsigned int type; > union { > - long sleep; /* Sleep time in msecs if sleep element, else 0 */ > + long sleep; /* Sleep time in microseconds if sleep element, else 0 */ > char *cdev; > char *cset; > char *exec; > Acked-by: Liam Girdwood