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

Add OPi NVME install docs #1608

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ArchdukeTim
Copy link
Contributor

@ArchdukeTim ArchdukeTim commented Nov 27, 2024

Verified working with latest beta

@ArchdukeTim ArchdukeTim requested a review from a team as a code owner November 27, 2024 02:07
@Juniormunk
Copy link
Contributor

Juniormunk commented Nov 27, 2024

Did we need to install anything back on the opi image? Do we need to tell them to install the os on the sd card then do these commands with the SSD installed? How long should the user expect this to take?

@ArchdukeTim
Copy link
Contributor Author

ArchdukeTim commented Nov 27, 2024

No, yes, 5 minutes (added)

@Juniormunk
Copy link
Contributor

Juniormunk commented Nov 27, 2024

Should we recommend a brand name SSD (or at least an example)? Make sure teams don't go buy that $10 SSD?

Copy link
Contributor

@Juniormunk Juniormunk left a comment

Choose a reason for hiding this comment

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

LGTM

@mcm001
Copy link
Contributor

mcm001 commented Dec 5, 2024

Do we want to qualify this any more strongly? A sd card -will- work, this is just another option.

@Juniormunk
Copy link
Contributor

Do we want to qualify this any more strongly? A sd card -will- work, this is just another option.

I think it's fine considering it's in the advanced install section and it says "MicroSD cards are not the best" which is true.

@mcm001
Copy link
Contributor

mcm001 commented Dec 5, 2024

I just don't want a bunch of teams cargo culting this because we published it. That might be true, but I don't think that people will necessarily read that sentence that way.

Copy link
Contributor

@Juniormunk Juniormunk left a comment

Choose a reason for hiding this comment

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

We shall please the gods with a note that industrial sd cards are still known to work well.

@stephenjust
Copy link
Contributor

stephenjust commented Dec 7, 2024

Should we call out that the uboot flash is only required once? Once that has been done, you can image the SSD directly with a USB M.2 writer. (I've been working that way the entire off-season, skipping the SD card entirely for updates)

@mcm001
Copy link
Contributor

mcm001 commented Dec 7, 2024

Yeah that would be good to mention.

@Juniormunk
Copy link
Contributor

How does this work with reflashing the opi?

@stephenjust
Copy link
Contributor

The key is that the uboot flash process to enable SSD boot writes to an SPI Flash chip soldered to the OPi board itself. It's not changing anything about the OS on the SD card or SSD. Think of it just like changing a BIOS setting to change the boot order. You would need to re-flash the SPI Flash memory explicitly to revert the change.

@ArchdukeTim
Copy link
Contributor Author

I found that the uboot flash step works only with the v2.4.0 of the Ubuntu-rockchip image.

Will photonvision be updated to be based off of these or should I update the step to flash one of these to a microsd card as the initial setup image?

@ArchdukeTim
Copy link
Contributor Author

ArchdukeTim commented Dec 19, 2024

We shall please the gods with a note that industrial sd cards are still known to work well.

@Juniormunk In our experience the sd card type wasn't the weak point. It was the form factor. The physical card self-ejected during competition. Industrial or not won't fix that issue.

@crschardt
Copy link
Contributor

I made a PR for that PhotonVision/photon-image-modifier#38. I just haven't pushed to get a tag made since there are a few other updates I'd like to make.

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.

5 participants