topic Re: uboot logo color is not normal in i.MX Processors

uboot logo color is not normal

fulinux — Thu, 12 May 2016 11:27:34 GMT

Dear Community,

board: imx6q;

version: u-boot-2009.08;

i did what this doc said: https://community.nxp.com/docs/DOC-98109

but the logo color is not normal, it si green like this:

what is wrong? and this is my head file:

//#define IPU_OUTPUT_MODE_HDMI

#define IPU_OUTPUT_MODE_LVDS

// #define IPU_OUTPUT_MODE_LCD

#ifdef IPU_OUTPUT_MODE_LVDS

// For LVDS, 1920*1080 resolution, dual channel

#define DISPLAY_WIDTH 1920

#define DISPLAY_HEIGHT 1080

#define DISPLAY_BPP 32

#define DISPLAY_IF_BPP 24 // RGB24 interface

#define DISPLAY_HSYNC_START 80

#define DISPLAY_HSYNC_END 68

#define DISPLAY_HSYNC_WIDTH 12

#define DISPLAY_VSYNC_START 15

#define DISPLAY_VSYNC_END 15

#define DISPLAY_VSYNC_WIDTH 8

#define DISPLAY_PIX_CLOCK 139526400 //(DISPLAY_HSYNC_START + DISPLAY_HSYNC_END + DISPLAY_HSYNC_WIDTH + DISPLAY_WIDTH) *

(DISPLAY_VSYNC_START + DISPLAY_VSYNC_END + DISPLAY_VSYNC_WIDTH + DISPLAY_HEIGHT) * refresh rate (60Hz)

#define LVDS_SPLIT_MODE // For dual channel split mode.

#define DISPLAY_VSYNC_POLARITY 1

#define DISPLAY_HSYNC_POLARITY 1

#define DISPLAY_CLOCK_POLARITY 1

#define DISPLAY_DATA_POLARITY 0

#define DISPLAY_DATA_ENABLE_POLARITY 1

#define IPU_NUM 1 // 1 for IPU1, 2 for IPU2.

#define DI_NUM 0 // 0 for DI0, 1 for DI1.

#define LVDS_PORT 0 // 0 for LVDS0, 1 for LVDS1.

#define DI_CLOCK_EXTERNAL_MODE //When clock external mode was defined, the DI clock root will be PLL3 PFD1,

//without this macro, the DI root clock is IPU internal clock.

nothing changed, just above, can you help me? what should i do? what is info do you need? tell me.

wating for your reply?

thank you.

fulinux.

Re: uboot logo color is green

fulinux — Fri, 13 May 2016 01:08:28 GMT

Dear all,

can anybody help me? how can i solve this problem?

fulinux.

Re: uboot logo color is not normal

qiang_li-mpu_se — Fri, 13 May 2016 02:55:53 GMT

How about your hardware connection?

Do you just used three pair of LVDS data lane? If so, you should set "#define DISPLAY_IF_BPP 24 " to 18.

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 03:16:32 GMT

Hi Li,

1、my kernel displays ok,, not hardware problem.

2、"Do you just used three pair of LVDS data lane?" what does this mean?

3、i haved tried set "#define DISPLAY_IF_BPP 24 " to 18, it is also green color , but color become lighter.

i thank split mode need di0 and di1, but i see this:

#ifdef LVDS_SPLIT_MODE

reg |= (1 << 4);

if (DI == 0)

reg |= (1 << 0);

else if(DI == 1)

reg |= (3 << 0);

#endif

should i delete "if and else" to set di0 and di1 together?

Re: uboot logo color is not normal

qiang_li-mpu_se — Fri, 13 May 2016 05:17:20 GMT

How about your video mode parameters in kernel?

And which patch do you use? Please also attach your panel datasheet, is it a YUV panel?

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 05:36:15 GMT

Hi,

kernel:

drivers/video/mxc/ldb.c

static struct fb_videomode ldb_modedb[] = {

...

{

"LDB-1080P60", 60, 1920, 1080, 12048,

80, 68,

15, 15,

12, 8,

FB_VMODE_NONINTERLACED,

FB_MODE_IS_DETAILED,},

{

//67.3MHZ 67326600

"1080P60", 60, 960, 1080, 14853,

0, 50,

0, 31,

66, 60,

FB_VMODE_NONINTERLACED,

FB_MODE_IS_DETAILED,},

arch/arm/mach-mx6/board-mx6q_sabresd.c:

static struct fsl_mxc_ldb_platform_data ldb_data = {

.ipu_id = 0,//1

.disp_id = 1,

.ext_ref = 1,

.mode = LDB_SEP1,

.sec_ipu_id = 0,//1

.sec_disp_id = 0,

};

i use this L3.0.35_4.1.0_uboot_logo_keep_patch_2013-11-22.zip,

[fulinux@ubuntu uboot-imx]$ git branch

* imx_v2009.08_3.0.35_4.1.0

[fulinux@ubuntu uboot-imx]$ git log

6bd825c ENGR00327364 iMX6x: Ensure that the bandgap self-bias circuit is disabled after boot. (Ranjani Vaidyanathan, 1 year, 7 month

5caea2f ENGR00329930-2 MMC: Fix debounce problem in the tuning algorithm (Ye.Li, 1 year, 8 months ago)

...

i have modified nothing before patch, but i add backlight power gpio code:

static void setup_lvds_iomux(void)

{

....

#if 1

gpio_direction_output(IMX_GPIO_NR(1, 2), 1);

udelay(10*1000);

gpio_set_value(IMX_GPIO_NR(1, 2), 0);

udelay(10*1000);

gpio_set_value(IMX_GPIO_NR(1, 2), 1);

gpio_direction_output(IMX_GPIO_NR(4, 5), 1);

udelay(10*1000);

gpio_set_value(IMX_GPIO_NR(4, 5), 0);

gpio_direction_output(IMX_GPIO_NR(1, 7), 1);

udelay(10*1000);

gpio_set_value(IMX_GPIO_NR(1, 7), 1);

#endif

}

anything info which you need, tell me. this problem makes me very anxious.

and when i change " #define DISPLAY_BPP 32" to 24, and change "logo_1024x768_24bpp.bmp" to "logo_1920x1080_24bpp.bmp", and then screen like this:

but i am sure my lcd panel support 32bpp, because uboot bootargs is :

"video=mxcfb0:dev=ldb,1920x1080M@60,if=RGB24,bpp=32 consoleblank=0 ldb=spl0 ldo_active=on fbmem=128M" and kernel displays ok!

this is panel schematic:

Waiting for you reply!

thankyou,

fulinux.

Re: uboot logo color is not normal

qiang_li-mpu_se — Fri, 13 May 2016 06:08:44 GMT

You should not use 24bpp bmp file, it should be original 32bpp "logo.bmp", and set DISPLAY_BPP to 32 to align with the bmp file.

From your kernel video mode setting, your panel is VSYNC low active, HSYNC low active, pixel clock high active, Date enable low active. So your UBoot setting should be followed:

#define DISPLAY_VSYNC_POLARITY 0

#define DISPLAY_HSYNC_POLARITY 0

#define DISPLAY_CLOCK_POLARITY 1

#define DISPLAY_DATA_POLARITY 0

#define DISPLAY_DATA_ENABLE_POLARITY 0

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 06:23:17 GMT

Hi,

i test as you said, it shows white, red and blue loop, this is my head file:

// For LVDS, 1920*1080 resolution, dual channel

#define DISPLAY_WIDTH 1920

#define DISPLAY_HEIGHT 1080

#define DISPLAY_BPP 32

#define DISPLAY_IF_BPP 24 // RGB24 interface

#define DISPLAY_HSYNC_START 80

#define DISPLAY_HSYNC_END 68

#define DISPLAY_HSYNC_WIDTH 12

#define DISPLAY_VSYNC_START 15

#define DISPLAY_VSYNC_END 15

#define DISPLAY_VSYNC_WIDTH 8

#define DISPLAY_PIX_CLOCK 139526400 //(DISPLAY_HSYNC_START + DISPLAY_HSYNC_END + DISPLAY_HSYNC_WIDTH + DISPLAY_WIDTH) *

(DISPLAY_VSYNC_START + DISPLAY_VSYNC_END + DISPLAY_VSYNC_WIDTH + DISPLAY_HEIGHT) * refresh rate (60Hz)

#define LVDS_SPLIT_MODE 1 // For dual channel split mode.

#endif

#define DISPLAY_VSYNC_POLARITY 0

#define DISPLAY_HSYNC_POLARITY 0

#define DISPLAY_CLOCK_POLARITY 1

#define DISPLAY_DATA_POLARITY 0

#define DISPLAY_DATA_ENABLE_POLARITY 0

#define IPU_NUM 1 // 1 for IPU1, 2 for IPU2.

#define DI_NUM 0 // 0 for DI0, 1 for DI1.

#define LVDS_PORT 0 // 0 for LVDS0, 1 for LVDS1.

#define DI_CLOCK_EXTERNAL_MODE //When clock external mode was defined, the DI clock root will be PLL3 PFD1,

//without this macro, the DI root clock is IPU internal clock.

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 07:53:42 GMT

Hi,

when i modified this:

#define DISPLAY_WIDTH 1920

#define DISPLAY_HEIGHT 1080

#define DISPLAY_BPP 32

#define DISPLAY_IF_BPP 24 // RGB24 interface

#define DISPLAY_HSYNC_START 80

#define DISPLAY_HSYNC_END 68

#define DISPLAY_HSYNC_WIDTH 12

#define DISPLAY_VSYNC_START 15

#define DISPLAY_VSYNC_END 15

#define DISPLAY_VSYNC_WIDTH 8

#define DISPLAY_PIX_CLOCK 139526400 //(DISPLAY_HSYNC_START + DISPLAY_HSYNC_END + DISPLAY_HSYNC_WIDTH + DISPLAY_WIDTH) *

(DISPLAY_VSYNC_START + DISPLAY_VSYNC_END + DISPLAY_VSYNC_WIDTH + DISPLAY_HEIGHT) * refresh rate (60Hz)

#define LVDS_SPLIT_MODE 1 // For dual channel split mode.

#endif

#define DISPLAY_VSYNC_POLARITY 0

#define DISPLAY_HSYNC_POLARITY 0

#define DISPLAY_CLOCK_POLARITY 1

#define DISPLAY_DATA_POLARITY 0

#define DISPLAY_DATA_ENABLE_POLARITY 1

#define IPU_NUM 1 // 1 for IPU1, 2 for IPU2.

#define DI_NUM 0 // 0 for DI0, 1 for DI1.

#define LVDS_PORT 1 // 0 for LVDS0, 1 for LVDS1.

#define DI_CLOCK_EXTERNAL_MODE //When clock external mode was defined, the DI clock root will be PLL3 PFD1,

//without this macro, the DI root clock is IPU internal clock.

and

bmpReady = 0;

if (bmpReady == 0) {

// Fill RGB frame buffer

// Red

for (i = 0; i < (DISPLAY_WIDTH * DISPLAY_HEIGHT * (DISPLAY_BPP / 8) / 3); i += (DISPLAY_BPP / 8)) {

#if (DISPLAY_BPP == 16)

pData[i + 0] = 0x00;

pData[i + 1] = 0xF8;

#else

and the lcd panel screen display like this:

you can look at this, the color is lighter;

when i change code like this, change "pData[] = 0x FF" to 0x00:

// Red

for (i = 0; i < (DISPLAY_WIDTH * DISPLAY_HEIGHT * (DISPLAY_BPP / 8) / 3); i += (DISPLAY_BPP / 8)) {

#if (DISPLAY_BPP == 16)

pData[i + 0] = 0x00;

pData[i + 1] = 0xF8;

#else

pData[i + 0] = 0x00;

pData[i + 1] = 0x00;

pData[i + 2] = 0x00;

pData[i + 3] = 0x00;

#endif

}

// Green

for (; i < (DISPLAY_WIDTH * DISPLAY_HEIGHT * (DISPLAY_BPP / 8) / 3) * 2; i += (DISPLAY_BPP / 8)) {

#if (DISPLAY_BPP == 16)

pData[i + 0] = 0xE0;

pData[i + 1] = 0x07;

#else

pData[i + 0] = 0x00;

pData[i + 1] = 0x00;

pData[i + 2] = 0x00;

pData[i + 3] = 0x00;

#endif

}

// Blue

for (; i < DISPLAY_WIDTH * DISPLAY_HEIGHT * (DISPLAY_BPP / 8); i += (DISPLAY_BPP / 8)) {

#if (DISPLAY_BPP == 16)

pData[i + 0] = 0x1F;

pData[i + 1] = 0x00;

#else

pData[i + 0] = 0x00;

pData[i + 1] = 0x00;

pData[i + 2] = 0x00;

pData[i + 3] = 0x00;

then like this:

it should be black,but it is whilt!

Re: uboot logo color is not normal

qiang_li-mpu_se — Fri, 13 May 2016 08:06:50 GMT

Yes, from your kernel setting, the DE is high active, so it should be "#define DISPLAY_DATA_ENABLE_POLARITY 1"

For black to white issue, you can change DISPLAY_DATA_POLARITY from 0 to 1 to invert the data line.

I think all these informations should be found from your panel's datasheet.

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 08:22:33 GMT

when it should be black it shows white, but when it should be whilt it shows more white, so it can't shows black.

when i change "#define IPU_NUM 1" to 2,it shows white, red and blue looply, why

Re: uboot logo color is not normal

qiang_li-mpu_se — Fri, 13 May 2016 08:31:48 GMT

Another thing can be tried: invert the clock polarity, change DISPLAY_CLOCK_POLARITY from 1 to 0.

The display shows white, red and blue looply, that means the panel hasn't found valid input signal and show its test pattern.

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 08:42:07 GMT

it dosen't change anything when i change DISPLAY_CLOCK_POLARITY from 1 to 0. this is logo picture（original picture）:

this is the screen shows：

Re: uboot logo color is not normal

qiang_li-mpu_se — Fri, 13 May 2016 08:47:28 GMT

Can you attach your bmp file and panel datasheet here?

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 08:47:29 GMT

hi,

i want try uboot-2015.04(on branch imx_v2015.04_4.1.15_1.0.0_ga), should i need any logo patchs for this version uboot?

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 08:53:38 GMT

Yes ^_^ !

ps.

logo.bmp-bak file: this file i have skiped 54bytes from logo.bmp;

Re: uboot logo color is not normal

qiang_li-mpu_se — Fri, 13 May 2016 09:09:52 GMT

From the datasheet, the setting should be followed, no other issues found.

#define DISPLAY_VSYNC_POLARITY 0

#define DISPLAY_HSYNC_POLARITY 0

#define DISPLAY_CLOCK_POLARITY 0

#define DISPLAY_DATA_POLARITY 0

#define DISPLAY_DATA_ENABLE_POLARITY 1

Re: uboot logo color is not normal

fulinux — Fri, 13 May 2016 09:31:09 GMT

Hi，

it doesn't change anything when i did as what you said. it still like this:

i am trying uboot-2015.04(on branch imx_v2015.04_4.1.15_1.0.0_ga), how can i enable split mode in uboot-2015.04?

Re: uboot logo color is not normal

qiang_li-mpu_se — Fri, 13 May 2016 09:41:56 GMT

Uboot version should not be related to this issue, I think you need find out the different setting between kernel and Uboot.

Re: uboot logo color is not normal

fulinux — Mon, 16 May 2016 03:02:23 GMT

hi Qiang,

is it wrong with my backlight and lcd power settings of gpios, this is my settings:

at setup_lvds_iomux() function in board/freescale/mx6q_sabresd/mx6q_sabresd.c:

#if defined CONFIG_MX6Q

/* GPIO backlight */

//mxc_iomux_v3_setup_pad(MX6Q_PAD_SD1_DAT3__PWM1_PWMO);

mxc_iomux_v3_setup_pad(MX6Q_PAD_GPIO_9__PWM1_PWMO);

/* LVDS panel CABC_EN */

mxc_iomux_v3_setup_pad(MX6Q_PAD_NANDF_CS2__GPIO_6_15);

mxc_iomux_v3_setup_pad(MX6Q_PAD_NANDF_CS3__GPIO_6_16);

mxc_iomux_v3_setup_pad(MX6Q_PAD_GPIO_2__GPIO_1_2);

mxc_iomux_v3_setup_pad(MX6Q_PAD_GPIO_7__GPIO_1_7);

mxc_iomux_v3_setup_pad(MX6Q_PAD_GPIO_19__GPIO_4_5);

reg = readl(GPIO6_BASE_ADDR + GPIO_DR);

reg &= ~(1 << 16);

writel(reg, GPIO6_BASE_ADDR + GPIO_DR);

#if 1

gpio_direction_output(IMX_GPIO_NR(1, 2), 1);

udelay(10 * 1000);

gpio_set_value(IMX_GPIO_NR(1, 2), 0);

udelay(10 * 1000);

gpio_set_value(IMX_GPIO_NR(1, 2), 1);

gpio_direction_output(IMX_GPIO_NR(4, 5), 1);

udelay(10 * 1000);

gpio_set_value(IMX_GPIO_NR(4, 5), 1);

gpio_direction_output(IMX_GPIO_NR(1, 7), 0);

udelay(10 * 1000);

gpio_set_value(IMX_GPIO_NR(1, 7), 0);

#endif

at board_late_init() function:

ipu_display_setup();

gpio_set_value(IMX_GPIO_NR(1, 7), 1);

udelay(10 * 1000);

gpio_set_value(IMX_GPIO_NR(4, 5), 0);

Do you think that was right?

with my attach file.