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

shield-specific behaviors #1218

Closed
tehn opened this issue Oct 27, 2020 · 8 comments
Closed

shield-specific behaviors #1218

tehn opened this issue Oct 27, 2020 · 8 comments
Labels

Comments

@tehn
Copy link
Member

@tehn tehn commented Oct 27, 2020

system awareness of shield vs. not-shield

for shield:

  • remove i2c headphone errors
  • "sleep" needs to be clarified because the device doesn't shut down
  • remove battery displays/etc
  • anything with wifi??
@tehn tehn added the enhancement label Oct 27, 2020
@catfact
Copy link
Collaborator

@catfact catfact commented Oct 27, 2020

how would we actually detect shields?

it's the same kernel and everything, right?

can we see something different in /proc/cpuinfo or etc?

query for presence of headphone driver on i2c?

@ericmoderbacher
Copy link
Contributor

@ericmoderbacher ericmoderbacher commented Oct 27, 2020

Maybe this thread will help:
https://www.raspberrypi.org/forums/viewtopic.php?t=192748

Looks like you can get a cpu revision.

@catfact
Copy link
Collaborator

@catfact catfact commented Oct 27, 2020

thanks!

capturing that: seems like both these paths should have something usable
/proc/cpuinfo, indeed (maybe messier)
/sys/firmware/devicetree/base/model (human-readable string built by rpi bootcode)

@tehn
Copy link
Member Author

@tehn tehn commented Oct 27, 2020

sorry I should've mentioned this is already figured out because we have to id the chip for updates. we do it in update.sh which is in the update folder.

other issues however remain, will post them here when i get a minute

@catfact
Copy link
Collaborator

@catfact catfact commented Oct 28, 2020

gotcha. so, here:
https://github.com/monome/norns/blob/main/update/update.sh#L56-L62

should also, i dunno, write a config file that matron can use.

@csboling
Copy link
Contributor

@csboling csboling commented Oct 30, 2020

I think it would probably work to configure all these things in the matronrc.lua file my screen-decoupling branch (for #1187) loads at startup to decide which type of screen / input source to use.

@tehn
Copy link
Member Author

@tehn tehn commented Nov 2, 2020

using matronrc.lua is a good idea--- but i'm wondering where/how the config step should happen. at some point matron will have to self-detect if it's on a CM3 or a shield, and that info can be written to the rc file... so should there always be a user config file?

for example, if the user config file doesn't exist, first copy the default, then append some extra information re: detecting architecture? i suppose since desktop is possibly an option it should check that as well?

@tehn
Copy link
Member Author

@tehn tehn commented Nov 30, 2020

fixed by #1250

@tehn tehn closed this Nov 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants