From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oa1-f49.google.com (mail-oa1-f49.google.com [209.85.160.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6248C7E for ; Mon, 6 Jun 2022 19:51:18 +0000 (UTC) Received: by mail-oa1-f49.google.com with SMTP id 586e51a60fabf-fb6b4da1dfso2461920fac.4 for ; Mon, 06 Jun 2022 12:51:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:in-reply-to:references:from:user-agent:date:message-id :subject:to:cc; bh=wUE0WYijKz/I9WB3BAXy+vr8DpFKXrMlnfOuumpeD0s=; b=cs4bfETKtnutHfKnMRa8EDDF/SGg0Ol3SZDgvq2v6rDjqMPEVozqvGlftR+7NtXnnx ZFfs0bZ5sKKXhTYSPNDrousmupAvxoLeIv0BCTQHDt393fs72j4NHYDc4HLULrjZSHX2 1tPgngy5jRf1rRTLFlCDhojXu+H93PWIkKFnw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:in-reply-to:references:from :user-agent:date:message-id:subject:to:cc; bh=wUE0WYijKz/I9WB3BAXy+vr8DpFKXrMlnfOuumpeD0s=; b=cIpo3U2i9KRiRBL+lOdUOnvwZ7tdYaSu4hQLV55Bvea57hBKxYDqAeZLZLCAMoRP8K abAKh4rdHtHLfPmlK4AfSK0Jut0tmTbkPPcOlv9ZiBcC7tMbRTrG9sEKCnWZotYeJkKJ C/nK6FyaMTivVqeXrsoJOdEeLYcq+s0QMkZeUl+zGPP3FEEOG+mLbK0Uv8IpaY/0upfS 5ZyLERBEP8vO+M7CNRJLal8EdmEB54X9sO3tAay+SG8hJ0hHAXcnPR0r5QGallD5h1LZ QlxtgQZPTkyVNxP6zl3MuDg1/EASqNmgITNkBkgOaTATCOeM2hOZtKPyVW5L6t20MYaT slFA== X-Gm-Message-State: AOAM531U9sE4IqGAwy85DXeIXHzDv/DdnNI+IMYg9iuIfxALzWCEJln0 4b1RH7gHGS4L8UiMm/mhs4BAzAyLVieXRPEks5e13Q== X-Google-Smtp-Source: ABdhPJyA+8bj5MPF+OqCsgGfdq8U0lMbu6ZpxmSpJ/QCRTM+wYpE9wn/YOFZUpgt/HDEEtcKS0IhaqQHFdgpGb1STd8= X-Received: by 2002:a05:6870:558e:b0:e1:db7c:26aa with SMTP id n14-20020a056870558e00b000e1db7c26aamr14772185oao.63.1654545077326; Mon, 06 Jun 2022 12:51:17 -0700 (PDT) Received: from 753933720722 named unknown by gmailapi.google.com with HTTPREST; Mon, 6 Jun 2022 12:51:16 -0700 Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 In-Reply-To: <1cbd12ad-4147-5d97-ef7f-4104b630244b@linaro.org> References: <20220517193307.3034602-1-swboyd@chromium.org> <1cbd12ad-4147-5d97-ef7f-4104b630244b@linaro.org> From: Stephen Boyd User-Agent: alot/0.10 Date: Mon, 6 Jun 2022 12:51:16 -0700 Message-ID: Subject: Re: [PATCH] arm64: dts: qcom: sc7180: Remove ipa_fw_mem node on trogdor To: Alex Elder , Andy Gross , Bjorn Andersson Cc: linux-kernel@vger.kernel.org, patches@lists.linux.dev, linux-arm-msm@vger.kernel.org, Matthias Kaehlcke Content-Type: text/plain; charset="UTF-8" Quoting Alex Elder (2022-05-17 13:56:06) > On 5/17/22 3:06 PM, Stephen Boyd wrote: > > Quoting Alex Elder (2022-05-17 12:51:34) > >> On 5/17/22 2:33 PM, Stephen Boyd wrote: > >>> We don't use this carveout on trogdor boards, and having it defined in > >>> the sc7180 SoC file causes an overlap message to be printed at boot. > >>> > >>> OF: reserved mem: OVERLAP DETECTED! > >>> memory@86000000 (0x0000000086000000--0x000000008ec00000) overlaps with memory@8b700000 (0x000000008b700000--0x000000008b710000) > >>> > >>> Delete the node in the trogdor dtsi file to fix the overlap problem and > >>> remove the error message.' > >> > >> Is the mpss_mem node (which defines 0x86000000-0x8ec00000) correct? > > > > I think it is. Modem is working for me. > > > >> > >> You're right, we don't use that memory on Chrome OS, but I just > >> wondered whether the other node might also have a problem. > > > > What sort of problem are you thinking about? > > At some point in the past I think some of these carved out > areas got updated (perhaps on sdm845?). I.e. they were > wrong, and got corrected later. Some things worked despite > that. So the problem, for example, might be that the > mpss region is bigger than it needs to be. > > I don't *assume* there's a problem, I was just asking, > and maybe Bjorn or someone else can verify what's right. > > I'm done for the day right now; otherwise I'd spend a > little more time trying to track the information down > myself... Did you find anything? Otherwise, can you provide a Reviewed-by tag?