[PATCH 2/6] drm/amdgpu: add info ioctl query for vce clock info

[Alex Deucher]

No, you can just check if it fails or not (i.e., the ioctl will return
-EINVAL if the query is not supported).

Alex

On Fri, Oct 7, 2016 at 2:33 PM, StDenis, Tom <Tom.StDenis at amd.com> wrote:

> Shouldn't there be a version bump if you're adding a new command to the
> ioctl?
>
>
> Tom
>
>
> ------------------------------
> *From:* amd-gfx <amd-gfx-bounces at lists.freedesktop.org> on behalf of Alex
> Deucher <alexdeucher at gmail.com>
> *Sent:* Friday, October 7, 2016 14:28
> *To:* amd-gfx at lists.freedesktop.org; Fang, Peter; Zhang, Boyuan
> *Cc:* Deucher, Alexander
> *Subject:* [PATCH 2/6] drm/amdgpu: add info ioctl query for vce clock info
>
> This is needed to set up the vce clock table in userspace
> for proper VCE DPM.
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>  include/uapi/drm/amdgpu_drm.h | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/include/uapi/drm/amdgpu_drm.h b/include/uapi/drm/amdgpu_drm.h
> index b6a04d4..7cb71f8 100644
> --- a/include/uapi/drm/amdgpu_drm.h
> +++ b/include/uapi/drm/amdgpu_drm.h
> @@ -491,6 +491,8 @@ struct drm_amdgpu_cs_chunk_data {
>  #define AMDGPU_INFO_NUM_EVICTIONS               0x18
>  /* Query memory about VRAM and GTT domains */
>  #define AMDGPU_INFO_MEMORY                      0x19
> +/* Query memory about VRAM and GTT domains */
> +#define AMDGPU_INFO_VCE_CLOCK_TABLE            0x1A
>
>  #define AMDGPU_INFO_MMR_SE_INDEX_SHIFT  0
>  #define AMDGPU_INFO_MMR_SE_INDEX_MASK   0xff
> @@ -677,6 +679,22 @@ struct drm_amdgpu_info_hw_ip {
>          __u32  _pad;
>  };
>
> +#define AMDGPU_VCE_CLOCK_TABLE_ENTRIES         6
> +
> +struct drm_amdgpu_info_vce_clock_table_entry {
> +       /** System clock */
> +       __u32 sclk;
> +       /** Memory clock */
> +       __u32 mclk;
> +       /** VCE clock */
> +       __u32 eclk;
> +       __u32 pad;
> +};
> +
> +struct drm_amdgpu_info_vce_clock_table {
> +       struct drm_amdgpu_info_vce_clock_table_entry
> entries[AMDGPU_VCE_CLOCK_TABLE_ENTRIES];
> +};
> +
>  /*
>   * Supported GPU families
>   */
> --
> 2.5.5
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> amd-gfx Info Page - lists.freedesktop.org
> <https://lists.freedesktop.org/mailman/listinfo/amd-gfx>
> lists.freedesktop.org
> To see the collection of prior postings to the list, visit the amd-gfx
> Archives. Using amd-gfx: To post a message to all the list members, send
> email ...
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161007/327de093/attachment.html>