Skip to content

cpu/efm32: remove i2c clock 'reset'#22077

Open
basilfx wants to merge 1 commit intoRIOT-OS:masterfrom
basilfx:feature/efm32_i2c_clock_init
Open

cpu/efm32: remove i2c clock 'reset'#22077
basilfx wants to merge 1 commit intoRIOT-OS:masterfrom
basilfx:feature/efm32_i2c_clock_init

Conversation

@basilfx
Copy link
Member

@basilfx basilfx commented Feb 13, 2026

Contribution description

This was copied from the Gecko SDK I2C driver back in the days, but it does not add any proven value:

  • It does not take into account clock frequency (there should be delays in between)
  • No other drivers have this
  • It works without

Removing this saves about 24-52 bytes (depending on series).

Testing procedure

Take any EFM32-based board, ant interact with I2C.

Issues/PRs references

None.

@github-actions github-actions bot added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: cpu Area: CPU/MCU ports labels Feb 13, 2026
@basilfx basilfx added the Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation label Feb 13, 2026
@crasbe crasbe added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 13, 2026
@riot-ci
Copy link

riot-ci commented Feb 13, 2026

Murdock results

✔️ PASSED

7942dd0 cpu/efm32: remove i2c clock 'reset'

Success Failures Total Runtime
11005 0 11005 08m:39s

Artifacts

@basilfx
Copy link
Member Author

basilfx commented Feb 13, 2026

A few tests results, just to prove that I did test this and the EFM32 driver does not depend on this.

SLTB001a (ignore the `ccs811` error, I damaged it) Scherm­afbeelding 2026-02-13 om 21 41 56
SLSTK3701a (with #22065 and #22052) Scherm­afbeelding 2026-02-13 om 21 45 12
SLSTK3401a Scherm­afbeelding 2026-02-13 om 21 47 07
SLSTK3402a Scherm­afbeelding 2026-02-13 om 21 47 51
STK3600 (with I2C display and U8g2 test application) ![IMG_6351](https://github.com/user-attachments/assets/4ee1ca3d-b168-42c3-bb89-646b6ff84808)

This was copied from the Gecko SDK I2C driver, but it does not add any
proven value:

- It does not take into account clock frequency
- No other drivers have this
- It works without
@basilfx basilfx force-pushed the feature/efm32_i2c_clock_init branch from ae92f18 to 7942dd0 Compare February 15, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: cpu Area: CPU/MCU ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants