On Thu, 13 Aug 2020 at 12:38, Bjorn Andersson
[off-list ref] wrote:

On Thu 06 Aug 15:31 PDT 2020, Konrad Dybcio wrote:

quoted

Subject: Re: [PATCH v4] arm64: dts: qcom: Add support for Xiaomi Poco F1 (Beryllium)

quoted

// This removed_region is needed to boot the device
              // TODO: Find out the user of this reserved memory
              removed_region: memory@88f00000 {

This region seems to belong to the Trust Zone. When Linux tries to access it, TZ bites and shuts the device down.

This is in line with what the documentation indicates and then it would
be better to just bump &tz_mem to a size of 0x4900000.

Hi, so just to be sure that I got this right, you want me to extend
&tz_mem to the size of 0x4900000 from the default size of 0x2D00000 by
including this downstream &removed_region (of size 0x1A00000) +
previously unreserved downstream memory region (of size 0x200000), to
align with the starting address of &qseecom_mem?

I just gave this &tz_mem change a spin and I do not see any obvious
regression in my limited smoke testing (Boots AOSP to UI with
v5.9-rc1. Touch/BT/WiFi works) so far, with 20+ out-of-tree patches.

Regards,
Amit Pundir

Regards,
Bjorn