Skip to content

Fix portTTICK_Period/portTICK_Rate, CONFIG_DAC_I2C_ADDR, tas5805m_fault_monitor_task#218

Open
Sturi2011 wants to merge 9 commits intoCarlosDerSeher:developfrom
Sturi2011:master
Open

Fix portTTICK_Period/portTICK_Rate, CONFIG_DAC_I2C_ADDR, tas5805m_fault_monitor_task#218
Sturi2011 wants to merge 9 commits intoCarlosDerSeher:developfrom
Sturi2011:master

Conversation

@Sturi2011
Copy link

Replace of portTTICK_Period/portTICK_Rate by pdMS_TO_TICKS in driver and board

fix for hard coded i2c address in tas5805m_reg_cfg.h

extend tas5805m_fault_monitor_task to 3072 because of ESP32S3 minimum Task Size is about 1,6K

@CarlosDerSeher CarlosDerSeher changed the base branch from master to develop March 18, 2026 20:03
@CarlosDerSeher
Copy link
Owner

Is 3072 really needed? Can't we do less?

@Sturi2011
Copy link
Author

I will do some more testing this weekend after i soldered my third testing board.
I expect the Size is depending on local strings of this task so after first fault (e.g. clock) the whole
tas5805m_decode_faults will be loaded into thread and the size is exploding
because the most worst stack Size is the size of all chains...

I will check....

Greetings Andreas

@CarlosDerSeher
Copy link
Owner

Maybe you could place the fault strings to flash memory? As global consts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants