Received: by 2002:a05:6a10:1d13:0:0:0:0 with SMTP id pp19csp1031547pxb; Wed, 1 Sep 2021 15:48:01 -0700 (PDT) X-Google-Smtp-Source: ABdhPJybVF9Yum4iTtCDV01kpTTuW1tjOt8NM9Su4xncu256TDdEjwpypSHb6QmhjV/nehmNAytN X-Received: by 2002:a05:6402:b65:: with SMTP id cb5mr235372edb.343.1630536481530; Wed, 01 Sep 2021 15:48:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1630536481; cv=none; d=google.com; s=arc-20160816; b=q/05ge0BZOxHShAhRxQbFVHFUsIKmaM4X4aks5s2kl7YaTYFWSB8p1bAZf1oDVDbem HuwGZG63QFYeD9r5lU8oMhLsMcib5Oxo0dqc13fEV40Nnk052z6XI3eAXs7tv8ZGKr7a fVnqLpgVmxr+Egcp4AzssVzBe8WWGnHL5KZ7ejXDangX+N0W9xJhJfzFHlyWx8fMY68u Xxd4/1VtgqVCyDYaoyCZ3jABPlQx/ap20/kFZGH9PKxAtZ/mp31I/DLEijs+1Rr7zRqS X8Ov/m64J5m6TAVFZjZw6iyRJR+vTVnyTOclZNcYKWqTVkqrriRpAQUCdG9+BrVBAx5J IgIQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject; bh=6Qu3mTkAiHLPO+GV5tlkTRQ/Sz9Aoex/Za+fXbxe1+Y=; b=NLmOtTBZIhWWTq8Vn9Wym92ry+J224vlu9PG5zVQ6Ii9de3F4PsgrA1/VZU1RdudRy DOvcZax+yPNBWBHp8PqCNMEDRUkWwbs8C0L/DIc9eztkmCcCoPhyav+/JNqZ4mhFO2XX UVKGlzmX6duTJxcv9zI7odyebJttsyycIEKtiVbpBE9gHDqL3Qe920Wri7Zmzy6MQ5OC PLKXDHvethQVtCRFGM3LiEjzzGa6FCui9HE9xb2RJg5L0i/GsgiA54490bqu6wgUPfhb VUP/l1TWCa4WF2Bgiw+QvxilJi3ljgBCqltf9UdQ6NQuhUpfJ6YR82dssP8b36gOvYGB WmCQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id mp24si916242ejc.731.2021.09.01.15.47.38; Wed, 01 Sep 2021 15:48:01 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243666AbhIAWUN (ORCPT + 99 others); Wed, 1 Sep 2021 18:20:13 -0400 Received: from relay02.th.seeweb.it ([5.144.164.163]:45711 "EHLO relay02.th.seeweb.it" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230168AbhIAWUI (ORCPT ); Wed, 1 Sep 2021 18:20:08 -0400 Received: from IcarusMOD.eternityproject.eu (unknown [2.237.20.237]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by m-r1.th.seeweb.it (Postfix) with ESMTPSA id D5DED1F614; Thu, 2 Sep 2021 00:19:08 +0200 (CEST) Subject: Re: [PATCH v4 1/5] interconnect: qcom: sdm660: Commonize RPM-QoS To: Dmitry Baryshkov , Yassine Oudjana , Andy Gross , Bjorn Andersson , Georgi Djakov , Rob Herring Cc: AngeloGioacchino Del Regno , linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Shawn Guo , Marijn Suijten , Konrad Dybcio References: <20210901121518.152481-1-y.oudjana@protonmail.com> <20210901121518.152481-2-y.oudjana@protonmail.com> <9af0f031-101e-53b4-514e-9ead44320f4e@somainline.org> From: AngeloGioacchino Del Regno Message-ID: <9f902b08-bdf2-13b5-acbe-215173e7929a@somainline.org> Date: Thu, 2 Sep 2021 00:19:08 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.9.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Il 02/09/21 00:01, Dmitry Baryshkov ha scritto: > On 01/09/2021 21:48, AngeloGioacchino Del Regno wrote: >> Il 01/09/21 14:15, Yassine Oudjana ha scritto: >>> SoCs such as MSM8996 also control bus QoS in a similar fashion to SDM660, >>> with some paths being controlled by RPM and others directly by the AP. >>> Move relevant functions and defines to a new object so that they can be used >>> in multiple drivers. >>> >>> Signed-off-by: Yassine Oudjana >> >> Hey guys! >> >> I'm waiting for the interconnect RPM-QoS commonization to be merged as I have fresh >> interconnect drivers for MSM8998 and MSM8976, ready to send, that are also using >> the very same QoS mechanism as SDM660. > > We were also looking onto this. I'd propose to merge sdm660 code into main > icc-rpm.c instead of splitting it into separate file. We have enabled QoS for > apq8096 (msm8916) and msm8939. See > https://lore.kernel.org/linux-arm-msm/20210818015732.1717810-1-dmitry.baryshkov@linaro.org/ > for the reference. I'm waiting for Shawn to publish v2 of his fix, then I can post > v2 of my patchset. > Merging it into icc-rpm would actually make sense, since the QoS mechanism won't ever be used by the RPM-Hardened icc... so yeah, on that I agree. Whenever you guys send the new patchsets, if you can please make sure to Cc me and/or the other SoMainline guys, we will make sure to review and test the sets as soon as possible, as a way to speed up the merge process, if that helps! Thank you, - Angelo >> >> Yassine, please check Shawn's recent patches for SDM660 interconnect, which are >> fixing some bits for the QoS implementation and adding some required clocks to the >> SDM660 interconnect driver. >> >> Adding Shawn to the Ccs as to make him aware of this patch; >> also adding Marijn and Konrad from SoMainline as probably interested parties. >> >> Cheers! >> - Angelo > >