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

Platform/OPi5: Fix platform vendor name #167

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abhisit
Copy link

@abhisit abhisit commented Oct 31, 2024

Reference to the schematic revision 1.0, the vendor name is Xunlong Co.,Limited

image

An official link to download the schematic on Google Drive

@abhisit abhisit force-pushed the opi5plus-correct-platform-vendor-name branch 2 times, most recently from fd53f70 to 3adaece Compare October 31, 2024 10:24
@Rippanda12
Copy link

Rippanda12 commented Oct 31, 2024

Xunlong is their company name yes but their brand is Orange Pi. I'm against this PR because if this merges then we need to update radxa's boards to Shenzhen Vamrs Technology Co.,Ltd

@microcai
Copy link

microcai commented Nov 1, 2024

@Rippanda12 is it a name issue? or because it's from a Rassua ally?

@Rippanda12
Copy link

@Rippanda12 is it a name issue? or because it's from a Rassua ally?

Name change, everyone is human regardless from where they are from pls don't bring up politics again

@abhisit
Copy link
Author

abhisit commented Nov 2, 2024

When I realized, it was when the Gnome duplicated the word. Should it better if it is "Xunlong Orange Pi 5 Plus"?

image

@abhisit abhisit force-pushed the opi5plus-correct-platform-vendor-name branch from 3adaece to 114683e Compare November 15, 2024 08:27
@abhisit abhisit marked this pull request as draft November 15, 2024 08:42
@abhisit abhisit force-pushed the opi5plus-correct-platform-vendor-name branch 2 times, most recently from daa8188 to ac2bb74 Compare November 15, 2024 08:55
@abhisit abhisit marked this pull request as ready for review November 15, 2024 08:57
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.

3 participants