Re: [PATCH] drm/etnaviv: provide more ID values via GET_PARAM ioctl.

From: Lucas Stach
Date: Mon Jan 25 2021 - 12:34:58 EST


Am Mittwoch, dem 16.12.2020 um 12:42 +0100 schrieb Christian Gmeiner:
> Make it possible for the user space to access these ID values.

Thanks, I've added this patch to my etnaviv/next branch.

Regards,
Lucas

> Signed-off-by: Christian Gmeiner <christian.gmeiner@xxxxxxxxx>
> ---
>  drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 12 ++++++++++++
>  include/uapi/drm/etnaviv_drm.h | 3 +++
>  2 files changed, 15 insertions(+)
>
> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c b/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
> index c6404b8d067f..ec16991ba8b6 100644
> --- a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
> +++ b/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
> @@ -156,6 +156,18 @@ int etnaviv_gpu_get_param(struct etnaviv_gpu *gpu, u32 param, u64 *value)
>   *value = ~0ULL;
>   break;
>  
>
>
>
> + case ETNAVIV_PARAM_GPU_PRODUCT_ID:
> + *value = gpu->identity.product_id;
> + break;
> +
> + case ETNAVIV_PARAM_GPU_CUSTOMER_ID:
> + *value = gpu->identity.customer_id;
> + break;
> +
> + case ETNAVIV_PARAM_GPU_ECO_ID:
> + *value = gpu->identity.eco_id;
> + break;
> +
>   default:
>   DBG("%s: invalid param: %u", dev_name(gpu->dev), param);
>   return -EINVAL;
> diff --git a/include/uapi/drm/etnaviv_drm.h b/include/uapi/drm/etnaviv_drm.h
> index 09d0df8b71c5..af024d90453d 100644
> --- a/include/uapi/drm/etnaviv_drm.h
> +++ b/include/uapi/drm/etnaviv_drm.h
> @@ -74,6 +74,9 @@ struct drm_etnaviv_timespec {
>  #define ETNAVIV_PARAM_GPU_NUM_CONSTANTS 0x19
>  #define ETNAVIV_PARAM_GPU_NUM_VARYINGS 0x1a
>  #define ETNAVIV_PARAM_SOFTPIN_START_ADDR 0x1b
> +#define ETNAVIV_PARAM_GPU_PRODUCT_ID 0x1c
> +#define ETNAVIV_PARAM_GPU_CUSTOMER_ID 0x1d
> +#define ETNAVIV_PARAM_GPU_ECO_ID 0x1e
>  
>
>
>
>  #define ETNA_MAX_PIPES 4
>  
>
>
>