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

VITIS-14240 Driver Version Detection via APIs #8709

Merged
merged 4 commits into from
Jan 23, 2025
Merged

Conversation

AShivangi
Copy link
Collaborator

@AShivangi AShivangi commented Jan 22, 2025

Problem solved by the commit

VITIS-13522 Add API to query system configuration which is a story under VITIS-14240 Driver Version Detection via APIs

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

This is a requested feature

How problem was solved, alternative solutions (if any) and why they were rejected

enhanced the existing API and marked some device properties as deprecated

Risks (if any) associated the changes in the commit

N/A

What has been tested and how, request additional testing if necessary

tested on linux and windows setup

xdna output:

$ ./query.exe -j
device host json info ==========================================
{
    "version": "2.19.0",
    "branch": "master",
    "hash": "4e8c32a9e5a2e51a8bdfa672885d213cbad6df5a",
    "build_date": "2025-01-15 11:39:37",
    "drivers": [
        {
            "name": "amdxdna",
            "version": "2.19.0_20250122",
            "hash": "..."
        }
    ]
}
device platform json info ==========================================
{
    "platforms": [
        {
            "static_region": {
                "name": "RyzenAI-npu4", # will be full name on MCDM
                "total_columns": "8"
            },
            "status": {
                "power_mode": "Default"
            },
            "electrical": {
                "power_consumption_watts": "1.234"
            }
        }
    ]
}
PASSED TEST

Documentation impact (if any)

Signed-off-by: AShivangi <shivangiagarwal53@gmail.com>
Signed-off-by: AShivangi <shivangiagarwal53@gmail.com>
@AShivangi AShivangi added the do not merge hold off on merging label Jan 22, 2025
@AShivangi AShivangi removed the do not merge hold off on merging label Jan 22, 2025
@stsoe stsoe merged commit 01989d3 into Xilinx:master Jan 23, 2025
20 checks passed
rchane pushed a commit to rchane/XRT that referenced this pull request Jan 24, 2025
* Add API to query system configuration

Signed-off-by: AShivangi <shivangiagarwal53@gmail.com>

* fix arg parsing

Signed-off-by: AShivangi <shivangiagarwal53@gmail.com>

* update copyright

* add total_cols to static region

---------

Signed-off-by: AShivangi <shivangiagarwal53@gmail.com>
@AShivangi AShivangi deleted the get_info branch February 4, 2025 16:45
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