From patchwork Mon Jun 18 17:22:29 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Doug Anderson X-Patchwork-Id: 10472383 X-Patchwork-Delegate: agross@codeaurora.org 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 B9F24601D7 for ; Mon, 18 Jun 2018 17:23:11 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 9FA9C28AFD for ; Mon, 18 Jun 2018 17:23:11 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 907AB28B18; Mon, 18 Jun 2018 17:23:11 +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=-7.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI, T_DKIM_INVALID autolearn=ham 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 2C56A28B05 for ; Mon, 18 Jun 2018 17:23:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935230AbeFRRXJ (ORCPT ); Mon, 18 Jun 2018 13:23:09 -0400 Received: from mail-pg0-f67.google.com ([74.125.83.67]:42790 "EHLO mail-pg0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935072AbeFRRXI (ORCPT ); Mon, 18 Jun 2018 13:23:08 -0400 Received: by mail-pg0-f67.google.com with SMTP id c10-v6so7837313pgu.9 for ; Mon, 18 Jun 2018 10:23:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id; bh=zYKY/M5v9IljymNM1No/utpSz2vumnyX/pJFyKFgmOY=; b=RYNou3E2SigxWaH8gsAmVqn5/wmx8GABr8L5JBh1nAoPVI5xvjZowkYu/dshn1Qe1c H18844maG45H+ybKfk9BKHl2W7cjfX6Hhk/KwVn8dyciky0KCd/ixN5gHe6KJaBa6L4m 3x38vS5cwQF1zdtfHFuWKvNCPArX0XuMaMrDU= 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; bh=zYKY/M5v9IljymNM1No/utpSz2vumnyX/pJFyKFgmOY=; b=Uk+VBnuB6Z4u+OhEvvuZY97cRC9GJcA6GGd4FLbD6jUaEN+UjRuxBTLkECM//YPpJc 0DiIz2sDOWKzcCS1w8qTcyHA8F9CDIshwox8ZOfovXqk5lUTwkpEELRIvzPxGBG+UAeR Ijh/DFrKpgOqfibS7pVIaNghgVx7ePKhg3zBTi1aHo1/J4Dl/hqFzEtjGFH+/T8KGmYH rSaLjENiystqf3Y4rC1pQqAcoEoD227xWgzcW1XYIVa6YoL8duLKdlybSPmZ3na0tFw2 J/bcqVTskjipz7H2jIfRMLI3hv5kKysflt297Zm1KYF9xAARNwFDkYLOSeVWIBLX1YL+ 6Xpg== X-Gm-Message-State: APt69E3TgQrS+VLvIbqB6nrazkfN2mQufbj+V7QP+Zl9wYYHVkb9rtxN KC0CXybaB84Ax2gP1bXPWEM2Q6bnLCo= X-Google-Smtp-Source: ADUXVKJY62YHmuds/GBLqNvvrZCU88nngNdFuFChdyjeV4ncvPQk7MgRvvkvlp64vDDsZAjHe234Cw== X-Received: by 2002:a62:4c88:: with SMTP id e8-v6mr9819135pfj.99.1529342588389; Mon, 18 Jun 2018 10:23:08 -0700 (PDT) Received: from tictac2.mtv.corp.google.com ([2620:0:1000:1501:38e4:86fe:ec0c:4007]) by smtp.gmail.com with ESMTPSA id z26-v6sm26849091pfd.60.2018.06.18.10.23.06 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 18 Jun 2018 10:23:07 -0700 (PDT) From: Douglas Anderson To: andy.gross@linaro.org Cc: tdas@codeaurora.org, collinsd@codeaurora.org, sboyd@kernel.org, ilina@codeaurora.org, broonie@kernel.org, rplsssn@codeaurora.org, Douglas Anderson , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, David Brown , linux-kernel@vger.kernel.org Subject: [PATCH] drivers: qcom: rpmh-rsc: Check cmd_db_ready() to help children Date: Mon, 18 Jun 2018 10:22:29 -0700 Message-Id: <20180618172230.241960-1-dianders@chromium.org> X-Mailer: git-send-email 2.18.0.rc1.244.gcf134e6275-goog Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Children of RPMh will need access to cmd_db. Rather than having each child have code to check if cmd_db is ready let's add the check to RPMh. Suggested-by: Stephen Boyd Signed-off-by: Douglas Anderson Acked-by: Lina Iyer Reviewed-by: Bjorn Andersson --- This code is based upon v11 of Lina and Raju's RPMh series. With this we'll be able to remove this boilerplate code from clk-rpmh.c and qcom-rpmh-regulator.c. Neither of these files has landed upstream yet but patches are pretty far along. drivers/soc/qcom/rpmh-rsc.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/drivers/soc/qcom/rpmh-rsc.c b/drivers/soc/qcom/rpmh-rsc.c index 144a058407c0..ad85523bdd3b 100644 --- a/drivers/soc/qcom/rpmh-rsc.c +++ b/drivers/soc/qcom/rpmh-rsc.c @@ -18,6 +18,7 @@ #include #include +#include #include #include @@ -621,6 +622,18 @@ static int rpmh_rsc_probe(struct platform_device *pdev) struct rsc_drv *drv; int ret, irq; + /* + * Even though RPMh doesn't directly use cmd-db, all of its children + * do. To avoid adding this check to our children we'll do it now. + */ + ret = cmd_db_ready(); + if (ret) { + if (ret != -EPROBE_DEFER) + dev_err(&pdev->dev, "Command DB not available (%d)\n", + ret); + return ret; + } + drv = devm_kzalloc(&pdev->dev, sizeof(*drv), GFP_KERNEL); if (!drv) return -ENOMEM;