beryllium-org / OS

Beryllium OS, a unix-like operating system for CircuitPython powered microcontrollers. (Formerly known as ljinux)
GNU General Public License v3.0
227 stars 14 forks source link

Add initial support for new board - Adafruit KB2040 #50

Closed markbirss closed 1 year ago

markbirss commented 1 year ago

Initial support for new board Adafruit KB2040

Kindly review and comment

bill88t commented 1 year ago

Changed the pin specified in the json, to match the neopixel number (9) in accordance to the pintab. Should you try to boot ljinux with these changes, you should see quite a lot of activity on that neopixel. Please verify it works.

markbirss commented 1 year ago

@bill88t im not sure if the pintab numbering is all correct ? The neopixel is on GPIO17 https://learn.adafruit.com/adafruit-kb2040/pinouts

bill88t commented 1 year ago

@bill88t im not sure if the pintab numbering is all correct ? The neopixel is on GPIO17 https://learn.adafruit.com/adafruit-kb2040/pinouts

Is it GPIO17 or NEOPIXEL in the board module? The "led" value is determined in accordance to the pintab, not based on real GPIO, and the pintab you gave set it to value 9.

bill88t commented 1 year ago

Adafruit says it's board.NEOPIXEL (https://learn.adafruit.com/adafruit-kb2040/pinouts#neopixel-led-3106797). Which is what you set as 9 in the pintab, so I am pretty sure the led is correct as is.

markbirss commented 1 year ago

Yea, this ordering scheme makes more sense, but shouldn't A0-A3 allocate 10-13?

I have taken the pinmap from this diagram, please advise image

markbirss commented 1 year ago

Ok, thank you

markbirss commented 1 year ago

Let me do a quick test again

bill88t commented 1 year ago

I have taken the pinmap from this diagram, please advise

Oh alright then. I agree it's best to stick to the pinout since we can. Now imma go make the pinout command's ascii art.

markbirss commented 1 year ago

Ok, i tested, and neopixel much more activity now

bill88t commented 1 year ago

Alrighty then, as soon as I get the ascii pinout done, I'm gonna merge it in. Cuz atm, the pinout command prolly will crash.

markbirss commented 1 year ago

@bill88t Ok, thank you, great project