Hi Patrick
On 11/15/21 4:32 PM, Patrick Delaunay wrote: > Add a new function to activate an existing simple frame buffer node > and add the associated reserved memory, with no-map properties. > > This device tree update is only done when the video device is active > and the video buffer is used. > > This patch uses '#if CONFIG_IS_ENABLED(DM_VIDEO)' because > gd->video_bottom and gd->video_top are only defined when CONFIG_DM_VIDEO > is activated. > > Signed-off-by: Patrick Delaunay <[email protected]> > --- > > common/fdt_simplefb.c | 27 +++++++++++++++++++++++++++ > include/fdt_simplefb.h | 1 + > 2 files changed, 28 insertions(+) > > diff --git a/common/fdt_simplefb.c b/common/fdt_simplefb.c > index 32173030ab..c52846f4bc 100644 > --- a/common/fdt_simplefb.c > +++ b/common/fdt_simplefb.c > @@ -89,3 +89,30 @@ int fdt_simplefb_enable_existing_node(void *blob) > > return fdt_simplefb_configure_node(blob, off); > } > + > +#if CONFIG_IS_ENABLED(DM_VIDEO) > +int fdt_simplefb_enable_and_mem_rsv(void *blob) > +{ > + struct fdt_memory mem; > + int ret; > + > + /* nothing to do when video is not active */ > + if (!video_is_active()) > + return 0; > + > + ret = fdt_simplefb_enable_existing_node(blob); > + if (ret) > + return ret; > + > + /* nothing to do when the frame buffer is not defined */ > + if (gd->video_bottom == gd->video_top) > + return 0; > + > + /* reserved with no-map tag the video buffer */ > + mem.start = gd->video_bottom; > + mem.end = gd->video_top - 1; > + > + return fdtdec_add_reserved_memory(blob, "framebuffer", &mem, NULL, 0, > NULL, > + FDTDEC_RESERVED_MEMORY_NO_MAP); > +} > +#endif > diff --git a/include/fdt_simplefb.h b/include/fdt_simplefb.h > index 7e54723591..41cd740ac0 100644 > --- a/include/fdt_simplefb.h > +++ b/include/fdt_simplefb.h > @@ -10,4 +10,5 @@ > #define _FDT_SIMPLEFB_H_ > int fdt_simplefb_add_node(void *blob); > int fdt_simplefb_enable_existing_node(void *blob); > +int fdt_simplefb_enable_and_mem_rsv(void *blob); > #endif > Reviewed-by: Patrice Chotard <[email protected]> Thanks Patrice
