nxp-archive / egui Goto Github PK

The CRTouch device has a TouchPending signal, used to signalize new data available. eGUI does not use the signal, it reads the RTSRDY field of the Resistive Touch Status Register 1 instead. That causes high load to the I2C bus.

User should be allowed to define the TouchPending signal which should be then used by eGUI to check for ready data.

When both double-buffering and DMA are enabled (i.e. d4dlcdhw_k70_lcdc_cfg.h contains the following definitions), the two buffers are not synchronized properly. When an object (not full screen) is invalidated, it is repainted to the back-buffer with deprecated content (previous screen). When the buffers are swapped, the screen content becomes broken.

#define D4DLCDHWFB_DOUBLE_BUFFER              1
#define D4DLCDHWFB_DMA_ENABLE                 1

I think there is an issue on line 234 of d4d_font.c. The following code:

#ifdef D4D_UNICODE
        while((pFntDesc != NULL) && (pFntDesc - d4d_FontTableRam[i].pFontDescriptor) <= fileSize)
        {
#endif

should be changed into:

#ifdef D4D_UNICODE
        while((pFntDesc != NULL) && ((LWord)pFntDesc - (LWord)d4d_FontTableRam[i].pFontDescriptor) <= fileSize)
        {
#endif

In order to compare the same quantities.
We have had a case where while loop was aborted before reaching the last page (with arm gcc).

Thanks

I guess the following declaration:

typedef D4D_TCHAR D4D_FONT_IX;

within d4d_font.h, active when D4D_UNICODE is defined, should be changed into:

typedef Word D4D_FONT_IX;

otherwise if you configure D4D_WCHAR (= D4D_TCHAR) 4 bytes long, reading of a binary font file produced by the eGUI Converter Utility won't work (fields of type D4D_FONT_IX are 2 bytes long in the file).

When using CRTouch as the touchscreen driver, some of the release events are not registered by eGUI. As a result, the buttons sometimes remain pressed without getting released until a new touch to the display is performed.

I have analysed the problem, it is caused by an incorrect use of the RTST field of the Resistive Touch Status register 1. I will deliver the fix soon.

It looks like line 234 of d4d_lcd_frame_buffer.c:

win_const1 = (signed long)(p_fbDesc->fb_start_addr + (p_fbDesc->lcd_x_max * win_x + p_fbDesc->lcd_x_max - win_y) * bpp_byte);

needs to be changed into:

win_const1 = (signed long)(p_fbDesc->fb_start_addr + (p_fbDesc->lcd_x_max * win_x + p_fbDesc->lcd_x_max - win_y - 1) * bpp_byte);

(-1 added) otherwise an LCD used with D4D_ORIENT_PORTRAIT180 orientation will show the first horizontal line of screen on the bottom (the whole screen is "rolled" up by 1 line).

Do you agree?

The D4D_UNICODE feature cannot be enabled if project compiled with CW, as the default wchar_t size in prebuilt CW standard libraries is 4 bytes, disallowing e.g. the following initialization in d4d_string.c:

D4D_TCHAR str_touch[10] = D4D_DEFSTR("X1:00000");

More information can be found in the discussion https://community.freescale.com/message/444007#444007 .