[U-Boot] [PATCH v2 00/10] common.lcd: remove globals and cleanup
Anatolij Gustschin
agust at denx.de
Fri Mar 29 12:03:13 CET 2013
Hi,
On Tue, 22 Jan 2013 21:44:06 +0100
Jeroen Hofstee <jeroen at myspectrum.nl> wrote:
> As mentioned in http://lists.denx.de/pipermail/u-boot/2013-January/143459.html
> there does not seem to be a good reason why the frame buffer driver
> provide the variables for the lcd internal logic. So lets just remove it.
> If there is any good reason for a framebuffer driver to "call up" it should
> do so by a function call and not change the lcd its internal state (and
> there is actually none doing so at the moment).
>
> This patchset first removes unused drivers and places drivers useing
> common/lcd.c in drivers/video. Then removes the unneeded globals and
> thereafter finishes with some cleanups.
>
> This patch goes after http://patchwork.ozlabs.org/patch/209692/
> "common/lcd.c: cleanup use of global variables"
>
> Version 2:
> Marek Vasut <marex at denx.de>: use format-patch -M to detect moves
> Adjust the commit message not to exceed the max line length
> Fix malformed email address.
> check-patch clean (adjusted a space in patch 6)
> remove bouncing email addresses and add acks
> patch 7 and 10 are unchanged and not reposted
>
> Jeroen Hofstee (10):
> lcd, amba: remove this driver since it is not used
> lcd, tegra: remove unused cursor functions
> lcd, mpc8xx: move the mpc8xx driver to drivers/video
> lcd, pxafb: move the pxafb to drivers/video
> common/lcd.c: cleanup use of global variables
> common/lcd.c: remove global lcd_base
> common/lcd: cosmetic: clean up a bit
> lcd, fb: remove duplicated prototypes and unused code
> api/api_display: use the getters for console size info
> common/lcd.c: move the macro's to the c file
>
> api/api_display.c | 4 +-
> arch/arm/cpu/pxa/Makefile | 1 -
> arch/powerpc/cpu/mpc8xx/Makefile | 1 -
> board/mcc200/lcd.c | 21 +---
> common/lcd.c | 117 +++++++++++---------
> drivers/video/Makefile | 3 +-
> drivers/video/amba.c | 77 -------------
> drivers/video/atmel_hlcdfb.c | 8 --
> drivers/video/atmel_lcdfb.c | 8 --
> drivers/video/exynos_fb.c | 14 +--
> .../cpu/mpc8xx/lcd.c => drivers/video/mpc8xx_lcd.c | 58 +---------
> .../arm/cpu/pxa/pxafb.c => drivers/video/pxa_lcd.c | 41 -------
> drivers/video/tegra.c | 64 +----------
> include/configs/R360MPI.h | 1 +
> include/configs/RBC823.h | 1 +
> include/configs/RPXlite_DW.h | 1 +
> include/configs/RRvision.h | 4 +-
> include/configs/TQM823L.h | 1 +
> include/configs/TQM823M.h | 1 +
> include/configs/lubbock.h | 1 +
> include/configs/lwmon.h | 1 +
> include/configs/palmld.h | 1 +
> include/configs/palmtc.h | 1 +
> include/configs/pxa255_idp.h | 1 +
> include/configs/svm_sc8xx.h | 1 +
> include/configs/v37.h | 1 +
> include/configs/zipitz2.h | 1 +
> include/lcd.h | 71 +++---------
> 28 files changed, 109 insertions(+), 396 deletions(-)
> delete mode 100644 drivers/video/amba.c
> rename arch/powerpc/cpu/mpc8xx/lcd.c => drivers/video/mpc8xx_lcd.c (92%)
> rename arch/arm/cpu/pxa/pxafb.c => drivers/video/pxa_lcd.c (93%)
Merged this series after rebasing, also merged patches
7 and 10 from previous series. Thanks!
Anatolij
More information about the U-Boot
mailing list