TouchGFX: advanceFrameBufferToRect() undefined reference

[Sandro_K]

Hello,

I have a STM32F777 with ST7789 Display controller over SPI.

To write the Framebuffer i have the following code:

 

void TouchGFXHAL::flushFrameBuffer(const touchgfx::Rect& rect)
{
 // Get Framebuffer Address
 uint8_t* frameBufferAddress = (uint8_t*)HAL::lockFrameBuffer();

 // Calc start address
 uint8_t* fbPtr = advanceFrameBufferToRect(frameBufferAddress, rect

 // Set window
 DisplaySetWindow(rect.x, rect.y, rect.width, rect.height);

 // Start DMA Transfer
 DisplaySendDataDMA(fbPtr, rect.width * rect.height * 2); 

 // Wait for DMA Transfer end
 if (osSemaphoreWait(DisplayTransferSemHandle, osWaitForever) == pdTRUE)
 {
 	HAL::unlockFrameBuffer();
 HAL::flushFrameBuffer(rect); 
 }
}

 

I can't build the code because the function advanceFrameBufferToRect() is defined as inline but without implementation in the header. The implementation in the source file is not accesible with the inline statement.

How can I fix this error? Is the code above correct?

Thanks for your reply!

 

[GaetanGodart]

Hello @Sandro_K ,

 

Have you looked at the TouchGFX SPI guide ?

 

Regards,

[Sandro_K]

Thanks for your reply. Yes, i have looked at the SPI guide but there ist not much information.

In the TouchGFXHAL.cpp file at function flushFrameBuffer(...)  there is some information:

 // Calling parent implementation of flushFrameBuffer(const touchgfx::Rect& rect).
 //
 // To overwrite the generated implementation, omit the call to the parent function
 // and implement the needed functionality here.
 // Please note, HAL::flushFrameBuffer(const touchgfx::Rect& rect) must
 // be called to notify the touchgfx framework that flush has been performed.
 // To calculate the start address of rect,
 // use advanceFrameBufferToRect(uint8_t* fbPtr, const touchgfx::Rect& rect)
 // defined in TouchGFXGeneratedHAL.cpp

I tried to implement these instructions. So I need the function advanceFrameBufferToRect(). 

And there comes the point. The function prototype is specified as inline function in the header file of the library. The implementation is in the .cpp file but this is not working because the function is defined as inline. I think there is a bug in the library. 

Either the function is implemented directly as an inline function in the header or the keyword inline is omitted.

[GaetanGodart]

Well you can check what you have in TouchGFXGeneratedHAL.hpp.
Personally, on the STM32C071 TBS which use an SPI display I have that:

The inline is not omitted and the function is not implemented in the hpp.

What do you have in your project?

 

Here is an example of implementation of flushFrameBuffer.

 

In the code you shared, on line 7, why is there no closing parenthesis and semicolon?

uint8_t* fbPtr = advanceFrameBufferToRect(frameBufferAddress, rect

 

 

Regards,

[Sandro_K]

This is my complete code:

 

void TouchGFXHAL::flushFrameBuffer(const touchgfx::Rect& rect)
{
 // Calling parent implementation of flushFrameBuffer(const touchgfx::Rect& rect).
 //
 // To overwrite the generated implementation, omit the call to the parent function
 // and implement the needed functionality here.
 // Please note, HAL::flushFrameBuffer(const touchgfx::Rect& rect) must
 // be called to notify the touchgfx framework that flush has been performed.
 // To calculate the start address of rect,
 // use advanceFrameBufferToRect(uint8_t* fbPtr, const touchgfx::Rect& rect)
 // defined in TouchGFXGeneratedHAL.cpp

 // Get framebuffer
 uint8_t* frameBufferAddress = (uint8_t*)HAL::lockFrameBuffer();

 // calculate start address
 uint8_t* fbPtr = advanceFrameBufferToRect(frameBufferAddress, rect);

 // Set the region on the display
 DisplaySetWindow(rect.x, rect.y, rect.width, rect.height);

 // Start DMA transfer
 DisplaySendDataDMA(fbPtr, rect.width * rect.height * 2); // 2 Bytes per Pixel (RGB565)

 // Wait for DMA transfer end
 if(osSemaphoreAcquire(DisplayTransferSemHandle, osWaitForever) == pdTRUE)
 {
 	// DMA transfer finished
 	HAL::unlockFrameBuffer(); 
 HAL::flushFrameBuffer(rect); 
 }
}

 

The closing parenthesis and semicolon was a copy & paste error. 

But with the function above i get the following error:

How can the code work if it is implemented as inline without implementation in the .hpp file? This is against the cpp definitions.

(I use STM32CubeIDE 1.17.0)

[Sandro_K]

See also: Inline Functions, C++ FAQ

 

 

How do you tell the compiler to make a member function inline? 
The declaration of an inline member function looks just like the declaration of a non-inline member function:

class Fred {
public:
 void f(int i, char c);
};
But when you define an inline member function (the {...} part), you prepend the member function’s definition with the keyword inline, and you (almost always) put the definition into a header file:

inline
void Fred::f(int i, char c)
{
 // ...
}
The reason you (almost always) put the definition (the {...} part) of an inline function in a header file is to avoid “unresolved external” errors from the linker. That error will occur if you put the inline function’s definition in a .cpp file and if that function is called from some other .cpp file.

 

 

  The hint in the article to the unresolved externals is exactly the error that occurs in my project