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

lg-hammerhead: init #144

Draft
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

thefloweringash
Copy link
Member

@thefloweringash thefloweringash commented May 6, 2020

This is a minimal configuration that allows booting and should serve as a starting point for anyone looking to work on this device. There are a few problems that still need resolving, but I don't believe that anything here is incorrect.

The display sometimes freezes with msm fd900000.mdss: pp done time out, lm=0. There was a proposed patch but applying it only seemed to help a little. See also postmarketOS/pmaports#51.

I did not add any peripheral firmwares.

@thefloweringash
Copy link
Member Author

I only just noticed #24

@samueldr samueldr added the 4. type: port New device port label May 6, 2020
@samueldr
Copy link
Member

samueldr commented May 6, 2020

How was all-modules.nix generated?

@thefloweringash
Copy link
Member Author

How was all-modules.nix generated?

I made a quick python script to parse a device tree for “compatible” strings and turn them into required modules. Putting all the hardware support into the initrd is a bit of a blunt method, but it does mean no difficult debugging if critical peripherals don’t work. I‘m assuming there’s enough free space here that precision isn’t important.

@samueldr
Copy link
Member

samueldr commented May 7, 2020

Putting all the hardware support into the initrd is a bit of a blunt method

And pretty much required if you were to set yourself up with a pre-built generic system image.


I believe this is a tool that is going to be helpful, as even if it doesn't end up required for end-users in the future, it's pretty good to be able to detect the modules and do something. I'm thinking e.g. automatically enabling kernel options.

Can you package it up either in Nixpkgs or in this overlay? I guess this overlay is fine for now if you don't think it's Nixpkgs material. I don't know if it should be part of the default shell.nix, but maybe. There's already stuff pulling in python in there.

Copy link
Member

@samueldr samueldr left a comment

Choose a reason for hiding this comment

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

With #152 merged, this PR needs some changes. They should be rather trivial to implement. Tell me if there is anything non-obvious or broken.

@samueldr
Copy link
Member

samueldr commented Aug 31, 2021

Hi!

We just had some major~ish changes, at least one affecting new ports. I would heavily suggest rebasing on top of the latest master update.

The main change affecting ports is that we now have required kernel configuration options, which set known good values to a lot of "difficult to guess" options. A simple bin/kernel-normalize-config should be all that is needed.

Hope this helps!

Though uh... This port is a bit old, don't worry if it gets hard to rebase, you could always ping me if you need help going forward.

@samueldr samueldr mentioned this pull request Aug 31, 2021
3 tasks
@samueldr samueldr marked this pull request as draft October 1, 2022 02:04
@samueldr samueldr added 1. device: other mainline Catch-all for devices that don't fit in another device category 9. status: WIP The PR is not ready for merge, work in progress labels Sep 14, 2023
@samueldr samueldr changed the base branch from master to development September 14, 2023 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. priority: (WIP) 1. device: other mainline Catch-all for devices that don't fit in another device category 4. type: port New device port 9. status: WIP The PR is not ready for merge, work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants