[U-Boot] [PATCH v2 2/3] LCD: add compressed 32bpp BMP image data

Minkyu Kang promsoft at gmail.com
Fri Apr 27 02:18:06 CEST 2012


Dear Wolfgang Denk,

On 26 April 2012 21:43, Wolfgang Denk <wd at denx.de> wrote:
> Dear Donghwa Lee,
>
> In message <4F99057F.5040907 at samsung.com> you wrote:
>> This patch is 32bpp BMP image data that is converted to header
>> after gzip. I had agonized about where these data located.
>> I think TIZEN is not specific on board and SoC, so I determined
>> that position is proper in include/.
>> If you have other opinions, please comment to me.
>>
>> Signed-off-by: Donghwa Lee <dh09.lee at samsung.com>
>> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
>> ---
>>  include/tizen_logo.h      | 5071 +++++++++++++++++++++++++++++++++++++++++++++
>>  include/tizen_logo_data.h |   30 +
>>  2 files changed, 5101 insertions(+), 0 deletions(-)
>>  create mode 100644 include/tizen_logo.h
>>  create mode 100644 include/tizen_logo_data.h
>
> Please make sure to include a changelog; this is a mandatory
> requirement, see
> http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions
>
>
> I reject your patch.  I will not accept to have such board and/or
> vendor specific data in the top level include/ directory.  Please move
> this to the respective board or vendor directory, as I have asked you
> do to before.
>

But I think TIZEN logo is not a board/vendor specific.
It's a platform specific.
How do you think that makes a new directory?
(e.g: include/video/ or include/video/logo/)

Thnaks.
Minkyu Kang.
-- 
from. prom.
www.promsoft.net


More information about the U-Boot mailing list