From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 572E0C05027 for ; Sat, 18 Feb 2023 00:26:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229744AbjBRA0k (ORCPT ); Fri, 17 Feb 2023 19:26:40 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40016 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229677AbjBRA0j (ORCPT ); Fri, 17 Feb 2023 19:26:39 -0500 Received: from mail-lf1-x136.google.com (mail-lf1-x136.google.com [IPv6:2a00:1450:4864:20::136]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4775668AC9 for ; Fri, 17 Feb 2023 16:26:36 -0800 (PST) Received: by mail-lf1-x136.google.com with SMTP id be32so3507795lfb.10 for ; Fri, 17 Feb 2023 16:26:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Iz+PpBCaN2eTF1bAhZ+2tgpCk6bNWDrT4b3aQDTLjn4=; b=C0d26VB/UL384OGHWCA1AXRo89XPr0JW09Y0bUgWGjPuT4pq2OCCGxWpqr9sDJiUkY MSB5QRvHJSGSDGrk/vlfxhtQCValpz9gpXANpLcU+p5H2Oc9VdszeuPZHz6MedQpKxIX eSXaDx5BLPhyMcKa854KKFdJjLpJxjvPMVCnCwO+Y0bl7TWn/TObzR7eYfwnYjku0U9N DAceDkHWSuiL+nvkuaHmkizuhe2PfiQ1lO+Xd42kKC3RV3J9mEE8mQ6bmGbEOECGX8mc XphU8SljyARdlMtk6jEg/5I4zBdHwKkCy30J+L9l/LUqaMgj1EELHqpcSvNxHs/YJogP d1OA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Iz+PpBCaN2eTF1bAhZ+2tgpCk6bNWDrT4b3aQDTLjn4=; b=W7Zl/crX2zx027kfcnmpohvtSicDKnC5R6HkAtKF/4FXEsdLubNnm0TWkk8IrIib/V 7t0sdKus6cgTltp08De0aXLJoTMhrvKT2ERCZQJ/4qVb1Mjg8j/gtBy3ClKLoPmsy6lX cTvUNBbUL1AdEeTHbYcWZzqJAkQ/doZQBXFsmwZnh2s0/p0pCXBu7p70BZ9E1ZPwN3Nn GaRIQG8EkDybrQs5dIcCeRB+2WnCiniXZ2Qoanyt4oZEOT2UDM99MRVrQE6Jk3G+cEHR HHi49DHeflvfM74Lwe3G9k+i+w2rcKoycAp6mMbhaXY4V/JRDzANVX3w5K5fHE2/vXkQ vs5g== X-Gm-Message-State: AO0yUKVKxva8WlrvR71GKJa++3QiHVxF2RgK/448bZZGcYTFBb+/UOw7 845pL8J3HHVuC2k8pYcl5BVOpQ== X-Google-Smtp-Source: AK7set/blEr5tuwakVvUBgDfcxlFT/0GmnlBbcRkf7FavIpGqWDHjeJsNPnY+VrSOFq8PpELXzUChg== X-Received: by 2002:ac2:4a87:0:b0:4db:2a28:6453 with SMTP id l7-20020ac24a87000000b004db2a286453mr745429lfp.25.1676679994477; Fri, 17 Feb 2023 16:26:34 -0800 (PST) Received: from [192.168.1.101] (abxh184.neoplus.adsl.tpnet.pl. [83.9.1.184]) by smtp.gmail.com with ESMTPSA id e1-20020a05651c038100b002950ce0ecd0sm143976ljp.114.2023.02.17.16.26.33 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 17 Feb 2023 16:26:34 -0800 (PST) Message-ID: Date: Sat, 18 Feb 2023 01:26:32 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.2 Subject: Re: [PATCH v10 2/6] dt-bindings: opp: v2-qcom-level: Document CPR3 open/closed loop volt adjustment To: Rob Herring Cc: AngeloGioacchino Del Regno , Andy Gross , Bjorn Andersson , Krzysztof Kozlowski , Viresh Kumar , Nishanth Menon , Stephen Boyd , Niklas Cassel , Liam Girdwood , Mark Brown , Robert Marko , linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-pm@vger.kernel.org References: <20230217-topic-cpr3h-v10-0-67aed8fdfa61@linaro.org> <20230217-topic-cpr3h-v10-2-67aed8fdfa61@linaro.org> <20230217231330.GA2238521-robh@kernel.org> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20230217231330.GA2238521-robh@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 18.02.2023 00:13, Rob Herring wrote: > On Fri, Feb 17, 2023 at 12:08:25PM +0100, Konrad Dybcio wrote: >> CPR3 and newer can be fed per-OPP voltage adjustment values for both >> open- and closed-loop paths to make better decisions about settling >> on the final voltage offset target. Document these properties. >> >> Signed-off-by: Konrad Dybcio >> --- >> .../devicetree/bindings/opp/opp-v2-qcom-level.yaml | 14 ++++++++++++++ >> 1 file changed, 14 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/opp/opp-v2-qcom-level.yaml b/Documentation/devicetree/bindings/opp/opp-v2-qcom-level.yaml >> index a30ef93213c0..93cc88434dfe 100644 >> --- a/Documentation/devicetree/bindings/opp/opp-v2-qcom-level.yaml >> +++ b/Documentation/devicetree/bindings/opp/opp-v2-qcom-level.yaml >> @@ -34,6 +34,20 @@ patternProperties: >> minItems: 1 >> maxItems: 2 >> >> + qcom,opp-cloop-vadj: >> + description: | >> + A value representing the closed-loop voltage adjustment value > > A value? > >> + associated with this OPP node. >> + $ref: /schemas/types.yaml#/definitions/int32-array >> + maxItems: 2 > > Or 2 values? Right, this description doesn't make any sense if you're just looking at the documentation without looking at the driver.. Generally, each CPR3 instance can have multiple "threads" (each one of which regulates voltage for some on-SoC IP or part of it). The nth entry in the qcom,opp-[co]loop-vadj array corresponds to a voltage offset for the nth thread. If the nth entry in the array is missing, the driver assumes the arr[0] one is "global" to this CPR3 instance at this OPP level and applies it to all threads. ...and looking at it again, this is sorta just bad design, especially if you take into account that there's no known user of CPR3 that employs more than 2 threads. I'll remove that from the driver and make the description clearer. Also, only noticed now.. "qcom,sdm630-cprh" was not documented, so that's to be fixed for the next submission as well! Konrad > >> + >> + qcom,opp-oloop-vadj: >> + description: | >> + A value representing the open-loop voltage adjustment value >> + associated with this OPP node. >> + $ref: /schemas/types.yaml#/definitions/int32-array >> + maxItems: 2 >> + >> required: >> - opp-level >> - qcom,opp-fuse-level >> >> -- >> 2.39.1 >>