[U-Boot] [PATCH v4 00/12] Introduce Samsung misc file and LCD menu.

[Przemyslaw Marczak]

Hello Hyungwon,

On 01/10/2014 07:45 AM, Hyungwon Hwang wrote:
> Hi Marczak,
>
> On Thu, 09 Jan 2014 12:23:04 +0100
> Przemyslaw Marczak <p.marczak at samsung.com> wrote:
>
>> This patch set includes changes required to:
>> - properly use of all gpios
>> - introduce common file for Samsung misc code
>> - keys support (PWR, VOL:UP,DOWN)
>> - console support on LCD
>> - 16bpp logo support
>> - introduce LCD menu on Samsung devices
>>
>> Each version changes are described in each patch commit msg.
>>
>> Przemyslaw Marczak (12):
>>    s5p: gpio: change gpio coding method for s5p gpio.
>>    trats2: Code cleanup.
>>    samsung: common: Add misc file and common function misc_init_r().
>>    samsung: misc: move display logo function to misc.c file.
>>    common: lcd.c: fix data abort exception when try to access bmp
>> header lib: tizen: change Tizen logo with the new one.
>>    video: exynos: fimd: add support for various display color modes
>>    samsung: boards: update display configs with 16bpp mode.
>>    samsung: misc: Add LCD download menu.
>>    Trats: add LCD download menu support
>>    trats2: add LCD download menu support
>>    universal: add LCD download menu support
>>
>>   arch/arm/include/asm/arch-exynos/gpio.h  |  245 +-
>>   arch/arm/include/asm/arch-s5pc1xx/gpio.h |   47 +-
>>   board/samsung/common/Makefile            |    1 +
>>   board/samsung/common/misc.c              |  407 ++
>>   board/samsung/common/misc.h              |   18 +
>>   board/samsung/trats/trats.c              |    5 +-
>>   board/samsung/trats2/trats2.c            |   33 +-
>>   board/samsung/universal_c210/universal.c |    6 +-
>>   common/lcd.c                             |   27 +-
>>   drivers/gpio/s5p_gpio.c                  |   15 +-
>>   drivers/power/battery/bat_trats2.c       |    2 +-
>>   drivers/video/exynos_fb.c                |   28 -
>>   drivers/video/exynos_fimd.c              |   15 +-
>>   include/configs/s5p_goni.h               |    4 +-
>>   include/configs/s5pc210_universal.h      |   41 +-
>>   include/configs/trats.h                  |   33 +-
>>   include/configs/trats2.h                 |   31 +-
>>   include/lcd.h                            |    2 +
>>   include/power/max77686_pmic.h            |    2 +
>>   include/power/pmic.h                     |    1 -
>>   lib/tizen/tizen.c                        |   21 +-
>>   lib/tizen/tizen_hd_logo.h                | 5057 ---------------
>>   lib/tizen/tizen_hd_logo_data.h           |   15 -
>>   lib/tizen/tizen_logo_16bpp.h             |10025
>> ++++++++++++++++++++++++++++++
>> lib/tizen/tizen_logo_16bpp_gzip.h        |  727 +++ 25 files changed,
>> 11437 insertions(+), 5371 deletions(-) create mode 100644
>> board/samsung/common/misc.c create mode 100644
>> board/samsung/common/misc.h delete mode 100644
>> lib/tizen/tizen_hd_logo.h delete mode 100644
>> lib/tizen/tizen_hd_logo_data.h create mode 100644
>> lib/tizen/tizen_logo_16bpp.h create mode 100644
>> lib/tizen/tizen_logo_16bpp_gzip.h
>>
>
> I downloaded the newest version of u-boot-samsung,
> applied all patches you sent, build it, and
> write it on M0 board.
> Logo image doesn't appear.
>
> On the console, I found the below error message.
> "Error: 16 bit/pixel mode, but BMP has 32 bit/pixel"
>
> Is there anything wrong I did? or should something
> be changed in your code?
>
> Thanks,
> ---
> Hyungwon Hwang
> Samsung SWC S/W Platform Team
> Smasung Electronics
> human.hwang at samsung.com
>

I assume that it was because you missed the big size patch, right?

Regards
-- 
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com