From patchwork Wed Oct 4 14:17:18 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephan Gerhold X-Patchwork-Id: 148318 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:612c:254a:b0:403:3b70:6f57 with SMTP id hf10csp166858vqb; Wed, 4 Oct 2023 07:17:43 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFFfxiwTuHyrvP+LpaQwqGHiAgtv7oERk/iiIDmlCT/pSvm4/yOsz/wPHoR9rRqnBlUuxZZ X-Received: by 2002:a05:6300:808c:b0:160:643f:ad54 with SMTP id ap12-20020a056300808c00b00160643fad54mr2044504pzc.20.1696429063258; Wed, 04 Oct 2023 07:17:43 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1696429063; cv=none; d=google.com; s=arc-20160816; b=jfguX4Z72P87tFu8/4Z/16kgDe/R8HZ9Wvtop4IRiox+slKWXPBBKwkyJWbUifSLI7 SIm7Jd82qTUcUE5KkliofNK9TvlrzDgQxR4TQ9ztSHel8EMXcTKJj2+renZRG5/o4Kt3 lmidd+Hyy5nB+QIgo8NML2hIj8EkBj0bwOTl+8VQhnE4vSzb33RzMS1f1A5Q6Bt/cLj1 3hgW9HHzT5C2ctVI27hc7pHyiwGK1JMI1grJ/rwQ5wxP8isDQQFCq3sg09Pp1kETswTg resJvuZKP7Kqy5cTM3mJpSczK4m8MT3WeoHEJBl72u+2M6jin0HCYIRLUfQAAoEOxyJ7 wVAQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:in-reply-to:references:message-id :content-transfer-encoding:mime-version:subject:date:from :dkim-signature; bh=uGFUKnshKK/Dlrx7eSaJi3UN15UGXc4Ye0y6BgFwE70=; fh=6ROhvpJrsh+iZGjEACun9ZXYVgb1LMu8Jj3+nzwGyNE=; b=gCTU2im/h7Ennqtute+OcD8FySgZctMISlEt9CLivL7fMNqUJItLtJJHcMT0VvJS05 mr1YEO5UnxiJ86b47lUYdXqcWN9hDzqk/4jBz0fpD98I7HPguPhrhpCf8T11Iy7q8V6+ xIfJSQRoVmrb/FKk+20kw5scicUQJFZircBVkr8MqhbNgl34kAS7aiZB/HT9MpnVcJYp FPPgU04cn6hRpeZgQMvER9d0sb6klsqO14+nuGwMtiI/uc9RoeuUB+QGvZtFHN2XhrKY 3cOTvlOSM/7sGQ59IxBbqPG9iiV+XyHySZwyANkbRwp5jlHeBIl6bCXcMmyUZCdLzeku Z+sA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernkonzept.com header.s=mx1 header.b="mcndqaF/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernkonzept.com Received: from snail.vger.email (snail.vger.email. [23.128.96.37]) by mx.google.com with ESMTPS id c2-20020a170902d48200b001c38666f77dsi3994399plg.596.2023.10.04.07.17.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 04 Oct 2023 07:17:43 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37; Authentication-Results: mx.google.com; dkim=pass header.i=@kernkonzept.com header.s=mx1 header.b="mcndqaF/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernkonzept.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id 1B196821A15E; Wed, 4 Oct 2023 07:17:42 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242888AbjJDORj (ORCPT + 18 others); Wed, 4 Oct 2023 10:17:39 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48736 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242880AbjJDORb (ORCPT ); Wed, 4 Oct 2023 10:17:31 -0400 Received: from mx.kernkonzept.com (serv1.kernkonzept.com [IPv6:2a01:4f8:1c1c:b490::2]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5421BBD; Wed, 4 Oct 2023 07:17:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=kernkonzept.com; s=mx1; h=Cc:To:In-Reply-To:References:Message-Id: Content-Transfer-Encoding:Content-Type:MIME-Version:Subject:Date:From: Reply-To:Content-ID:Content-Description; bh=uGFUKnshKK/Dlrx7eSaJi3UN15UGXc4Ye0y6BgFwE70=; b=mcndqaF/9YtRUFqpPsJk35hOjM ssdRCGiQRptp2Mz9hSJoC9ceNzNnFVX7rU3X2vCznKlRa20t5axk4+9w1KYS2AJnf5G+kV5Sp8w2m nsnBkhJLiNVv2U1cyj8wzbOpWopwDszXGdCxzxGRihGf80lMpvoXi7jlVEX3EPBqlQcIYbaiFWRDj qw3b8W3M/r7XSOkeQO6X6bWQgEp0r8pZW19fsVnHK6qOXiUpyF6g5A5WYbCFfZarXAIY61mBT88gS 33Fj7IaVRnTJRmJx6wPWoG263EUDVqgVKI/furH2T2IjWSehykKB2ykrNXMI2M1VBhmIfOkD77Jq8 JVKqIm2g==; Received: from [10.22.3.24] (helo=serv1.dd1.int.kernkonzept.com) by mx.kernkonzept.com with esmtpsa (TLS1.3:ECDHE_X25519__RSA_PSS_RSAE_SHA256__AES_256_GCM:256) (Exim 4.96) id 1qo2gl-0071hO-1J; Wed, 04 Oct 2023 16:17:23 +0200 From: Stephan Gerhold Date: Wed, 04 Oct 2023 16:17:18 +0200 Subject: [PATCH RFC 2/2] regulator: qcom_smd: Disable unused regulators MIME-Version: 1.0 Message-Id: <20231004-reg-smd-unused-v1-2-5d682493d555@kernkonzept.com> References: <20231004-reg-smd-unused-v1-0-5d682493d555@kernkonzept.com> In-Reply-To: <20231004-reg-smd-unused-v1-0-5d682493d555@kernkonzept.com> To: Mark Brown Cc: Liam Girdwood , Andy Gross , Bjorn Andersson , Konrad Dybcio , linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Stephan Gerhold , Stephan Gerhold X-Mailer: b4 0.12.3 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Wed, 04 Oct 2023 07:17:42 -0700 (PDT) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1778834797704786702 X-GMAIL-MSGID: 1778834801354924743 The RPM firmware on Qualcomm platforms does not provide a way to check if a regulator is on during boot using the SMD interface. If the regulators are already on during boot and Linux does not make use of them they will currently stay enabled forever. The regulator core does not know these regulators are on and cannot clean them up together with the other unused regulators. Fix this by setting the initial enable state to -EINVAL similar to qcom-rpmh-regulator.c. The regulator core will then also explicitly disable all unused regulators with unknown status. Signed-off-by: Stephan Gerhold --- NOTE: This has a slight potential of breaking boards that rely on having unused regulators permanently enabled (without regulator-always-on). However, this is always a mistake in the device tree so it's probably better to risk some breakage now, add the missing regulators and avoid this problem for all future boards. --- drivers/regulator/qcom_smd-regulator.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/regulator/qcom_smd-regulator.c b/drivers/regulator/qcom_smd-regulator.c index f53ada076252..0bbfba2e17ff 100644 --- a/drivers/regulator/qcom_smd-regulator.c +++ b/drivers/regulator/qcom_smd-regulator.c @@ -53,14 +53,14 @@ static int rpm_reg_write_active(struct qcom_rpm_reg *vreg) reqlen++; } - if (vreg->uv_updated && vreg->is_enabled) { + if (vreg->uv_updated && vreg->is_enabled > 0) { req[reqlen].key = cpu_to_le32(RPM_KEY_UV); req[reqlen].nbytes = cpu_to_le32(sizeof(u32)); req[reqlen].value = cpu_to_le32(vreg->uV); reqlen++; } - if (vreg->load_updated && vreg->is_enabled) { + if (vreg->load_updated && vreg->is_enabled > 0) { req[reqlen].key = cpu_to_le32(RPM_KEY_MA); req[reqlen].nbytes = cpu_to_le32(sizeof(u32)); req[reqlen].value = cpu_to_le32(vreg->load / 1000); @@ -1377,6 +1377,7 @@ static int rpm_regulator_init_vreg(struct qcom_rpm_reg *vreg, struct device *dev vreg->rpm = rpm; vreg->type = rpm_data->type; vreg->id = rpm_data->id; + vreg->is_enabled = -EINVAL; memcpy(&vreg->desc, rpm_data->desc, sizeof(vreg->desc)); vreg->desc.name = rpm_data->name;