AORUS Radeon™ RX 6800 XT MASTER 16G (AMD Radeon RX 6800 XT)#61
Open
ksaadDE wants to merge 5 commits intoCalcProgrammer1:masterfrom
Open
AORUS Radeon™ RX 6800 XT MASTER 16G (AMD Radeon RX 6800 XT)#61ksaadDE wants to merge 5 commits intoCalcProgrammer1:masterfrom
ksaadDE wants to merge 5 commits intoCalcProgrammer1:masterfrom
Conversation
…00_XT_MASTER_16G changed pci_id definiton name and the shown "title" of the GPU
caused the error requiring seven arguments but only six given
Author
cd7b90e to
90e4da1
Compare
Author
Atm no time, you may reference to this and copy it over to gitlab. Also, I don't have a gitlab acc. :) /E: if the force push does not solve it 👍 |
ceb9ff0 to
f948d6e
Compare
79b7751 to
ba91bf4
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Added detection for
AORUS Radeon™ RX 6800 XT MASTER 16G(shown asAMD Radeon RX 6800 XT).I hope the naming is correct, as per project's standards.
Two files changed:
pci_ids.hGigabyteRGBFusion2GPUControllerDetect.cppAdded the hardware details in both files.
Background: After opening OpenRGB a few days ago, just to play around, I found my GPU is not detected which bugged me. Not sure but I remember this might have worked years ago, thus suddenly did not work. Bothered me, so I opted for adding support for it.
@CalcProgrammer1 pointed me to the right i2c IDs and hardware IDs, how to extract them and read the output, next to showing me the Controller that could be used, of which I had to edit the detection script.
Essentially
Process for identification:
lspci -vnnthe looking at the SubSystem of theVGA compatible controller(e.g. additionally amd-smi list or rocm-smi -i)The 0x70 was gussed based on this process:
i2cdetect -lSo i2c-9 was selected, as it is the OEM Bus.
Thus
i2cdetect -y 9returns:So its either 0x61 or 0x70 for RGB control via the controller. For most similiar cards OpenRGB seems to use 0x70, so that address is tested first. It works fine.