Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <a60b5954-c9ac-0f2b-aef4-ad34b8f3abe7@kernel.org>
Date:   Tue, 5 Jul 2022 17:03:21 +0300
MIME-Version: 1.0
Subject: Re: [PATCH v4 0/5] interconnect: qcom: icc-rpm: Support bucket
Content-Language: en-US
To:     Leo Yan <leo.yan@linaro.org>, Andy Gross <agross@kernel.org>,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20220705072336.742703-1-leo.yan@linaro.org>
From:   Georgi Djakov <djakov@kernel.org>
In-Reply-To: <20220705072336.742703-1-leo.yan@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 5.07.22 10:23, Leo Yan wrote:
> This patch set is to support bucket in icc-rpm driver, so it implements
> the similar mechanism in the icc-rpmh driver.
> 
> We can use interconnect path tag to indicate the bandwidth voting is for
> which buckets, and there have three kinds of buckets: AWC, WAKE and
> SLEEP, finally the wake and sleep bucket values are used to set the
> corresponding clock (active and sleep clocks).  So far, we keep the AWC
> bucket but doesn't really use it.
> 
> Patches 01, 02, 03 enable interconnect path tag and update the DT
> binding document; patches 04 and 05 support bucket and use bucket values
> to set the bandwidth and clock rates.
> 
> Changes from v3:
> - Removed $ref and redundant sentence in DT binding document for
>    '#interconnect-cells' (Krzysztof Kozlowski).
> 
> Changes from v2:
> - Fixed for DT checker error for command ''make DT_CHECKER_FLAGS=-m
>    dt_binding_check' (Rob Herring).
> 
> Changes from v1:
> - Added description for property "#interconnect-cells" (Rob Herring);
> - Added Dimtry's reviewed tags for patches 02 and 03 (Dmitry Baryshkov);
> - Rebased on the latest mainline kernel and resolved conflict.
> 
> 
> Leo Yan (5):
>    dt-bindings: interconnect: Update property for icc-rpm path tag
>    interconnect: qcom: Move qcom_icc_xlate_extended() to a common file
>    interconnect: qcom: icc-rpm: Change to use qcom_icc_xlate_extended()
>    interconnect: qcom: icc-rpm: Support multiple buckets
>    interconnect: qcom: icc-rpm: Set bandwidth and clock for bucket values

Hi Leo,

Some patches do not apply clean on next. On which tree is this based? Do you have any additional 
patches?

Thanks,
Georgi