* [PATCH] graphics_utils: Let fb_open allocate the screen
@ 2015-07-13 6:17 Sascha Hauer
0 siblings, 0 replies; only message in thread
From: Sascha Hauer @ 2015-07-13 6:17 UTC (permalink / raw)
To: Barebox List
Allocate the screen dynamically in fb_open. This opens the way to create
a fb_create_screen function which takes a struct fb_info * instead of a
filename. This is suitable for the framebuffer console which already has
a struct fb_info *.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
commands/splash.c | 27 ++++++++---------
include/gui/graphic_utils.h | 3 +-
include/gui/gui.h | 2 +-
lib/gui/graphic_utils.c | 73 +++++++++++++++++++++++++++++----------------
4 files changed, 63 insertions(+), 42 deletions(-)
diff --git a/commands/splash.c b/commands/splash.c
index 04562e3..90f0a0c 100644
--- a/commands/splash.c
+++ b/commands/splash.c
@@ -10,9 +10,9 @@
static int do_splash(int argc, char *argv[])
{
struct surface s;
- struct screen sc;
+ struct screen *sc;
int ret = 0;
- int opt, fd;
+ int opt;
char *fbdev = "/dev/fb0";
char *image_file;
int offscreen = 0;
@@ -20,7 +20,6 @@ static int do_splash(int argc, char *argv[])
bool do_bg = false;
memset(&s, 0, sizeof(s));
- memset(&sc, 0, sizeof(sc));
s.x = -1;
s.y = -1;
@@ -53,29 +52,29 @@ static int do_splash(int argc, char *argv[])
}
image_file = argv[optind];
- fd = fb_open(fbdev, &sc, offscreen);
- if (fd < 0) {
+ sc = fb_open(fbdev, offscreen);
+ if (IS_ERR(sc)) {
perror("fd_open");
- return fd;
+ return PTR_ERR(sc);
}
- if (sc.offscreenbuf) {
+ if (sc->offscreenbuf) {
if (do_bg)
- gu_memset_pixel(&sc.info, sc.offscreenbuf, bg_color,
- sc.s.width * sc.s.height);
+ gu_memset_pixel(sc->info, sc->offscreenbuf, bg_color,
+ sc->s.width * sc->s.height);
else
- memcpy(sc.offscreenbuf, sc.fb, sc.fbsize);
+ memcpy(sc->offscreenbuf, sc->fb, sc->fbsize);
} else if (do_bg) {
- gu_memset_pixel(&sc.info, sc.fb, bg_color, sc.s.width * sc.s.height);
+ gu_memset_pixel(sc->info, sc->fb, bg_color, sc->s.width * sc->s.height);
}
- ret = image_renderer_file(&sc, &s, image_file);
+ ret = image_renderer_file(sc, &s, image_file);
if (ret > 0)
ret = 0;
- gu_screen_blit(&sc);
+ gu_screen_blit(sc);
- fb_close(&sc);
+ fb_close(sc);
return ret;
}
diff --git a/include/gui/graphic_utils.h b/include/gui/graphic_utils.h
index 5c5c40a..ea33f70 100644
--- a/include/gui/graphic_utils.h
+++ b/include/gui/graphic_utils.h
@@ -17,7 +17,8 @@ void gu_set_pixel(struct fb_info *info, void *adr, u32 px);
void gu_set_rgb_pixel(struct fb_info *info, void *adr, u8 r, u8 g, u8 b);
void gu_set_rgba_pixel(struct fb_info *info, void *adr, u8 r, u8 g, u8 b, u8 a);
void gu_memset_pixel(struct fb_info *info, void* buf, u32 color, size_t size);
-int fb_open(const char * fbdev, struct screen *sc, bool offscreen);
+struct screen *fb_create_screen(struct fb_info *info, bool offscreen);
+struct screen *fb_open(const char *fbdev, bool offscreen);
void fb_close(struct screen *sc);
void gu_screen_blit(struct screen *sc);
void gu_invert_area(struct fb_info *info, void *buf, int startx, int starty, int width,
diff --git a/include/gui/gui.h b/include/gui/gui.h
index 59ff590..03e60aa 100644
--- a/include/gui/gui.h
+++ b/include/gui/gui.h
@@ -18,7 +18,7 @@ struct surface {
struct screen {
int fd;
- struct fb_info info;
+ struct fb_info *info;
struct surface s;
diff --git a/lib/gui/graphic_utils.c b/lib/gui/graphic_utils.c
index 0d985a6..df318d8 100644
--- a/lib/gui/graphic_utils.c
+++ b/lib/gui/graphic_utils.c
@@ -215,51 +215,72 @@ void gu_rgba_blend(struct fb_info *info, struct image *img, void* buf, int heigh
}
}
-int fb_open(const char * fbdev, struct screen *sc, bool offscreen)
+struct screen *fb_create_screen(struct fb_info *info, bool offscreen)
{
- int ret;
+ struct screen *sc;
- sc->fd = open(fbdev, O_RDWR);
- if (sc->fd < 0)
- return sc->fd;
-
- sc->fb = memmap(sc->fd, PROT_READ | PROT_WRITE);
- if (sc->fb == (void *)-1) {
- ret = -ENOMEM;
- goto failed_memmap;
- }
-
- ret = ioctl(sc->fd, FBIOGET_SCREENINFO, &sc->info);
- if (ret) {
- goto failed_memmap;
- }
+ sc = xzalloc(sizeof(*sc));
sc->s.x = 0;
sc->s.y = 0;
- sc->s.width = sc->info.xres;
- sc->s.height = sc->info.yres;
- sc->fbsize = sc->info.line_length * sc->s.height;
+ sc->s.width = info->xres;
+ sc->s.height = info->yres;
+ sc->fbsize = info->line_length * sc->s.height;
+ sc->fb = info->screen_base;
if (offscreen) {
- /* Don't fail if malloc fails, just continue rendering directly
+ /*
+ * Don't fail if malloc fails, just continue rendering directly
* on the framebuffer
*/
sc->offscreenbuf = malloc(sc->fbsize);
}
- return sc->fd;
+ return sc;
+}
+
+struct screen *fb_open(const char * fbdev, bool offscreen)
+{
+ int fd, ret;
+ struct fb_info info;
+ struct screen *sc;
-failed_memmap:
- sc->fb = NULL;
- close(sc->fd);
+ fd = open(fbdev, O_RDWR);
+ if (fd < 0)
+ return ERR_PTR(fd);
- return ret;
+ ret = ioctl(fd, FBIOGET_SCREENINFO, &info);
+ if (ret) {
+ goto failed_screeninfo;
+ }
+
+ sc = fb_create_screen(&info, offscreen);
+ if (IS_ERR(sc)) {
+ ret = PTR_ERR(sc);
+ goto failed_create;
+ }
+
+ sc->fd = fd;
+
+ return sc;
+
+failed_create:
+ free(sc->offscreenbuf);
+ free(sc);
+failed_screeninfo:
+ close(fd);
+
+ return ERR_PTR(ret);
}
void fb_close(struct screen *sc)
{
free(sc->offscreenbuf);
- close(sc->fd);
+
+ if (sc->fd > 0)
+ close(sc->fd);
+
+ free(sc);
}
void gu_screen_blit(struct screen *sc)
--
2.1.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2015-07-13 6:18 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-13 6:17 [PATCH] graphics_utils: Let fb_open allocate the screen Sascha Hauer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox