From patchwork Tue Aug 30 18:38:35 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tony Lindgren X-Patchwork-Id: 9306115 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 4409B60756 for ; Tue, 30 Aug 2016 21:37:06 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 32F5A28D1F for ; Tue, 30 Aug 2016 21:37:06 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 25BAD28D86; Tue, 30 Aug 2016 21:37:06 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_NONE autolearn=unavailable version=3.3.1 Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7190528D83 for ; Tue, 30 Aug 2016 21:37:04 +0000 (UTC) Received: by alsa0.perex.cz (Postfix, from userid 1000) id 95D94266EFA; Tue, 30 Aug 2016 23:37:03 +0200 (CEST) Received: from alsa0.perex.cz (localhost [127.0.0.1]) by alsa0.perex.cz (Postfix) with ESMTP id 6AB66266EEE; Tue, 30 Aug 2016 23:25:30 +0200 (CEST) X-Original-To: alsa-devel@alsa-project.org Delivered-To: alsa-devel@alsa-project.org Received: by alsa0.perex.cz (Postfix, from userid 1000) id 34FF8266EE0; Tue, 30 Aug 2016 23:25:27 +0200 (CEST) Received: from muru.com (muru.com [72.249.23.125]) by alsa0.perex.cz (Postfix) with ESMTP id EFE202673E8 for ; Tue, 30 Aug 2016 20:38:39 +0200 (CEST) Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 66769813A; Tue, 30 Aug 2016 18:39:24 +0000 (UTC) Date: Tue, 30 Aug 2016 11:38:35 -0700 From: Tony Lindgren To: kbuild test robot Message-ID: <20160830183834.6u73eb5ujgm5emlk@atomide.com> References: <20160829182746.53vrgbxadqodzzug@atomide.com> <201608300739.JrTVlIyd%fengguang.wu@intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <201608300739.JrTVlIyd%fengguang.wu@intel.com> User-Agent: Mutt/1.6.2-neo (2016-07-23) Cc: alsa-devel@alsa-project.org, Ivaylo Dimitrov , Aaro Koskinen , Liam Girdwood , Sebastian Reichel , Peter Ujfalusi , Mark Brown , Jarkko Nikula , kbuild-all@01.org, Pavel Machel , Pali =?utf-8?B?Um9ow6Fy?= , linux-omap@vger.kernel.org Subject: Re: [alsa-devel] [PATCH] ASoC: omap-mcbsp: Add PM QoS support for McBSP to prevent glitches X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org X-Virus-Scanned: ClamAV using ClamSMTP * kbuild test robot [160829 21:31]: > vim +/dev_pm_qos_request_active +1108 sound/soc/omap/mcbsp.c > > 1102 clk_put(mcbsp->fclk); > 1103 return ret; > 1104 } > 1105 > 1106 void omap_mcbsp_cleanup(struct omap_mcbsp *mcbsp) > 1107 { > > 1108 if (dev_pm_qos_request_active(&mcbsp->pm_qos_req)) > 1109 pm_qos_remove_request(&mcbsp->pm_qos_req); > 1110 > 1111 if (mcbsp->pdata->buffer_size) Oops sorry about that. Looks like I completely missed that warning after a copy-paste adding of the check to omap_mcbsp_cleanup. Looks like implementing dev_pm_qos needs more work as it needs the set_latency_tolerance implemented somewhere. Updated patch below, Mark let me know if you need an incremental patch to fix the warning. Regards, Tony 8< ----------------- From: Tony Lindgren Date: Mon, 29 Aug 2016 11:27:46 -0700 Subject: [PATCHv2] ASoC: omap-mcbsp: Add PM QoS support for McBSP to prevent glitches We can get audio errors if hitting deeper idle states on omaps: [alsa.c:230] error: Fatal problem with alsa output, error -5. [audio.c:614] error: Error in writing audio (Input/output error?)! This seems to happen with off mode idle enabled as power for the whole SoC may get cut off between filling the McBSP fifo using DMA. While active DMA blocks deeper idle states in hardware, McBSP activity does not seem to do so. Let's fix the issue by adding PM QoS latency requirement for McBSP. Based on my experiments a working latency value is somewhere around 35 ms with 40 ms breaking. So let's use a safer value of 30 ms. Note that this is different from snd_pcm_hw_constraint_step() as that can configure things based on the buffer and period size. However, that does not help to block idle between the fifo fills. Note that in theory some omaps are capable of playing audio while hitting deeper idle states. If somebody needs that and gets it working, we can set the latency requirements accordingly. Signed-off-by: Tony Lindgren diff --git a/sound/soc/omap/mcbsp.c b/sound/soc/omap/mcbsp.c --- a/sound/soc/omap/mcbsp.c +++ b/sound/soc/omap/mcbsp.c @@ -25,6 +25,7 @@ #include #include #include +#include #include @@ -643,6 +644,10 @@ void omap_mcbsp_start(struct omap_mcbsp *mcbsp, int tx, int rx) int enable_srg = 0; u16 w; + /* Prevent omap hardware from hitting off between fifo fills */ + pm_qos_add_request(&mcbsp->pm_qos_req, PM_QOS_CPU_DMA_LATENCY, + 30 * 1000); + if (mcbsp->st_data) omap_st_start(mcbsp); @@ -731,6 +736,8 @@ void omap_mcbsp_stop(struct omap_mcbsp *mcbsp, int tx, int rx) if (mcbsp->st_data) omap_st_stop(mcbsp); + + pm_qos_remove_request(&mcbsp->pm_qos_req); } int omap2_mcbsp_set_clks_src(struct omap_mcbsp *mcbsp, u8 fck_src_id) @@ -1098,6 +1105,9 @@ err_thres: void omap_mcbsp_cleanup(struct omap_mcbsp *mcbsp) { + if (pm_qos_request_active(&mcbsp->pm_qos_req)) + pm_qos_remove_request(&mcbsp->pm_qos_req); + if (mcbsp->pdata->buffer_size) sysfs_remove_group(&mcbsp->dev->kobj, &additional_attr_group); diff --git a/sound/soc/omap/mcbsp.h b/sound/soc/omap/mcbsp.h --- a/sound/soc/omap/mcbsp.h +++ b/sound/soc/omap/mcbsp.h @@ -325,6 +325,8 @@ struct omap_mcbsp { unsigned int in_freq; int clk_div; int wlen; + + struct pm_qos_request pm_qos_req; }; void omap_mcbsp_config(struct omap_mcbsp *mcbsp,