From mboxrd@z Thu Jan 1 00:00:00 1970 From: Donghwa Lee Date: Tue, 24 Apr 2012 11:17:33 +0900 Subject: [U-Boot] [PATCH 2/2] EXYNOS: draw 32bpp bitmap TIZEN logo In-Reply-To: <20120423092216.603FF200261@gemini.denx.de> References: <4F94B49E.5020802@samsung.com> <20120423092216.603FF200261@gemini.denx.de> Message-ID: <4F960D3D.4010703@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, On 23 April 2012 18:22, Wolfgang Denk wrote: > Dear Donghwa Lee, > > In message <4F94B49E.5020802@samsung.com> you wrote: >> This patches support drawing 32bpp bitmap TIZEN logo in exynos fb. >> asm/arch/trats_logo.h data is compressed by zip and decomressed at >> the exynos fb driver. >> >> And vl_bpix of vidinfo_t is changed proper value for u-boot. >> >> Signed-off-by: Donghwa Lee >> Signed-off-by: Kyungmin park >> --- >> arch/arm/include/asm/arch-exynos/trats_logo.h | 5070 +++++++++++++++++++++++++ >> board/samsung/trats/trats.c | 2 +- >> drivers/video/exynos_fb.c | 40 +- >> drivers/video/exynos_fimd.c | 6 +- >> include/configs/trats.h | 4 +- >> 5 files changed, 5116 insertions(+), 6 deletions(-) >> create mode 100644 arch/arm/include/asm/arch-exynos/trats_logo.h >> >> diff --git a/arch/arm/include/asm/arch-exynos/trats_logo.h b/arch/arm/include/asm/arch-exynos/trats_logo.h >> new file mode 100644 >> index 0000000..a595abd >> --- /dev/null >> +++ b/arch/arm/include/asm/arch-exynos/trats_logo.h > > This is not a good location for the logo data. Either this is vendor > specific, in which place it should go to the vendor directory, or it's > board specific, in which case it should go into the board directory. > > In no case I want to have this in arch/arm/include/ > > thanks. > > Best regards, > > Wolfgang Denk > Thank you for your comment. This logo data is TIZEN logo for TRATS board. Current TIZEN reference board is TRATS based on EXYNOS SoC. But, I don'know exactly any others boards that loaded TIZEN based on others SoC will be appeared in the future. So, I think this logo data is not board(TRATS) and vendor(EXYNOS) specific. So, I want to move this data in include/ How do you think about that? Plese tell me your opinions. Thank you, Donghwa Lee