From patchwork Fri Dec 16 00:03:51 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sarangdhar Joshi X-Patchwork-Id: 9477117 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 0DA6360827 for ; Fri, 16 Dec 2016 00:05:36 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id F27102889E for ; Fri, 16 Dec 2016 00:05:35 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id E5E89288A3; Fri, 16 Dec 2016 00:05:35 +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=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 5A5F82889E for ; Fri, 16 Dec 2016 00:05:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757258AbcLPAD7 (ORCPT ); Thu, 15 Dec 2016 19:03:59 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:48602 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757253AbcLPAD6 (ORCPT ); Thu, 15 Dec 2016 19:03:58 -0500 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 0BD63615C2; Fri, 16 Dec 2016 00:03:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1481846637; bh=oxgylKAfM9QhgprlnPN8xxAWxXdHLA+HaMKR6mNgYkU=; h=From:To:Cc:Subject:Date:From; b=eDR1o+rEPuniCUuD2zN5p1VDjaNJGhODTtYqBYIkjkcRBL5kNB7G8dOYoyIermPS+ Y/BabzcUBKVelnrNoTNRkG4qWHRJFYbyaTVXVOCK1IW1qu//87lh1xh3O7xB6S1Ne5 HGERLVeJTUW+tHLqJV56ijE3rudPxo2VifyiAAHA= Received: from spjoshi-linux.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: spjoshi@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 5D016600EC; Fri, 16 Dec 2016 00:03:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1481846635; bh=oxgylKAfM9QhgprlnPN8xxAWxXdHLA+HaMKR6mNgYkU=; h=From:To:Cc:Subject:Date:From; b=chtQQKLW3/x/arbbaVqJg/zdtUDIM4GoHItSEqxjn0eX8/ZFYHZ3XffnsQX5jm/HJ N7DPMqF6DrDR3ESM5I/djYO1b+JBWWw685EZJK0n30sxTEtMsQ2iUGQNlafpOHPRwI qgqrCmn3HzQ5B3kQdz/07DWvxQW9DidBi3nmvrLs= DMARC-Filter: OpenDMARC Filter v1.3.1 smtp.codeaurora.org 5D016600EC Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=pass smtp.mailfrom=spjoshi@codeaurora.org From: Sarangdhar Joshi To: Ohad Ben-Cohen , Bjorn Andersson , Santosh Shilimkar Cc: Sarangdhar Joshi , linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, Stephen Boyd , Trilok Soni , Dave Gerlach , Suman Anna Subject: [PATCH 1/2] soc: ti: Use remoteproc auto_boot feature Date: Thu, 15 Dec 2016 16:03:51 -0800 Message-Id: <1481846632-4778-1-git-send-email-spjoshi@codeaurora.org> X-Mailer: git-send-email 1.9.1 Sender: linux-remoteproc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-remoteproc@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP The function wkup_m3_rproc_boot_thread waits for asynchronous firmware loading to complete successfully before calling rproc_boot(). The same can be achieved by just setting rproc->auto_boot flag. Change this. As a result this change removes wkup_m3_rproc_boot_thread and moves m3_ipc->sync_complete initialization to the wkup_m3_ipc_probe(). Other than the current usage, the firmware_loading_complete is only used in rproc_del() where it's no longer needed. This change is in preparation for removing firmware_loading_complete completely. CC: Dave Gerlach CC: Suman Anna CC: Bjorn Andersson Signed-off-by: Sarangdhar Joshi --- Hi Suman, Unfortunately, I don't have a TI device and couldn't test this change. Is it possible for you to test this change on TI device? Thanks in advance. Regards, Sarang drivers/remoteproc/wkup_m3_rproc.c | 2 +- drivers/soc/ti/wkup_m3_ipc.c | 35 ++--------------------------------- 2 files changed, 3 insertions(+), 34 deletions(-) diff --git a/drivers/remoteproc/wkup_m3_rproc.c b/drivers/remoteproc/wkup_m3_rproc.c index 18175d0..79ea022 100644 --- a/drivers/remoteproc/wkup_m3_rproc.c +++ b/drivers/remoteproc/wkup_m3_rproc.c @@ -167,7 +167,7 @@ static int wkup_m3_rproc_probe(struct platform_device *pdev) goto err; } - rproc->auto_boot = false; + rproc->auto_boot = true; wkupm3 = rproc->priv; wkupm3->rproc = rproc; diff --git a/drivers/soc/ti/wkup_m3_ipc.c b/drivers/soc/ti/wkup_m3_ipc.c index 8823cc8..31090d70 100644 --- a/drivers/soc/ti/wkup_m3_ipc.c +++ b/drivers/soc/ti/wkup_m3_ipc.c @@ -17,7 +17,6 @@ #include #include -#include #include #include #include @@ -365,22 +364,6 @@ void wkup_m3_ipc_put(struct wkup_m3_ipc *m3_ipc) } EXPORT_SYMBOL_GPL(wkup_m3_ipc_put); -static void wkup_m3_rproc_boot_thread(struct wkup_m3_ipc *m3_ipc) -{ - struct device *dev = m3_ipc->dev; - int ret; - - wait_for_completion(&m3_ipc->rproc->firmware_loading_complete); - - init_completion(&m3_ipc->sync_complete); - - ret = rproc_boot(m3_ipc->rproc); - if (ret) - dev_err(dev, "rproc_boot failed\n"); - - do_exit(0); -} - static int wkup_m3_ipc_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev; @@ -388,7 +371,6 @@ static int wkup_m3_ipc_probe(struct platform_device *pdev) phandle rproc_phandle; struct rproc *m3_rproc; struct resource *res; - struct task_struct *task; struct wkup_m3_ipc *m3_ipc; m3_ipc = devm_kzalloc(dev, sizeof(*m3_ipc), GFP_KERNEL); @@ -402,6 +384,8 @@ static int wkup_m3_ipc_probe(struct platform_device *pdev) return PTR_ERR(m3_ipc->ipc_mem_base); } + init_completion(&m3_ipc->sync_complete); + irq = platform_get_irq(pdev, 0); if (!irq) { dev_err(&pdev->dev, "no irq resource\n"); @@ -449,25 +433,10 @@ static int wkup_m3_ipc_probe(struct platform_device *pdev) m3_ipc->ops = &ipc_ops; - /* - * Wait for firmware loading completion in a thread so we - * can boot the wkup_m3 as soon as it's ready without holding - * up kernel boot - */ - task = kthread_run((void *)wkup_m3_rproc_boot_thread, m3_ipc, - "wkup_m3_rproc_loader"); - - if (IS_ERR(task)) { - dev_err(dev, "can't create rproc_boot thread\n"); - goto err_put_rproc; - } - m3_ipc_state = m3_ipc; return 0; -err_put_rproc: - rproc_put(m3_rproc); err_free_mbox: mbox_free_channel(m3_ipc->mbox); return ret;