Skip to content

Added Robotics-Masters and the MM1 Hat #421

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 3 commits into from
May 22, 2019

Conversation

hmic
Copy link
Contributor

@hmic hmic commented Apr 26, 2019

We support a set of different bootloaders and firmware choices for our board to make it as accessible as possible for the different communities, for this reason we request differnet VID for each of the boards usecases.
We are working on adding Ardupilot and Dronecode support at the moment and would like to request continuous VIDs 0x4D46 and 0x4D47 for the programming USB device in those i the near future.

Copy link
Contributor

@Arachnid Arachnid left a comment

Choose a reason for hiding this comment

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

Please make sure all of your linked repositories exist and have appropriate LICENSE files.

Also, can you clarify why the hat itself needs a VID, in addition to all the VIDs for specific applications?

@hmic
Copy link
Contributor Author

hmic commented May 7, 2019

Hi Nick,
sorry for messing things up.

I fixed the changed repo URL, renamed the LICENCE to LICENSE in the schematics repo and have Cian add a LICENSE file to the bootloader repo.

Regarding the board VID, it will be used in production with an automated testing device, but we will flash the bootloader before shipping, so it will not be seen in the wild.

@Arachnid
Copy link
Contributor

If the PID for the board won't be used in the wild, could you use one of our test PIDs, instead of allocating a public one for it?

@Arachnid Arachnid merged commit cd6b326 into pidcodes:master May 22, 2019
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.

2 participants