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. -- from. prom. www.promsoft.net _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
