Hi Stefano, > -----Original Message----- > From: Stefano Stabellini <sstabell...@kernel.org> > Subject: Re: [PATCH 1/2] docs, xen/arm: Introduce reserved heap memory > > On Wed, 24 Aug 2022, Henry Wang wrote: > > --- > > The name of the device tree property was chosen because we want to > > reuse as much as the device tree parsing helpers from the static > > memory allocation feature, but we would like to hear the upstream > > reviewers' opinion about if using "xen,static-heap" is better. > > --- > > +The dtb property should look like as follows: > > + > > +- property name > > + > > + "xen,static-mem" (Should be used in the `chosen` node) > > + > > +- cells > > + > > + Specify the start address and the length of the reserved heap memory. > > + The number of cells for the address and the size should be defined > > + using the properties `#xen,static-mem-address-cells` and > > + `#xen,static-mem-size-cells` respectively. > > I would choose a different name for the property not to be confused with > a domain's xen,static-mem property which is for a different purpose: the > memory of the domain.
Sure, thank you for the input. I will correct these in v2. > > --- > > - xen,static-heap > > Property under the top-level "chosen" node. It specifies the address > and size of Xen reserved heap memory. > > > - #xen,static-heap-address-cells and #xen,static-heap-size-cells > > Specify the number of cells used for the address and size of the > xen,static-heap property under "chosen". Thanks for these! Kind regards, Henry
