lkml.org 
[lkml]   [2022]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    SubjectRe: [PATCH] arm64: dts: qcom: sc7180: Remove ipa_fw_mem node on trogdor
    From
    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?

    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.

    I don't actually know... Bjorn, do you?

    -Alex

    > Cc: Alex Elder <elder@linaro.org>
    > Cc: Matthias Kaehlcke <mka@chromium.org>
    > Fixes: 310b266655a3 ("arm64: dts: qcom: sc7180: define ipa_fw_mem node")
    > Signed-off-by: Stephen Boyd <swboyd@chromium.org>
    > ---
    > arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 1 +
    > 1 file changed, 1 insertion(+)
    >
    > diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
    > index 732e1181af48..262224504921 100644
    > --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
    > +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
    > @@ -42,6 +42,7 @@ charger-crit {
    > */
    >
    > /delete-node/ &hyp_mem;
    > +/delete-node/ &ipa_fw_mem;
    > /delete-node/ &xbl_mem;
    > /delete-node/ &aop_mem;
    > /delete-node/ &sec_apps_mem;
    >
    > base-commit: 42226c989789d8da4af1de0c31070c96726d990c

    \
     
     \ /
      Last update: 2022-05-17 21:53    [W:3.982 / U:0.188 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site