zmk icon indicating copy to clipboard operation
zmk copied to clipboard

feat(boards) Kinesis Advantage 360 pro

Open ReFil opened this issue 2 years ago • 8 comments

This PR adds support for the kinesis advantage 360 pro split, an nrf52840 based board https://kinesis-ergo.com/keyboards/advantage360

I've tested this on real production hardware and everything works as expected, I'm not 100% sure about the metadata file though as I've never done one before

Board/Shield Check-list

  • [x] This board/shield is tested working on real hardware
  • [x] Definitions follow the general style of other shields/boards upstream (Reference)
  • [x] .zmk.yml metadata file added
  • [x] Proper Copyright + License headers added to applicable files (Generally, we stick to "The ZMK Contributors" for copyrights to help avoid churn when files get edited)
  • [x] General consistent formatting of DeviceTree files
  • [x] Keymaps do not use deprecated key defines (Check using the upgrader tool)
  • [x] &pro_micro used in favor of &pro_micro_d/a if applicable (N/A)
  • [x] If split, no name added for the right/peripheral half
  • [x] Kconfig.defconfig file correctly wraps all configuration in conditional on the shield symbol
  • [x] .conf file has optional extra features commented out
  • [x] Keyboard/PCB is part of a shipped group buy or is generally available in stock to purchase (OSH/personal projects without general availability should create a zmk-config repo instead)

ReFil avatar Sep 13 '22 12:09 ReFil

I've merged just this PR into main branch, have successfully built firmwares and install them into my adv360. But LED indicators are not lit.

I found some PRs which you issued on this PR list.
Is there any PRs which have to merge into source code to work LED indicators correctly?

ypsilon-takai avatar Sep 20 '22 15:09 ypsilon-takai

LED indicators need a lot of custom RGB code to make them work, right now they're just set up as underglow defaulting to off. They won't work as intended for the foreseeable future. If you want the indicators to work properly you need to use the custom fork

ReFil avatar Sep 20 '22 16:09 ReFil

My setup that works with adv360-beta branch gives the following. I might just be doing something silly, I dunno. Any tips appreciated.

devicetree error: /__w/zmk-config-adv360/zmk-config-adv360/config/adv360.keymap:56 (column 47): parse error: expected number or parenthesized expression

ccarter avatar Sep 21 '22 20:09 ccarter

My setup that works with adv360-beta branch gives the following. I might just be doing something silly, I dunno. Any tips appreciated.

devicetree error: /__w/zmk-config-adv360/zmk-config-adv360/config/adv360.keymap:56 (column 47): parse error: expected number or parenthesized expression

The adv360-beta branch has features that are not present in zmk main, including behaviours. At a guess check the last layer and replace the &rgb_ug RGB_MEFS_CMD 5 with &none

ReFil avatar Sep 21 '22 20:09 ReFil

My setup that works with adv360-beta branch gives the following. I might just be doing something silly, I dunno. Any tips appreciated. devicetree error: /__w/zmk-config-adv360/zmk-config-adv360/config/adv360.keymap:56 (column 47): parse error: expected number or parenthesized expression

The adv360-beta branch has features that are not present in zmk main, including behaviours. At a guess check the last layer and replace the &rgb_ug RGB_MEFS_CMD 5 with &none

Ya, that got me further. Now I'm getting. devicetree error: 'color-mapping' is marked as required in 'properties:' in /__w/zmk-config-adv360/zmk-config-adv360/zephyr/dts/bindings/led_strip/worldsemi,ws2812-spi.yaml, but does not appear in <Node /soc/spi@40004000/ws2812@0 in '/__w/zmk-config-adv360/zmk-config-adv360/zephyr/misc/empty_file.c'>

ccarter avatar Sep 21 '22 20:09 ccarter

My setup that works with adv360-beta branch gives the following. I might just be doing something silly, I dunno. Any tips appreciated. devicetree error: /__w/zmk-config-adv360/zmk-config-adv360/config/adv360.keymap:56 (column 47): parse error: expected number or parenthesized expression

The adv360-beta branch has features that are not present in zmk main, including behaviours. At a guess check the last layer and replace the &rgb_ug RGB_MEFS_CMD 5 with &none

Ya, that got me further. Now I'm getting. devicetree error: 'color-mapping' is marked as required in 'properties:' in /__w/zmk-config-adv360/zmk-config-adv360/zephyr/dts/bindings/led_strip/worldsemi,ws2812-spi.yaml, but does not appear in <Node /soc/spi@40004000/ws2812@0 in '/__w/zmk-config-adv360/zmk-config-adv360/zephyr/misc/empty_file.c'>

You can't use a config branch for the adv360-beta branch with this. It simply won't work. Use this branch instead. The devicetree is all configured correctly

https://github.com/PolarityWorks/zmk-config-adv360/tree/zmk-main

ReFil avatar Sep 21 '22 20:09 ReFil

My setup that works with adv360-beta branch gives the following. I might just be doing something silly, I dunno. Any tips appreciated. devicetree error: /__w/zmk-config-adv360/zmk-config-adv360/config/adv360.keymap:56 (column 47): parse error: expected number or parenthesized expression

The adv360-beta branch has features that are not present in zmk main, including behaviours. At a guess check the last layer and replace the &rgb_ug RGB_MEFS_CMD 5 with &none

Ya, that got me further. Now I'm getting. devicetree error: 'color-mapping' is marked as required in 'properties:' in /__w/zmk-config-adv360/zmk-config-adv360/zephyr/dts/bindings/led_strip/worldsemi,ws2812-spi.yaml, but does not appear in <Node /soc/spi@40004000/ws2812@0 in '/__w/zmk-config-adv360/zmk-config-adv360/zephyr/misc/empty_file.c'>

You can't use a config branch for the adv360-beta branch with this. It simply won't work. Use this branch instead. The devicetree is all configured correctly

https://github.com/PolarityWorks/zmk-config-adv360/tree/zmk-main

very good. Thanks for your help. Good work BTW, and the keyboard is amazing IMO.

ccarter avatar Sep 21 '22 20:09 ccarter

My setup that works with adv360-beta branch gives the following. I might just be doing something silly, I dunno. Any tips appreciated. devicetree error: /__w/zmk-config-adv360/zmk-config-adv360/config/adv360.keymap:56 (column 47): parse error: expected number or parenthesized expression

The adv360-beta branch has features that are not present in zmk main, including behaviours. At a guess check the last layer and replace the &rgb_ug RGB_MEFS_CMD 5 with &none

Ya, that got me further. Now I'm getting. devicetree error: 'color-mapping' is marked as required in 'properties:' in /__w/zmk-config-adv360/zmk-config-adv360/zephyr/dts/bindings/led_strip/worldsemi,ws2812-spi.yaml, but does not appear in <Node /soc/spi@40004000/ws2812@0 in '/__w/zmk-config-adv360/zmk-config-adv360/zephyr/misc/empty_file.c'>

You can't use a config branch for the adv360-beta branch with this. It simply won't work. Use this branch instead. The devicetree is all configured correctly

https://github.com/PolarityWorks/zmk-config-adv360/tree/zmk-main

very good. Thanks for your help. Good work BTW, and the keyboard is amazing IMO.

Thanks for the support. It's much appreciated 😊

ReFil avatar Sep 21 '22 20:09 ReFil

Isn't this PR missing from the left and right .defconfig files at least the following which are found in these files in https://github.com/KinesisCorporation/Adv360-Pro-ZMK ?

CONFIG_USB_DEVICE_VID=0x29EA CONFIG_USB_DEVICE_PID=0x0362 CONFIG_USB_DEVICE_MANUFACTURER="Kinesis Corporation" CONFIG_BT_DIS_MANUF="Kinesis Corporation"

yanshay avatar Mar 05 '23 11:03 yanshay

Any chance of us seeing this merged?

douglascamata avatar Mar 29 '23 23:03 douglascamata

howdy! I'm also stuck on an another branch until this gets merged.

Is this PR missing something to be able to be merged?

elisiano avatar Apr 10 '23 12:04 elisiano

Since the zephyr 3.2 update I need to rebase and bugfix before this can be merged. I'll work on it asap

ReFil avatar Apr 10 '23 13:04 ReFil

Since the zephyr 3.2 update I need to rebase and bugfix before this can be merged. I'll work on it asap

I didn't realize that, thank you :heart:

elisiano avatar Apr 10 '23 13:04 elisiano

Just retested it because quite a few things changed through the review process and everything seems to still be working

ReFil avatar Nov 21 '23 09:11 ReFil