Message ID | 20240203023645.31105-51-quic_wcheng@quicinc.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel+bounces-50840-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:9bc1:b0:106:209c:c626 with SMTP id op1csp818347dyc; Fri, 2 Feb 2024 18:46:36 -0800 (PST) X-Google-Smtp-Source: AGHT+IGcpzQdrXmU2jR06QoEQB1W15lLY1g5fiEHaKLZUwH6eSkx3FR8sGMmV1OOUJGGBr50NRh/ X-Received: by 2002:ac8:5401:0:b0:42c:1080:a0f with SMTP id b1-20020ac85401000000b0042c10800a0fmr94218qtq.60.1706928396625; Fri, 02 Feb 2024 18:46:36 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706928396; cv=pass; d=google.com; s=arc-20160816; b=lyt/nsfI1KGFNNnZDTEBUofikeZSYjGan3XauQBOLvQrN9OtnANTjtfuh/vlqSfmX2 P6+uXZiTg4J7GcuTW9UCLSjliFNvzrPW5UBOgfjaWNs9PDhfgg4IG2xlC0zKE78BGjS/ 00GjGB+CHjg99A4iG9zXBfZXVza622I/jEDU7Zu9iZPstDRwtLAhDHA0hUagV3u3ala8 J+IJwSE8XaeI/JWU8C2e4sCc1Dz4RVq+LSOZ0i3r5Jp5EuWowRHczcjsWi7JsXD50C7L D4dQtvsnGVT+9j9eYGJSwo36gqxzRK8GBMWGhUEJiKbisReLklLBmFwAPV2BlIEbSKCU JLZg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:list-unsubscribe:list-subscribe:list-id:precedence :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=B6HqoGAf59UDCE9RA1dgQJWtkxf5nc1TAsdGr04J9VA=; fh=NCiZC+LkA7Z++D1Sw+I1V0W+yA5qiZcc33yXbLlmf4o=; b=pAKZicXL0G5VTls37haJCdoSr/WuLp1xhv2TH8CQmDwnY9QSJ9sY1K8GfNH/U9hMGI gBW1cpFLbQF0QV2Khtt+GHIwzqb63GFUG6OF/V9JzG14Yng9nSFNJb1igXbteKo3fFNT 99bjoW03FvAdsBh84TwVtEmFOeC2HZKV0XVNSPU4Y6clfqLqUILifSSVJz4fb3QDza7/ fFdDQ7MenV2ORKgl3L361RzfqA5st0BE+h01iCNuGIijgwEHE+HQprLsmH7v6Jkbzkla KlzQpNt1GmoQl4vgZ4XWVB23XGUpbWPOU7RKmEFsplEe+lHpjcloaFqiIotbeQHIXEPq S5RQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=bnzrSkkQ; arc=pass (i=1 spf=pass spfdomain=quicinc.com dkim=pass dkdomain=quicinc.com dmarc=pass fromdomain=quicinc.com); spf=pass (google.com: domain of linux-kernel+bounces-50840-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-50840-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com X-Forwarded-Encrypted: i=1; AJvYcCWzNJFwQK15YI1pCcwYTn0UZ+pM0YN4CQiCebk1FMQMWZPYAE8Ol6jOnneo//AEcUIzRSNRiO607F+xfU+9x687c9WYDQ== Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id w15-20020a05622a134f00b0042bf8e02246si3495129qtk.614.2024.02.02.18.46.36 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 02 Feb 2024 18:46:36 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-50840-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=bnzrSkkQ; arc=pass (i=1 spf=pass spfdomain=quicinc.com dkim=pass dkdomain=quicinc.com dmarc=pass fromdomain=quicinc.com); spf=pass (google.com: domain of linux-kernel+bounces-50840-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-50840-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 6F7141C242B5 for <ouuuleilei@gmail.com>; Sat, 3 Feb 2024 02:46:05 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A4A8142A9E; Sat, 3 Feb 2024 02:38:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=quicinc.com header.i=@quicinc.com header.b="bnzrSkkQ" Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 31123111B2; Sat, 3 Feb 2024 02:37:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706927872; cv=none; b=VyQzaQ5RwJryx5Qk8gyuC/PgczIhQnQBSpHgR2UNKlL8BwyvT5PJ+RnOir9cZ8h/C3GW4f8DOW3xyO0OVc1UvPKc7FiPRRiSw5JZznavVoRnGv8fmwCBMnT506SVbWfCjq76SSqW/CWP/T9kWpdCXyHdfMw0ShuwV4ejAe2vL40= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706927872; c=relaxed/simple; bh=xCYa/T1UZkUzSPguiIm4X9Udx/Zl2KAFEFl4QY/9Y1o=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=OESp//i9EZHabjb1wBZPq2u/aKugdAYnLlvfYoTml7c+qMAMbStGPgpQ1Zhg7IiAfQVdwJV1osABZJ8mOqTpBPEDtnVV6mXilXhzai29IL0YuXooZcKTX0N1T3fmgzqoKAd4kOaJRNppLsdWXOtb4hGWRtR5IhEmf6lyqSdmcBA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=quicinc.com; spf=pass smtp.mailfrom=quicinc.com; dkim=pass (2048-bit key) header.d=quicinc.com header.i=@quicinc.com header.b=bnzrSkkQ; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=quicinc.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=quicinc.com Received: from pps.filterd (m0279870.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.24/8.17.1.24) with ESMTP id 4132bYTt031016; Sat, 3 Feb 2024 02:37:34 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-type; s=qcppdkim1; bh=B6HqoGAf59UDCE9RA1dg QJWtkxf5nc1TAsdGr04J9VA=; b=bnzrSkkQe7dxR6mY/xbWi9umXyBdHISQ381Q 3zVbSgW82fnM6YuhcA8e24CT5yb1tZAG0UaffSHjkeg6Q75iHp/1rhX59pqb2BTA F8jIB73wGWvg0qkD+tX+63nLxwOmkMzsT9TFjEegltPMu1aVbBwBGG1Z1pwhIH6O sbW/sh85YCprAmlcz23Xt0xyojWSRvGqtyXe8KDn6HzahGpP/iE0HZI1WrTkNQEH VaxbmQgRpEIVk3qsf99gQ1G1QxLrXslSEkaWY8gvxJiKqQ15YYqc130So1OySutR COPVSRdkW3rEdGEvwsUMGFo0kJIwJtJmGQTkS+gJENrQiqVuFQ== Received: from nalasppmta05.qualcomm.com (Global_NAT1.qualcomm.com [129.46.96.20]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3w0ptwawrg-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Sat, 03 Feb 2024 02:37:33 +0000 (GMT) Received: from nalasex01b.na.qualcomm.com (nalasex01b.na.qualcomm.com [10.47.209.197]) by NALASPPMTA05.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 4132bWaK003624 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Sat, 3 Feb 2024 02:37:32 GMT Received: from hu-wcheng-lv.qualcomm.com (10.49.16.6) by nalasex01b.na.qualcomm.com (10.47.209.197) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.40; Fri, 2 Feb 2024 18:37:32 -0800 From: Wesley Cheng <quic_wcheng@quicinc.com> To: <srinivas.kandagatla@linaro.org>, <mathias.nyman@intel.com>, <perex@perex.cz>, <conor+dt@kernel.org>, <corbet@lwn.net>, <lgirdwood@gmail.com>, <andersson@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>, <gregkh@linuxfoundation.org>, <Thinh.Nguyen@synopsys.com>, <broonie@kernel.org>, <bgoswami@quicinc.com>, <tiwai@suse.com>, <robh+dt@kernel.org>, <konrad.dybcio@linaro.org> CC: <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>, <linux-sound@vger.kernel.org>, <linux-usb@vger.kernel.org>, <linux-arm-msm@vger.kernel.org>, <linux-doc@vger.kernel.org>, <alsa-devel@alsa-project.org>, Wesley Cheng <quic_wcheng@quicinc.com> Subject: [PATCH v13 50/53] ALSA: usb-audio: Allow for rediscovery of connected USB SND devices Date: Fri, 2 Feb 2024 18:36:42 -0800 Message-ID: <20240203023645.31105-51-quic_wcheng@quicinc.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20240203023645.31105-1-quic_wcheng@quicinc.com> References: <20240203023645.31105-1-quic_wcheng@quicinc.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Type: text/plain X-ClientProxiedBy: nalasex01b.na.qualcomm.com (10.47.209.197) To nalasex01b.na.qualcomm.com (10.47.209.197) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-ORIG-GUID: jm1gF90H2k3KN9lIsynYLe7DAn5vfYVP X-Proofpoint-GUID: jm1gF90H2k3KN9lIsynYLe7DAn5vfYVP X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2024-02-02_16,2024-01-31_01,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 bulkscore=0 malwarescore=0 mlxscore=0 lowpriorityscore=0 phishscore=0 clxscore=1015 mlxlogscore=999 adultscore=0 impostorscore=0 suspectscore=0 priorityscore=1501 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2401310000 definitions=main-2402030015 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1789844150075629036 X-GMAIL-MSGID: 1789844150075629036 |
Series |
Introduce QC USB SND audio offloading support
|
|
Commit Message
Wesley Cheng
Feb. 3, 2024, 2:36 a.m. UTC
In case of notifying SND platform drivers of connection events, some of
these use cases, such as offloading, require an ASoC USB backend device to
be initialized before the events can be handled. If the USB backend device
has not yet been probed, this leads to missing initial USB audio device
connection events.
Expose an API that traverses the usb_chip array for connected devices, and
to call the respective connection callback registered to the SND platform
driver.
Signed-off-by: Wesley Cheng <quic_wcheng@quicinc.com>
---
sound/usb/card.c | 19 +++++++++++++++++++
sound/usb/card.h | 2 ++
sound/usb/qcom/qc_audio_offload.c | 2 ++
3 files changed, 23 insertions(+)
Comments
On 2/3/2024 3:36 AM, Wesley Cheng wrote: > In case of notifying SND platform drivers of connection events, some of > these use cases, such as offloading, require an ASoC USB backend device to > be initialized before the events can be handled. If the USB backend device > has not yet been probed, this leads to missing initial USB audio device > connection events. > > Expose an API that traverses the usb_chip array for connected devices, and > to call the respective connection callback registered to the SND platform > driver. > > Signed-off-by: Wesley Cheng <quic_wcheng@quicinc.com> > --- > sound/usb/card.c | 19 +++++++++++++++++++ > sound/usb/card.h | 2 ++ > sound/usb/qcom/qc_audio_offload.c | 2 ++ > 3 files changed, 23 insertions(+) > > diff --git a/sound/usb/card.c b/sound/usb/card.c > index 11b827b7a2a5..995b2df676ab 100644 > --- a/sound/usb/card.c > +++ b/sound/usb/card.c > @@ -202,6 +202,25 @@ struct snd_usb_stream *snd_usb_find_suppported_substream(int card_idx, > } > EXPORT_SYMBOL_GPL(snd_usb_find_suppported_substream); > > +/* > + * in case the platform driver was not ready at the time of USB SND > + * device connect, expose an API to discover all connected USB devices > + * so it can populate any dependent resources/structures. > + */ > +void snd_usb_rediscover_devices(void) > +{ > + int i; > + > + mutex_lock(®ister_mutex); > + for (i = 0; i < SNDRV_CARDS; i++) { > + if (usb_chip[i]) > + if (platform_ops && platform_ops->connect_cb) > + platform_ops->connect_cb(usb_chip[i]); if inside if, it can just be && or maybe move callback check before mutex lock and just return early if it is not present? > + } > + mutex_unlock(®ister_mutex); > +} > +EXPORT_SYMBOL_GPL(snd_usb_rediscover_devices); > + > /* > * disconnect streams > * called from usb_audio_disconnect() > diff --git a/sound/usb/card.h b/sound/usb/card.h > index 6d59995440c3..3a0d68f453a1 100644 > --- a/sound/usb/card.h > +++ b/sound/usb/card.h > @@ -222,11 +222,13 @@ int snd_usb_unregister_platform_ops(void); > #if IS_ENABLED(CONFIG_SND_USB_AUDIO) > struct snd_usb_stream *snd_usb_find_suppported_substream(int card_idx, > struct snd_pcm_hw_params *params, int direction); > +void snd_usb_rediscover_devices(void); > #else > static struct snd_usb_stream *snd_usb_find_suppported_substream(int card_idx, > struct snd_pcm_hw_params *params, int direction) > { > return NULL; > } > +static void snd_usb_rediscover_devices(void) { } > #endif /* IS_ENABLED(CONFIG_SND_USB_AUDIO) */ > #endif /* __USBAUDIO_CARD_H */ > diff --git a/sound/usb/qcom/qc_audio_offload.c b/sound/usb/qcom/qc_audio_offload.c > index 08af82ec22ad..9b0f98600e58 100644 > --- a/sound/usb/qcom/qc_audio_offload.c > +++ b/sound/usb/qcom/qc_audio_offload.c > @@ -1867,6 +1867,8 @@ static int __init qc_usb_audio_offload_init(void) > if (ret < 0) > goto release_qmi; > > + snd_usb_rediscover_devices(); > + > return 0; > > release_qmi: >
Hi Amadeusz, On 2/5/2024 1:01 AM, Amadeusz Sławiński wrote: > On 2/3/2024 3:36 AM, Wesley Cheng wrote: >> In case of notifying SND platform drivers of connection events, some of >> these use cases, such as offloading, require an ASoC USB backend >> device to >> be initialized before the events can be handled. If the USB backend >> device >> has not yet been probed, this leads to missing initial USB audio device >> connection events. >> >> Expose an API that traverses the usb_chip array for connected devices, >> and >> to call the respective connection callback registered to the SND platform >> driver. >> >> Signed-off-by: Wesley Cheng <quic_wcheng@quicinc.com> >> --- >> sound/usb/card.c | 19 +++++++++++++++++++ >> sound/usb/card.h | 2 ++ >> sound/usb/qcom/qc_audio_offload.c | 2 ++ >> 3 files changed, 23 insertions(+) >> >> diff --git a/sound/usb/card.c b/sound/usb/card.c >> index 11b827b7a2a5..995b2df676ab 100644 >> --- a/sound/usb/card.c >> +++ b/sound/usb/card.c >> @@ -202,6 +202,25 @@ struct snd_usb_stream >> *snd_usb_find_suppported_substream(int card_idx, >> } >> EXPORT_SYMBOL_GPL(snd_usb_find_suppported_substream); >> +/* >> + * in case the platform driver was not ready at the time of USB SND >> + * device connect, expose an API to discover all connected USB devices >> + * so it can populate any dependent resources/structures. >> + */ >> +void snd_usb_rediscover_devices(void) >> +{ >> + int i; >> + >> + mutex_lock(®ister_mutex); >> + for (i = 0; i < SNDRV_CARDS; i++) { >> + if (usb_chip[i]) >> + if (platform_ops && platform_ops->connect_cb) >> + platform_ops->connect_cb(usb_chip[i]); > > if inside if, it can just be && or maybe move callback check before > mutex lock and just return early if it is not present? > Thanks for pointing that out. Makes sense, and will update it. Thanks Wesley Cheng
On Mon, 05 Feb 2024 10:01:03 +0100, Amadeusz Sławiński wrote: > > On 2/3/2024 3:36 AM, Wesley Cheng wrote: > > In case of notifying SND platform drivers of connection events, some of > > these use cases, such as offloading, require an ASoC USB backend device to > > be initialized before the events can be handled. If the USB backend device > > has not yet been probed, this leads to missing initial USB audio device > > connection events. > > > > Expose an API that traverses the usb_chip array for connected devices, and > > to call the respective connection callback registered to the SND platform > > driver. > > > > Signed-off-by: Wesley Cheng <quic_wcheng@quicinc.com> > > --- > > sound/usb/card.c | 19 +++++++++++++++++++ > > sound/usb/card.h | 2 ++ > > sound/usb/qcom/qc_audio_offload.c | 2 ++ > > 3 files changed, 23 insertions(+) > > > > diff --git a/sound/usb/card.c b/sound/usb/card.c > > index 11b827b7a2a5..995b2df676ab 100644 > > --- a/sound/usb/card.c > > +++ b/sound/usb/card.c > > @@ -202,6 +202,25 @@ struct snd_usb_stream *snd_usb_find_suppported_substream(int card_idx, > > } > > EXPORT_SYMBOL_GPL(snd_usb_find_suppported_substream); > > +/* > > + * in case the platform driver was not ready at the time of USB SND > > + * device connect, expose an API to discover all connected USB devices > > + * so it can populate any dependent resources/structures. > > + */ > > +void snd_usb_rediscover_devices(void) > > +{ > > + int i; > > + > > + mutex_lock(®ister_mutex); > > + for (i = 0; i < SNDRV_CARDS; i++) { > > + if (usb_chip[i]) > > + if (platform_ops && platform_ops->connect_cb) > > + platform_ops->connect_cb(usb_chip[i]); > > if inside if, it can just be && or maybe move callback check before > mutex lock and just return early if it is not present? The callback check must be inside mutex; otherwise you'll get a race about the platform_ops registration. thanks, Takashi
diff --git a/sound/usb/card.c b/sound/usb/card.c index 11b827b7a2a5..995b2df676ab 100644 --- a/sound/usb/card.c +++ b/sound/usb/card.c @@ -202,6 +202,25 @@ struct snd_usb_stream *snd_usb_find_suppported_substream(int card_idx, } EXPORT_SYMBOL_GPL(snd_usb_find_suppported_substream); +/* + * in case the platform driver was not ready at the time of USB SND + * device connect, expose an API to discover all connected USB devices + * so it can populate any dependent resources/structures. + */ +void snd_usb_rediscover_devices(void) +{ + int i; + + mutex_lock(®ister_mutex); + for (i = 0; i < SNDRV_CARDS; i++) { + if (usb_chip[i]) + if (platform_ops && platform_ops->connect_cb) + platform_ops->connect_cb(usb_chip[i]); + } + mutex_unlock(®ister_mutex); +} +EXPORT_SYMBOL_GPL(snd_usb_rediscover_devices); + /* * disconnect streams * called from usb_audio_disconnect() diff --git a/sound/usb/card.h b/sound/usb/card.h index 6d59995440c3..3a0d68f453a1 100644 --- a/sound/usb/card.h +++ b/sound/usb/card.h @@ -222,11 +222,13 @@ int snd_usb_unregister_platform_ops(void); #if IS_ENABLED(CONFIG_SND_USB_AUDIO) struct snd_usb_stream *snd_usb_find_suppported_substream(int card_idx, struct snd_pcm_hw_params *params, int direction); +void snd_usb_rediscover_devices(void); #else static struct snd_usb_stream *snd_usb_find_suppported_substream(int card_idx, struct snd_pcm_hw_params *params, int direction) { return NULL; } +static void snd_usb_rediscover_devices(void) { } #endif /* IS_ENABLED(CONFIG_SND_USB_AUDIO) */ #endif /* __USBAUDIO_CARD_H */ diff --git a/sound/usb/qcom/qc_audio_offload.c b/sound/usb/qcom/qc_audio_offload.c index 08af82ec22ad..9b0f98600e58 100644 --- a/sound/usb/qcom/qc_audio_offload.c +++ b/sound/usb/qcom/qc_audio_offload.c @@ -1867,6 +1867,8 @@ static int __init qc_usb_audio_offload_init(void) if (ret < 0) goto release_qmi; + snd_usb_rediscover_devices(); + return 0; release_qmi: