| Message ID | 1356086122-4489-4-git-send-email-ajaykumar.rs@samsung.com |
|---|---|
| State | Changes Requested |
| Delegated to: | Minkyu Kang |
| Headers | show |
Dear Ajay Kumar, On 21/12/12 19:35, Ajay Kumar wrote: > Currently, exynos FIMD driver is being used to support only TIZEN LOGOs. > In order to get LCD console, we need to enable half word swap feature > of FIMD and use 16 BPP. > LCD console and proprietary Logo cannot be used simultaneously. > We use logo_on field inside vidinfo_t structure to decide whether > user wants Logo or Console. > > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > --- Please add changelog here about what you changed since last version. > drivers/video/exynos_fb.c | 7 +++++++ > drivers/video/exynos_fimd.c | 12 ++++++++---- > 2 files changed, 15 insertions(+), 4 deletions(-) > > diff --git a/drivers/video/exynos_fb.c b/drivers/video/exynos_fb.c > index d9a3f9a..ee916be 100644 > --- a/drivers/video/exynos_fb.c > +++ b/drivers/video/exynos_fb.c > @@ -45,6 +45,13 @@ short console_row; > > static unsigned int panel_width, panel_height; > > +#ifndef CONFIG_CMD_BMP > +int bmp_display(ulong addr, int x, int y) > +{ > + return 0; > +} > +#endif It's a common function of bmp command. Please do not redefine such a function. --- Thanks, Minkyu Kang.
Hi Minkyu, On Tue, Jan 8, 2013 at 8:05 AM, Minkyu Kang <mk7.kang@samsung.com> wrote: > Dear Ajay Kumar, > > On 21/12/12 19:35, Ajay Kumar wrote: >> Currently, exynos FIMD driver is being used to support only TIZEN LOGOs. >> In order to get LCD console, we need to enable half word swap feature >> of FIMD and use 16 BPP. >> LCD console and proprietary Logo cannot be used simultaneously. >> We use logo_on field inside vidinfo_t structure to decide whether >> user wants Logo or Console. >> >> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> >> --- > > Please add changelog here about what you changed since last version. I will send V5 soon. Please see my explanation below. >> drivers/video/exynos_fb.c | 7 +++++++ >> drivers/video/exynos_fimd.c | 12 ++++++++---- >> 2 files changed, 15 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/video/exynos_fb.c b/drivers/video/exynos_fb.c >> index d9a3f9a..ee916be 100644 >> --- a/drivers/video/exynos_fb.c >> +++ b/drivers/video/exynos_fb.c >> @@ -45,6 +45,13 @@ short console_row; >> >> static unsigned int panel_width, panel_height; >> >> +#ifndef CONFIG_CMD_BMP >> +int bmp_display(ulong addr, int x, int y) >> +{ >> + return 0; >> +} >> +#endif > It's a common function of bmp command. > Please do not redefine such a function. I am using CONFIG_CMD_BMP to differentiate between LCD Logo and LCD console. I select CONFIG_CMD_BMP only when I need Logo, and I will not select CONFIG_CMD_BMP when I need console. Lets consider that we want console. So we don't define CONFIG_CMD_BMP now. And, exynos_fb.c has linking time dependency on "bmp_display", which is not defined! In such a case, I am left with 2 options: 1) Place an #ifdef CONFIG_CMD_BMP inside exynos_fb, where it makes a call to "bmp_display". 2) Redefine "bmp_display" to do nothing inside a header file(That's how it is done in this patch!) Kindly let me know if you have a better way to do this. > --- > Thanks, > Minkyu Kang. > ___________ Regards, Ajay Kumar
On 08/01/13 14:31, Ajay kumar wrote: > Hi Minkyu, > > On Tue, Jan 8, 2013 at 8:05 AM, Minkyu Kang <mk7.kang@samsung.com> wrote: >> Dear Ajay Kumar, >> >> On 21/12/12 19:35, Ajay Kumar wrote: >>> Currently, exynos FIMD driver is being used to support only TIZEN LOGOs. >>> In order to get LCD console, we need to enable half word swap feature >>> of FIMD and use 16 BPP. >>> LCD console and proprietary Logo cannot be used simultaneously. >>> We use logo_on field inside vidinfo_t structure to decide whether >>> user wants Logo or Console. >>> >>> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> >>> --- >> >> Please add changelog here about what you changed since last version. > I will send V5 soon. Please see my explanation below. >>> drivers/video/exynos_fb.c | 7 +++++++ >>> drivers/video/exynos_fimd.c | 12 ++++++++---- >>> 2 files changed, 15 insertions(+), 4 deletions(-) >>> >>> diff --git a/drivers/video/exynos_fb.c b/drivers/video/exynos_fb.c >>> index d9a3f9a..ee916be 100644 >>> --- a/drivers/video/exynos_fb.c >>> +++ b/drivers/video/exynos_fb.c >>> @@ -45,6 +45,13 @@ short console_row; >>> >>> static unsigned int panel_width, panel_height; >>> >>> +#ifndef CONFIG_CMD_BMP >>> +int bmp_display(ulong addr, int x, int y) >>> +{ >>> + return 0; >>> +} >>> +#endif > > >> It's a common function of bmp command. >> Please do not redefine such a function. > > I am using CONFIG_CMD_BMP to differentiate between LCD Logo and LCD console. > I select CONFIG_CMD_BMP only when I need Logo, and > I will not select CONFIG_CMD_BMP when I need console. > Lets consider that we want console. So we don't define CONFIG_CMD_BMP now. > And, exynos_fb.c has linking time dependency on "bmp_display", which > is not defined! > In such a case, I am left with 2 options: > 1) Place an #ifdef CONFIG_CMD_BMP inside exynos_fb, where it makes a > call to "bmp_display". > 2) Redefine "bmp_display" to do nothing inside a header file(That's > how it is done in this patch!) > Kindly let me know if you have a better way to do this. > I think, 1) is better. Thanks. Minkyu Kang.
diff --git a/drivers/video/exynos_fb.c b/drivers/video/exynos_fb.c index d9a3f9a..ee916be 100644 --- a/drivers/video/exynos_fb.c +++ b/drivers/video/exynos_fb.c @@ -45,6 +45,13 @@ short console_row; static unsigned int panel_width, panel_height; +#ifndef CONFIG_CMD_BMP +int bmp_display(ulong addr, int x, int y) +{ + return 0; +} +#endif + static void exynos_lcd_init_mem(void *lcdbase, vidinfo_t *vid) { unsigned long palette_size; diff --git a/drivers/video/exynos_fimd.c b/drivers/video/exynos_fimd.c index 06eae2e..f957dc8 100644 --- a/drivers/video/exynos_fimd.c +++ b/drivers/video/exynos_fimd.c @@ -88,14 +88,18 @@ static void exynos_fimd_set_par(unsigned int win_id) /* DATAPATH is DMA */ cfg |= EXYNOS_WINCON_DATAPATH_DMA; - /* bpp is 32 */ - cfg |= EXYNOS_WINCON_WSWP_ENABLE; + if (pvid->logo_on) /* To get proprietary LOGO */ + cfg |= EXYNOS_WINCON_WSWP_ENABLE; + else /* To get output console on LCD */ + cfg |= EXYNOS_WINCON_HAWSWP_ENABLE; /* dma burst is 16 */ cfg |= EXYNOS_WINCON_BURSTLEN_16WORD; - /* pixel format is unpacked RGB888 */ - cfg |= EXYNOS_WINCON_BPPMODE_24BPP_888; + if (pvid->logo_on) /* To get proprietary LOGO */ + cfg |= EXYNOS_WINCON_BPPMODE_24BPP_888; + else /* To get output console on LCD */ + cfg |= EXYNOS_WINCON_BPPMODE_16BPP_565; writel(cfg, (unsigned int)&fimd_ctrl->wincon0 + EXYNOS_WINCON(win_id));
Currently, exynos FIMD driver is being used to support only TIZEN LOGOs. In order to get LCD console, we need to enable half word swap feature of FIMD and use 16 BPP. LCD console and proprietary Logo cannot be used simultaneously. We use logo_on field inside vidinfo_t structure to decide whether user wants Logo or Console. Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> --- drivers/video/exynos_fb.c | 7 +++++++ drivers/video/exynos_fimd.c | 12 ++++++++---- 2 files changed, 15 insertions(+), 4 deletions(-)