From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S969226AbdAITkD (ORCPT ); Mon, 9 Jan 2017 14:40:03 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:50178 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966206AbdAITj6 (ORCPT ); Mon, 9 Jan 2017 14:39:58 -0500 DMARC-Filter: OpenDMARC Filter v1.3.1 smtp.codeaurora.org EF195613A9 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 Subject: Re: [PATCH v3] soc: ti: wkup_m3_ipc: Drop wait from wkup_m3_rproc_boot_thread To: Tony Lindgren References: <1483653615-24378-1-git-send-email-spjoshi@codeaurora.org> <20170109193328.GO2630@atomide.com> Cc: Ohad Ben-Cohen , Bjorn Andersson , Santosh Shilimkar , Trilok Soni , linux-arm-msm@vger.kernel.org, Dave Gerlach , Stephen Boyd , linux-kernel@vger.kernel.org, linux-remoteproc@vger.kernel.org, linux-arm-kernel@lists.infradead.org From: Sarangdhar Joshi Message-ID: <10f36506-e148-2e00-4b83-dfe46463ce5b@codeaurora.org> Date: Mon, 9 Jan 2017 11:39:52 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <20170109193328.GO2630@atomide.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/09/2017 11:33 AM, Tony Lindgren wrote: > * Sarangdhar Joshi [170105 14:01]: >> The function wkup_m3_rproc_boot_thread waits for asynchronous >> firmware loading to parse the resource table before calling >> rproc_boot(). However, as the resource table parsing has been >> moved to rproc_boot(), there's no need to wait for the >> asynchronous firmware loading completion. So, drop this. >> >> CC: Dave Gerlach >> CC: Bjorn Andersson >> Tested-by: Suman Anna >> Signed-off-by: Sarangdhar Joshi >> --- >> >> This patch seems to be doing an independent clean up now. Hence >> removing it from the series. >> >> Changes from v2: >> >> * Updated the subject line as per Suman's comment > > FYI, I already have v2 applied with subject: > > "soc: ti: wkup_m3_ipc: Drop wait from wkup_m3_rproc_boot_thread" Oh great. Thanks a lot Tony. Regards, Sarang > > Tony > > >> drivers/soc/ti/wkup_m3_ipc.c | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/drivers/soc/ti/wkup_m3_ipc.c b/drivers/soc/ti/wkup_m3_ipc.c >> index 8823cc8..8bfa44b 100644 >> --- a/drivers/soc/ti/wkup_m3_ipc.c >> +++ b/drivers/soc/ti/wkup_m3_ipc.c >> @@ -370,8 +370,6 @@ 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); >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >> a Linux Foundation Collaborative Project >> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >> -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project