Skip to content

Add 2 boards nRF52840 M.2 devkit and M60 Keyboard #3136

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

Merged
merged 10 commits into from
Jul 14, 2020

Conversation

xiongyihui
Copy link

@ladyada
Copy link
Member

ladyada commented Jul 9, 2020

hi - do you have permission from nordic to use their VID?

https://devzone.nordicsemi.com/f/nordic-q-a/26196/obtain-a-pid-for-an-nrf24lu1-based-product

@xiongyihui
Copy link
Author

Has discussed the VID issue before. It seems Nordic allows customers to use its VID in a product with its chip. Of course, there is a risk.

You can use our Nordic's Vendor ID. However, you will have a case that other customer also use our Vendor ID and use the same Product ID that you choose.

from https://devzone.nordicsemi.com/f/nordic-q-a/33851/nrf52840---usb-vendor-id-and-usb-product-id

@ladyada
Copy link
Member

ladyada commented Jul 9, 2020

considering the number of products you are making, we think you should get a VID at this point - other companies such as seeed and sparkfun and arduino and adafruit all have a VID

@xiongyihui
Copy link
Author

Got your thought. I will try to get a unique VID & PID.

@tannewt
Copy link
Member

tannewt commented Jul 9, 2020

Once you get a VID for MakerDiary you'll have 2**16 PIDs for all of your products and more. Also, I'm available on WeChat as tannewt if you want help over chat.

@xiongyihui
Copy link
Author

Our Seeed friends have kindly allowed us to use their VID with a range of PIDs.
@ladyada Is it OK that I use Seeed's VID 0x2886 and the PID 0xf001 for our products in this project? One PID should be enough as all these products are based on nRF52840 and have the same driver for an OS.
You may ask why not buy a VID. Although being comparing to seeed, sparkfun, arduino and adafruit, we are still a small team with 3 persons and can not afford a USB VID yet. It doesn't break any rule by using Nordic's VID or Seeed's VID.

@tannewt have sent a request to add me on wechat

@ladyada
Copy link
Member

ladyada commented Jul 10, 2020

we purchased a VID when we were only a few people too, about 10 years ago. if you are selling USB products, it is the cost of doing business, just like it costs money to buy components and PCBs :)

you can totally use VID/PIDs donated by others. it is important that each circuitpython board has unique VID/PID. having duplicates will cause problems for customers. please ask seeed to grant you individual PID's for each item so they are unique.

@ladyada
Copy link
Member

ladyada commented Jul 10, 2020

thank you :)

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Just a couple questions and suggestion about using the RGB led for CircuitPython status. Here is an example: https://github.com/adafruit/circuitpython/blob/main/ports/nrf/boards/particle_argon/mpconfigboard.h#L35-L37

@xiongyihui
Copy link
Author

Are MICROPY_HW_RGB_LED_RED, MICROPY_HW_RGB_LED_GREEN and MICROPY_HW_RGB_LED_BLUE necessary? Didn't find them being used.

@tannewt
Copy link
Member

tannewt commented Jul 11, 2020

Are MICROPY_HW_RGB_LED_RED, MICROPY_HW_RGB_LED_GREEN and MICROPY_HW_RGB_LED_BLUE necessary? Didn't find them being used.

Looks like I have the wrong defines. Looks like it should be CP_RGB_STATUS_R: https://github.com/adafruit/circuitpython/blob/main/supervisor/shared/rgb_led_status.c#L147

tannewt
tannewt previously approved these changes Jul 13, 2020
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Thank you!

@tannewt tannewt changed the base branch from master to main July 13, 2020 22:52
@tannewt tannewt dismissed their stale review July 13, 2020 22:52

The base branch was changed.

@tannewt
Copy link
Member

tannewt commented Jul 13, 2020

FYI, I changed the base branch to main. We no longer are using master.

@tannewt
Copy link
Member

tannewt commented Jul 13, 2020

I also did the merge in your master branch so make sure and pull from GitHub if you use that branch locally.

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Thank you! We're excited for these boards!

@tannewt tannewt merged commit 90ad9be into adafruit:main Jul 14, 2020
@xiongyihui
Copy link
Author

OK. Many thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants