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 47ED7C4320E 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 1600960F35 for ; Sun, 25 Jul 2021 06:55:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230236AbhGYGPJ (ORCPT ); Sun, 25 Jul 2021 02:15:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33110 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230195AbhGYGPH (ORCPT ); Sun, 25 Jul 2021 02:15:07 -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 3942CC06175F for ; Sat, 24 Jul 2021 23:55:37 -0700 (PDT) Received: by mail-ot1-x32f.google.com with SMTP id z6-20020a9d24860000b02904d14e47202cso6629682ota.4 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=pEmTKGR5k13A4OPlCT2TphW2amiaUR54jva/H1NM2hL5eSfFjswm3gFw1qj8pVJGW5 gdoWVMkmuOHL1u8YXLZh77d/XZ1wkdx/ymadX6qV4J7xW2UUEwJw24Q7dc63VDdb1Mez qXFCO2BMAyMobRzS8evLhZktsmvEj83qfcUfxFZRWY4l/jW/lqw7Ote1oF4pKFMQ5chw ZRXC6KpNuOqyz9y7VWU/BH06oozvgSwenWUKHVdNIPITuUcKiHzrok43uPZUj9+46Ium /XZjBW2yqHPcUXooZm5I30tQLB/YMaNRFEqDgFeLU54TCRoCTFnmIXEbhvl58K9xy7/5 NQqg== X-Gm-Message-State: AOAM5313/Aq3xSOwVjR+0JY4eg9IioPZAMBoABUxMtd8KmqHAcEwExgK jJHPdiS0skN0uepS3OfOkfmktw== 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: linux-arm-msm@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