Re: [PATCH 1/3] arm64: dts: qcom: milos: Reduce rmtfs_mem size to 2.5MiB

[Luca Weiss]

On Mon Mar 23, 2026 at 10:51 AM CET, Konrad Dybcio wrote:
> On 3/21/26 6:00 PM, Alexander Koskovich wrote:
>> The rmtfs_mem region is currently sized at 6MiB but the default for
>> milos downstream is 2.5MiB. This causes remoteproc crashes on devices
>> that expect the smaller size:
>> 
>> modem_ac.c:281:Access Control Error: Could not protect the region specified:Start:e1f00000 End:e2180000, PID:1
>> 
>> Reduce the default to 2.5MiB to match the QCOM downstream config, and
>> override the size for FP6.
>> 
>> Signed-off-by: Alexander Koskovich <akoskovich@xxxxx>
>> ---
>>  arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts | 4 ++++
>>  arch/arm64/boot/dts/qcom/milos.dtsi              | 2 +-
>>  2 files changed, 5 insertions(+), 1 deletion(-)
>> 
>> diff --git a/arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts b/arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts
>> index 52895dd9e4fa..c0981ab731c4 100644
>> --- a/arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts
>> +++ b/arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts
>> @@ -724,6 +724,10 @@ &remoteproc_wpss {
>>  	status = "okay";
>>  };
>>  
>> +&rmtfs_mem {
>> +	reg = <0x0 0xe1f00000 0x0 0x600000>;
>
> +Luca

You didn't actually Cc me, but I found it on the mailing list anyways :)

>
> Could you please confirm that this was intentional in the first place?
>
> FWIW I see that volcano.dtsi indeed only requests 0x280_000 on a "clean"
> Qualcomm tree

Yes, it's intentional that it's 6 MiB on Fairphone (Gen. 6):

https://gerrit-public.fairphone.software/plugins/gitiles/platform/vendor/qcom/proprietary/devicetree/+/b3744ffef87e1f2a232b26e9cb270a1f82554bc3%5E!/

I think Fixes is appropriate since the default for Milos should be
0x280_000 and not 0x600_000?

Fixes: d9d59d105f98 ("arm64: dts: qcom: Add initial Milos dtsi")

And maybe a comment like this on top of the node?

/* Increase the size from 2.5 MiB to 6 MiB */

Anyways:

Reviewed-by: Luca Weiss <luca.weiss@xxxxxxxxxxxxx>


Regards
Luca