Re: [PATCH] fbdev: simplefb: add support for 'memory-region' property on DT node

From: Rob Herring
Date: Mon Feb 05 2018 - 01:12:05 EST


On Thu, Feb 01, 2018 at 04:56:08PM +0100, Bartlomiej Zolnierkiewicz wrote:
>
> Hi,
>
> On Tuesday, January 23, 2018 08:34:56 PM Kunihiko Hayashi wrote:
> > Enables 'memory-region' property referring to the memory description on
> > the reserved-memory node in case of devicetree use.
> > If there is no 'reg' property that specifies the address and size of
> > the framebuffer, the address and size written in the memory description
> > on the reserved-memory node can be used for the framebuffer.
> >
> > Furthermore, the reserved-memory node needs to have "no-map" attributes
> > because simplefb driver maps the region by ioremap_wc().
> >
> > Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@xxxxxxxxxxxxx>
>
> This needs an ACK from Rob or Mark (DT bindings Maintainers).
>
> > ---
> > .../bindings/display/simple-framebuffer.txt | 3 ++
> > drivers/video/fbdev/simplefb.c | 32 ++++++++++++++++++++++
> > 2 files changed, 35 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/display/simple-framebuffer.txt b/Documentation/devicetree/bindings/display/simple-framebuffer.txt
> > index 5a9ce51..be5139f 100644
> > --- a/Documentation/devicetree/bindings/display/simple-framebuffer.txt
> > +++ b/Documentation/devicetree/bindings/display/simple-framebuffer.txt
> > @@ -56,6 +56,9 @@ Optional properties:
> > framebuffer remains active.
> >
> > - display : phandle pointing to the primary display hardware node
> > +- memory-region: phandle to a node describing memory region as framebuffer
> > + memory instead of reg property. The node should include
> > + 'no-map'.

This should also state when it's appropriate to use this instead of reg.
The memory would only be reclaimed if reg is used.

Though I'm wondering what keeps the simple fb memory from getting used
by the OS if reserved memory is not always used.

Rob