From: Wang Huan-B18965 <B18965@freescale.com>
To: linux-arm-kernel@lists.infradead.org
Subject: RE: [PATCH 4/5] fb: Add DCU framebuffer driver for Vybrid VF610 platform
Date: Tue, 06 Aug 2013 07:20:58 +0000 [thread overview]
Message-ID: <81BA6E5E0BC2344391CABCEE22D1B6D83CEAA6@039-SN1MPN1-002.039d.mgd.msft.net> (raw)
In-Reply-To: <1375712322.4276.12.camel@weser.hi.pengutronix.de>
SGksDQoNCj4gQW0gTW9udGFnLCBkZW4gMDUuMDguMjAxMywgMTU6MDkgKzAyMDAgc2NocmllYiBS
b2JlcnQgU2Nod2ViZWw6DQo+ID4gT24gTW9uLCBBdWcgMDUsIDIwMTMgYXQgMDk6NTE6NDBBTSAr
MDAwMCwgV2FuZyBIdWFuLUIxODk2NSB3cm90ZToNCj4gPiA+ID4gT24gRnJpLCBKdWwgMTIsIDIw
MTMgYXQgMDI6MDc6NTVQTSArMDgwMCwgQWxpc29uIFdhbmcgd3JvdGU6DQo+ID4gPiA+ID4gVGhl
IERpc3BsYXkgQ29udHJvbGxlciBVbml0IChEQ1UpIG1vZHVsZSBpcyBhIHN5c3RlbSBtYXN0ZXIN
Cj4gdGhhdA0KPiA+ID4gPiA+IGZldGNoZXMgZ3JhcGhpY3Mgc3RvcmVkIGluIGludGVybmFsIG9y
IGV4dGVybmFsIG1lbW9yeSBhbmQNCj4gPiA+ID4gPiBkaXNwbGF5cyB0aGVtIG9uIGEgVEZUIExD
RCBwYW5lbC4gQSB3aWRlIHJhbmdlIG9mIHBhbmVsIHNpemVzDQo+IGlzDQo+ID4gPiA+ID4gc3Vw
cG9ydGVkIGFuZCB0aGUgdGltaW5nIG9mIHRoZSBpbnRlcmZhY2Ugc2lnbmFscyBpcyBoaWdobHkN
Cj4gY29uZmlndXJhYmxlLg0KPiA+ID4gPiA+IEdyYXBoaWNzIGFyZSByZWFkIGRpcmVjdGx5IGZy
b20gbWVtb3J5IGFuZCB0aGVuIGJsZW5kZWQgaW4NCj4gPiA+ID4gPiByZWFsLXRpbWUsIHdoaWNo
IGFsbG93cyBmb3IgZHluYW1pYyBjb250ZW50IGNyZWF0aW9uIHdpdGgNCj4gPiA+ID4gPiBtaW5p
bWFsIENQVQ0KPiA+ID4gPiBpbnRlcnZlbnRpb24uDQo+ID4gPiA+DQo+ID4gPiA+IE9ubHkgYSBy
ZXZpZXcgb2YgdGhlIGNvZGUgaW5saW5lLg0KPiA+ID4gPg0KPiA+ID4gPiBNYXliZSB0aGUgcmVh
bCBxdWVzdGlvbiBpcyB3aGV0aGVyIHdlIHdhbnQgdG8gaW50cm9kdWNlIGFub3RoZXINCj4gPiA+
ID4gZnJhbWVidWZmZXIgZHJpdmVyIGF0IGFsbCBpbnN0ZWFkIG9mIG1ha2luZyBpdCBhIERSTSBk
cml2ZXIuDQo+ID4gPiBbQWxpc29uIFdhbmddIEkgdGhpbmsgRENVIG1vZHVsZSBpcyBtb3JlIHN1
aXRhYmxlIHRvIGJlIGRlc2lnbmVkIGFzDQo+IGEgZnJhbWVidWZmZXIgZHJpdmVyIHRoYW4gYSBE
Uk0gZHJpdmVyLiBKdXN0IGxpa2UgRElVIGZyYW1lYnVmZmVyDQo+IGRyaXZlciBmb3IgUG93ZXJQ
Qy4NCj4gPg0KPiA+IFdlIGxvb2tlZCBhdCB0aGUgVnlicmlkIGRhdGFzaGVldCBhbmQgaXQncyBE
Q1Ugc2VjdGlvbiBsYXN0IHdlZWssIGFuZA0KPiA+IHdpdGggaXRzIDY0IHBsYW5lcywgdGhlIGNv
bnRyb2xsZXIgcmVhbGx5IHdhbnRzIHRvIGdldCBhIERSTSBkcml2ZXIuDQo+ID4NCj4gRXhhY3Rs
eSwgd2l0aCBpdCdzIGV4dGVuc2l2ZSBoYXJkd2FyZSBjb21wb3NpdGlvbiBjYXBhYmlsaXRpZXMg
dGhlDQo+IGNvbnRyb2xsZXIgYmVncyBmb3IgYmVpbmcgZHJpdmVuIGJ5IGEgcHJvcGVyIERSTSBk
cml2ZXIuIFRoZXJlIGlzIG5vDQo+IHdheSBpbiBmYmRldiB0byBwcm9wZXJseSBzdXBwb3J0IGFs
bCB0aG9zZSBoYXJkd2FyZSBwbGFuZXMgd2l0aG91dA0KPiBpbnRyb2R1Y2luZyBhIGxvdCBvZiBu
b24gc3RhbmRhcmQgaW9jdGxzLCB3aGljaCBpbiB0dXJuIHdpbGwgZm9yY2UgeW91DQo+IGludG8g
d3JpdGluZyBhIGxvdCBvZiBjdXN0b20gdXNlcnNwYWNlIGNvZGUgaW5zdGVhZCBvZiBydW5uaW5n
IHByb3Zlbg0KPiB0ZWNobm9sb2d5IHRoYXQgc2l0cyBvbiB0b3Agb2YgS01TLg0KPiANCj4gRG9p
bmcgdGhpbmdzIGluIERSTSBtaWdodCBiZSBzbGlnaHRseSBtb3JlIHdvcmsgZm9yIHRoZSBpbml0
aWFsIGJyaW5nDQo+IHVwLCBidXQgd2lsbCBwYXkgb2ZmIGluIHRoZSBsb25nIHJ1biB3aGVuIHlv
dSBhcmUgZ29pbmcgdG8gcmVhbGx5IHVzZQ0KPiB0aGUgaGFyZHdhcmUgY2FwcyBvZiB0aGUgY29u
dHJvbGxlci4NCj4gDQpbQWxpc29uIFdhbmddIFRoYW5rcyBmb3IgeW91ciBjb21tZW50cyBhbmQg
ZXhwbGFuYXRpb25zLiBJIHVuZGVyc3RhbmQgeW91ciBjb25zaWRlcmF0aW9ucy4NCg0KSW4gRENV
IG1vZHVsZSwgdGhlcmUgYXJlIDY0IGdyYXBoaWNzIGxheWVycywgYnV0IHRoZSBtYXhpbXVtIHNv
dXJjZSBsYXllcnMgZm9yIGJsZW5kaW5nIG9mIGVhY2ggcGl4ZWwgaXMgb25seSA2LiBUaGUgZmIg
ZHJpdmVyIGNvdWxkIHNhdGlzZnkgdGhlIGN1cnJlbnQgYXBwbGljYXRpb24gcmVxdWlyZW1lbnRz
Lg0KDQpTbyBJIHdpbGwgcHVzaCBvdXQgdGhpcyBmYiBkcml2ZXIgZmlyc3QsIGFuZCB0aGVuIEkg
d2lsbCBhZGQgYSBEUk0gZHJpdmVyIGZvciBtb3JlIGFkdmFuY2VkIHJlcXVpcmVtZW50cy4NCg0K
RG8geW91IGhhdmUgbW9yZSBjb21tZW50cz8NCg0KDQpCZXN0IFJlZ2FyZHMsDQpBbGlzb24gV2Fu
Zw0K
WARNING: multiple messages have this Message-ID (diff)
From: B18965@freescale.com (Wang Huan-B18965)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/5] fb: Add DCU framebuffer driver for Vybrid VF610 platform
Date: Tue, 6 Aug 2013 07:20:58 +0000 [thread overview]
Message-ID: <81BA6E5E0BC2344391CABCEE22D1B6D83CEAA6@039-SN1MPN1-002.039d.mgd.msft.net> (raw)
In-Reply-To: <1375712322.4276.12.camel@weser.hi.pengutronix.de>
Hi,
> Am Montag, den 05.08.2013, 15:09 +0200 schrieb Robert Schwebel:
> > On Mon, Aug 05, 2013 at 09:51:40AM +0000, Wang Huan-B18965 wrote:
> > > > On Fri, Jul 12, 2013 at 02:07:55PM +0800, Alison Wang wrote:
> > > > > The Display Controller Unit (DCU) module is a system master
> that
> > > > > fetches graphics stored in internal or external memory and
> > > > > displays them on a TFT LCD panel. A wide range of panel sizes
> is
> > > > > supported and the timing of the interface signals is highly
> configurable.
> > > > > Graphics are read directly from memory and then blended in
> > > > > real-time, which allows for dynamic content creation with
> > > > > minimal CPU
> > > > intervention.
> > > >
> > > > Only a review of the code inline.
> > > >
> > > > Maybe the real question is whether we want to introduce another
> > > > framebuffer driver at all instead of making it a DRM driver.
> > > [Alison Wang] I think DCU module is more suitable to be designed as
> a framebuffer driver than a DRM driver. Just like DIU framebuffer
> driver for PowerPC.
> >
> > We looked at the Vybrid datasheet and it's DCU section last week, and
> > with its 64 planes, the controller really wants to get a DRM driver.
> >
> Exactly, with it's extensive hardware composition capabilities the
> controller begs for being driven by a proper DRM driver. There is no
> way in fbdev to properly support all those hardware planes without
> introducing a lot of non standard ioctls, which in turn will force you
> into writing a lot of custom userspace code instead of running proven
> technology that sits on top of KMS.
>
> Doing things in DRM might be slightly more work for the initial bring
> up, but will pay off in the long run when you are going to really use
> the hardware caps of the controller.
>
[Alison Wang] Thanks for your comments and explanations. I understand your considerations.
In DCU module, there are 64 graphics layers, but the maximum source layers for blending of each pixel is only 6. The fb driver could satisfy the current application requirements.
So I will push out this fb driver first, and then I will add a DRM driver for more advanced requirements.
Do you have more comments?
Best Regards,
Alison Wang
next prev parent reply other threads:[~2013-08-06 7:20 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-12 6:07 [PATCH v2 0/5] ARM: vf610: Add DCU framebuffer driver for Vybrid VF610 platform Alison Wang
2013-07-12 6:07 ` Alison Wang
2013-07-12 6:07 ` [PATCH 1/5] ARM: dts: vf610: Add DCU and TCON nodes Alison Wang
2013-07-12 6:07 ` Alison Wang
2013-07-12 6:07 ` [PATCH 2/5] ARM: dts: vf610-twr: Enable DCU and TCON devices Alison Wang
2013-07-12 6:07 ` Alison Wang
2013-07-12 6:07 ` [PATCH 3/5] ARM: clk: vf610: Add DCU and TCON clock support Alison Wang
2013-07-12 6:07 ` Alison Wang
2013-07-12 6:07 ` [PATCH 4/5] fb: Add DCU framebuffer driver for Vybrid VF610 platform Alison Wang
2013-07-12 6:07 ` Alison Wang
2013-07-29 11:14 ` Sascha Hauer
2013-07-29 11:14 ` Sascha Hauer
2013-08-05 9:51 ` Wang Huan-B18965
2013-08-05 9:51 ` Wang Huan-B18965
2013-08-05 10:06 ` Sascha Hauer
2013-08-05 10:06 ` Sascha Hauer
2013-08-06 3:42 ` Wang Huan-B18965
2013-08-06 3:42 ` Wang Huan-B18965
2013-08-05 13:09 ` Robert Schwebel
2013-08-05 13:09 ` Robert Schwebel
2013-08-05 14:18 ` Lucas Stach
2013-08-05 14:18 ` Lucas Stach
2013-08-06 7:20 ` Wang Huan-B18965 [this message]
2013-08-06 7:20 ` Wang Huan-B18965
2014-01-06 18:50 ` Bill Pringlemeir
2014-01-07 9:18 ` Lucas Stach
2014-01-07 20:52 ` Bill Pringlemeir
2014-01-07 21:00 ` Bill Pringlemeir
2013-08-06 8:47 ` Lucas Stach
2013-08-06 8:47 ` Lucas Stach
2013-08-07 8:07 ` Wang Huan-B18965
2013-08-07 8:07 ` Wang Huan-B18965
2013-07-12 6:07 ` [PATCH 5/5] Documentation: DT: Add DCU framebuffer driver Alison Wang
2013-07-12 6:07 ` Alison Wang
2013-07-19 3:49 ` [PATCH v2 0/5] ARM: vf610: Add DCU framebuffer driver for Vybrid VF610 platform Wang Huan-B18965
2013-07-19 3:49 ` Wang Huan-B18965
2013-07-29 6:13 ` Wang Huan-B18965
2013-07-29 6:13 ` Wang Huan-B18965
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=81BA6E5E0BC2344391CABCEE22D1B6D83CEAA6@039-SN1MPN1-002.039d.mgd.msft.net \
--to=b18965@freescale.com \
--cc=linux-arm-kernel@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 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.