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 X-Spam-Level: X-Spam-Status: No, score=-18.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1DD38C432BE for ; Sun, 25 Jul 2021 06:55:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D8A9B60E78 for ; Sun, 25 Jul 2021 06:55:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229538AbhGYGPG (ORCPT ); Sun, 25 Jul 2021 02:15:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33108 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229460AbhGYGPG (ORCPT ); Sun, 25 Jul 2021 02:15:06 -0400 Received: from mail-ot1-x32f.google.com (mail-ot1-x32f.google.com [IPv6:2607:f8b0:4864:20::32f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 33646C061757 for ; Sat, 24 Jul 2021 23:55:37 -0700 (PDT) Received: by mail-ot1-x32f.google.com with SMTP id a5-20020a05683012c5b029036edcf8f9a6so6657009otq.3 for ; Sat, 24 Jul 2021 23:55:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kali.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding:content-language; bh=BGn+EjMqq99Bj04hcp2WO+QUSqyG5NK9Xv1j68OtcH0=; b=BcLvcUE6Fwt13XrkUHN/uz/j8woPZS1aYLDsNjLOmdnms6EFWkIR7tihdK7Cgu+W4f CtRNrqXqCFzAiJg/Hqw/gw/88rA9P3SZn8N+aa9UMAtwRO7XBB+lfQo19TTuEnIVm/HW ql28wyYFMjwuDLG3zxczw8JfDhu/B2hIQzKh7E4jxu4lr5GQBlLXaESVfCYGg65NDNdM FigSk0gsyw2IsCg8l6mMZd2DvGanx4Iwc+tBg3lzMXprgIUTJHWMR70btuO12b04QUFP mYdlaAcP9/TOZUy6e45RKfClfSv/qL7a94Dt37dx3Iq4k/o0BU3xLfpYOSUN0CmISJN7 o46A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding :content-language; bh=BGn+EjMqq99Bj04hcp2WO+QUSqyG5NK9Xv1j68OtcH0=; b=bjxFHOnvGKu6tAdctYDKHdpBI31jBrGz6/yF0mw46Zah06Yi/4/oMcsFUazSbb4UI7 MOpA8ObLGjMDNnXGbSYIT/3WuXBN6ZBzwdES+Hx/vf8RQISXa1VZR9IqEa0bO6LHYiHQ va/CXgKAjt5SppK8mLRbRiTbBK2UxwHFngFYM5c4fa0YXshKF7WX5tIQhD+kLkoB0SUQ 2yuU+7dGVzfNmuBdZCTrb5qZGtZHJwiiO8j4ogUEZTNr8RaHkeKy4Kcnl+3pRY/myLBs IB1k7WuB+lHyQg592wJ8qq2YcCrTJEhSmPkRxFl3e79dVlCZ0WNkOieC1pSsKhxcWJ37 bu7A== X-Gm-Message-State: AOAM530bXt5ChTxr8ZSbkljYCsrkBnMh/KP7koxAfoTpLWNodsYdgYdA a3/YbqMXJwX0PH7m2NPxtheJXQ== X-Google-Smtp-Source: ABdhPJynVQWZnbcHqSx6DNemrpNukJu/Vnj0fuSC7MDxJlHlvE364cXzRFvZwCEOS69yYR8t0McbnQ== X-Received: by 2002:a9d:7f98:: with SMTP id t24mr8484242otp.366.1627196136433; Sat, 24 Jul 2021 23:55:36 -0700 (PDT) Received: from MacBook-Pro.hackershack.net (cpe-173-173-107-246.satx.res.rr.com. [173.173.107.246]) by smtp.gmail.com with ESMTPSA id r26sm4288163ooh.32.2021.07.24.23.55.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 24 Jul 2021 23:55:35 -0700 (PDT) Subject: Re: [PATCH] arm64: dts: qcom: sdm850-yoga: Enable IPA To: Bjorn Andersson , Andy Gross , Rob Herring , Alex Elder Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20210615232816.835325-1-bjorn.andersson@linaro.org> From: Steev Klimaszewski Message-ID: Date: Sun, 25 Jul 2021 01:55:34 -0500 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0) Gecko/20100101 Thunderbird/78.12.0 MIME-Version: 1.0 In-Reply-To: <20210615232816.835325-1-bjorn.andersson@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Content-Language: en-US Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 6/15/21 6:28 PM, Bjorn Andersson wrote: > Shuffle memory regions to make firmware loading succeed and then enable > the ipa device. > > Signed-off-by: Bjorn Andersson > --- > arch/arm64/boot/dts/qcom/sdm845.dtsi | 21 +++++++------------ > .../boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 5 +++++ > 2 files changed, 13 insertions(+), 13 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi > index 1796ae8372be..49624eadce84 100644 > --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi > +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi > @@ -128,28 +128,23 @@ camera_mem: memory@8bf00000 { > no-map; > }; > > - ipa_fw_mem: memory@8c400000 { > - reg = <0 0x8c400000 0 0x10000>; > + wlan_msa_mem: memory@8c400000 { > + reg = <0 0x8c400000 0 0x100000>; > no-map; > }; > > - ipa_gsi_mem: memory@8c410000 { > - reg = <0 0x8c410000 0 0x5000>; > + gpu_mem: memory@8c515000 { > + reg = <0 0x8c515000 0 0x2000>; > no-map; > }; > > - gpu_mem: memory@8c415000 { > - reg = <0 0x8c415000 0 0x2000>; > + ipa_fw_mem: memory@8c517000 { > + reg = <0 0x8c517000 0 0x5a000>; > no-map; > }; > > - adsp_mem: memory@8c500000 { > - reg = <0 0x8c500000 0 0x1a00000>; > - no-map; > - }; > - > - wlan_msa_mem: memory@8df00000 { > - reg = <0 0x8df00000 0 0x100000>; > + adsp_mem: memory@8c600000 { > + reg = <0 0x8c600000 0 0x1a00000>; > no-map; > }; > > diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts > index c2a709a384e9..3eaa42dc3794 100644 > --- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts > +++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts > @@ -415,6 +415,11 @@ ecsh: hid@5c { > }; > }; > > +&ipa { > + status = "okay"; > + memory-region = <&ipa_fw_mem>; > +}; > + > &mdss { > status = "okay"; > }; Hi, Thank you for this! Tested - one thing that end users/distro packagers may need to be aware of, if the module is loaded in an initramfs, the firmware file also needs to be loaded in the initramfs.  If the module is in an initramfs and the firmware is not, the module will need to be removed and re-inserted for the device to come up. Tested-by: Steev Klimaszewski