Oshel Posted August 2, 2017 Report Share Posted August 2, 2017 (edited) Hello, At the begining I'd like to say that I am really shocked how versatile yet easy to use is this library. In fact, when I become somehow more free with using it I will try to push it into the company I am working for- using Linux based SBCs for product with simple GUI is not the best solution... Anyway- after spending some time with setting everything up and testing diffrent modules I wanted to start writing some basic app using examples etc. to get the uGFX better known for me. I am working with STM32F103 uC in HY-Mini board (yes, it's old but I had it in my drawer). I studied your example with a widget and I started to write something similar- in fact it is the same approach. The problem is when I call gwinSetVisible function I get HardFault error. The problem is with redraw timer and queue as call stack says so (look at attachment). Here is the code: typedef struct { GWidgetObject widget; char time[9]; uint8_t sec; uint8_t min; uint8_t hrs; char date[9]; uint8_t day; uint8_t mth; uint8_t yrs; gdispImage* icon1; gdispImage* icon2; } tStatusbar; static GHandle ghStatusbar; tStatusbar statusbarObject; // Functions for VMT void statusbarDraw ( GWidgetObject* gw, void* param); static void mousePressDown( GWidgetObject* gw, coord_t x, coord_t y); // End of Functions for VMT // The statusbar VMT static const gwidgetVMT statusbarVMT = { { "Statusbar", // The classname sizeof(statusbarObject), // The object size _gwidgetDestroy, // The destroy routine _gwidgetRedraw, // The redraw routine 0, // The after-clear routine }, statusbarDraw, // The default drawing routine #if GINPUT_NEED_MOUSE { mousePressDown, // Process mouse down events 0, // Process mouse up events 0, // Process mouse move events }, #endif #if GINPUT_NEED_KEYBOARD || GWIN_NEED_KEYBOARD { 0 // Process keyboard events }, #endif #if GINPUT_NEED_TOGGLE { 0, // Toggle role 0, // Assign Toggles 0, // Get Toggles 0, // Process toggle off events 0, // Process toggle on events }, #endif #if GINPUT_NEED_DIAL { 0, // No dial roles 0, // Assign Dials 0, // Get Dials 0, // Process dial move events }, #endif }; void guiCreate ( void) { GWidgetInit widget; gwinSetDefaultBgColor(HTML2COLOR(0xFBDAAA)); gwinSetDefaultFont(gdispOpenFont("UI2")); gwinSetDefaultStyle(&WhiteWidgetStyle, FALSE); gwinWidgetClearInit(&widget); widget.text = "Statusbar"; widget.g.width = gdispGetWidth(); widget.g.height = 15; widget.g.x = 0; widget.g.y = 0; widget.g.show = TRUE; // widget of GWidgetInit is initiating structre with basic info // statusbarObject is an actual widget with GWidgetObject structure // statusbarVMT is a structure with methods shit tStatusbar* pointer; // Create pointer to point on the created widget on the heap pointer = (tStatusbar*) _gwidgetCreate( NULL, // basic display &statusbarObject.widget, // actual widget object &widget, // widget init object for init &statusbarVMT); // virtual widget handlers pointer->time[0] = '1'; pointer->time[1] = '\0'; pointer->date[0] = '\0'; gwinSetVisible((GHandle) pointer, TRUE); ghStatusbar = (GHandle) pointer; } The gfxconf.h looks like this: /** * This file has a different license to the rest of the uGFX system. * You can copy, modify and distribute this file as you see fit. * You do not need to publish your source modifications to this file. * The only thing you are not permitted to do is to relicense it * under a different license. */ /** * Copy this file into your project directory and rename it as gfxconf.h * Edit your copy to turn on the uGFX features you want to use. * The values below are the defaults. * * Only remove the comments from lines where you want to change the * default value. This allows definitions to be included from * driver makefiles when required and provides the best future * compatibility for your project. * * Please use spaces instead of tabs in this file. */ #ifndef _GFXCONF_H #define _GFXCONF_H /////////////////////////////////////////////////////////////////////////// // GOS - One of these must be defined, preferably in your Makefile // /////////////////////////////////////////////////////////////////////////// //#define GFX_USE_OS_CHIBIOS FALSE #define GFX_USE_OS_FREERTOS TRUE // #define GFX_FREERTOS_USE_TRACE FALSE //#define GFX_USE_OS_WIN32 FALSE //#define GFX_USE_OS_LINUX FALSE //#define GFX_USE_OS_OSX FALSE //#define GFX_USE_OS_ECOS FALSE //#define GFX_USE_OS_RAWRTOS FALSE //#define GFX_USE_OS_ARDUINO FALSE //#define GFX_USE_OS_KEIL TRUE //#define GFX_USE_OS_CMSIS FALSE //#define GFX_USE_OS_RAW32 TRUE //#define GFX_USE_OS_NIOS FALSE //#define GFX_USE_OS_QT FALSE // #define INTERRUPTS_OFF() optional_code // #define INTERRUPTS_ON() optional_code // Options that (should where relevant) apply to all operating systems // #define GFX_NO_INLINE FALSE #define GFX_COMPILER GFX_COMPILER_KEIL // #define GFX_SHOW_COMPILER FALSE #define GFX_CPU GFX_CPU_CORTEX_M3 // #define GFX_CPU_NO_ALIGNMENT_FAULTS FALSE // #define GFX_CPU_ENDIAN GFX_CPU_ENDIAN_UNKNOWN #define GFX_OS_HEAP_SIZE 4096 #define GFX_OS_NO_INIT TRUE // #define GFX_OS_INIT_NO_WARNING FALSE // #define GFX_OS_PRE_INIT_FUNCTION myHardwareInitRoutine // #define GFX_OS_EXTRA_INIT_FUNCTION myOSInitRoutine // #define GFX_OS_EXTRA_DEINIT_FUNCTION myOSDeInitRoutine // #define GFX_EMULATE_MALLOC FALSE /////////////////////////////////////////////////////////////////////////// // GDISP // /////////////////////////////////////////////////////////////////////////// #define GFX_USE_GDISP TRUE //#define GDISP_NEED_AUTOFLUSH FALSE //#define GDISP_NEED_TIMERFLUSH FALSE #define GDISP_NEED_VALIDATION TRUE #define GDISP_NEED_CLIP TRUE #define GDISP_NEED_CIRCLE TRUE //#define GDISP_NEED_DUALCIRCLE FALSE //#define GDISP_NEED_ELLIPSE FALSE //#define GDISP_NEED_ARC FALSE //#define GDISP_NEED_ARCSECTORS FALSE //#define GDISP_NEED_CONVEX_POLYGON FALSE //#define GDISP_NEED_SCROLL FALSE //#define GDISP_NEED_PIXELREAD FALSE #define GDISP_NEED_CONTROL TRUE //#define GDISP_NEED_QUERY FALSE #define GDISP_NEED_MULTITHREAD TRUE //#define GDISP_NEED_STREAMING FALSE #define GDISP_NEED_TEXT TRUE // #define GDISP_NEED_TEXT_WORDWRAP FALSE // #define GDISP_NEED_ANTIALIAS FALSE // #define GDISP_NEED_UTF8 FALSE // #define GDISP_NEED_TEXT_KERNING FALSE // #define GDISP_INCLUDE_FONT_UI1 FALSE #define GDISP_INCLUDE_FONT_UI2 TRUE // The smallest preferred font. // #define GDISP_INCLUDE_FONT_LARGENUMBERS FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS10 FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS12 FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS16 FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS20 FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS24 FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS32 FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANSBOLD12 FALSE // #define GDISP_INCLUDE_FONT_FIXED_10X20 FALSE // #define GDISP_INCLUDE_FONT_FIXED_7X14 FALSE // #define GDISP_INCLUDE_FONT_FIXED_5X8 FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS12_AA FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS16_AA FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS20_AA FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS24_AA FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANS32_AA FALSE // #define GDISP_INCLUDE_FONT_DEJAVUSANSBOLD12_AA FALSE // #define GDISP_INCLUDE_USER_FONTS FALSE #define GDISP_NEED_IMAGE TRUE // #define GDISP_NEED_IMAGE_NATIVE FALSE // #define GDISP_NEED_IMAGE_GIF FALSE #define GDISP_NEED_IMAGE_BMP TRUE // #define GDISP_NEED_IMAGE_BMP_1 TRUE // #define GDISP_NEED_IMAGE_BMP_4 TRUE // #define GDISP_NEED_IMAGE_BMP_4_RLE TRUE #define GDISP_NEED_IMAGE_BMP_8 TRUE // #define GDISP_NEED_IMAGE_BMP_8_RLE TRUE // #define GDISP_NEED_IMAGE_BMP_16 TRUE // #define GDISP_NEED_IMAGE_BMP_24 TRUE // #define GDISP_NEED_IMAGE_BMP_32 TRUE // #define GDISP_NEED_IMAGE_JPG FALSE // #define GDISP_NEED_IMAGE_PNG TRUE // #define GDISP_NEED_IMAGE_PNG_INTERLACED FALSE // #define GDISP_NEED_IMAGE_PNG_TRANSPARENCY TRUE // #define GDISP_NEED_IMAGE_PNG_BACKGROUND TRUE // #define GDISP_NEED_IMAGE_PNG_ALPHACLIFF 32 // #define GDISP_NEED_IMAGE_PNG_PALETTE_124 TRUE // #define GDISP_NEED_IMAGE_PNG_PALETTE_8 TRUE // #define GDISP_NEED_IMAGE_PNG_GRAYSCALE_124 TRUE // #define GDISP_NEED_IMAGE_PNG_GRAYSCALE_8 TRUE // #define GDISP_NEED_IMAGE_PNG_GRAYSCALE_16 TRUE // #define GDISP_NEED_IMAGE_PNG_GRAYALPHA_8 TRUE // #define GDISP_NEED_IMAGE_PNG_GRAYALPHA_16 TRUE // #define GDISP_NEED_IMAGE_PNG_RGB_8 TRUE // #define GDISP_NEED_IMAGE_PNG_RGB_16 TRUE // #define GDISP_NEED_IMAGE_PNG_RGBALPHA_8 TRUE // #define GDISP_NEED_IMAGE_PNG_RGBALPHA_16 TRUE // #define GDISP_NEED_IMAGE_ACCOUNTING FALSE //#define GDISP_NEED_PIXMAP FALSE // #define GDISP_NEED_PIXMAP_IMAGE FALSE //#define GDISP_DEFAULT_ORIENTATION GDISP_ROTATE_LANDSCAPE // If not defined the native hardware orientation is used. //#define GDISP_LINEBUF_SIZE 128 //#define GDISP_STARTUP_COLOR Black #define GDISP_NEED_STARTUP_LOGO FALSE //#define GDISP_TOTAL_DISPLAYS 1 //#define GDISP_DRIVER_LIST GDISPVMT_Win32, GDISPVMT_Win32 // #ifdef GDISP_DRIVER_LIST // // For code and speed optimization define as TRUE or FALSE if all controllers have the same capability // #define GDISP_HARDWARE_STREAM_WRITE FALSE // #define GDISP_HARDWARE_STREAM_READ FALSE // #define GDISP_HARDWARE_STREAM_POS FALSE // #define GDISP_HARDWARE_DRAWPIXEL FALSE // #define GDISP_HARDWARE_CLEARS FALSE // #define GDISP_HARDWARE_FILLS FALSE // #define GDISP_HARDWARE_BITFILLS FALSE // #define GDISP_HARDWARE_SCROLL FALSE // #define GDISP_HARDWARE_PIXELREAD FALSE // #define GDISP_HARDWARE_CONTROL FALSE // #define GDISP_HARDWARE_QUERY FALSE // #define GDISP_HARDWARE_CLIP FALSE // #define GDISP_PIXELFORMAT GDISP_PIXELFORMAT_RGB888 // #endif //#define GDISP_USE_GFXNET FALSE // #define GDISP_GFXNET_PORT 13001 // #define GDISP_GFXNET_CUSTOM_LWIP_STARTUP FALSE // #define GDISP_DONT_WAIT_FOR_NET_DISPLAY FALSE // #define GDISP_GFXNET_UNSAFE_SOCKETS FALSE /////////////////////////////////////////////////////////////////////////// // GWIN // /////////////////////////////////////////////////////////////////////////// #define GFX_USE_GWIN TRUE #define GWIN_NEED_WINDOWMANAGER TRUE // #define GWIN_REDRAW_IMMEDIATE FALSE // #define GWIN_REDRAW_SINGLEOP FALSE // #define GWIN_NEED_FLASHING FALSE // #define GWIN_FLASHING_PERIOD 250 //#define GWIN_NEED_CONSOLE FALSE // #define GWIN_CONSOLE_USE_HISTORY FALSE // #define GWIN_CONSOLE_HISTORY_AVERAGING FALSE // #define GWIN_CONSOLE_HISTORY_ATCREATE FALSE // #define GWIN_CONSOLE_ESCSEQ FALSE // #define GWIN_CONSOLE_USE_BASESTREAM FALSE // #define GWIN_CONSOLE_USE_FLOAT FALSE //#define GWIN_NEED_GRAPH FALSE //#define GWIN_NEED_GL3D FALSE #define GWIN_NEED_WIDGET TRUE //#define GWIN_FOCUS_HIGHLIGHT_WIDTH 1 // #define GWIN_NEED_LABEL FALSE // #define GWIN_LABEL_ATTRIBUTE FALSE #define GWIN_NEED_BUTTON TRUE // #define GWIN_BUTTON_LAZY_RELEASE TRUE // #define GWIN_NEED_SLIDER FALSE // #define GWIN_SLIDER_NOSNAP FALSE // #define GWIN_SLIDER_DEAD_BAND 5 // #define GWIN_SLIDER_TOGGLE_INC 20 // #define GWIN_NEED_CHECKBOX FALSE // #define GWIN_NEED_IMAGE FALSE // #define GWIN_NEED_IMAGE_ANIMATION FALSE // #define GWIN_NEED_RADIO FALSE // #define GWIN_NEED_LIST FALSE // #define GWIN_NEED_LIST_IMAGES FALSE // #define GWIN_NEED_PROGRESSBAR FALSE // #define GWIN_PROGRESSBAR_AUTO FALSE // #define GWIN_NEED_KEYBOARD FALSE // #define GWIN_KEYBOARD_DEFAULT_LAYOUT VirtualKeyboard_English1 // #define GWIN_NEED_KEYBOARD_ENGLISH1 TRUE // #define GWIN_NEED_TEXTEDIT FALSE // #define GWIN_FLAT_STYLING FALSE // #define GWIN_WIDGET_TAGS FALSE //#define GWIN_NEED_CONTAINERS FALSE // #define GWIN_NEED_CONTAINER FALSE // #define GWIN_NEED_FRAME FALSE // #define GWIN_NEED_TABSET FALSE // #define GWIN_TABSET_TABHEIGHT 18 /////////////////////////////////////////////////////////////////////////// // GTRANS // /////////////////////////////////////////////////////////////////////////// //#define GFX_USE_GTRANS FALSE /////////////////////////////////////////////////////////////////////////// // GEVENT // /////////////////////////////////////////////////////////////////////////// #define GFX_USE_GEVENT TRUE //#define GEVENT_ASSERT_NO_RESOURCE FALSE #define GEVENT_MAXIMUM_SIZE 32 #define GEVENT_MAX_SOURCE_LISTENERS 32 /////////////////////////////////////////////////////////////////////////// // GTIMER // /////////////////////////////////////////////////////////////////////////// #define GFX_USE_GTIMER TRUE #define GTIMER_THREAD_PRIORITY NORMAL_PRIORITY #define GTIMER_THREAD_WORKAREA_SIZE 256 /////////////////////////////////////////////////////////////////////////// // GQUEUE // /////////////////////////////////////////////////////////////////////////// #define GFX_USE_GQUEUE TRUE #define GQUEUE_NEED_ASYNC TRUE //#define GQUEUE_NEED_GSYNC TRUE //#define GQUEUE_NEED_FSYNC TRUE //#define GQUEUE_NEED_BUFFERS TRUE /////////////////////////////////////////////////////////////////////////// // GINPUT // /////////////////////////////////////////////////////////////////////////// #define GFX_USE_GINPUT TRUE #define GINPUT_NEED_MOUSE TRUE #define GINPUT_TOUCH_STARTRAW TRUE // #define GINPUT_TOUCH_NOTOUCH FALSE // #define GINPUT_TOUCH_NOCALIBRATE FALSE // #define GINPUT_TOUCH_NOCALIBRATE_GUI FALSE // #define GINPUT_MOUSE_POLL_PERIOD 25 // #define GINPUT_MOUSE_CLICK_TIME 300 // #define GINPUT_TOUCH_CXTCLICK_TIME 700 // #define GINPUT_TOUCH_USER_CALIBRATION_LOAD FALSE // #define GINPUT_TOUCH_USER_CALIBRATION_SAVE FALSE // #define GMOUSE_DRIVER_LIST GMOUSEVMT_Win32, GMOUSEVMT_Win32 //#define GINPUT_NEED_KEYBOARD FALSE // #define GINPUT_KEYBOARD_POLL_PERIOD 200 // #define GKEYBOARD_DRIVER_LIST GKEYBOARDVMT_Win32, GKEYBOARDVMT_Win32 // #define GKEYBOARD_LAYOUT_OFF FALSE // #define GKEYBOARD_LAYOUT_SCANCODE2_US FALSE //#define GINPUT_NEED_TOGGLE FALSE //#define GINPUT_NEED_DIAL FALSE /////////////////////////////////////////////////////////////////////////// // GFILE // /////////////////////////////////////////////////////////////////////////// #define GFX_USE_GFILE TRUE #define GFILE_NEED_PRINTG TRUE //#define GFILE_NEED_SCANG FALSE #define GFILE_NEED_STRINGS TRUE #define GFILE_NEED_FILELISTS TRUE //#define GFILE_NEED_STDIO FALSE //#define GFILE_NEED_NOAUTOMOUNT FALSE //#define GFILE_NEED_NOAUTOSYNC FALSE //#define GFILE_NEED_MEMFS FALSE //#define GFILE_NEED_ROMFS FALSE //#define GFILE_NEED_RAMFS FALSE #define GFILE_NEED_FATFS TRUE //#define GFILE_NEED_NATIVEFS FALSE //#define GFILE_NEED_CHBIOSFS FALSE //#define GFILE_ALLOW_FLOATS FALSE //#define GFILE_ALLOW_DEVICESPECIFIC FALSE //#define GFILE_MAX_GFILES 3 /////////////////////////////////////////////////////////////////////////// // GADC // /////////////////////////////////////////////////////////////////////////// //#define GFX_USE_GADC FALSE // #define GADC_MAX_LOWSPEED_DEVICES 4 /////////////////////////////////////////////////////////////////////////// // GAUDIO // /////////////////////////////////////////////////////////////////////////// //#define GFX_USE_GAUDIO FALSE // #define GAUDIO_NEED_PLAY FALSE // #define GAUDIO_NEED_RECORD FALSE /////////////////////////////////////////////////////////////////////////// // GMISC // /////////////////////////////////////////////////////////////////////////// //#define GFX_USE_GMISC FALSE //#define GMISC_NEED_ARRAYOPS FALSE //#define GMISC_NEED_FASTTRIG FALSE //#define GMISC_NEED_FIXEDTRIG FALSE //#define GMISC_NEED_INVSQRT FALSE // #define GMISC_INVSQRT_MIXED_ENDIAN FALSE // #define GMISC_INVSQRT_REAL_SLOW FALSE //#define GMISC_NEED_MATRIXFLOAT2D FALSE //#define GMISC_NEED_MATRIXFIXED2D FALSE //#define GMISC_NEED_HITTEST_POLY FALSE #endif /* _GFXCONF_H */ What may be important is I am using the FreeRTOS with following init (according to Wiki): static void taskInit(void* pvParameters) { gfxInit(); guiCreate(); while(1); } int main(void) { // Create the initialization task xTaskCreate(taskInit, "Initialization", STACK_SIZE_MIN, 0, tskIDLE_PRIORITY, 0); // Initialize FreeRTOS. This will start the scheduler. vTaskStartScheduler(); } Could you please look at it? Maybe I am doing some weird stuff or I am operating with pointers in a wrong maner. Thank you in advance P.S Forgot to mention- statusbarDraw and mousePressDown are of course declared- mousePressDown although is an empty function. Edited August 2, 2017 by Oshel Link to comment Share on other sites More sharing options...
inmarket Posted August 3, 2017 Report Share Posted August 3, 2017 The first thing I noticed from a quick scan of the above code is that you have call _widgetCreate with a first parameter of NULL. I can't remember if this is internally checked and replaced with GDISP (the first display) but I suspect not. Try replacing the NULL with GDISP. Another trick that will help is to try compiling on window/linux/osx first as you can use the full debugging capabilities on those platforms to chase bugs. Most uGFX programs should be fully portable if written appropriately eg most of the provided demos are portable to any uGFX supported platforms with no code changes. Link to comment Share on other sites More sharing options...
Oshel Posted August 3, 2017 Author Report Share Posted August 3, 2017 Hello, Thank you for the reply. I will give it a shot at home. But as I now look into the libraries I can see that function _gwindowCreate (which is called after _gwidgetCreate) takes g argument without replacing: GHandle _gwindowCreate(GDisplay *g, GWindowObject *pgw, const GWindowInit *pInit, const gwinVMT *vmt, uint32_t flags) { // Allocate the structure if necessary if (!pgw) { if (!(pgw = gfxAlloc(vmt->size))) return 0; pgw->flags = flags|GWIN_FLG_DYNAMIC; } else pgw->flags = flags; // Initialise all basic fields pgw->display = g; ... } It might be it! I will also try compiling on win32- first I need to find example projects or tutorials how to start with it Link to comment Share on other sites More sharing options...
Joel Bodenmann Posted August 3, 2017 Report Share Posted August 3, 2017 Hello @Oshel and welcome to the µGFX community! Thank you very much for your positive feedback - we appreciate it a lot! Regarding the win32 stuff: I can upload a bare win32 demo project (basically a hello world) that uses Makefiles if you'd like. It's well tested. Link to comment Share on other sites More sharing options...
Oshel Posted August 3, 2017 Author Report Share Posted August 3, 2017 Hello @Joel Bodenmann! It would be nice of you if you do that- it will spare me some time Link to comment Share on other sites More sharing options...
Joel Bodenmann Posted August 3, 2017 Report Share Posted August 3, 2017 Here you go. Let me know whether that worked out for you. Link to comment Share on other sites More sharing options...
Oshel Posted August 3, 2017 Author Report Share Posted August 3, 2017 (edited) Thank you Joel- I will try it later. I tried the solution given by @inmarket and everything started to work. Thank you very much I have now another question- let's say that I want to store pointers to images just like you did in the example with gdispImage*. But I want to store this image on SD Card. I wrote following code: void guiCreate ( void) { ... pointer->time[0] = '\0'; pointer->date[0] = '\0'; gfileMount('F', ""); imgFile = gfileOpen("icon1.bmp", "r"); gdispImageOpenGFile(pointer->icon1, imgFile); ... } Question 1 is if the (GFILE* imgFile) handler can be local? I mean when I open the file and then using this handler I open the image do I still need this handler? Question 2 is if it is right approach to store gdispImage* handlers in structure or better solution is to store for example images names and when widget has to be updated then open proper file and draw it. Question 3 is that in the code above gdispImageOpenGFile function causes HardFault (again <3). I gues I am doing something wrong again and I can not use pointer->icon1 that way. Btw. imgFile is global. And it normaly works when using normal, global GFILE and gdispImage objects. Edited August 3, 2017 by Oshel Link to comment Share on other sites More sharing options...
Oshel Posted August 3, 2017 Author Report Share Posted August 3, 2017 Well, just after writing "Btw" I've noticed that actually *gdispImage icon1 has not been pointing anywhere... Now it works- I have changed struct to: typedef struct { GWidgetObject widget; char time[9]; uint8_t sec; uint8_t min; uint8_t hrs; char date[9]; uint8_t day; uint8_t mth; uint8_t yrs; gdispImage icon1; gdispImage icon2; } tStatusbar; But still- is it better to store pointers or objects? If pointers- should these objects be declared as globals normally? Link to comment Share on other sites More sharing options...
inmarket Posted August 3, 2017 Report Share Posted August 3, 2017 1. The open file is required by the image during rendering so the file should only be closed once you are completely finished with the image. Its been a long time since I looked at the image stuff but from memory when you destroy the image object it will close the file for you - in which case you won't need the intermediary GFILE pointer again. Check the demo programs or the uGFX code (or even the doco) to be sure. 2. Keeping a file open keeps the GFILE allocated (and there is a fixed number of them in the system) however it is definitely faster than reopening/recreating the image each time. In the end I would typically keep things open until they are definitely not needed again. Next post: there is no need for them to be global. Whether to store a static object or to dynamically allocate the object and store the pointer is up to you. Using a static object saves calls to the memory allocator Note: If loading images from SD-CARD remember the issue of the card speed. uGFX generally does not load the image into RAM before rendering it thus saving lots of RAM. uGFX is the only image handling library that I know of that is capable of doing that. We custom wrote our own renderers for exactly that reason. The downside is that the image has to be reread from the card everytime it is rendered which affects the rendering speed. This is also the reason the image GFILE needs to be kept open for rendering. uGFX has the gdispImageCache call which can prevent the image being reread from disk each render by caching the image in RAM. When you call gdispImageCache the system tries to allocate memory for the image. If it succeeds you have a nice fast image and a lot less ram. If it fails to allocate enough ram you get the image still rendering (slowly) from the card but nothing stops working. It is therefore possible for the programmer to control what images are held in ram, in rom, or on sdcard and the performance compromises associated with that. Link to comment Share on other sites More sharing options...
Oshel Posted August 3, 2017 Author Report Share Posted August 3, 2017 Well, since last post I've managed to write some simple GUI, everything works smooth, even from SD card. I am using images only for icons etc- additionally these are bmps couse I don't have enough RAM to handle PNG/GIF (at least errors say that). I was asking you about GFILEs objects couse I did not understood them well- now I can see that it doesn't matter if I declare GFILE* pointer because... this is just a pointer to created GFILE object. Everything's clear now. But now I've encountered new problem- I opened like 20 files from SD card in one task of FreeRTOS (the Init task- so far the only task). When there were 19 objects- everything went well. But when I opened another file I got error that there is no such file in the filesystem (GDISP_IMAGE_ERR_NOSUCHFILE). I was sure that there was that file couse I could render it if I ignore other images. I figured out that increasing configTOTAL_HEAP_SIZE parameter (from 17 kB to 20 kB) helped to render all images. Could you explain this phenomena to me? How uGFX's heap size corresponds with declared FreeRTOS's one. And how does is correspond with declared heap and stack size of uC and RAM capacity at all. I am little bit confused. I don't want to write GUIs which consumes too much RAM than it needs only because I don't understand something. Well, I go sleep now- enough for today Link to comment Share on other sites More sharing options...
inmarket Posted August 4, 2017 Report Share Posted August 4, 2017 On FreeRTOS uGFX uses the FreeRTOS heap functions so the heap on uGFX is the FreeRTOS heap. As for what FreeRTOS does I am not really able to answer that. With 17K heap I am guessing you just ran out of memory. Heap memory is obviosly needed by FreeRTOS for file management and by all sorts of bits and pieces for uGFX. Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now