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@denx.de wrote:
Dear Donghwa Lee,
In message 4F99057F.5040907@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@samsung.com Signed-off-by: Kyungmin Park kyungmin.park@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.