From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexandre Rusev Subject: Re: [PATCH 0/8] pxafb cleanup Date: Wed, 27 Feb 2008 11:58:16 +0300 Message-ID: <47C52628.8010005@ru.mvista.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.arm.linux.org.uk Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org+linux-arm-kernel=m.gmane.org@lists.arm.linux.org.uk To: eric miao Cc: linux-fbdev-devel@lists.sourceforge.net, linux-kernel eric miao wrote: > The following series of patches try to address several issues of the > current PXA framebuffer driver. Some of these patches are already > submitted for review, here are more: > > pxa: un-nest pxafb_parse_options() to cleanup the coding style issue > pxa: fix various coding style issues for pxafb > pxa: purge unnecessary pr_debug and comments from pxafb > pxa: sanitize the usage of #ifdef .. processing pxafb parameters > pxa: convert fb driver to use ioremap() and __raw_{readl, writel} > pxa: introduce "struct pxafb_dma_buff" for palette and dma descriptors > pxa: introduce register independent LCD connection type for pxafb > pxa: make lubbock/mainstone/zylonite/littleton to use new LCD connection type > > Board maintainers are encouraged to use the new way for LCD connection > types. > > To further clean-up the driver, I suggest to make the following changes: > 1. removal of set_ctrlr_state(), the original code is written so because > the fb_blank() is called within interrupt context in 2.4 kernel, but this is > no longer the case in 2.6. And states can be simplified. > > 2. introduce a "vram" module parameter or boot option to indicate the > size of the video memory, so that frame buffer offset can be specified > within this video memory range, and PAN_DISPLAY can be done. > And the start offset of each overlay can also be specified. > > 3. a "pxafb_layer" structure describing each possible layers, > base, overlay1 and overlay2 for pxa27x can be described. From this > But what about support of DirectFB applications? DirectFB as we know needs framebuffer memory to be allocated and mapped only once taking in account the greatest values of colordeepth and resolution. Furthermore for support of overlays DirectFB needs a capability to map all overlays and baseplane to be mapped continuously in memory (taking in account greatest bpp and resolution values again) I'd prefer to see DirectFB fixed against this, yet the fulfillment of these a bit strange rules looks like to be the only way to make it possible the DirectFB to work with pxafb driver :( > layer information, frame buffer device can be dynamically allocated > and registered. Palette manipulation, parameter checking and > layer activation code can be generalized. > > ------------------------------------------------------------------- List admin: http://lists.arm.linux.org.uk/mailman/listinfo/linux-arm-kernel FAQ: http://www.arm.linux.org.uk/mailinglists/faq.php Etiquette: http://www.arm.linux.org.uk/mailinglists/etiquette.php