LCD_DEVFUNC_READPIXEL and LCD_DEVFUNC_READMPIXELS not used

    This site uses cookies. By continuing to browse this site, you are agreeing to our Cookie Policy.

    • LCD_DEVFUNC_READPIXEL and LCD_DEVFUNC_READMPIXELS not used

      I'm trying to specify custom functions to read pixels using the LCD_SetDevFunc function, but this does not seem to work.
      I see the PortAPI specified function are always called, not my custom function.

      Relevant part of my LCDConf.c is below, is anything wrong with it?



      C Source Code

      1. #include "GUI.h"
      2. #include "GUIDRV_FlexColor.h"
      4. /* Physical display size */
      5. #define XSIZE_PHYS 320
      6. #define YSIZE_PHYS 240
      8. /*********************************************************************
      9. *
      10. * Configuration checking
      11. *
      12. **********************************************************************
      13. */
      14. #ifndef VXSIZE_PHYS
      15. #define VXSIZE_PHYS XSIZE_PHYS
      16. #endif
      17. #ifndef VYSIZE_PHYS
      18. #define VYSIZE_PHYS YSIZE_PHYS
      19. #endif
      20. #ifndef XSIZE_PHYS
      21. #error Physical X size of display is not defined!
      22. #endif
      23. #ifndef YSIZE_PHYS
      24. #error Physical Y size of display is not defined!
      25. #endif
      26. #ifndef GUICC_M565
      27. #error Color conversion not defined!
      28. #endif
      29. #ifndef GUIDRV_FLEXCOLOR
      30. #error No display driver defined!
      31. #endif
      33. /*********************************************************************
      34. *
      35. * Local functions
      36. *
      37. **********************************************************************
      38. */
      40. static U16 _ReadPixel(int LayerIndex) {
      41. GUI_USE_PARA(LayerIndex);
      42. U16 Data = LCD_IO_ReadData();
      43. return ((Data & 0xFF00) >> 8) | ((Data & 0x00FF) << 8);
      44. }
      46. static void _ReadMPixels(int LayerIndex, U16* pBuffer, U32 NumPixels) {
      47. for (int pixel = 0; pixel < NumPixels; ++pixel) {
      48. pBuffer[pixel] = _ReadPixel(LayerIndex);
      49. }
      50. }
      52. /*********************************************************************
      53. *
      54. * Public functions
      55. *
      56. **********************************************************************
      57. */
      59. /*********************************************************************
      60. *
      61. * LCD_X_Config
      62. *
      63. * Function description:
      64. * Called during the initialization process in order to set up the
      65. * display driver configuration.
      66. *
      67. */
      68. void LCD_X_Config(void)
      69. {
      70. GUI_DEVICE * pDevice;
      71. CONFIG_FLEXCOLOR Config = {0};
      72. GUI_PORT_API PortAPI = {0};
      74. /* Set display driver and color conversion */
      75. pDevice = GUI_DEVICE_CreateAndLink(GUIDRV_FLEXCOLOR, GUICC_M565, 0, 0);
      76. //make FlexColor orientation match BSP orientation
      77. Config.Orientation = GUI_SWAP_XY | GUI_MIRROR_Y;
      78. GUIDRV_FlexColor_Config(pDevice, &Config);
      79. /* Display driver configuration */
      80. LCD_SetSizeEx (0, XSIZE_PHYS , YSIZE_PHYS);
      81. LCD_SetVSizeEx(0, VXSIZE_PHYS, VYSIZE_PHYS);
      83. /* Set controller and operation mode */
      84. PortAPI.pfWrite16_A0 = LCD_IO_WriteReg;
      85. PortAPI.pfWrite16_A1 = LCD_IO_WriteData;
      86. PortAPI.pfWriteM16_A1 = LCD_IO_WriteMultipleData;
      87. PortAPI.pfReadM16_A1 = LCD_IO_ReadMultipleData;
      88. GUIDRV_FlexColor_SetFunc(pDevice, &PortAPI, GUIDRV_FLEXCOLOR_F66709, GUIDRV_FLEXCOLOR_M16C0B16);
      90. /* Custom reading functions */
      91. LCD_SetDevFunc(0, LCD_DEVFUNC_READPIXEL, (void(*)(void))_ReadPixel);
      92. LCD_SetDevFunc(0, LCD_DEVFUNC_READMPIXELS, (void(*)(void))_ReadMPixels);
      93. }
      95. /*********************************************************************
      96. *
      97. * LCD_X_DisplayDriver
      98. *
      99. * Function description:
      100. * This function is called by the display driver for several purposes.
      101. * To support the according task the routine needs to be adapted to
      102. * the display controller. Please note that the commands marked with
      103. * 'optional' are not cogently required and should only be adapted if
      104. * the display controller supports these features.
      105. *
      106. * Parameter:
      107. * LayerIndex - Index of layer to be configured
      108. * Cmd - Please refer to the details in the switch statement below
      109. * pData - Pointer to a LCD_X_DATA structure
      110. *
      111. * Return Value:
      112. * < -1 - Error
      113. * -1 - Command not handled
      114. * 0 - Ok
      115. */
      116. int LCD_X_DisplayDriver(unsigned LayerIndex, unsigned Cmd, void * pData) {
      117. int r;
      118. (void) LayerIndex;
      119. (void) pData;
      120. switch (Cmd) {
      121. case LCD_X_INITCONTROLLER: {
      122. BSP_LCD_Init();
      123. return 0;
      124. }
      125. default:
      126. r = -1;
      127. }
      128. return r;
      129. }
      131. /************************ (C) COPYRIGHT STMicroelectronics *****END OF FILE****/
      Display All

    • Hi,

      Unfortunately, I can not say why the functions are not getting called. On my end it is working.

      Which version of emWin are you using?
      The exact version can be found in GUI_Version.h.

      The possibility to set custom read functions was introduced with version 5.22 of emWin.
      But, I doubt that someone uses such an old version (at least I hope no one does).

      Regards,
      Sven
      Please read the forum rules before posting.

      Keep in mind, this is *not* a support forum.
      Our engineers will try to answer your questions between their projects if possible but this can be delayed by longer periods of time.
      Should you be entitled to support you can contact us via our support system: segger.com/ticket/
      Or you can contact us via e-mail.