Hi, On Tue, 22 Jan 2013 21:44:06 +0100 Jeroen Hofstee <jer...@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 <ma...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot