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

Update Raspberry Pi samples suggesting modifying config.txt settings #2297

Open
ChayoteJarocho opened this issue Mar 21, 2024 · 5 comments
Open
Labels
bug Something isn't working Priority:2 Work that is important, but not critical for the release up-for-grabs Good issue for external contributors to iot

Comments

@ChayoteJarocho
Copy link

The latest available version of Raspberry Pi OS (Bookworm) has changed the location of the config.txt file where you can enable the SPI hardware interface. It used to be in /boot/config.txt, and now it is located in /boot/firmware/config.txt. Also, the setting went through different names, but the latest name is dtparam=spi=on.

Some of the sample docs still have the old instructions and need updating. For example, the nRF24L01 sample: https://github.com/dotnet/iot/blob/main/src/devices/README.md#important

@ChayoteJarocho ChayoteJarocho added the bug Something isn't working label Mar 21, 2024
@pgrawehr
Copy link
Contributor

When was that change made and where is it documented? I set up a version of bookworm about a month ago and there everything was thr same as before.

@krwq krwq added the Needs: Author Feedback We are waiting for author to react to feedback (action required) label Mar 28, 2024
@pgrawehr
Copy link
Contributor

pgrawehr commented Apr 3, 2024

I looked at it and can confirm the change. The only change appears to be the place of the file, though. Also, there's a file in the original place that contains the information to look in the new place - people should be able to understand that.

@Ellerbach
Copy link
Member

[Triage] If you are willing to adjust the documentation in the repository, feel free. In that case, please make sure you are clear about the fact that depending on the OS version, it's one or the other. The params should be already good on what we've been checking. We ❤️ PR so looking forward to reviewing it 😉

Copy link
Contributor

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment.

@pgrawehr pgrawehr added up-for-grabs Good issue for external contributors to iot and removed Needs: Author Feedback We are waiting for author to react to feedback (action required) Status: No Recent Activity labels Apr 9, 2024
@krwq krwq added the Priority:2 Work that is important, but not critical for the release label Apr 11, 2024
@krwq
Copy link
Member

krwq commented Apr 11, 2024

[Triage] on top of docs we need to update Board class config reading logic as part of this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority:2 Work that is important, but not critical for the release up-for-grabs Good issue for external contributors to iot
Projects
None yet
Development

No branches or pull requests

4 participants