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

dietpi-config does not change i2c_baudrate correctly #2966

Closed
flashspys opened this issue Jul 7, 2019 · 4 comments
Closed

dietpi-config does not change i2c_baudrate correctly #2966

flashspys opened this issue Jul 7, 2019 · 4 comments
Labels
Milestone

Comments

@flashspys
Copy link

Creating a bug report/issue

Required Information

  • DietPi version | G_DIETPI_VERSION_CORE=6 G_DIETPI_VERSION_SUB=25 G_DIETPI_VERSION_RC=3 G_GITBRANCH='master' G_GITOWNER='MichaIng'
  • Distro version | stretch
  • Kernel version | Linux link 4.19.42-v7+ #1219 SMP Tue May 14 21:20:58 BST 2019 armv7l GNU/Linux
  • SBC device | RPi 3 Model B (armv7l)
  • Power supply used | 5V 2A
  • SDcard used | Samsung 32GB

Steps to reproduce

  1. Set the i2c_arm_baudrate via dietpi-config to e.g. 10kHz

Expected behaviour

  • As described in e.g. the official doc or in various other sources, there should be an entry in config.txt like dtparam=i2c_arm_baudrate=10000

Actual behaviour

  • The entry is wrong: i2c_arm_baudrate=10000. The dtparam= is missing. Because of this, dietpi-config does not change the baudrate effectively.

I changed the entry in /DietPi/config.txt manually and then the baudrate changed.

@MichaIng
Copy link
Owner

MichaIng commented Jul 7, 2019

@flashspys
Many thanks for your report. I mark this for investigation.

@MichaIng
Copy link
Owner

MichaIng commented Jul 7, 2019

Resolved for v6.26: #2967

Very good find, this wrong handling is already in our code since loooong time, at least down to v6.10. Seems like not much users needed any different baudrate than the default 100,000 😄.

@flashspys
Copy link
Author

I just switched to the dev branch, works as expected 👍

@MichaIng
Copy link
Owner

MichaIng commented Jul 7, 2019

@flashspys
Many thanks for testing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants