From patchwork Fri Dec 18 17:32:14 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mathieu Poirier X-Patchwork-Id: 345482 Delivered-To: patch@linaro.org Received: by 2002:a02:85a7:0:0:0:0:0 with SMTP id d36csp2335829jai; Fri, 18 Dec 2020 09:35:28 -0800 (PST) X-Google-Smtp-Source: ABdhPJzXhvHIR0fzC/LMv1Xxz8e6O9D1R0i2bsOmJQedOhCoNrFXM5TW9DItl/rP5yBRfgABnLUv X-Received: by 2002:aa7:cb10:: with SMTP id s16mr5485649edt.304.1608312928476; Fri, 18 Dec 2020 09:35:28 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1608312928; cv=none; d=google.com; s=arc-20160816; b=qeCLS0a6xQRhGrNr4m1Cw1zF9bAEzLwCKznZE+nw8taYv6HtQsmLvFUOOpV4NrTCCF l5OuC9xJGqb05hXSXTLXHHqBtbYfQjj+6LiPeRkbuOL3vfZsDolPUv/uXB6QnsRdriRA HVBaOYlAV35ppw0P1FZFGBr/gYG0l+eqrM8T8kIGoabowXlS1uTmSDlSAmyF2+dUdMIp Aj1QEohyoGNkkPMh2R1XbdrhWGUIXro5lrCwpCVWzNnBIet5NYUcRV7t1FCMWF1ml9x9 fUWpybSuzI6qtS5xR/nPU/AAig0gk5nPqFnmHicIC8/0GZ7bccbopcl2uq+IeHItz4iN DTCg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=Dhrgj6KlTsZx4Q0NhpQquJvgjz/fkM6eRvngZm9P8MI=; b=INaILiDWXyUL7kzPGy4o34AtsYrurxrOlZ/04JZAGYX7IjOzeUJGsOyV5XKhJxpa4O d/VP+qLagMb6Jo12qBBmcljeyioxHur1mpw9c371owD3xgMb2VqFXqOHB6fbx2kW/Z4U hqFA0bwVCRgd+tIix/xyK9mos1NytagfonjrSi7rNjtp2kpjFHXcAOsY0pVRb4OJ1+Yf uoYoHwDMfF7iGGt2B22LTLZa32tDFvayDeaaBUutaoN/qNoMzQ8TFuROmq7O0oRZOo0/ HFXK0NTypNc2alyuXGerTm0QzB5Iy0loSulGGnFZkSYKNpv52OFVACcBN9et+anROyIt ZdJQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=CCTWyUnz; spf=pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id t20si4902939ejd.719.2020.12.18.09.35.28; Fri, 18 Dec 2020 09:35:28 -0800 (PST) Received-SPF: pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=CCTWyUnz; spf=pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732497AbgLRRdw (ORCPT + 7 others); Fri, 18 Dec 2020 12:33:52 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60082 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732414AbgLRRdv (ORCPT ); Fri, 18 Dec 2020 12:33:51 -0500 Received: from mail-pj1-x102c.google.com (mail-pj1-x102c.google.com [IPv6:2607:f8b0:4864:20::102c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2B5B9C0611CC for ; Fri, 18 Dec 2020 09:32:34 -0800 (PST) Received: by mail-pj1-x102c.google.com with SMTP id iq13so1666143pjb.3 for ; Fri, 18 Dec 2020 09:32:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=Dhrgj6KlTsZx4Q0NhpQquJvgjz/fkM6eRvngZm9P8MI=; b=CCTWyUnzE1ZanX60E/iLMHT0I8ZKWfPXwPWUJWfADUGe1anX5MYdsbtqq9PzKYM9/j n7OFo3P1s+0nskqFjWV2eOnXUxh+q3oIlb7YjcMN9vV5TS0FghgN1ctFHNf7Rs5OaNz0 a8foVoXrny67rmJYDx25RebAwa6Lrn/essn4udioOyBENIujDvSTMTEVSTk8Qvua6HIX F6eS1tPhvPc0tLr2Gt4TMUD4Gxhq6n9EZAt+jGOgvH+PUKd8fKp4GcO0SWVYVXMomcsO 94YO06VeiG0HGk4RyldO1QnAYqBw1oQ3vmmw06P0vDdLEFM5eh910MItWLsNhZOUaYT8 Lc9g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=Dhrgj6KlTsZx4Q0NhpQquJvgjz/fkM6eRvngZm9P8MI=; b=cYtxJBJlVKKYvM2WdeEtOjOpYKbAr2J+M4fQQo6muAJCTuT9QCzaZzbMKVSYojHNAa U2X4H9PEKFzn33cTSRut7t/XHmRbOdkIDIEj3CmmBfvzVEj0eWMzxNyz8KBT+GRwilIM tDsnC5NeEeeOCI/uiYv3QqNjrIGoc022mjPNb/OkkPSnSwWRJB4mhX+7yTRKeH8PJhGg W59u7n+Q7kB/94W5z9h0J2CNitbYj9/GW3ni5KWtrDfIRGj1gbpIW0X13HXz+eMK321Q GUqPPUqg2TtdwV1aRz4JQWqHJPYKqdV+zUjDMvBCrF3bdMP7LZ+1sHNITjM0BfdNvPdk H6WA== X-Gm-Message-State: AOAM530CElmV8yB/YizQwNMxu4aLIQm2FYdncq7D0Qq6C8+Ia7PKygf6 4+vQdv3b/ntEitooeKLBpe90tg== X-Received: by 2002:a17:902:e9c5:b029:db:d1ae:46ba with SMTP id 5-20020a170902e9c5b02900dbd1ae46bamr5424989plk.38.1608312753708; Fri, 18 Dec 2020 09:32:33 -0800 (PST) Received: from xps15.cg.shawcable.net (S0106889e681aac74.cg.shawcable.net. [68.147.0.187]) by smtp.gmail.com with ESMTPSA id l11sm9892957pgt.79.2020.12.18.09.32.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 18 Dec 2020 09:32:33 -0800 (PST) From: Mathieu Poirier To: ohad@wizery.com, bjorn.andersson@linaro.org, robh+dt@kernel.org Cc: arnaud.pouliquen@st.com, linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v4 03/17] remoteproc: Remove useless check in rproc_del() Date: Fri, 18 Dec 2020 10:32:14 -0700 Message-Id: <20201218173228.2277032-4-mathieu.poirier@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20201218173228.2277032-1-mathieu.poirier@linaro.org> References: <20201218173228.2277032-1-mathieu.poirier@linaro.org> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Whether started at probe() time or thereafter from the command line, a remote processor needs to be shutdown before the final cleanup phases can happen. Otherwise the system may be left in an unpredictable state where the remote processor is expecting the remoteproc core to be providing services when in fact it no longer exist. Invariably calling rproc_shutdown() is fine since it will return immediately if the remote processor has already been switched off. Signed-off-by: Mathieu Poirier Reviewed-by: Peng Fan Reviewed-by: Arnaud Pouliquen --- drivers/remoteproc/remoteproc_core.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) -- 2.25.1 diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c index e55568d1e7e2..f36786b47a4f 100644 --- a/drivers/remoteproc/remoteproc_core.c +++ b/drivers/remoteproc/remoteproc_core.c @@ -2283,10 +2283,8 @@ int rproc_del(struct rproc *rproc) if (!rproc) return -EINVAL; - /* if rproc is marked always-on, rproc_add() booted it */ /* TODO: make sure this works with rproc->power > 1 */ - if (rproc->auto_boot) - rproc_shutdown(rproc); + rproc_shutdown(rproc); mutex_lock(&rproc->lock); rproc->state = RPROC_DELETED;