Jump to content

gwinSetVisible generated HardFault


Oshel

Recommended Posts

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.

aaaaa.png

Edited by Oshel
Link to comment
Share on other sites

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

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

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 by Oshel
Link to comment
Share on other sites

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

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

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

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

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 account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...