Re: [PATCH v1] arm64: dts: qcom: sc7180-trogdor: Enable IPA on LTE only SKUs

[Alex Elder]

On 9/20/21 1:32 PM, Sujit Kautkar wrote:
> Enable the IPA node for LTE and skip for wifi-only SKUs
>
> Signed-off-by: Sujit Kautkar <sujitka@xxxxxxxxxxxx>

Looks good to me.  Thanks Sujit.

Reviewed-by: Alex Elder <elder@xxxxxxxxxx>

> ---
>
>   arch/arm64/boot/dts/qcom/sc7180-trogdor-lte-sku.dtsi | 11 +++++++++++
>   arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi         | 11 -----------
>   2 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lte-sku.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lte-sku.dtsi
> index 469aad4e5948c..fd4b712037542 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-lte-sku.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-lte-sku.dtsi
> @@ -17,3 +17,14 @@ &remoteproc_mpss {
>   	firmware-name = "qcom/sc7180-trogdor/modem/mba.mbn",
>   			"qcom/sc7180-trogdor/modem/qdsp6sw.mbn";
>   };
> +
> +&ipa {
> +	status = "okay";
> +
> +	/*
> +	 * Trogdor doesn't have QHEE (Qualcomm's EL2 blob), so the
> +	 * modem needs to cover certain init steps (GSI init), and
> +	 * the AP needs to wait for it.
> +	 */
> +	modem-init;
> +};
> diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> index 6e96024a487cd..3d8b9c6b21a85 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> @@ -754,17 +754,6 @@ alc5682: codec@1a {
>   	};
>   };
>   
> -&ipa {
> -	status = "okay";
> -
> -	/*
> -	 * Trogdor doesn't have QHEE (Qualcomm's EL2 blob), so the
> -	 * modem needs to cover certain init steps (GSI init), and
> -	 * the AP needs to wait for it.
> -	 */
> -	modem-init;
> -};
> -
>   &lpass_cpu {
>   	status = "okay";
>