Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing initializer for ext_clk_freq_hz (IDFGH-14501) #15271

Closed
3 tasks done
floitsch opened this issue Jan 23, 2025 · 3 comments
Closed
3 tasks done

Missing initializer for ext_clk_freq_hz (IDFGH-14501) #15271

floitsch opened this issue Jan 23, 2025 · 3 comments
Assignees
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally

Comments

@floitsch
Copy link
Contributor

floitsch commented Jan 23, 2025

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

The I2S_STD_CLK_DEFAULT_CONFIG macro is not initializing the field ext_clk_freq_hz which is available on SOC_I2S_HW_VERSION_2. When compiling for platforms with that hw-version (like the S3), there are now warnigs:

warning: missing initializer for member 'i2s_std_clk_config_t::ext_clk_freq_hz' [-Wmissing-field-initializers]

#define I2S_STD_CLK_DEFAULT_CONFIG(rate) { \

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jan 23, 2025
@github-actions github-actions bot changed the title Missing initializer for ext_clk_freq_hz Missing initializer for ext_clk_freq_hz (IDFGH-14501) Jan 23, 2025
@L-KAYA
Copy link
Collaborator

L-KAYA commented Jan 24, 2025

@floitsch Thanks for reporting! May I ask your environment? Arduino or using idf.py directly?

@espressif-bot espressif-bot added Status: In Progress Work is in progress and removed Status: Opened Issue is new labels Jan 24, 2025
@floitsch
Copy link
Contributor Author

idf.py directly.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: In Progress Work is in progress labels Jan 27, 2025
@wakwak-koba
Copy link

I applied the fix, but the error

i2s_std.h:220:1: error: designator order for field 'i2s_std_clk_config_t::ext_clk_freq_hz' does not match declaration order in 'i2s_std_clk_config_t'

thelazt@2bb9fb2#diff-a1137c1c92942c76ba853b95de0e85e14fbc3898d5490e3c02baba63d02ff4e6R218-R219

Do you need to change the order ?

.ext_clk_freq_hz = 0,
.mclk_multiple = I2S_MCLK_MULTIPLE_256,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

4 participants