-
-
Notifications
You must be signed in to change notification settings - Fork 658
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
pine64-rock64: init #1280
base: master
Are you sure you want to change the base?
pine64-rock64: init #1280
Conversation
"pcie_rockchip_host" | ||
"phy_rockchip_pcie" | ||
"sdhci_pci" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this not detected by nixos-generate-config?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mhm. but I guess our installer wouldn't boot, so the normal install workflow wouldn't boot.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nixos-hardware/pine64/rockpro64/default.nix
Lines 3 to 8 in b12e314
boot.initrd.kernelModules = [ | |
# PCIe/NVMe | |
"nvme" | |
"pcie_rockchip_host" | |
"phy_rockchip_pcie" | |
]; |
my local config (and, in turn, what was done in this PR) is inspired by how the RockPro64 is configured
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm currently using a half-functional build of u-boot/tow-boot, generated from this draft PR Tow-Boot/Tow-Boot#322, so I don't really know how the usual NixOS install process would go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can leave those modules in here if you also add a README how to bootstrap the rest of the configuration i.e. instructions to build a installer image.
Without this context it would have little value to have this module in nixos-hardware.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's just the generic aarch64 uefi installer iso
I guess it'd be more common to go the u-boot+sd-card image route, like it says on the wiki
which is unlike what I have -- tow-boot+normal EFI installer
I wouldn't really recommend people use a draft PR for their own stuff though
"phy_rockchip_pcie" | ||
"sdhci_pci" | ||
]; | ||
kernelModules = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is any of this needed? I don't remember needing to specify this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I remember some of them not being detected automatically, but I can remove that boot.kernelModules
list entirely if it's out-of-scope for nixos-hardware
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just copied it from my local config
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So it won't show any graphics if you attach a monitor, or what does this fix? Usually the kernel will match pci/usb vendor ids with drivers and load them as needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am fine with adding modules if they are actually not recognize by the hardware or if there is a good reason to add them to early boot
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense, then I'll boot once without this list, and I'll compare which modules are loaded
might take a bit, I won't have access to the SBC for a few days
Description of changes
Added
pine64-rock64
Things done
nixos-hardware
andimporting it via
<nixos-hardware>
or Flake input