va_config: Rename functions arguments for more clarity

Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
This commit is contained in:
Paul Kocialkowski
2018-04-23 14:13:15 +02:00
parent 1d7d92bf1c
commit add3854179
2 changed files with 64 additions and 77 deletions
+43 -44
View File
@@ -39,11 +39,11 @@
* correspondence between v4l and VA video formats. * correspondence between v4l and VA video formats.
*/ */
VAStatus SunxiCedrusQueryConfigProfiles(VADriverContextP ctx, VAStatus SunxiCedrusQueryConfigProfiles(VADriverContextP context,
VAProfile *profile_list, int *num_profiles) VAProfile *profiles, int *profiles_count)
{ {
struct sunxi_cedrus_driver_data *driver_data = struct sunxi_cedrus_driver_data *driver_data =
(struct sunxi_cedrus_driver_data *) ctx->pDriverData; (struct sunxi_cedrus_driver_data *) context->pDriverData;
int i = 0; int i = 0;
struct v4l2_fmtdesc vid_fmtdesc; struct v4l2_fmtdesc vid_fmtdesc;
memset(&vid_fmtdesc, 0, sizeof(vid_fmtdesc)); memset(&vid_fmtdesc, 0, sizeof(vid_fmtdesc));
@@ -54,57 +54,56 @@ VAStatus SunxiCedrusQueryConfigProfiles(VADriverContextP ctx,
{ {
switch(vid_fmtdesc.pixelformat) { switch(vid_fmtdesc.pixelformat) {
case V4L2_PIX_FMT_MPEG2_FRAME: case V4L2_PIX_FMT_MPEG2_FRAME:
profile_list[i++] = VAProfileMPEG2Simple; profiles[i++] = VAProfileMPEG2Simple;
profile_list[i++] = VAProfileMPEG2Main; profiles[i++] = VAProfileMPEG2Main;
break; break;
} }
vid_fmtdesc.index++; vid_fmtdesc.index++;
} }
assert(i <= SUNXI_CEDRUS_MAX_PROFILES); assert(i <= SUNXI_CEDRUS_MAX_PROFILES);
*num_profiles = i; *profiles_count = i;
return VA_STATUS_SUCCESS; return VA_STATUS_SUCCESS;
} }
VAStatus SunxiCedrusQueryConfigEntrypoints(VADriverContextP ctx, VAStatus SunxiCedrusQueryConfigEntrypoints(VADriverContextP context,
VAProfile profile, VAEntrypoint *entrypoint_list, VAProfile profile, VAEntrypoint *entrypoints, int *entrypoints_count)
int *num_entrypoints)
{ {
switch (profile) { switch (profile) {
case VAProfileMPEG2Simple: case VAProfileMPEG2Simple:
case VAProfileMPEG2Main: case VAProfileMPEG2Main:
*num_entrypoints = 2; *entrypoints_count = 2;
entrypoint_list[0] = VAEntrypointVLD; entrypoints[0] = VAEntrypointVLD;
entrypoint_list[1] = VAEntrypointMoComp; entrypoints[1] = VAEntrypointMoComp;
break; break;
default: default:
*num_entrypoints = 0; *entrypoints_count = 0;
break; break;
} }
assert(*num_entrypoints <= SUNXI_CEDRUS_MAX_ENTRYPOINTS); assert(*entrypoints_count <= SUNXI_CEDRUS_MAX_ENTRYPOINTS);
return VA_STATUS_SUCCESS; return VA_STATUS_SUCCESS;
} }
VAStatus SunxiCedrusGetConfigAttributes(VADriverContextP ctx, VAStatus SunxiCedrusGetConfigAttributes(VADriverContextP context,
VAProfile profile, VAEntrypoint entrypoint, VAProfile profile, VAEntrypoint entrypoint, VAConfigAttrib *attributes,
VAConfigAttrib *attrib_list, int num_attribs) int attributes_count)
{ {
int i; int i;
for (i = 0; i < num_attribs; i++) for (i = 0; i < attributes_count; i++)
{ {
switch (attrib_list[i].type) switch (attributes[i].type)
{ {
case VAConfigAttribRTFormat: case VAConfigAttribRTFormat:
attrib_list[i].value = VA_RT_FORMAT_YUV420; attributes[i].value = VA_RT_FORMAT_YUV420;
break; break;
default: default:
/* Do nothing */ /* Do nothing */
attrib_list[i].value = VA_ATTRIB_NOT_SUPPORTED; attributes[i].value = VA_ATTRIB_NOT_SUPPORTED;
break; break;
} }
} }
@@ -112,8 +111,8 @@ VAStatus SunxiCedrusGetConfigAttributes(VADriverContextP ctx,
return VA_STATUS_SUCCESS; return VA_STATUS_SUCCESS;
} }
VAStatus SunxiCedrusupdate_attribute(struct object_config *obj_config, VAStatus sunxi_cedrus_update_attribute(struct object_config *obj_config,
VAConfigAttrib *attrib) VAConfigAttrib *attrib)
{ {
int i; int i;
/* Check existing attributes */ /* Check existing attributes */
@@ -137,12 +136,12 @@ VAStatus SunxiCedrusupdate_attribute(struct object_config *obj_config,
return VA_STATUS_ERROR_MAX_NUM_EXCEEDED; return VA_STATUS_ERROR_MAX_NUM_EXCEEDED;
} }
VAStatus SunxiCedrusCreateConfig(VADriverContextP ctx, VAProfile profile, VAStatus SunxiCedrusCreateConfig(VADriverContextP context, VAProfile profile,
VAEntrypoint entrypoint, VAConfigAttrib *attrib_list, VAEntrypoint entrypoint, VAConfigAttrib *attributes,
int num_attribs, VAConfigID *config_id) int attributes_count, VAConfigID *config_id)
{ {
struct sunxi_cedrus_driver_data *driver_data = struct sunxi_cedrus_driver_data *driver_data =
(struct sunxi_cedrus_driver_data *) ctx->pDriverData; (struct sunxi_cedrus_driver_data *) context->pDriverData;
VAStatus vaStatus; VAStatus vaStatus;
int configID; int configID;
struct object_config *obj_config; struct object_config *obj_config;
@@ -181,9 +180,9 @@ VAStatus SunxiCedrusCreateConfig(VADriverContextP ctx, VAProfile profile,
obj_config->attributes[0].value = VA_RT_FORMAT_YUV420; obj_config->attributes[0].value = VA_RT_FORMAT_YUV420;
obj_config->attributes_count = 1; obj_config->attributes_count = 1;
for(i = 0; i < num_attribs; i++) for(i = 0; i < attributes_count; i++)
{ {
vaStatus = sunxi_cedrus_update_attribute(obj_config, &(attrib_list[i])); vaStatus = sunxi_cedrus_update_attribute(obj_config, &(attributes[i]));
if (VA_STATUS_SUCCESS != vaStatus) if (VA_STATUS_SUCCESS != vaStatus)
break; break;
} }
@@ -197,10 +196,11 @@ VAStatus SunxiCedrusCreateConfig(VADriverContextP ctx, VAProfile profile,
return vaStatus; return vaStatus;
} }
VAStatus SunxiCedrusDestroyConfig(VADriverContextP ctx, VAConfigID config_id) VAStatus SunxiCedrusDestroyConfig(VADriverContextP context,
VAConfigID config_id)
{ {
struct sunxi_cedrus_driver_data *driver_data = struct sunxi_cedrus_driver_data *driver_data =
(struct sunxi_cedrus_driver_data *) ctx->pDriverData; (struct sunxi_cedrus_driver_data *) context->pDriverData;
VAStatus vaStatus; VAStatus vaStatus;
struct object_config *obj_config; struct object_config *obj_config;
@@ -215,13 +215,12 @@ VAStatus SunxiCedrusDestroyConfig(VADriverContextP ctx, VAConfigID config_id)
return VA_STATUS_SUCCESS; return VA_STATUS_SUCCESS;
} }
VAStatus SunxiCedrusQueryConfigAttributes(VADriverContextP ctx, VAStatus SunxiCedrusQueryConfigAttributes(VADriverContextP context,
VAConfigID config_id, VAProfile *profile, VAConfigID config_id, VAProfile *profile, VAEntrypoint *entrypoint,
VAEntrypoint *entrypoint, VAConfigAttrib *attrib_list, VAConfigAttrib *attributes, int *attributes_count)
int *num_attribs)
{ {
struct sunxi_cedrus_driver_data *driver_data = struct sunxi_cedrus_driver_data *driver_data =
(struct sunxi_cedrus_driver_data *) ctx->pDriverData; (struct sunxi_cedrus_driver_data *) context->pDriverData;
VAStatus vaStatus = VA_STATUS_SUCCESS; VAStatus vaStatus = VA_STATUS_SUCCESS;
struct object_config *obj_config; struct object_config *obj_config;
int i; int i;
@@ -231,22 +230,22 @@ VAStatus SunxiCedrusQueryConfigAttributes(VADriverContextP ctx,
*profile = obj_config->profile; *profile = obj_config->profile;
*entrypoint = obj_config->entrypoint; *entrypoint = obj_config->entrypoint;
*num_attribs = obj_config->attributes_count; *attributes_count = obj_config->attributes_count;
for(i = 0; i < obj_config->attributes_count; i++) for(i = 0; i < obj_config->attributes_count; i++)
attrib_list[i] = obj_config->attributes[i]; attributes[i] = obj_config->attributes[i];
return vaStatus; return vaStatus;
} }
/* sunxi-cedrus doesn't support display attributes */ /* sunxi-cedrus doesn't support display attributes */
VAStatus SunxiCedrusQueryDisplayAttributes (VADriverContextP ctx, VAStatus SunxiCedrusQueryDisplayAttributes(VADriverContextP context,
VADisplayAttribute *attr_list, int *num_attributes) VADisplayAttribute *attributes, int *attributes_count)
{ return VA_STATUS_ERROR_UNKNOWN; } { return VA_STATUS_ERROR_UNKNOWN; }
VAStatus SunxiCedrusGetDisplayAttributes (VADriverContextP ctx, VAStatus SunxiCedrusGetDisplayAttributes(VADriverContextP context,
VADisplayAttribute *attr_list, int num_attributes) VADisplayAttribute *attributes, int *attributes_count)
{ return VA_STATUS_ERROR_UNKNOWN; } { return VA_STATUS_ERROR_UNKNOWN; }
VAStatus SunxiCedrusSetDisplayAttributes (VADriverContextP ctx, VAStatus SunxiCedrusSetDisplayAttributes(VADriverContextP context,
VADisplayAttribute *attr_list, int num_attributes) VADisplayAttribute *attributes, int *attributes_count)
{ return VA_STATUS_ERROR_UNKNOWN; } { return VA_STATUS_ERROR_UNKNOWN; }
+21 -33
View File
@@ -42,38 +42,26 @@ struct object_config {
int attributes_count; int attributes_count;
}; };
VAStatus SunxiCedrusQueryConfigProfiles(VADriverContextP ctx, VAStatus SunxiCedrusQueryConfigProfiles(VADriverContextP context,
VAProfile *profile_list, int *num_profiles); VAProfile *profiles, int *profiles_count);
VAStatus SunxiCedrusQueryConfigEntrypoints(VADriverContextP context,
VAStatus SunxiCedrusQueryConfigEntrypoints(VADriverContextP ctx, VAProfile profile, VAEntrypoint *entrypoints, int *entrypoints_count);
VAProfile profile, VAEntrypoint *entrypoint_list, VAStatus SunxiCedrusGetConfigAttributes(VADriverContextP context,
int *num_entrypoints); VAProfile profile, VAEntrypoint entrypoint, VAConfigAttrib *attributes,
int attributes_count);
VAStatus SunxiCedrusGetConfigAttributes(VADriverContextP ctx, VAStatus SunxiCedrusCreateConfig(VADriverContextP context, VAProfile profile,
VAProfile profile, VAEntrypoint entrypoint, VAEntrypoint entrypoint, VAConfigAttrib *attributes,
VAConfigAttrib *attrib_list, int num_attribs); int attributes_count, VAConfigID *config_id);
VAStatus SunxiCedrusDestroyConfig(VADriverContextP context,
VAStatus SunxiCedrusupdate_attribute(struct object_config *obj_config, VAConfigID config_id);
VAConfigAttrib *attrib); VAStatus SunxiCedrusQueryConfigAttributes(VADriverContextP context,
VAConfigID config_id, VAProfile *profile, VAEntrypoint *entrypoint,
VAStatus SunxiCedrusCreateConfig(VADriverContextP ctx, VAProfile profile, VAConfigAttrib *attributes, int *attributes_count);
VAEntrypoint entrypoint, VAConfigAttrib *attrib_list, VAStatus SunxiCedrusQueryDisplayAttributes(VADriverContextP context,
int num_attribs, VAConfigID *config_id); VADisplayAttribute *attributes, int *attributes_count);
VAStatus SunxiCedrusGetDisplayAttributes(VADriverContextP context,
VAStatus SunxiCedrusDestroyConfig(VADriverContextP ctx, VAConfigID config_id); VADisplayAttribute *attributes, int *attributes_count);
VAStatus SunxiCedrusSetDisplayAttributes(VADriverContextP context,
VAStatus SunxiCedrusQueryConfigAttributes(VADriverContextP ctx, VADisplayAttribute *attributes, int *attributes_count);
VAConfigID config_id, VAProfile *profile,
VAEntrypoint *entrypoint, VAConfigAttrib *attrib_list,
int *num_attribs);
VAStatus SunxiCedrusQueryDisplayAttributes (VADriverContextP ctx,
VADisplayAttribute *attr_list, int *num_attributes);
VAStatus SunxiCedrusGetDisplayAttributes (VADriverContextP ctx,
VADisplayAttribute *attr_list, int num_attributes);
VAStatus SunxiCedrusSetDisplayAttributes (VADriverContextP ctx,
VADisplayAttribute *attr_list, int num_attributes);
#endif /* _CONFIG_H_ */ #endif /* _CONFIG_H_ */