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

[Donghwa Lee]

Dear Wolfgang Denk,

Mr.Kang and I think TIZEN logo is not a board/vendor specific although samsung 
will release it. So, Mr.kang had proposed new directory include/video or 
include/video/logo/.

How do you think about that? When you receive this e-mail, please replay with
your opinions.

Thank you,
Donghwa Lee


On 27 April 2012 09:18, Minkyu Kang wrote:

> 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.