All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2 3/4] MX51: Add video support
Date: Fri, 22 Oct 2010 06:59:30 +0200	[thread overview]
Message-ID: <4CC11A32.6080001@denx.de> (raw)
In-Reply-To: <AANLkTi=Wjcd7MvPPOfTNUVUBjiuO6-U-9Ya0aXbqDwb=@mail.gmail.com>

On 10/22/2010 05:54 AM, Liu Ying wrote:
> Hello, Stefano,

Hi Liu,

> 
> It looks that your framebuffer patch supports one framebuffer only on IPU DI0.

That is correct. As I stated in the commit message, the goal was to add
LCD support to allow to display a splashscreen in u-boot. On the board I
used to test, LCD is connected to DI0.

> Could you please consider to add the support for framebuffer on IPU
> DI1, as someone may design a board whose primary display device is
> connected with IPU DI1.

Well, now the driver is in mainline. Patches are welcome !

> And, it will be better if the following two points can be supported:
> 1) Two framebuffers on IPU DI0 and DI1 can be support at the same
> time, that is to say, users may see two pictures on two display
> devices respectively.

I understand this is a feature for Linux, but in u-boot ? Two
splashscreens at the same time ? At the moment I do not know a board in
u-boot managing two display and I do not see a reason.

> 2) IPU DP module supports to combine two layers. The overlay layer can
> have a smaller resolution than the bottom layer. And, color space
> conversion, color key, global alpha and local alpha are supported by
> DP, too. I 'm not sure if these features are really needed in u-boot
> or not.

All these features were dropped because they are not used in u-boot. We
have no GUI programming, and only a small subset of commands to manage
the display. I understand they are a bunch of important features in
Linux, but in my understanding we need in u-boot only to show more or
less fixed images.

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

  reply	other threads:[~2010-10-22  4:59 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <ActxlXt7tSZQfD4NQ92G+5+Fd5jtsgAAUkCg>
     [not found] ` <7629BC1F9408B943B87E2821E11D7E02120F13@zch01exm24.fsl.freescale.net>
2010-10-22  3:54   ` [U-Boot] [PATCH V2 3/4] MX51: Add video support Liu Ying
2010-10-22  4:59     ` Stefano Babic [this message]
2010-10-12  9:25 [U-Boot] [PATCH " Stefano Babic
2010-10-13 10:17 ` [U-Boot] [PATCH V2 " Stefano Babic
2010-10-21 22:18   ` Anatolij Gustschin

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=4CC11A32.6080001@denx.de \
    --to=sbabic@denx.de \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.