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

Create a PID for LilyGO T-Display rp2040 #827

Merged
merged 7 commits into from May 12, 2023
Merged

Conversation

kreier
Copy link
Contributor

@kreier kreier commented Mar 9, 2023

As LilyGO does not have their own VID for their boards and the application for a PID with the 0x2E8A VID has not been successful for a year this was the suggested solution by @tannewt already in November 16th, 2022 (adafruit/circuitpython#6037). I don't work for LilyGO, but as a teacher would like to use this board in education because of the integrated display and input buttons (current solution: https://github.com/ssis-aa/rvr2023).

kreier added a commit to kreier/circuitpython that referenced this pull request Mar 13, 2023
See pull request pidcodes/pidcodes.github.com#827

The original values from @erongd in https://github.com/erongd/circuitpython/blob/lilygo_t_display_rp2040/ports/raspberrypi/boards/lilygo_t_display_rp2040/mpconfigboard.mk point to a T-Display with esp32 (VID 0x303A is for espressif) but this board uses a raspberry pico (VID 0x2E8A) but it has not been granted for this product a PID. The pid.codes is a workaround
---
layout: pid
title: T-Display rp2040
owner: LilyGO
Copy link
Collaborator

Choose a reason for hiding this comment

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

Mind having yourself as the owner and link to the fork of CircuitPython with the changes? I want it to be clear that this isn't for LilyGo. It is for a third party using their hardware. Thanks!

Choose a reason for hiding this comment

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

@kreier Awaiting your changes as requested by @tannewt. Then we can proceed with the board PR. Thanks.

Copy link
Contributor Author

@kreier kreier Apr 3, 2023

Choose a reason for hiding this comment

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

@tannewt sure, no problem. Yet I have a lot of options and possible answers, maybe you can point to the best one. Following the instructions on https://pid.codes/howto/ I have

2. Set up your organization page

a) I keep org/LilyGO/index.md
b) I create a new page for myself as org/kreier/index.md , linking to https://kreier.org
c) I create one for our school club org/SSISRobotics/index.md and link to https://github.com/ssis-robotics

3. Find a VID and PID - 1209/2023/index.md

a) Keep the title in 1209/2023/index.md
b) Change title to 'SSIS:bit with T-Display rp2040' in reference to my older project https://github.com/ssisbit/ssis.bit
c) Change the owner to Matthias Kreier
d) Change the site to adafruit/circuitpython#6037
e) Keep the source with https://github.com/Xinyuan-LilyGO/LILYGO-T-display-RP2040
f) Change the source to https://github.com/kreier/t-display where I collect ideas since 2020
g) Change the title to 'SSIS.bit with T-Display rp2040' and reference https://github.com/ssis-robotics/ssis.bit

Choose a reason for hiding this comment

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

@tannewt any update on @kreier 's outstanding questions?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@tannewt sure, no problem. Yet I have a lot of options and possible answers, maybe you can point to the best one. Following the instructions on https://pid.codes/howto/ I have

2. Set up your organization page

a) I keep org/LilyGO/index.md b) I create a new page for myself as org/kreier/index.md , linking to https://kreier.org c) I create one for our school club org/SSISRobotics/index.md and link to https://github.com/ssis-robotics

b or c is fine. Whichever you'd like.

3. Find a VID and PID - 1209/2023/index.md

a) Keep the title in 1209/2023/index.md b) Change title to 'SSIS:bit with T-Display rp2040' in reference to my older project https://github.com/ssisbit/ssis.bit c) Change the owner to Matthias Kreier d) Change the site to adafruit/circuitpython#6037 e) Keep the source with https://github.com/Xinyuan-LilyGO/LILYGO-T-display-RP2040 f) Change the source to https://github.com/kreier/t-display where I collect ideas since 2020 g) Change the title to 'SSIS.bit with T-Display rp2040' and reference https://github.com/ssis-robotics/ssis.bit

The title can be the same but I'd switch the site to the CP PR. Please change the owner to match 2.

My main goal is to not give the impression that the manufacturer of the closed source board was granted a PID.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@tannewt Thanks for the feedback, that's a good solution to make a proper use of the pidcodes project. I updated both the organization description (LilyGO is out and a new SSIS-Robotics in) and the 1209/2023/index.md. I left the title as T-Display rp2040 since the organization name will be prefixed in the final list.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

BTW: Thanks for volunteering to run this project. Please don't burn out. 🙂

Copy link

Choose a reason for hiding this comment

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

@kreier First of all thanks for get this through. Do you mind if I add a PID request for the T-Display PICOC3 board under your org "SSS Robotics" just like the T-Display RP2040 so I can try to contribute the board definition to CircuitPython as well. That board has wifi build-in should be able to power some very interesting project. Or if you have other suggestion for obtain a official PID?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @erongd , sure, no problem. You can copy the SSIS Robotics information for another PID request. Let's see if that is necessary. I think you should be able to use the same PID for the PicoC3 (depending how you plug in the USB-C cable) since in one side it's a regular T-Display rp2040 (only 2 serial pins connected to another onboard MCU) and the other way it is covered by Espressif - and not relevant for us, since we won't use it for CircuitPython on the ESP32 C3.

@rcarteraz
Copy link

@kreier Thanks for taking this up. Hope you can make the changes requested and we can get support for this device soon!

@tannewt tannewt merged commit 99fea00 into pidcodes:master May 12, 2023
2 checks passed
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.

None yet

5 participants