From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hyungwon Hwang Date: Thu, 26 Dec 2013 08:39:16 +0900 Subject: [U-Boot] [PATCH v2 07/13] lib: tizen: change Tizen logo with the new one. In-Reply-To: <52B42EF3.2020602@samsung.com> References: <23bcd95f5c542315d31fc910b4b0abe83998e2d4.1387390491.git.p.marczak@samsung.com> <001f01cefc7c$c53a7550$4faf5ff0$@samsung.com> <52B2CD1A.1030007@samsung.com> <20131220140755.365063ce@hwh-ubuntu> <52B42EF3.2020602@samsung.com> Message-ID: <20131226083916.6b63a474@hwh-ubuntu> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Marczak, On Fri, 20 Dec 2013 12:50:11 +0100 Przemyslaw Marczak wrote: > Dear Hyungwon, > > On 12/20/2013 06:07 AM, Hyungwon Hwang wrote: > > Hi, Marczak > > > > On Thu, 19 Dec 2013 11:40:26 +0100 > > Przemyslaw Marczak wrote: > > > >> Hello Hyungwon, > >> > >> On 12/19/2013 06:40 AM, ??? wrote: > >>> Hi, Marczak. > >>> > >>> Is this logo image what I sent you before? > >>> > >>> It's a little different what we use, > >>> and also the logo image is not aligned center horizontally. > >>> > >>> Best regards, > >>> Hyungwon Hwang > >>> > >> > >> This is not a logo which you sent me. Your logo has 24BPP and > >> actually it has displayed correctly because of fimd 24bpp mode. > >> When I tried to convert it to 16bpp with gimp, it was not > >> displayed correctly in 16bpp mode, there was some lines shift on > >> display so I took logo from tizen.org and resized it to 500x160 - > >> now it's looking good. > >> > >> This is the logo location: > >> https://download.tizen.org/misc/Tizen-Brand/01-Primary-Assets/Lockup/On-Dark/01-RGB/Tizen-Lockup-On-Dark-RGB.png > >> > >> I can add some x and y offsets to make word "TIZEN" aligned center. > >> If this logo is really bad, then can you share logo in PNG format? > >> Or BMP 16bpp? > > > > The logo file is OK. Could you adjust alignment? > > Also, I think that anti-aliasing is needed. > > We can see stair-stepping appearance in diagonal lines. > > Can you fix it? > > > >> > >> Regards. > > > > > > > > --- > > Hyungwon Hwang > > Samsung SWC S/W Platform Team > > Smasung Electronics > > human.hwang at samsung.com > > > > You're right the picture size scalling method which I have chosen was > not good enough. I fixed it and now logo quality is much better. I > also introduced logo x and y offset in struct vidinfo, so function > get logo will be looking like this: > > file: lib/tizen/tizen.c > > void get_tizen_logo_info(vidinfo_t *vid) > { > - switch (vid->resolution) { > - case HD_RESOLUTION: > - vid->logo_width = TIZEN_HD_LOGO_WIDTH; > - vid->logo_height = TIZEN_HD_LOGO_HEIGHT; > - vid->logo_addr = (ulong)tizen_hd_logo; > + switch (vid->vl_bpix) { > + case 4: > + vid->logo_width = TIZEN_LOGO_16BPP_WIDTH; > + vid->logo_height = TIZEN_LOGO_16BPP_HEIGHT; > + vid->logo_x_offset = TIZEN_LOGO_16BPP_X_OFFSET; > + vid->logo_y_offset = TIZEN_LOGO_16BPP_Y_OFFSET; > +#if defined(CONFIG_VIDEO_BMP_GZIP) > + vid->logo_addr = (ulong)tizen_logo_16bpp_gzip; > +#else > + vid->logo_addr = (ulong)tizen_logo_16bpp; > +#endif > break; > default: > + vid->logo_addr = 0; > break; > } > } > > and set logo position before display: > file: board/samsung/common/misc.c > > x = ((panel_info.vl_width - panel_info.logo_width) >> 1); > + x += panel_info.logo_x_offset; /* For X center align */ > > y = ((panel_info.vl_height - panel_info.logo_height) >> 1); > + y += panel_info.logo_y_offset; /* For Y center align */ > > So every new logo in future could be simply center-aligned. > Is it ok? > Regards, > Can you send the patch? I think that the code above omit the added variable declaration. So I can't test it. Thank you. Best regards, Hyungwon Hwang --- Hyungwon Hwang Samsung SWC S/W Platform Team Smasung Electronics human.hwang at samsung.com