Message ID | 1506938433-681-1-git-send-email-ed.blake@sondrel.com (mailing list archive) |
---|---|
State | Accepted |
Commit | c70458890ff15d858bd347fa9f563818bcd6e457 |
Headers | show
Return-Path: <alsa-devel-bounces@alsa-project.org> 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 3B35560365 for <patchwork-alsa-devel@patchwork.kernel.org>; Mon, 2 Oct 2017 10:04:54 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 2D0442881B for <patchwork-alsa-devel@patchwork.kernel.org>; Mon, 2 Oct 2017 10:04:54 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 2200928947; Mon, 2 Oct 2017 10:04:54 +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.8 required=2.0 tests=BAD_ENC_HEADER,BAYES_00, DKIM_SIGNED, RCVD_IN_DNSWL_NONE, T_DKIM_INVALID autolearn=no 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 6AB492881B for <patchwork-alsa-devel@patchwork.kernel.org>; Mon, 2 Oct 2017 10:04:53 +0000 (UTC) Received: from alsa0.perex.cz (localhost [127.0.0.1]) by alsa0.perex.cz (Postfix) with ESMTP id F19BB267214; Mon, 2 Oct 2017 12:01:48 +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 D59B22671A8; Mon, 2 Oct 2017 12:00:57 +0200 (CEST) Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-db5eur01on0043.outbound.protection.outlook.com [104.47.2.43]) by alsa0.perex.cz (Postfix) with ESMTP id 2B90326719A for <alsa-devel@alsa-project.org>; Mon, 2 Oct 2017 12:00:54 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sondrel.onmicrosoft.com; s=selector1-sondrel-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=ybn83eOtA/66oq+JGH6clNpSEmJMrmfMWldZyK1TRp4=; b=Rrp8pJEQ6foe8L3KRncBhmTC9zabStjBLYmgE+8z8aBVUAzM+PutqC0RrQPbuu+ugdGTU7vJ87DcLtJhy7UVLs6Nr6imBh92b+ENkGNUiyCIk0eIQeWqEsPy4W5FrcwrXcgck5h+wGpk+X+SrDv61GWG/BuuM94ZnwB3T4nV9ac= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=ed.blake@sondrel.com; Received: from iw-build-2.sondrel.com (195.88.9.101) by VI1P191MB0015.EURP191.PROD.OUTLOOK.COM (2603:10a6:800:a4::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Mon, 2 Oct 2017 10:00:53 +0000 From: Ed Blake <ed.blake@sondrel.com> To: lgirdwood@gmail.com, broonie@kernel.org Date: Mon, 2 Oct 2017 11:00:33 +0100 Message-Id: <1506938433-681-1-git-send-email-ed.blake@sondrel.com> X-Mailer: git-send-email 1.9.1 MIME-Version: 1.0 X-Originating-IP: [195.88.9.101] X-ClientProxiedBy: HE1P190CA0043.EURP190.PROD.OUTLOOK.COM (2603:10a6:7:52::32) To VI1P191MB0015.EURP191.PROD.OUTLOOK.COM (2603:10a6:800:a4::14) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 7888a6b1-887c-41d7-26e5-08d5097c77ef X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:VI1P191MB0015; X-Microsoft-Exchange-Diagnostics: 1; VI1P191MB0015; 3:lRNPjhwjypVT17+7ykMvxP4evUH1pvqoe5v5PpzCAwneQtkNFJT3+1vk4hZKQUrEkzu+GLOrlw5/bZ86shnuSUH2T9g3McxAbnAy3OnSCB2eZyMGzUPOoG+lIVmCQuZq4sCJdzOXwPUGM3qfB7cj8tya9VfgJzwtNrBRVnvRUip0pSVByC6Yz04dA1t0VW9pHW1uNY5ZzZXFhtcEaVc47OKRE3mQIaSfjcTYX6qm89WQ/j/vz/AAVR/B6CzwUu0a; 25:KQZglUlonMGvsWj4VwRppRklx0yNLajaiZ05OMRbGP19PowwN5+WqVmt1jAG18LIUKgsOncqaMkGwyC0K2jRf6rt741YwtAoxRKeJ6tGx0ZB1jfnZCzy7FYfiF6Mmc4j7/sOSFukSH8J5CQ+KFdiT2OFf262369q6A/9J2U7NLXYD52JVvIuKdRJNKTTxFzxoT71zkGptdcDUUiPOfq02fLC1BB/9ibN7+m36XeTH2r9dIhyVYovxYCEbLqey2ods9pQdc636HgqnbOyTOsZd/yhXvQuxrICK2EvIk2XlOj2J2/Sbadpaskynq60gVD4jWYrj6VYO+mkVcdRiQKifg==; 31:rnk6utgBlSmBEU+4vhLTmO14aHWoKhGBj05ITkLggcY5y40aAM6DIMOQY7UBpZMljSRXTyBPleDcvTCxMzPZCtcXCxzwpAT0bOLlsAxQXYmpG+Ikud4Uof6xFfLSQhgh5KyoPb3+rtS1XJ0oQ4wxelwYneXoHIBa4lX0r1DyVHZHyvdOg4KX7+XfA64NiMyYX5a0XGKW1aRGlgA4g0yepAM7k+gtNzNnJRBt1BrayEk= X-MS-TrafficTypeDiagnostic: VI1P191MB0015: X-Microsoft-Exchange-Diagnostics: 1; VI1P191MB0015; 20:Mwb72DEbwBchYB3rIqM+dma+a9Fy3iIX6FhMqvs+lpizTb6MfgTJAjl5fv3a7AW7onTc1tEnHp2b3RKklCeIhMCgQULxGvngrswCc8wAZ+L2wkzE+mTDSdPosnvP16QhbEDI6uSYIyNiTeUJse+U4ZDSE+3NyAQPAMH7X9rdwpbxHHT+cVmdfGPtJnvAy9EZtHgI2dLwWuipuh4+TFHXlXs3GL9ctpG4CIM11tfQR6CTxe0hE6xIl4JrfdXesJ3t; 4:c37sXhHDyWhdqocNkeCycGo5DuxZKz1v6ZzBYXLf9wfjXgu1qtpXDlyVojfKcuTWM8Mjw9hAAsjDVkNu/ggR090aDGOxATXJdyD9KztcaJZQF39GBXtas3xvXL0eHgI/WbmhdVc1KPakucVKnfszxhM0a7MbE+RS83DLYXhBqoc3bRIyvY3+skDQdWOEI41r49fapZAFyWi0nsEe4PvnId6GAcR4iwsu9LB8vVreYx4KOROAi9CvrhJoDze5SFAD X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: <VI1P191MB00153E87F329835E077AE0F3967D0@VI1P191MB0015.EURP191.PROD.OUTLOOK.COM> X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(3002001)(10201501046)(100000703101)(100105400095)(93006095)(93001095)(6041248)(20161123555025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123560025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:VI1P191MB0015; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:VI1P191MB0015; X-Forefront-PRVS: 0448A97BF2 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(346002)(39830400002)(376002)(189002)(199003)(230783001)(50466002)(39060400002)(16526017)(48376002)(575784001)(47776003)(50226002)(16586007)(6116002)(4326008)(97736004)(86362001)(316002)(5003940100001)(33646002)(3846002)(68736007)(101416001)(305945005)(25786009)(478600001)(105586002)(106356001)(107886003)(2906002)(50986999)(5660300001)(66066001)(8936002)(36756003)(81166006)(81156014)(8676002)(53416004)(53936002)(6486002)(7736002)(6666003)(69596002)(189998001); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1P191MB0015; H:iw-build-2.sondrel.com; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: None (protection.outlook.com: sondrel.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; VI1P191MB0015; 23:msz73KrJldwg8n5SuyCQQOOZKsqgdts0kcqgl233E?= =?us-ascii?Q?u3q7h0XpYYeZr2zr8XTGQNzDlayWFLBpx4uYPMMwQsr6Cs0Jj89+EMec5pY7?= =?us-ascii?Q?mvD39PJ/ylLIMtSQUOGliPo+IcSb9mMin4oVoFehitBQL1IvLwOMPyB3PAJp?= =?us-ascii?Q?7YenxuhqtKPdzmvE5tc8cGYFh6KvEUH5uPNadSXUP0s+Ly5jCtwW58l/kJvS?= =?us-ascii?Q?eXbCBhXSRiY1awRSMvulOncJVD5vRnKa1UlFgPNnJCoNHn287goLfRYxeWWp?= =?us-ascii?Q?BVs+jmbWgQZjFMQR9UnX0dsBEvozQv8hBF/0bi8PZ0IAomUn69BhxGbAUSEw?= =?us-ascii?Q?2GO0EYLVTBjsWlBDgxskDJErh6zF7tPOuyFs10zxPwI3L+xpHV/8Qxe6XZ/x?= =?us-ascii?Q?jutVz6ck4BBqd006bEEr5SUJPNEfq3ScTKv+CMs22AhPrGpjkTKG9nkzhh1w?= =?us-ascii?Q?5XYSkrULo+upGW00rLKZDpKU50v/gfOLsn6sngNKhplFGvxSlvAqxOLdBvc6?= =?us-ascii?Q?I0IRT6zpB5i9GIemycrlAj5/0tsICkZh5oO4YpzJ3L1A2jEvsH7Ex96TnOTG?= =?us-ascii?Q?0FF9EpN046zzaVn4A5qPZdrZb18rtF8ig057drasTmXRWRO3xN/Lmcwjjya0?= =?us-ascii?Q?tnvb/smwRwHu2FkbbPTXgETRISYPpriYQ3SzH2hckCPrfIjUl4nLXwz29bVm?= =?us-ascii?Q?VUd8zDLTWc5H3NEwooVDARfoxQDXC9TrqVReYFXIPJl5Awe++nlRijnxII1M?= =?us-ascii?Q?JZifzWl24edItN2wnCAQdKYkXKP/hjETjLZh9I3jNR8TXeuncoN0I9Dyj9Yx?= =?us-ascii?Q?3UNPquQQe4BJ56LvAS6GsvBvKqvscQqaBBT0LChx3jTM+1WmKiW/+XwSUaSr?= =?us-ascii?Q?FS2pR0kR/szgy9lMZhx9BRiVMLDoSNWkv5MHS343LNym3db6d7rGnex/nLcO?= =?us-ascii?Q?sBj4Blw/gPs8Y95ry7U7jorm/fAqAGkCqnVdeuNCcvlkAF8AWtRp1rrUN0c9?= =?us-ascii?Q?u7Sloxb3bsKLBQg+jGdOzOqLO9Jr5LRX5XCbL92nBw6tST+FDqtYbbqz9ysz?= =?us-ascii?Q?p1cXnBexPlHHKnkoguptu/FE10OJ8tSBHqCOAGxdBMOYhmQHwe1aK2bFLjLb?= =?us-ascii?Q?BAjbg3TP0w=3D?= X-Microsoft-Exchange-Diagnostics: 1; VI1P191MB0015; 6:fTEJ1q3QGXSAdZNzSIyC5FScr0HHaS+tFiLrlq22uUdYVjCibWVCwYAIJA5KR2D2KGa7s3W7F12iXKu8O1JUUxzqY8IuVSMfxdTxxVcWMHtnrI8Ac35TOUM+rP0VCjIU7RGxY+CeriSfS15mmTGV8xUyaNvNZSEM0FVoedC3zsgJYQWqFJxnkGrCORB38KB3iS56T79yK3lY7+24qOqgUvCAu5fBURGEdDfOjQZl1Z069E4/moBDI3kNZEUsEyeXTJuuG3xYDDHdUjoaZjMmlEdVAh6XEf8XUFQToQgHRUfwH88ZIJLio8ugMDYqCYHnxh0e2vpzUqW3cPgtLUxKqQ==; 5:VBeHRmLoZiE04L7MWFaHNtGGDGIT56GlDw3g6um+6U7uxFhSKuXEkB+GK7VR/1R4BxEme2YNtnmhL8B4Jp0BAc+ltMmf1KcmnaAOQ4Dc6ks8GBLcbJEm/CScyr1MpUWa/YJ1AKZxJ+PBqsI1+JDOcQ==; 24:vzKcRpYwB8/Ds/bYCAChY4vTg48oJ6lucS1ST58saSVe66npEr6zz61qxo7VQB0kd+yvXgxemYCud1rVbqy5d+nNv3PrI8zHPv6xFkJAXXo=; 7:KHpePWdhQfGjR+eUB+1rutHa/O5FhxjYl4AfGYY35awOUZchIXzEBZ942sBwKx/GEGK4fqL1lMJPKO0M+cUnVpec7AEJWRlj6Pbe0AUNYRecty8LDLnhKFEA2+a11yb+jX4JBV1jaulp+VhS13MFsIopbOMFy+UXOI59wQFHU6pSH1TAuHwLE6RtNrOk0eQujb1HhbxEmrQYdOkH0EdxehOxco7lXNDLp+8llSvWoUk= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: sondrel.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Oct 2017 10:00:53.0237 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 4faa3872-698e-4896-80ec-148b916cb1ba X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1P191MB0015 X-Mailman-Approved-At: Mon, 02 Oct 2017 12:01:38 +0200 Cc: alsa-devel@alsa-project.org, Ed Blake <ed.blake@sondrel.com> Subject: [alsa-devel] [PATCH] ASoC: img-parallel-out: Add pm_runtime_get/put to set_fmt callback 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" <alsa-devel.alsa-project.org> List-Unsubscribe: <http://mailman.alsa-project.org/mailman/options/alsa-devel>, <mailto:alsa-devel-request@alsa-project.org?subject=unsubscribe> List-Archive: <http://mailman.alsa-project.org/pipermail/alsa-devel/> List-Post: <mailto:alsa-devel@alsa-project.org> List-Help: <mailto:alsa-devel-request@alsa-project.org?subject=help> List-Subscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>, <mailto:alsa-devel-request@alsa-project.org?subject=subscribe> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org X-Virus-Scanned: ClamAV using ClamSMTP |
On Mon, Oct 02, 2017 at 11:00:33AM +0100, Ed Blake wrote:
> + pm_runtime_get_sync(prl->dev);
I'll apply this since the driver already has runtime PM and it makes
things better but please in a followup patch add error handling.
diff --git a/sound/soc/img/img-parallel-out.c b/sound/soc/img/img-parallel-out.c index 23b0f0f..2fc8a63 100644 --- a/sound/soc/img/img-parallel-out.c +++ b/sound/soc/img/img-parallel-out.c @@ -164,9 +164,11 @@ static int img_prl_out_set_fmt(struct snd_soc_dai *dai, unsigned int fmt) return -EINVAL; } + pm_runtime_get_sync(prl->dev); reg = img_prl_out_readl(prl, IMG_PRL_OUT_CTL); reg = (reg & ~IMG_PRL_OUT_CTL_EDGE_MASK) | control_set; img_prl_out_writel(prl, reg, IMG_PRL_OUT_CTL); + pm_runtime_put(prl->dev); return 0; }
Add pm_runtime_get_sync and pm_runtime_put calls to set_fmt callback function. This fixes a bus error during boot when CONFIG_SUSPEND is defined when this function gets called while the device is runtime disabled and device registers are accessed while the clock is disabled. Signed-off-by: Ed Blake <ed.blake@sondrel.com> --- sound/soc/img/img-parallel-out.c | 2 ++ 1 file changed, 2 insertions(+)