[U-Boot] [PATCH 00/10] common.lcd: remove globals and cleanup

Jeroen Hofstee jeroen at myspectrum.nl
Sat Jan 12 23:07:49 CET 2013


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"

Jeroen Hofstee (10):
  lcd, amba: remove this frame buffer driver since it is not used
  lcd, tegra: remove unused cursor functions
  lcd, mpc8xx: move the mpc8xx frame buffer 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: cleanup, remove prototypes already in lcd.h 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/arm/cpu/pxa/pxafb.c         |  650 --------------------------------------
 arch/powerpc/cpu/mpc8xx/Makefile |    1 -
 arch/powerpc/cpu/mpc8xx/lcd.c    |  618 ------------------------------------
 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 +-
 drivers/video/mpc8xx_lcd.c       |  564 +++++++++++++++++++++++++++++++++
 drivers/video/pxa_lcd.c          |  609 +++++++++++++++++++++++++++++++++++
 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 +----
 30 files changed, 1280 insertions(+), 1567 deletions(-)
 delete mode 100644 arch/arm/cpu/pxa/pxafb.c
 delete mode 100644 arch/powerpc/cpu/mpc8xx/lcd.c
 delete mode 100644 drivers/video/amba.c
 create mode 100644 drivers/video/mpc8xx_lcd.c
 create mode 100644 drivers/video/pxa_lcd.c

-- 
1.7.9.5



More information about the U-Boot mailing list