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 instructions for use with rpi4 and 1.6 revision motherboard #6

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

MarioMasta64
Copy link

update instructions for use with rpi4 and 1.6

the pinout for 1.6 is the same however the window for messing with the eeprom is basically "run program then power on console" since the system usually turns off soon after turning on

removed the part about 1.6 needing extra connections that are tricky (this is referring to an lpc rebuild which is not actually required for eeprom management.)

i added the bcm source for rpi4 the compile instructions for 3 and 4 are the same besides the file used

risk510 and others added 10 commits January 3, 2020 18:09
note: you may also have to bridge 9 and 15 to use ground, if you need me to verify i can (its not in here just making a note in the additional details section)

the pinout for 1.6 is the same however the window for messing with the eeprom is basically non-existant
@@ -77,6 +91,8 @@ I added my own line into this. Use mine until the original OP fixes/merges my ch
v1 B+ | ```make p1b```
v2 B | ```make p2```
v3 B | ```make p3```
v4 | ```make p3```
=======
Copy link
Owner

@grimdoomer grimdoomer Feb 18, 2022

Choose a reason for hiding this comment

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

Remove line 95 since it shows up in the table cells.

sudo make check
sudo make install
```

Next we are going to download the latest source for PiPROM and compile it. Run the following commands which will move you back into the PiPROM folder and download the latest source code for it.

I added my own line into this. Use mine until the original OP fixes/merges my changes.
Copy link
Owner

Choose a reason for hiding this comment

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

Since these changes have been merged in as part of your PR this part should be removed since it is no longer applicable

@@ -41,7 +44,7 @@ GND | 6 | 2
cd ./PiPROM
```

Next we will setup bcm2835, a C library for the broadcom processor the Raspberry Pi uses. It is what PiPROM is built on and you will need to it compile PiPROM. Run the following commands to download and install bcm2835 v1.42. If you run into trouble please refer to the [bcm2835 website](http://www.airspayce.com/mikem/bcm2835/index.html).
Next we will setup bcm2835, a C library for the broadcom processor the Raspberry Pi uses. It is what PiPROM is built on and you will need to it compile PiPROM. If you have a Raspberry Pi 4 please use the second set of instructions instead. Run the following commands to download and install bcm2835 v1.42. If you run into trouble please refer to the [bcm2835 website](http://www.airspayce.com/mikem/bcm2835/index.html).
```
Copy link
Owner

Choose a reason for hiding this comment

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

There doesn't need to be two separate instructions here, just update the link/folder name for the existing set so it is the same for all pi versions

Copy link
Owner

@grimdoomer grimdoomer left a comment

Choose a reason for hiding this comment

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

Please address the comments I added and request another review. Once fixed I'll merge the changes in.

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

Successfully merging this pull request may close these issues.

3 participants