Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory

[Bertrand Marquis <Bertrand.Marquis@xxxxxxx>]

Hi Henry,

> On 7 Sep 2022, at 14:49, Henry Wang <Henry.Wang@xxxxxxx> wrote:
> 
> Hi Bertrand and Michal,
> 
> I don't want to spam the email so I just reply here...
> 
>> -----Original Message-----
>> From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
>>>>>> But in any case we should only add one pair here for sure, as you say
>> the
>>>>>> only implication is to add a couple of 0 in the worst case.
>>>>> I agree. The only drawback is the need to modify the already introduced
>> properties
>>>>> to be coherent.
>>>> 
>>>> Agree, someone will need to do a pass on the whole doc which might be
>> easier with all things in.
>>>> 
>>> Well, not only docs. If we decide to use a single pair of #address-cells and
>> #size-cells, then
>>> we need to modify the code that expects different properties e.g.
>> xen,static-mem-{address/size}-cells.
>> 
>> Right I forgot that some parts are already in.
>> So we will need an extra patch to handle those.
> 
> I think I've addressed all comments from Julien regarding my series,
> so I think I've got some bandwidth to do the clean-up patch tomorrow
> after the agreement, unless someone would like to do it himself?

This is very nice of you.
We need to confirm first if something is actually needed or not so we need 
Stefano’s view here.
If so then please do it :-)

Cheers
Bertrand

> 
> Kind regards,
> Henry
> 
>> 
>> Bertrand
>> 
>>> 
>>>> Cheers
>>>> Bertrand
>>>> 
>>>>> 
>>>>>> 
>>>>>> Cheers
>>>>>> Bertrand
>>>>>> 
>>>>>>> 
>>>>>>>> 
>>>>>>>> Cheers,
>>>>>>>> 
>>>>>>>> --
>>>>>>>> Julien Grall