2020-05-24 01:17 AM
Hello,
i used the STM32CubeIDE Version: 1.1.0 with TouchGFX designer 4.12.3 to build a project for the F746G-Disco. Everything worked fine, i was able to display animations etc.
Then, i made the changes to my custom board (based on Disco schematics) to use a 5 Inches (800x480) TFT and i was able to display some boxes/images but when i tried to display some animated images, the animation was not working (seems frozen).
The Touch panel is not configured yet, due to different driver than Disco.
Could this be a hardware issue? I am attaching the screen datasheet and the project, in case someone could help.
Project:
https://drive.google.com/file/d/1ibuGvqsCGqzeZYnflSGMnCdHznQdUkfN/view?usp=sharing
Small video:
https://www.youtube.com/watch?v=aE4VB7f4UFw
Solved! Go to Solution.
2020-05-27 07:36 AM
it should be related to the amount of framebuffers you're using. tGFX allows you to use 1,2 or 3. the 3rd is also called 'animation storage' so enabling that might help you a bit.
First I would make sure that your RAM is actually working - one can never be too sure ;)
when you have verified that it works you should have a look at the linker scrips and verify that you are using the external RAM for storage.
Then you can look into the amount of buffers you assign to the project. The 746-Disco can do all 3 buffers using the SDRAM so you should be able to do that as well.
2020-05-27 09:00 AM
it should be related to the amount of framebuffers you're using. tGFX allows you to use 1,2 or 3. the 3rd is also called 'animation storage' so enabling that might help you a bit.
I am not able to enable the 2nd framebuffer.
The .ioc is uploaded here in case someone can check it: https://drive.google.com/file/d/1foGZArtejr9RZqs7vX4vzj8qzmRgUweo/view?usp=sharing
First I would make sure that your RAM is actually working - one can never be too sure ;)
I ran the "STM32Cube_FW_F7_V1.15.0\Projects\STM32746G-Discovery\Examples\FMC\FMC_SDRAM\EWARM" and seems RAM is working for the defined area. I do not know how to check the whole RAM area.
2020-05-27 11:52 PM
Layers are not the same as framebuffers
With that said, you need to find the following:
setFrameBufferStartAddresses((void*)frameBuf, (void*)(frameBuf + sizeof(frameBuf)/(sizeof(uint32_t)*2)), (void*)0);
In my project it is located in:
/STM32H7B3I_DISCO/Application/User/TouchGFX/target/generated/TouchGFXGeneratedHAL.cpp
the method takes 3 arguments:
/**
* @fn virtual void HAL::setFrameBufferStartAddresses(void* frameBuffer, void* doubleBuffer, void* animationStorage)
*
* @brief Sets frame buffer start addresses.
*
* Sets individual frame buffer start addresses.
*
* @param [in] frameBuffer Buffer for frame buffer data, must be non-null.
* @param [in] doubleBuffer If non-null, buffer for double buffer data. If null double buffering is disabled.
* @param [in] animationStorage If non-null, the animation storage. If null animation storage is disabled.
*/
so you'd want to add the adress of the animation storage instead of the (void*)0 in the last argument. you can calculate the address the same way the second framebuffer is calculated.
2020-05-28 12:59 AM
On TouchGFX\target\BoardConfiguration.cpp i added:
static uint32_t frameBuf0 = (uint32_t)(0xc0000000);
static uint32_t frameBuf1 = (uint32_t)(0xc0400000); // Added
static uint32_t frameBuf2 = (uint32_t)(0xc0800000); // Added
void touchgfx_init()
{
uint16_t dispWidth = 800;
uint16_t dispHeight = 480;
HAL& hal = touchgfx_generic_init<STM32F7HAL>(dma, display, tc, dispWidth, dispHeight,(uint16_t*) 0,
0, 0);
/*hal.setFrameBufferStartAddress((uint16_t*)frameBuf0, bitdepth ,true , true);*/
hal.setFrameBufferStartAddresses((uint16_t*)frameBuf0, (uint16_t*)frameBuf1 , (uint16_t*)frameBuf2); // Added
.
.
.
Still the same issue
2020-05-28 01:38 AM
how do you calculate the address for the frameBuf1 and 2?
2020-05-28 01:54 AM
I saw the 0xC0400000 address from the Cube when i tried to define second Layer..
Based on the calculations i tried, 800*480*3 = 0x00119400 so the frameBuf1 should be on 0xC0119400 and the frameBuf2 on 0xC0232800?
Tried it too without success
2020-05-28 12:21 PM
Do you see the LTDC interrupt getting fired: LTDC_IRQHandler -> HAL_LTDC_LineEvenCallback()?
2020-05-28 05:42 PM
One more test: place the assets into the intenal flash: use two/three images for the animation instead of 10 then place them into the internal flash and see if things change. This test would rule out any issue with QSPI NOR memory.
2020-05-29 12:47 PM
Do you see the LTDC interrupt getting fired: LTDC_IRQHandler -> HAL_LTDC_LineEvenCallback()?
Yes it gets fired
One more test: place the assets into the intenal flash: use two/three images for the animation instead of 10 then place them into the internal flash and see if things change. This test would rule out any issue with QSPI NOR memory.
Done it, still the same issue
As i tested, issues not only on animated images... If i set 2 screens to change 1->2 and 2->1 using interactions, it works only on RGB565...! On RGB888 i just see the first screen
2020-05-30 01:57 AM
In order to make the QSPI NOR Flash work, i made some changes on Flash .ld file. Should i change Ram .ld too? I am trying to find some example or guide how/what to change to define my external RAM.
STM32F746NGHX_RAM.ld:
/*
******************************************************************************
**
** File : LinkerScript.ld (debug in RAM dedicated)
**
** Author : Auto-generated by STM32CubeIDE
**
** Abstract : Linker script for STM32F746G-DISCO Board embedding STM32F746NGHx Device from STM32F7 series
** 1024Kbytes FLASH
** 320Kbytes RAM
**
** Set heap size, stack size and stack location according
** to application requirements.
**
** Set memory bank area and size if external memory is used.
**
** Target : STMicroelectronics STM32
**
** Distribution: The file is distributed as is without any warranty
** of any kind.
**
*****************************************************************************
** @attention
**
** <h2><center>© COPYRIGHT(c) 2020 STMicroelectronics</center></h2>
**
** Redistribution and use in source and binary forms, with or without modification,
** are permitted provided that the following conditions are met:
** 1. Redistributions of source code must retain the above copyright notice,
** this list of conditions and the following disclaimer.
** 2. Redistributions in binary form must reproduce the above copyright notice,
** this list of conditions and the following disclaimer in the documentation
** and/or other materials provided with the distribution.
** 3. Neither the name of STMicroelectronics nor the names of its contributors
** may be used to endorse or promote products derived from this software
** without specific prior written permission.
**
** THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
** AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
** IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
** DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
** FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
** DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
** SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
** CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
** OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
** OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
**
*****************************************************************************
*/
/* Entry Point */
ENTRY(Reset_Handler)
/* Highest address of the user mode stack */
_estack = ORIGIN(RAM) + LENGTH(RAM); /* end of "RAM" Ram type memory */
_Min_Heap_Size = 0x200; /* required amount of heap */
_Min_Stack_Size = 0x400; /* required amount of stack */
/* Memories definition */
MEMORY
{
RAM (xrw) : ORIGIN = 0x20000000, LENGTH = 320K
FLASH (rx) : ORIGIN = 0x8000000, LENGTH = 1024K
}
/* Sections */
SECTIONS
{
/* The startup code into "RAM" Ram type memory */
.isr_vector :
{
. = ALIGN(4);
KEEP(*(.isr_vector)) /* Startup code */
. = ALIGN(4);
} >RAM
/* The program code and other data into "RAM" Ram type memory */
.text :
{
. = ALIGN(4);
*(.text) /* .text sections (code) */
*(.text*) /* .text* sections (code) */
*(.glue_7) /* glue arm to thumb code */
*(.glue_7t) /* glue thumb to arm code */
*(.eh_frame)
KEEP (*(.init))
KEEP (*(.fini))
. = ALIGN(4);
_etext = .; /* define a global symbols at end of code */
} >RAM
/* Constant data into "RAM" Ram type memory */
.rodata :
{
. = ALIGN(4);
*(.rodata) /* .rodata sections (constants, strings, etc.) */
*(.rodata*) /* .rodata* sections (constants, strings, etc.) */
. = ALIGN(4);
} >RAM
.ARM.extab : {
. = ALIGN(4);
*(.ARM.extab* .gnu.linkonce.armextab.*)
. = ALIGN(4);
} >RAM
.ARM : {
. = ALIGN(4);
__exidx_start = .;
*(.ARM.exidx*)
__exidx_end = .;
. = ALIGN(4);
} >RAM
.preinit_array :
{
. = ALIGN(4);
PROVIDE_HIDDEN (__preinit_array_start = .);
KEEP (*(.preinit_array*))
PROVIDE_HIDDEN (__preinit_array_end = .);
. = ALIGN(4);
} >RAM
.init_array :
{
. = ALIGN(4);
PROVIDE_HIDDEN (__init_array_start = .);
KEEP (*(SORT(.init_array.*)))
KEEP (*(.init_array*))
PROVIDE_HIDDEN (__init_array_end = .);
. = ALIGN(4);
} >RAM
.fini_array :
{
. = ALIGN(4);
PROVIDE_HIDDEN (__fini_array_start = .);
KEEP (*(SORT(.fini_array.*)))
KEEP (*(.fini_array*))
PROVIDE_HIDDEN (__fini_array_end = .);
. = ALIGN(4);
} >RAM
/* Used by the startup to initialize data */
_sidata = LOADADDR(.data);
/* Initialized data sections into "RAM" Ram type memory */
.data :
{
. = ALIGN(4);
_sdata = .; /* create a global symbol at data start */
*(.data) /* .data sections */
*(.data*) /* .data* sections */
. = ALIGN(4);
_edata = .; /* define a global symbol at data end */
} >RAM
/* Uninitialized data section into "RAM" Ram type memory */
. = ALIGN(4);
.bss :
{
/* This is used by the startup in order to initialize the .bss section */
_sbss = .; /* define a global symbol at bss start */
__bss_start__ = _sbss;
*(.bss)
*(.bss*)
*(COMMON)
. = ALIGN(4);
_ebss = .; /* define a global symbol at bss end */
__bss_end__ = _ebss;
} >RAM
/* User_heap_stack section, used to check that there is enough "RAM" Ram type memory left */
._user_heap_stack :
{
. = ALIGN(8);
PROVIDE ( end = . );
PROVIDE ( _end = . );
. = . + _Min_Heap_Size;
. = . + _Min_Stack_Size;
. = ALIGN(8);
} >RAM
/* Remove information from the compiler libraries */
/DISCARD/ :
{
libc.a ( * )
libm.a ( * )
libgcc.a ( * )
}
.ARM.attributes 0 : { *(.ARM.attributes) }
}