From mboxrd@z Thu Jan 1 00:00:00 1970 From: Donghwa Lee Date: Mon, 30 Apr 2012 15:06:21 +0900 Subject: [U-Boot] [PATCH v2 2/3] LCD: add compressed 32bpp BMP image data In-Reply-To: References: <4F99057F.5040907@samsung.com> <20120426124345.C830A200246@gemini.denx.de> Message-ID: <4F9E2BDD.3030904@samsung.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 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 >>> Signed-off-by: Kyungmin Park >>> --- >>> 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.