media: venus: core: delete not used buffer mode flags
authorStanimir Varbanov <stanimir.varbanov@linaro.org>
Thu, 5 Jul 2018 13:03:50 +0000 (09:03 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Wed, 25 Jul 2018 12:52:04 +0000 (08:52 -0400)
Delete not used flag for capture buffer allocation mode and
no longer used cap_bufs_mode_dynamic from instance structure.

Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>
Reviewed-by: Alexandre Courbot <acourbot@chromium.org>
Tested-by: Alexandre Courbot <acourbot@chromium.org>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/platform/qcom/venus/core.h
drivers/media/platform/qcom/venus/hfi_parser.c

index b995d16..1d1a59a 100644 (file)
@@ -255,8 +255,6 @@ struct venus_buffer {
  * @priv:      a private for HFI operations callbacks
  * @session_type:      the type of the session (decoder or encoder)
  * @hprop:     a union used as a holder by get property
- * @cap_bufs_mode_static:      buffers allocation mode capability
- * @cap_bufs_mode_dynamic:     buffers allocation mode capability
  */
 struct venus_inst {
        struct list_head list;
@@ -305,8 +303,6 @@ struct venus_inst {
        const struct hfi_inst_ops *ops;
        u32 session_type;
        union hfi_get_property hprop;
-       bool cap_bufs_mode_static;
-       bool cap_bufs_mode_dynamic;
 };
 
 #define IS_V1(core)    ((core)->res->hfi_version == HFI_VERSION_1XX)
index afc0db3..2293d93 100644 (file)
@@ -60,8 +60,7 @@ fill_buf_mode(struct venus_caps *cap, const void *data, unsigned int num)
 }
 
 static void
-parse_alloc_mode(struct venus_core *core, struct venus_inst *inst, u32 codecs,
-                u32 domain, void *data)
+parse_alloc_mode(struct venus_core *core, u32 codecs, u32 domain, void *data)
 {
        struct hfi_buffer_alloc_mode_supported *mode = data;
        u32 num_entries = mode->num_entries;
@@ -74,13 +73,9 @@ parse_alloc_mode(struct venus_core *core, struct venus_inst *inst, u32 codecs,
 
        while (num_entries--) {
                if (mode->buffer_type == HFI_BUFFER_OUTPUT ||
-                   mode->buffer_type == HFI_BUFFER_OUTPUT2) {
-                       if (*type == HFI_BUFFER_MODE_DYNAMIC && inst)
-                               inst->cap_bufs_mode_dynamic = true;
-
+                   mode->buffer_type == HFI_BUFFER_OUTPUT2)
                        for_each_codec(core->caps, ARRAY_SIZE(core->caps),
                                       codecs, domain, fill_buf_mode, type, 1);
-               }
 
                type++;
        }
@@ -267,7 +262,7 @@ u32 hfi_parser(struct venus_core *core, struct venus_inst *inst, void *buf,
                        parse_profile_level(core, codecs, domain, data);
                        break;
                case HFI_PROPERTY_PARAM_BUFFER_ALLOC_MODE_SUPPORTED:
-                       parse_alloc_mode(core, inst, codecs, domain, data);
+                       parse_alloc_mode(core, codecs, domain, data);
                        break;
                default:
                        break;