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;
From:   Bjorn Andersson <andersson@kernel.org>
To:     agross@kernel.org, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, elder@linaro.org,
        konrad.dybcio@linaro.org
Cc:     mka@chromium.org, quic_sibis@quicinc.com, elder@kernel.org,
        linux-arm-msm@vger.kernel.org, dianders@chromium.org,
        linux-kernel@vger.kernel.org, devicetree@vger.kernel.org
Subject: Re: [PATCH v3] arm64: dts: qcom: sc7280: only enable IPA for boards with a modem
Date:   Thu, 29 Dec 2022 11:23:24 -0600
Message-Id: <167233461763.1099840.13987684672071387995.b4-ty@kernel.org>
In-Reply-To: <20221224002126.1518552-1-elder@linaro.org>
References: <20221224002126.1518552-1-elder@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Fri, 23 Dec 2022 18:21:26 -0600, Alex Elder wrote:
> IPA is only needed on a platform if it includes a modem, and not all
> SC7280 SoC variants do.  The file "sc7280-herobrine-lte-sku.dtsi" is
> used to encapsulate definitions related to Chrome OS SC7280 devices
> where a modem is present, and that's the proper place for the IPA
> node to be enabled.
> 
> Currently IPA is enabled in "sc7280-idp.dtsi", which is included by
> DTS files for Qualcomm reference platforms (all of which include the
> modem).  That also includes "sc7280-herobrine-lte-sku.dtsi", so
> enabling IPA there would make it unnecessary for "sc7280-idp.dtsi"
> to enable it.
> 
> [...]

Applied, thanks!

[1/1] arm64: dts: qcom: sc7280: only enable IPA for boards with a modem
      commit: 9472edb3e7ea08ada9d19a9cfc1bee7de6edee75

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>