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 <w...@denx.de> wrote: >> Dear Donghwa Lee, >> >> In message <4f99057f.5040...@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....@samsung.com> >>> Signed-off-by: Kyungmin Park <kyungmin.p...@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. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot