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

Typo in the Documentation #2364

Closed
mikulik86 opened this issue Nov 16, 2024 · 2 comments
Closed

Typo in the Documentation #2364

mikulik86 opened this issue Nov 16, 2024 · 2 comments
Labels
Priority:3 Work that is nice to have up-for-grabs Good issue for external contributors to iot

Comments

@mikulik86
Copy link

mikulik86 commented Nov 16, 2024

Hi, i think you have a typo in this article: https://github.com/dotnet/iot/blob/main/Documentation/raspi-pwm.md#enabling-hardware-pwm

In the note there is supposed to be

Adjust the previous line to be sudo nano /boot/config.txt

istead of

Adjust the previous line to be sudo nano /boot/firmware/config.txt

The typo also repeats later on.

@raffaeler
Copy link
Contributor

Good catch! This changed right before June 2023 according to the official documentation:
https://www.raspberrypi.com/documentation/computers/config_txt.html

Instead of the BIOS found on a conventional PC, Raspberry Pi devices use a configuration file called config.txt. The GPU reads config.txt before the Arm CPU and Linux initialise. Raspberry Pi OS looks for this file in the boot partition, located at /boot/firmware/.

Note:
Prior to Raspberry Pi OS Bookworm, Raspberry Pi OS stored the boot partition at /boot/.

I believe it's worth to mention both.
Are you interested in submitting a pull request for this fix?

@krwq krwq added up-for-grabs Good issue for external contributors to iot Priority:2 Work that is important, but not critical for the release Priority:3 Work that is nice to have and removed untriaged Priority:2 Work that is important, but not critical for the release labels Nov 21, 2024
@krwq
Copy link
Member

krwq commented Jan 23, 2025

[Triage] /boot/firmware/config.txt is correct and /boot/config.txt is the old path. We also have a note mentioning that in the README. Closing as done

@krwq krwq closed this as completed Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:3 Work that is nice to have up-for-grabs Good issue for external contributors to iot
Projects
None yet
Development

No branches or pull requests

3 participants