Re: [PATCH v3 3/4] arm64: dts: qcom: sdm630: describe adsp_mem region properly

[Nickolay Goppen <setotau@mainlining.org>]

27.04.2026 15:08, Konrad Dybcio пишет:
> On 4/27/26 2:00 PM, Nickolay Goppen wrote:
>> 27.04.2026 14:49, Konrad Dybcio пишет:
>>> On 4/22/26 5:39 PM, Nickolay Goppen wrote:
>>>> Downstream [1] this region is marked as shared, reusable and dynamic so
>>>> describe it that way.
>>>>
>>>> [1]: https://github.com/xiaomi-sdm660/android_kernel_xiaomi_sdm660/blob/11-EAS/arch/arm/boot/dts/qcom/sdm660.dtsi#L448
>>>>
>>>> Signed-off-by: Nickolay Goppen <setotau@mainlining.org>
>>>> ---
>>>>    arch/arm64/boot/dts/qcom/sdm630.dtsi | 9 ++++++---
>>>>    1 file changed, 6 insertions(+), 3 deletions(-)
>>>>
>>>> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
>>>> index 4b47efdb57b2..36b419dea153 100644
>>>> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
>>>> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
>>>> @@ -494,9 +494,12 @@ venus_region: venus@9f800000 {
>>>>                no-map;
>>>>            };
>>>>    -        adsp_mem: adsp-region@f6000000 {
>>>> -            reg = <0x0 0xf6000000 0x0 0x800000>;
>>>> -            no-map;
>>>> +        adsp_mem: adsp-region {
>>>> +            compatible = "shared-dma-pool";
>>>> +            alloc-ranges = <0x0 0x00000000 0x0 0xffffffff>;
>>> Please make this = <0x0 0x80000000 0x0 0x80000000>; instead, since the RAM
>> Maybe you meant <0x0 0x80000000 0x0 0x8fffffff>?
> No, <0x0 0x80000000 0x0 0x80000000> means "start at 2 GiB, until the end of
> the 32-bit address range"
Thanks, I've misunderstood.
> Konrad

-- 
Best regards,
Nickolay