mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Antony Pavlov <antonynpavlov@gmail.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH] fixup! WIP: video: implement quick-n-dirty framebuffer console
Date: Mon, 13 Jul 2015 08:26:48 +0200	[thread overview]
Message-ID: <20150713062648.GD18700@pengutronix.de> (raw)
In-Reply-To: <1436695632-20270-1-git-send-email-antonynpavlov@gmail.com>

Hi Antony,

On Sun, Jul 12, 2015 at 01:07:12PM +0300, Antony Pavlov wrote:
> ---
> Sascha moved fbconsole registration to common fb code.
> So we can drop this supefluous code.
> ---
>  drivers/video/sdl.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)

Squashed this into the fbconsole patch. Also I applied the patch I just
sent to the list which adds a fb_create_screen function which creates
a screen from a struct fb_info *. With this we get rid of the hardcoded
opening of /dev/fb0.

What's missing now is that fbc_set_active() should check the flags
argument and actually disable the fbconsole and free the resources.

Also I'm not sure if the framebuffer should be activated when the
framebuffer console is activated. While this is convenient for the
user we get into a reference counting problem. What if the framebuffer
was activated before the fbconsole? Should we skip enabling the
framebuffer in this case? When disabling the fbconsole then we must
now if the fbconsole code activated the framebuffer and only then
disable the framebuffer again.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

      reply	other threads:[~2015-07-13  6:27 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-12 10:07 Antony Pavlov
2015-07-13  6:26 ` Sascha Hauer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150713062648.GD18700@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=antonynpavlov@gmail.com \
    --cc=barebox@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox