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

Explode the Pi 5 documentation all over the place #3522

Merged
merged 3 commits into from
Feb 19, 2024

Conversation

nathan-contino
Copy link
Collaborator

  • Closes Kill the Pi 5 page with 🔥  #3479
  • Minor updates to other sections so the new content doesn't mess with the flow
  • Deleted a table in the Power Supplies section that... existed twice? For some reason?
  • Mostly stripped-down copy from the Pi 5 page. Mostly.
  • And of course, removed the whole Pi 5 section!

On the main Pi site, I found a link to this page on our Pi 5 product page (reachable with just one Pi 5 click from the home page). Ctrl+F for "Raspberry Pi 5 documentation".

But it might be a good idea to add a redirect from https://www.raspberrypi.com/documentation/computers/raspberry-pi-5.html to https://www.raspberrypi.com/documentation/computers/raspberry-pi.html because I'm sure there are other places that link here and we'd rather not introduce a lot of 404s.

@aallan aallan added pi5 5️⃣ It's the everything computer. optimised. update Things change with time, this is a change labels Feb 1, 2024
Copy link
Contributor

@tdewey-rpi tdewey-rpi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aside from the extraneous instructions around PCIe connector enablement, LGTM

The other three pins on the Touch Display are used to connect the display to an original Raspberry Pi 1 Model A or B. Refer to our documentation on xref:display.adoc#legacy-support[legacy support] for more information.

NOTE: To identify an original Raspberry Pi, check the GPIO header connector. Only the original model has a 26-pin GPIO header connector; subsequent models have 40 pins.


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this newline serving a purpose, or is it just some cruft that's found it's way in?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cruft probably. It should affect the rendered output.

@@ -353,7 +353,7 @@ Hitting the temperature limit is not harmful to the SoC, but it will cause the C

When the core temperature is between 80°C and 85°C, the ARM cores will be throttled back. If the temperature exceeds 85°C, the ARM cores and the GPU will be throttled back.

For the Raspberry Pi 3 Model B+, the PCB technology has been changed to provide better heat dissipation and increased thermal mass. In addition, a soft temperature limit has been introduced, with the goal of maximising the time for which a device can "sprint" before reaching the hard limit at 85°C. When the soft limit is reached, the clock speed is reduced from 1.4GHz to 1.2GHz, and the operating voltage is reduced slightly. This reduces the rate of temperature increase: we trade a short period at 1.4GHz for a longer period at 1.2GHz. By default, the soft limit is 60°C. This can be changed via the `temp_soft_limit` setting in config.txt.
For the Raspberry Pi 3 Model B+, the PCB technology has been changed to provide better heat dissipation and increased thermal mass. In addition, a soft temperature limit has been introduced, with the goal of maximising the time for which a device can "sprint" before reaching the hard limit at 85°C. When the soft limit is reached, the clock speed is reduced from 1.4GHz to 1.2GHz, and the operating voltage is reduced slightly. This reduces the rate of temperature increase: we trade a short period at 1.4GHz for a longer period at 1.2GHz. By default, the soft limit is 60°C. This can be changed via the `temp_soft_limit` setting in `config.txt`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was surprised to see this in here, after the commit message referred to Pi5, but the change is certainly an improvement.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you look, the change here was Nate added quote marks around the config.txt file to make the filename monopaced. The text was already there. Shouldn't really have been in this PR!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes - that's why I was surprised!


At boot the fan is turned on, and the tachometer input is checked to see if the fan is spinning. If it is, then the `cooling_fan` device tree overlay is enabled. This overlay is in `bcm2712-rpi-5-b.dtb` by default, but with `status=disabled`.

The official fan is a Coolcox https://www.coolcox.com/products/pdf/CC3007.pdf[CC3007H05S] part.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Official fan in which configuration - case, Active Cooler, or both?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. Fixed.

Comment on lines 12 to 21
By default the PCIe connector is not enabled. To enable it, add the following line to `/boot/firmware/config.txt`:

[source]
----
dtparam=pciex1
----

Reboot with `sudo reboot` for the configuration changes to take effect.

NOTE: You can also use the alias `nvme`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we certain this is required? I've not had to do this on any recent firmware or Pi OS combo.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nathan-contino Can you confirm this is still necessary with the latest firmware?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately I don't have hardware to test this on. But if Tom says it isn't necessary for him, I believe it. I'll remove this bit.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I think the difference is whether we are using the PCIe as the boot disk or not. If we are booting from an NVMe drive then this isn't needed. You just need to make sure you have the boot order correct, and a recent firmware.

If you're booting from an SD Card you'll need this, so that PCIe is activated and you can use it as a drive. Alternatively, you'll need it if you're using PCIe for something that isn't NVMe.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a negative, @aallan. Booting from the SDcard on a Pi5b, I'm testing 'secure' NVMe drives via our M2 HAT+, and have not needed to use this directive to format and generally punish the candidate drives.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah. If you're using our hardware, then that's a HAT+. The HAT+ configures things for you. If you're not on our hardware you can't guarantee it's a HAT+. So what this section really needs is rewriting from that perspective. I think there's a NOTE block in there somewhere about this, but it needs to be cleaned up and all that put inline.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On it.


When you plug your Raspberry Pi into power for the first time, it will automatically turn on and boot into the operating system without having to push the button.

If you run Raspberry Pi Desktop, you can initiate a clean shutdown by briefly pressing the power button. A menu will appear asking whether you want to shutdown, reboot, or logout.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strictly speaking, you get a window rather than a menu. Not sure it's worth a change, though.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed! TY.

Very high-current devices, or devices which can draw a surge current such as certain modems and USB hard disks, will still require an external powered USB hub. The power requirements of the Raspberry Pi increase as you make use of the various interfaces on the Raspberry Pi. The GPIO pins can draw 50mA safely (note that that means 50mA distributed across all the pins: an individual GPIO pin can only safely draw 16mA), the HDMI port uses 50mA, the Camera Module requires 250mA, and keyboards and mice can take as little as 100mA or as much as 1000mA! Check the power rating of the devices you plan to connect to the Raspberry Pi and purchase a power supply accordingly. If you're not sure, we would advise you to buy a powered USB hub.

This is the typical amount of power (in amps) drawn by different Raspberry Pi models during standard processes:
The following table describes the amount of power (in amps) drawn by different Raspberry Pi models during various workloads:

|===
| | | Raspberry Pi 1B+ | Raspberry Pi 2B | Raspberry Pi 3B | Raspberry Pi Zero | Raspberry Pi 4B
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did we also characterise Raspberry Pi 5 B, or do we not intend to / intend to later?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't believe that this work has been done yet? Or at least I haven't seen numbers. @nathan-contino Can you follow up with the relevant folks and find out whether we have this data?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Who should I ping about this? Not sure who did this testing in the first place. I can do some of the testing myself for the 5, but it might be a good idea to re-run these tests with the latest Raspberry Pi OS for each device.

=== Power supply warnings

On all models of Raspberry Pi since the Raspberry Pi B+ (2014) except the Zero range, there is low-voltage detection circuitry that will detect if the supply voltage drops below 4.63V (±5%). This will result in an entry being added to the kernel log.

If you are seeing warnings, you should improve the power supply and/or cable, as low power can cause problems with corruption of SD cards, or erratic behaviour of the Raspberry Pi itself.
If you see warnings, switch to a better power supply and cable. Low power can corrupt storage or cause unpredictable behaviour within the Raspberry Pi.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I realise this is just the rework of the existing text - but I'm sort of uncomfortable with using 'better' and 'low' here - presumably we actually mean 'higher quality' and 'lower quality', and that comes through clearly enough, but it just reads a little too blunt.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. I think this need... softening. @nathan-contino Can you take another pass here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated and softened.

@nathan-contino
Copy link
Collaborator Author

Should be good to go, except for those benchmarks.

Copy link
Contributor

@tdewey-rpi tdewey-rpi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Largely LGTM - just one query about the syntax annotations


[source]
[source,perl]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this tagged as perl?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This syntax doesn't have a reasonable highlighting equivalent in our toolbox, so I just chose something that renders equal signs a different color so it's slightly easier to read large blocks of text with equal signs in between. Perl was a slightly arbitrary choice, but I tested bash and a couple of others and the square bracket [] syntax seems to mess up the highlighting. Perl, as we know, tolerates just about any string of characters without breaking, so I went with that 😄

@@ -52,16 +35,29 @@ $ sudo rpi-eeprom-config --edit

Replace the `BOOT_ORDER` line with the following line:

[source]
[source,perl]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As above - why perl?

@aallan
Copy link
Contributor

aallan commented Feb 19, 2024

YOLO!

@aallan aallan merged commit 2759c44 into develop Feb 19, 2024
1 check passed
@aallan aallan deleted the kill-pi-5-page-with-🔥 branch February 19, 2024 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pi5 5️⃣ It's the everything computer. optimised. update Things change with time, this is a change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kill the Pi 5 page with 🔥
3 participants