2024-01-21 10:22:14

by Luca Weiss

[permalink] [raw]
Subject: [PATCH] ARM: dts: qcom: msm8926-htc-memul: Add rmtfs memory node

Add the rmtfs-mem node which was part of one of the "unknown" memory
reservation. Split that one, make sure the reserved-memory in total
still covers the same space.

Signed-off-by: Luca Weiss <[email protected]>
---
arch/arm/boot/dts/qcom/qcom-msm8926-htc-memul.dts | 15 ++++++++++++++-
1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8926-htc-memul.dts b/arch/arm/boot/dts/qcom/qcom-msm8926-htc-memul.dts
index ed328b24335f..3037344eb240 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8926-htc-memul.dts
+++ b/arch/arm/boot/dts/qcom/qcom-msm8926-htc-memul.dts
@@ -107,7 +107,20 @@ smem_region: smem@fa00000 {
};

unknown@fb00000 {
- reg = <0x0fb00000 0x1b00000>;
+ reg = <0x0fb00000 0x280000>;
+ no-map;
+ };
+
+ rmtfs@fd80000 {
+ compatible = "qcom,rmtfs-mem";
+ reg = <0x0fd80000 0x180000>;
+ no-map;
+
+ qcom,client-id = <1>;
+ };
+
+ unknown@ff00000 {
+ reg = <0x0ff00000 0x1700000>;
no-map;
};
};

---
base-commit: ad5c60d66016e544c51ed98635a74073f761f45d
change-id: 20240121-memul-rmtfs-9aa9b54f200a

Best regards,
--
Luca Weiss <[email protected]>



2024-01-22 10:54:35

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: qcom: msm8926-htc-memul: Add rmtfs memory node

On 21.01.2024 11:21, Luca Weiss wrote:
> Add the rmtfs-mem node which was part of one of the "unknown" memory
> reservation. Split that one, make sure the reserved-memory in total
> still covers the same space.
>
> Signed-off-by: Luca Weiss <[email protected]>
> ---

Could you please test dynamic rmtfs alloc, which should be possible
on some (most?) boards after 9265bc6bce6919c771970e5a425a66551a1c78a0?

Konrad

2024-01-22 17:47:10

by Luca Weiss

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: qcom: msm8926-htc-memul: Add rmtfs memory node

On Montag, 22. J?nner 2024 11:53:33 CET Konrad Dybcio wrote:
> On 21.01.2024 11:21, Luca Weiss wrote:
> > Add the rmtfs-mem node which was part of one of the "unknown" memory
> > reservation. Split that one, make sure the reserved-memory in total
> > still covers the same space.
> >
> > Signed-off-by: Luca Weiss <[email protected]>
> > ---
>
> Could you please test dynamic rmtfs alloc, which should be possible
> on some (most?) boards after 9265bc6bce6919c771970e5a425a66551a1c78a0?

To be honest, I'd like to not continue to experiment with reserved-memory on
this board, I've already spent way too much time figuring out how to not make
the phone crash under some circumstances, and now it seems relatively stable.
I might've even put my eMMC into some weird read-only mode where any write to
it just weirdly fails (also original software) by writing to random locations
in the RAM (or well, what Linux thought was non-special RAM).

Regards
Luca

>
> Konrad





2024-01-28 17:47:38

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: qcom: msm8926-htc-memul: Add rmtfs memory node


On Sun, 21 Jan 2024 11:21:54 +0100, Luca Weiss wrote:
> Add the rmtfs-mem node which was part of one of the "unknown" memory
> reservation. Split that one, make sure the reserved-memory in total
> still covers the same space.
>
>

Applied, thanks!

[1/1] ARM: dts: qcom: msm8926-htc-memul: Add rmtfs memory node
commit: 713bc594c6334a36d0caf4b98510ba3b6795616a

Best regards,
--
Bjorn Andersson <[email protected]>