This is the implementation of HFI. It is charged with the
responsibility to comunicate with the firmware through an
interface commands and messages.

 - hfi.c has interface functions used by the core, decoder
and encoder parts to comunicate with the firmware. For example
there are functions for session and core initialisation.

 - hfi_cmds has packetization operations which preparing
packets to be send from host to firmware.

 - hfi_msgs takes care of messages sent from firmware to the
host.

Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org>
---
 drivers/media/platform/qcom/venus/hfi.c        |  506 ++++++++++
 drivers/media/platform/qcom/venus/hfi.h        |  174 ++++
 drivers/media/platform/qcom/venus/hfi_cmds.c   | 1256 ++++++++++++++++++++++++
 drivers/media/platform/qcom/venus/hfi_cmds.h   |  304 ++++++
 drivers/media/platform/qcom/venus/hfi_helper.h | 1045 ++++++++++++++++++++
 drivers/media/platform/qcom/venus/hfi_msgs.c   | 1057 ++++++++++++++++++++
 drivers/media/platform/qcom/venus/hfi_msgs.h   |  283 ++++++
 7 files changed, 4625 insertions(+)
 create mode 100644 drivers/media/platform/qcom/venus/hfi.c
 create mode 100644 drivers/media/platform/qcom/venus/hfi.h
 create mode 100644 drivers/media/platform/qcom/venus/hfi_cmds.c
 create mode 100644 drivers/media/platform/qcom/venus/hfi_cmds.h
 create mode 100644 drivers/media/platform/qcom/venus/hfi_helper.h
 create mode 100644 drivers/media/platform/qcom/venus/hfi_msgs.c
 create mode 100644 drivers/media/platform/qcom/venus/hfi_msgs.h

diff --git a/drivers/media/platform/qcom/venus/hfi.c 
b/drivers/media/platform/qcom/venus/hfi.c
new file mode 100644
index 000000000000..5da35f4ad2c7
--- /dev/null
+++ b/drivers/media/platform/qcom/venus/hfi.c
@@ -0,0 +1,506 @@
+/*
+ * Copyright (c) 2012-2016, The Linux Foundation. All rights reserved.
+ * Copyright (C) 2017 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+#include <linux/slab.h>
+#include <linux/mutex.h>
+#include <linux/list.h>
+#include <linux/completion.h>
+#include <linux/platform_device.h>
+#include <linux/videodev2.h>
+
+#include "core.h"
+#include "hfi.h"
+#include "hfi_cmds.h"
+#include "hfi_venus.h"
+
+#define TIMEOUT                msecs_to_jiffies(1000)
+
+static u32 to_codec_type(u32 pixfmt)
+{
+       switch (pixfmt) {
+       case V4L2_PIX_FMT_H264:
+       case V4L2_PIX_FMT_H264_NO_SC:
+               return HFI_VIDEO_CODEC_H264;
+       case V4L2_PIX_FMT_H263:
+               return HFI_VIDEO_CODEC_H263;
+       case V4L2_PIX_FMT_MPEG1:
+               return HFI_VIDEO_CODEC_MPEG1;
+       case V4L2_PIX_FMT_MPEG2:
+               return HFI_VIDEO_CODEC_MPEG2;
+       case V4L2_PIX_FMT_MPEG4:
+               return HFI_VIDEO_CODEC_MPEG4;
+       case V4L2_PIX_FMT_VC1_ANNEX_G:
+       case V4L2_PIX_FMT_VC1_ANNEX_L:
+               return HFI_VIDEO_CODEC_VC1;
+       case V4L2_PIX_FMT_VP8:
+               return HFI_VIDEO_CODEC_VP8;
+       case V4L2_PIX_FMT_XVID:
+               return HFI_VIDEO_CODEC_DIVX;
+       default:
+               return 0;
+       }
+}
+
+int hfi_core_init(struct venus_core *core)
+{
+       int ret = 0;
+
+       mutex_lock(&core->lock);
+
+       if (core->state >= CORE_INIT)
+               goto unlock;
+
+       reinit_completion(&core->done);
+
+       ret = core->ops->core_init(core);
+       if (ret)
+               goto unlock;
+
+       ret = wait_for_completion_timeout(&core->done, TIMEOUT);
+       if (!ret) {
+               ret = -ETIMEDOUT;
+               goto unlock;
+       }
+
+       ret = 0;
+
+       if (core->error != HFI_ERR_NONE) {
+               ret = -EIO;
+               goto unlock;
+       }
+
+       core->state = CORE_INIT;
+unlock:
+       mutex_unlock(&core->lock);
+       return ret;
+}
+
+int hfi_core_deinit(struct venus_core *core)
+{
+       struct device *dev = core->dev;
+       int ret = 0;
+
+       mutex_lock(&core->lock);
+
+       if (core->state == CORE_UNINIT)
+               goto unlock;
+
+       if (!list_empty(&core->instances)) {
+               ret = -EBUSY;
+               goto unlock;
+       }
+
+       ret = core->ops->core_deinit(core);
+       if (ret)
+               dev_err(dev, "core deinit failed: %d\n", ret);
+
+       core->state = CORE_UNINIT;
+
+unlock:
+       mutex_unlock(&core->lock);
+       return ret;
+}
+
+int hfi_core_suspend(struct venus_core *core)
+{
+       if (core->state != CORE_INIT)
+               return 0;
+
+       return core->ops->suspend(core);
+}
+
+int hfi_core_resume(struct venus_core *core, bool force)
+{
+       if (!force && core->state != CORE_INIT)
+               return 0;
+
+       return core->ops->resume(core);
+}
+
+int hfi_core_trigger_ssr(struct venus_core *core, u32 type)
+{
+       return core->ops->core_trigger_ssr(core, type);
+}
+
+int hfi_core_ping(struct venus_core *core)
+{
+       int ret;
+
+       mutex_lock(&core->lock);
+
+       ret = core->ops->core_ping(core, 0xbeef);
+       if (ret)
+               return ret;
+
+       ret = wait_for_completion_timeout(&core->done, TIMEOUT);
+       if (!ret) {
+               ret = -ETIMEDOUT;
+               goto unlock;
+       }
+       ret = 0;
+       if (core->error != HFI_ERR_NONE)
+               ret = -ENODEV;
+unlock:
+       mutex_unlock(&core->lock);
+       return ret;
+}
+
+static int wait_session_msg(struct venus_inst *inst)
+{
+       int ret;
+
+       ret = wait_for_completion_timeout(&inst->done, TIMEOUT);
+       if (!ret)
+               return -ETIMEDOUT;
+
+       if (inst->error != HFI_ERR_NONE)
+               return -EIO;
+
+       return 0;
+}
+
+int hfi_session_create(struct venus_inst *inst, const struct hfi_inst_ops *ops)
+{
+       struct venus_core *core = inst->core;
+
+       if (!ops)
+               return -EINVAL;
+
+       inst->state = INST_UNINIT;
+       init_completion(&inst->done);
+       inst->ops = ops;
+
+       mutex_lock(&core->lock);
+       list_add_tail(&inst->list, &core->instances);
+       mutex_unlock(&core->lock);
+
+       return 0;
+}
+EXPORT_SYMBOL(hfi_session_create);
+
+int hfi_session_init(struct venus_inst *inst, u32 pixfmt)
+{
+       struct venus_core *core = inst->core;
+       const struct hfi_ops *ops = core->ops;
+       u32 codec;
+       int ret;
+
+       codec = to_codec_type(pixfmt);
+       reinit_completion(&inst->done);
+
+       ret = ops->session_init(inst, inst->session_type, codec);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       inst->state = INST_INIT;
+
+       return 0;
+}
+EXPORT_SYMBOL(hfi_session_init);
+
+void hfi_session_destroy(struct venus_inst *inst)
+{
+       struct venus_core *core = inst->core;
+
+       mutex_lock(&core->lock);
+       list_del(&inst->list);
+       mutex_unlock(&core->lock);
+}
+EXPORT_SYMBOL(hfi_session_destroy);
+
+int hfi_session_deinit(struct venus_inst *inst)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       if (inst->state == INST_UNINIT)
+               return 0;
+
+       if (inst->state < INST_INIT)
+               return -EINVAL;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_end(inst);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       inst->state = INST_UNINIT;
+
+       return 0;
+}
+EXPORT_SYMBOL(hfi_session_deinit);
+
+int hfi_session_start(struct venus_inst *inst)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       if (inst->state != INST_LOAD_RESOURCES)
+               return -EINVAL;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_start(inst);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       inst->state = INST_START;
+
+       return 0;
+}
+
+int hfi_session_stop(struct venus_inst *inst)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       if (inst->state != INST_START)
+               return -EINVAL;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_stop(inst);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       inst->state = INST_STOP;
+
+       return 0;
+}
+
+int hfi_session_continue(struct venus_inst *inst)
+{
+       struct venus_core *core = inst->core;
+
+       if (core->res->hfi_version != HFI_VERSION_3XX)
+               return 0;
+
+       return core->ops->session_continue(inst);
+}
+EXPORT_SYMBOL(hfi_session_continue);
+
+int hfi_session_abort(struct venus_inst *inst)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_abort(inst);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       return 0;
+}
+
+int hfi_session_load_res(struct venus_inst *inst)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       if (inst->state != INST_INIT)
+               return -EINVAL;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_load_res(inst);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       inst->state = INST_LOAD_RESOURCES;
+
+       return 0;
+}
+
+int hfi_session_unload_res(struct venus_inst *inst)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       if (inst->state != INST_STOP)
+               return -EINVAL;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_release_res(inst);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       inst->state = INST_RELEASE_RESOURCES;
+
+       return 0;
+}
+
+int hfi_session_flush(struct venus_inst *inst)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_flush(inst, HFI_FLUSH_ALL);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       return 0;
+}
+
+int hfi_session_set_buffers(struct venus_inst *inst, struct hfi_buffer_desc 
*bd)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+
+       return ops->session_set_buffers(inst, bd);
+}
+
+int hfi_session_unset_buffers(struct venus_inst *inst,
+                             struct hfi_buffer_desc *bd)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_unset_buffers(inst, bd);
+       if (ret)
+               return ret;
+
+       if (!bd->response_required)
+               return 0;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       return 0;
+}
+
+int hfi_session_get_property(struct venus_inst *inst, u32 ptype,
+                            union hfi_get_property *hprop)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+       int ret;
+
+       if (inst->state < INST_INIT || inst->state >= INST_STOP)
+               return -EINVAL;
+
+       reinit_completion(&inst->done);
+
+       ret = ops->session_get_property(inst, ptype);
+       if (ret)
+               return ret;
+
+       ret = wait_session_msg(inst);
+       if (ret)
+               return ret;
+
+       *hprop = inst->hprop;
+
+       return 0;
+}
+EXPORT_SYMBOL(hfi_session_get_property);
+
+int hfi_session_set_property(struct venus_inst *inst, u32 ptype, void *pdata)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+
+       if (inst->state < INST_INIT || inst->state >= INST_STOP)
+               return -EINVAL;
+
+       return ops->session_set_property(inst, ptype, pdata);
+}
+EXPORT_SYMBOL(hfi_session_set_property);
+
+int hfi_session_process_buf(struct venus_inst *inst, struct hfi_frame_data *fd)
+{
+       const struct hfi_ops *ops = inst->core->ops;
+
+       if (fd->buffer_type == HFI_BUFFER_INPUT)
+               return ops->session_etb(inst, fd);
+       else if (fd->buffer_type == HFI_BUFFER_OUTPUT)
+               return ops->session_ftb(inst, fd);
+
+       return -EINVAL;
+}
+
+irqreturn_t hfi_isr_thread(int irq, void *dev_id)
+{
+       struct venus_core *core = dev_id;
+
+       return core->ops->isr_thread(core);
+}
+
+irqreturn_t hfi_isr(int irq, void *dev)
+{
+       struct venus_core *core = dev;
+
+       return core->ops->isr(core);
+}
+
+int hfi_create(struct venus_core *core, const struct hfi_core_ops *ops)
+{
+       int ret;
+
+       if (!ops)
+               return -EINVAL;
+
+       mutex_lock(&core->lock);
+       core->core_ops = ops;
+       core->state = CORE_UNINIT;
+       init_completion(&core->done);
+       pkt_set_version(core->res->hfi_version);
+       ret = venus_hfi_create(core);
+       mutex_unlock(&core->lock);
+
+       return ret;
+}
+
+void hfi_destroy(struct venus_core *core)
+{
+       mutex_lock(&core->lock);
+       venus_hfi_destroy(core);
+       mutex_unlock(&core->lock);
+}
diff --git a/drivers/media/platform/qcom/venus/hfi.h 
b/drivers/media/platform/qcom/venus/hfi.h
new file mode 100644
index 000000000000..96046ceb8248
--- /dev/null
+++ b/drivers/media/platform/qcom/venus/hfi.h
@@ -0,0 +1,174 @@
+/*
+ * Copyright (c) 2012-2016, The Linux Foundation. All rights reserved.
+ * Copyright (C) 2017 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+#ifndef __HFI_H__
+#define __HFI_H__
+
+#include <linux/interrupt.h>
+
+#include "hfi_helper.h"
+
+#define VIDC_SESSION_TYPE_VPE                  0
+#define VIDC_SESSION_TYPE_ENC                  1
+#define VIDC_SESSION_TYPE_DEC                  2
+
+#define VIDC_RESOURCE_NONE                     0
+#define VIDC_RESOURCE_OCMEM                    1
+#define VIDC_RESOURCE_VMEM                     2
+
+struct hfi_buffer_desc {
+       u32 buffer_type;
+       u32 buffer_size;
+       u32 num_buffers;
+       u32 device_addr;
+       u32 extradata_addr;
+       u32 extradata_size;
+       u32 response_required;
+};
+
+struct hfi_frame_data {
+       u32 buffer_type;
+       u32 device_addr;
+       u32 extradata_addr;
+       u64 timestamp;
+       u32 flags;
+       u32 offset;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 mark_target;
+       u32 mark_data;
+       u32 clnt_data;
+       u32 extradata_size;
+};
+
+union hfi_get_property {
+       struct hfi_profile_level profile_level;
+       struct hfi_buffer_requirements bufreq[HFI_BUFFER_TYPE_MAX];
+};
+
+/* HFI events */
+#define EVT_SYS_EVENT_CHANGE                   1
+#define EVT_SYS_WATCHDOG_TIMEOUT               2
+#define EVT_SYS_ERROR                          3
+#define EVT_SESSION_ERROR                      4
+
+/* HFI event callback structure */
+struct hfi_event_data {
+       u32 error;
+       u32 height;
+       u32 width;
+       u32 event_type;
+       u32 packet_buffer;
+       u32 extradata_buffer;
+       u32 profile;
+       u32 level;
+};
+
+/* define core states */
+#define CORE_UNINIT                            0
+#define CORE_INIT                              1
+
+/* define instance states */
+#define INST_UNINIT                            2
+#define INST_INIT                              3
+#define INST_LOAD_RESOURCES                    4
+#define INST_START                             5
+#define INST_STOP                              6
+#define INST_RELEASE_RESOURCES                 7
+
+struct venus_core;
+struct venus_inst;
+
+struct hfi_core_ops {
+       void (*event_notify)(struct venus_core *core, u32 event);
+};
+
+struct hfi_inst_ops {
+       void (*buf_done)(struct venus_inst *inst, unsigned int buf_type,
+                        u32 tag, u32 bytesused, u32 data_offset, u32 flags,
+                        u64 timestamp_us);
+       void (*event_notify)(struct venus_inst *inst, u32 event,
+                            struct hfi_event_data *data);
+};
+
+struct hfi_ops {
+       int (*core_init)(struct venus_core *core);
+       int (*core_deinit)(struct venus_core *core);
+       int (*core_ping)(struct venus_core *core, u32 cookie);
+       int (*core_trigger_ssr)(struct venus_core *core, u32 trigger_type);
+
+       int (*session_init)(struct venus_inst *inst, u32 session_type,
+                           u32 codec);
+       int (*session_end)(struct venus_inst *inst);
+       int (*session_abort)(struct venus_inst *inst);
+       int (*session_flush)(struct venus_inst *inst, u32 flush_mode);
+       int (*session_start)(struct venus_inst *inst);
+       int (*session_stop)(struct venus_inst *inst);
+       int (*session_continue)(struct venus_inst *inst);
+       int (*session_etb)(struct venus_inst *inst, struct hfi_frame_data *fd);
+       int (*session_ftb)(struct venus_inst *inst, struct hfi_frame_data *fd);
+       int (*session_set_buffers)(struct venus_inst *inst,
+                                  struct hfi_buffer_desc *bd);
+       int (*session_unset_buffers)(struct venus_inst *inst,
+                                    struct hfi_buffer_desc *bd);
+       int (*session_load_res)(struct venus_inst *inst);
+       int (*session_release_res)(struct venus_inst *inst);
+       int (*session_parse_seq_hdr)(struct venus_inst *inst, u32 seq_hdr,
+                                    u32 seq_hdr_len);
+       int (*session_get_seq_hdr)(struct venus_inst *inst, u32 seq_hdr,
+                                  u32 seq_hdr_len);
+       int (*session_set_property)(struct venus_inst *inst, u32 ptype,
+                                   void *pdata);
+       int (*session_get_property)(struct venus_inst *inst, u32 ptype);
+
+       int (*resume)(struct venus_core *core);
+       int (*suspend)(struct venus_core *core);
+
+       /* interrupt operations */
+       irqreturn_t (*isr)(struct venus_core *core);
+       irqreturn_t (*isr_thread)(struct venus_core *core);
+};
+
+int hfi_create(struct venus_core *core, const struct hfi_core_ops *ops);
+void hfi_destroy(struct venus_core *core);
+
+int hfi_core_init(struct venus_core *core);
+int hfi_core_deinit(struct venus_core *core);
+int hfi_core_suspend(struct venus_core *core);
+int hfi_core_resume(struct venus_core *core, bool force);
+int hfi_core_trigger_ssr(struct venus_core *core, u32 type);
+int hfi_core_ping(struct venus_core *core);
+int hfi_session_create(struct venus_inst *inst, const struct hfi_inst_ops 
*ops);
+void hfi_session_destroy(struct venus_inst *inst);
+int hfi_session_init(struct venus_inst *inst, u32 pixfmt);
+int hfi_session_deinit(struct venus_inst *inst);
+int hfi_session_start(struct venus_inst *inst);
+int hfi_session_stop(struct venus_inst *inst);
+int hfi_session_continue(struct venus_inst *inst);
+int hfi_session_abort(struct venus_inst *inst);
+int hfi_session_load_res(struct venus_inst *inst);
+int hfi_session_unload_res(struct venus_inst *inst);
+int hfi_session_flush(struct venus_inst *inst);
+int hfi_session_set_buffers(struct venus_inst *inst,
+                           struct hfi_buffer_desc *bd);
+int hfi_session_unset_buffers(struct venus_inst *inst,
+                             struct hfi_buffer_desc *bd);
+int hfi_session_get_property(struct venus_inst *inst, u32 ptype,
+                            union hfi_get_property *hprop);
+int hfi_session_set_property(struct venus_inst *inst, u32 ptype, void *pdata);
+int hfi_session_process_buf(struct venus_inst *inst, struct hfi_frame_data *f);
+irqreturn_t hfi_isr_thread(int irq, void *dev_id);
+irqreturn_t hfi_isr(int irq, void *dev);
+
+#endif
diff --git a/drivers/media/platform/qcom/venus/hfi_cmds.c 
b/drivers/media/platform/qcom/venus/hfi_cmds.c
new file mode 100644
index 000000000000..f6c65b361105
--- /dev/null
+++ b/drivers/media/platform/qcom/venus/hfi_cmds.c
@@ -0,0 +1,1256 @@
+/*
+ * Copyright (c) 2012-2016, The Linux Foundation. All rights reserved.
+ * Copyright (C) 2017 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+#include <linux/errno.h>
+#include <linux/hash.h>
+
+#include "hfi_cmds.h"
+
+static enum hfi_version hfi_ver;
+
+void pkt_sys_init(struct hfi_sys_init_pkt *pkt, u32 arch_type)
+{
+       pkt->hdr.size = sizeof(*pkt);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_INIT;
+       pkt->arch_type = arch_type;
+}
+
+void pkt_sys_pc_prep(struct hfi_sys_pc_prep_pkt *pkt)
+{
+       pkt->hdr.size = sizeof(*pkt);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_PC_PREP;
+}
+
+void pkt_sys_idle_indicator(struct hfi_sys_set_property_pkt *pkt, u32 enable)
+{
+       struct hfi_enable *hfi = (struct hfi_enable *) &pkt->data[1];
+
+       pkt->hdr.size = sizeof(*pkt) + sizeof(*hfi) + sizeof(u32);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_SET_PROPERTY;
+       pkt->num_properties = 1;
+       pkt->data[0] = HFI_PROPERTY_SYS_IDLE_INDICATOR;
+       hfi->enable = enable;
+}
+
+void pkt_sys_debug_config(struct hfi_sys_set_property_pkt *pkt, u32 mode,
+                         u32 config)
+{
+       struct hfi_debug_config *hfi;
+
+       pkt->hdr.size = sizeof(*pkt) + sizeof(*hfi) + sizeof(u32);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_SET_PROPERTY;
+       pkt->num_properties = 1;
+       pkt->data[0] = HFI_PROPERTY_SYS_DEBUG_CONFIG;
+       hfi = (struct hfi_debug_config *) &pkt->data[1];
+       hfi->config = config;
+       hfi->mode = mode;
+}
+
+void pkt_sys_coverage_config(struct hfi_sys_set_property_pkt *pkt, u32 mode)
+{
+       pkt->hdr.size = sizeof(*pkt) + sizeof(u32);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_SET_PROPERTY;
+       pkt->num_properties = 1;
+       pkt->data[0] = HFI_PROPERTY_SYS_CONFIG_COVERAGE;
+       pkt->data[1] = mode;
+}
+
+int pkt_sys_set_resource(struct hfi_sys_set_resource_pkt *pkt, u32 id, u32 
size,
+                        u32 addr, void *cookie)
+{
+       pkt->hdr.size = sizeof(*pkt);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_SET_RESOURCE;
+       pkt->resource_handle = hash32_ptr(cookie);
+
+       switch (id) {
+       case VIDC_RESOURCE_OCMEM:
+       case VIDC_RESOURCE_VMEM: {
+               struct hfi_resource_ocmem *res =
+                       (struct hfi_resource_ocmem *) &pkt->resource_data[0];
+
+               res->size = size;
+               res->mem = addr;
+               pkt->resource_type = HFI_RESOURCE_OCMEM;
+               pkt->hdr.size += sizeof(*res) - sizeof(u32);
+               break;
+       }
+       case VIDC_RESOURCE_NONE:
+       default:
+               return -ENOTSUPP;
+       }
+
+       return 0;
+}
+
+int pkt_sys_unset_resource(struct hfi_sys_release_resource_pkt *pkt, u32 id,
+                          u32 size, void *cookie)
+{
+       pkt->hdr.size = sizeof(*pkt);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_RELEASE_RESOURCE;
+       pkt->resource_handle = hash32_ptr(cookie);
+
+       switch (id) {
+       case VIDC_RESOURCE_OCMEM:
+       case VIDC_RESOURCE_VMEM:
+               pkt->resource_type = HFI_RESOURCE_OCMEM;
+               break;
+       case VIDC_RESOURCE_NONE:
+               break;
+       default:
+               return -ENOTSUPP;
+       }
+
+       return 0;
+}
+
+void pkt_sys_ping(struct hfi_sys_ping_pkt *pkt, u32 cookie)
+{
+       pkt->hdr.size = sizeof(*pkt);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_PING;
+       pkt->client_data = cookie;
+}
+
+void pkt_sys_power_control(struct hfi_sys_set_property_pkt *pkt,
+                                 u32 enable)
+{
+       struct hfi_enable *hfi = (struct hfi_enable *) &pkt->data[1];
+
+       pkt->hdr.size = sizeof(*pkt) + sizeof(*hfi) + sizeof(u32);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_SET_PROPERTY;
+       pkt->num_properties = 1;
+       pkt->data[0] = HFI_PROPERTY_SYS_CODEC_POWER_PLANE_CTRL;
+       hfi->enable = enable;
+}
+
+int pkt_sys_ssr_cmd(struct hfi_sys_test_ssr_pkt *pkt, u32 trigger_type)
+{
+       switch (trigger_type) {
+       case HFI_TEST_SSR_SW_ERR_FATAL:
+       case HFI_TEST_SSR_SW_DIV_BY_ZERO:
+       case HFI_TEST_SSR_HW_WDOG_IRQ:
+               break;
+       default:
+               return -EINVAL;
+       }
+
+       pkt->hdr.size = sizeof(*pkt);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_TEST_SSR;
+       pkt->trigger_type = trigger_type;
+
+       return 0;
+}
+
+void pkt_sys_image_version(struct hfi_sys_get_property_pkt *pkt)
+{
+       pkt->hdr.size = sizeof(*pkt);
+       pkt->hdr.pkt_type = HFI_CMD_SYS_GET_PROPERTY;
+       pkt->num_properties = 1;
+       pkt->data[0] = HFI_PROPERTY_SYS_IMAGE_VERSION;
+}
+
+int pkt_session_init(struct hfi_session_init_pkt *pkt, void *cookie,
+                    u32 session_type, u32 codec)
+{
+       if (!pkt || !cookie || !codec)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SYS_SESSION_INIT;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->session_domain = session_type;
+       pkt->session_codec = codec;
+
+       return 0;
+}
+
+void pkt_session_cmd(struct hfi_session_pkt *pkt, u32 pkt_type, void *cookie)
+{
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = pkt_type;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+}
+
+int pkt_session_set_buffers(struct hfi_session_set_buffers_pkt *pkt,
+                           void *cookie, struct hfi_buffer_desc *bd)
+{
+       int i;
+
+       if (!cookie || !pkt || !bd)
+               return -EINVAL;
+
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_SET_BUFFERS;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->buffer_size = bd->buffer_size;
+       pkt->min_buffer_size = bd->buffer_size;
+       pkt->num_buffers = bd->num_buffers;
+
+       if (bd->buffer_type == HFI_BUFFER_OUTPUT ||
+           bd->buffer_type == HFI_BUFFER_OUTPUT2) {
+               struct hfi_buffer_info *bi;
+
+               pkt->extradata_size = bd->extradata_size;
+               pkt->shdr.hdr.size = sizeof(*pkt) - sizeof(u32) +
+                       (bd->num_buffers * sizeof(*bi));
+               bi = (struct hfi_buffer_info *) pkt->buffer_info;
+               for (i = 0; i < pkt->num_buffers; i++) {
+                       bi->buffer_addr = bd->device_addr;
+                       bi->extradata_addr = bd->extradata_addr;
+               }
+       } else {
+               pkt->extradata_size = 0;
+               pkt->shdr.hdr.size = sizeof(*pkt) +
+                       ((bd->num_buffers - 1) * sizeof(u32));
+               for (i = 0; i < pkt->num_buffers; i++)
+                       pkt->buffer_info[i] = bd->device_addr;
+       }
+
+       pkt->buffer_type = bd->buffer_type;
+
+       return 0;
+}
+
+int pkt_session_unset_buffers(struct hfi_session_release_buffer_pkt *pkt,
+                             void *cookie, struct hfi_buffer_desc *bd)
+{
+       int i;
+
+       if (!cookie || !pkt || !bd)
+               return -EINVAL;
+
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_RELEASE_BUFFERS;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->buffer_size = bd->buffer_size;
+       pkt->num_buffers = bd->num_buffers;
+
+       if (bd->buffer_type == HFI_BUFFER_OUTPUT ||
+           bd->buffer_type == HFI_BUFFER_OUTPUT2) {
+               struct hfi_buffer_info *bi;
+
+               bi = (struct hfi_buffer_info *) pkt->buffer_info;
+               for (i = 0; i < pkt->num_buffers; i++) {
+                       bi->buffer_addr = bd->device_addr;
+                       bi->extradata_addr = bd->extradata_addr;
+               }
+               pkt->shdr.hdr.size =
+                               sizeof(struct hfi_session_set_buffers_pkt) -
+                               sizeof(u32) + (bd->num_buffers * sizeof(*bi));
+       } else {
+               for (i = 0; i < pkt->num_buffers; i++)
+                       pkt->buffer_info[i] = bd->device_addr;
+
+               pkt->extradata_size = 0;
+               pkt->shdr.hdr.size =
+                               sizeof(struct hfi_session_set_buffers_pkt) +
+                               ((bd->num_buffers - 1) * sizeof(u32));
+       }
+
+       pkt->response_req = bd->response_required;
+       pkt->buffer_type = bd->buffer_type;
+
+       return 0;
+}
+
+int pkt_session_etb_decoder(struct hfi_session_empty_buffer_compressed_pkt 
*pkt,
+                           void *cookie, struct hfi_frame_data *in_frame)
+{
+       if (!cookie || !in_frame->device_addr)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_EMPTY_BUFFER;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->time_stamp_hi = upper_32_bits(in_frame->timestamp);
+       pkt->time_stamp_lo = lower_32_bits(in_frame->timestamp);
+       pkt->flags = in_frame->flags;
+       pkt->mark_target = in_frame->mark_target;
+       pkt->mark_data = in_frame->mark_data;
+       pkt->offset = in_frame->offset;
+       pkt->alloc_len = in_frame->alloc_len;
+       pkt->filled_len = in_frame->filled_len;
+       pkt->input_tag = in_frame->clnt_data;
+       pkt->packet_buffer = in_frame->device_addr;
+
+       return 0;
+}
+
+int pkt_session_etb_encoder(
+               struct hfi_session_empty_buffer_uncompressed_plane0_pkt *pkt,
+               void *cookie, struct hfi_frame_data *in_frame)
+{
+       if (!cookie || !in_frame->device_addr)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_EMPTY_BUFFER;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->view_id = 0;
+       pkt->time_stamp_hi = upper_32_bits(in_frame->timestamp);
+       pkt->time_stamp_lo = lower_32_bits(in_frame->timestamp);
+       pkt->flags = in_frame->flags;
+       pkt->mark_target = in_frame->mark_target;
+       pkt->mark_data = in_frame->mark_data;
+       pkt->offset = in_frame->offset;
+       pkt->alloc_len = in_frame->alloc_len;
+       pkt->filled_len = in_frame->filled_len;
+       pkt->input_tag = in_frame->clnt_data;
+       pkt->packet_buffer = in_frame->device_addr;
+       pkt->extradata_buffer = in_frame->extradata_addr;
+
+       return 0;
+}
+
+int pkt_session_ftb(struct hfi_session_fill_buffer_pkt *pkt, void *cookie,
+                   struct hfi_frame_data *out_frame)
+{
+       if (!cookie || !out_frame || !out_frame->device_addr)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_FILL_BUFFER;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+
+       if (out_frame->buffer_type == HFI_BUFFER_OUTPUT)
+               pkt->stream_id = 0;
+       else if (out_frame->buffer_type == HFI_BUFFER_OUTPUT2)
+               pkt->stream_id = 1;
+
+       pkt->output_tag = out_frame->clnt_data;
+       pkt->packet_buffer = out_frame->device_addr;
+       pkt->extradata_buffer = out_frame->extradata_addr;
+       pkt->alloc_len = out_frame->alloc_len;
+       pkt->filled_len = out_frame->filled_len;
+       pkt->offset = out_frame->offset;
+       pkt->data[0] = out_frame->extradata_size;
+
+       return 0;
+}
+
+int pkt_session_parse_seq_header(
+               struct hfi_session_parse_sequence_header_pkt *pkt,
+               void *cookie, u32 seq_hdr, u32 seq_hdr_len)
+{
+       if (!cookie || !seq_hdr || !seq_hdr_len)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_PARSE_SEQUENCE_HEADER;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->header_len = seq_hdr_len;
+       pkt->packet_buffer = seq_hdr;
+
+       return 0;
+}
+
+int pkt_session_get_seq_hdr(struct hfi_session_get_sequence_header_pkt *pkt,
+                           void *cookie, u32 seq_hdr, u32 seq_hdr_len)
+{
+       if (!cookie || !seq_hdr || !seq_hdr_len)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_GET_SEQUENCE_HEADER;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->buffer_len = seq_hdr_len;
+       pkt->packet_buffer = seq_hdr;
+
+       return 0;
+}
+
+int pkt_session_flush(struct hfi_session_flush_pkt *pkt, void *cookie, u32 
type)
+{
+       switch (type) {
+       case HFI_FLUSH_INPUT:
+       case HFI_FLUSH_OUTPUT:
+       case HFI_FLUSH_OUTPUT2:
+       case HFI_FLUSH_ALL:
+               break;
+       default:
+               return -EINVAL;
+       }
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_FLUSH;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->flush_type = type;
+
+       return 0;
+}
+
+static int pkt_session_get_property_1x(struct hfi_session_get_property_pkt 
*pkt,
+                                      void *cookie, u32 ptype)
+{
+       switch (ptype) {
+       case HFI_PROPERTY_PARAM_PROFILE_LEVEL_CURRENT:
+       case HFI_PROPERTY_CONFIG_BUFFER_REQUIREMENTS:
+               break;
+       default:
+               return -EINVAL;
+       }
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_GET_PROPERTY;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->num_properties = 1;
+       pkt->data[0] = ptype;
+
+       return 0;
+}
+
+static int pkt_session_set_property_1x(struct hfi_session_set_property_pkt 
*pkt,
+                                      void *cookie, u32 ptype, void *pdata)
+{
+       void *prop_data = &pkt->data[1];
+       int ret = 0;
+
+       if (!pkt || !cookie || !pdata)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_SET_PROPERTY;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->num_properties = 1;
+
+       switch (ptype) {
+       case HFI_PROPERTY_CONFIG_FRAME_RATE: {
+               struct hfi_framerate *in = pdata, *frate = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_FRAME_RATE;
+               frate->buffer_type = in->buffer_type;
+               frate->framerate = in->framerate;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*frate);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_UNCOMPRESSED_FORMAT_SELECT: {
+               struct hfi_uncompressed_format_select *in = pdata;
+               struct hfi_uncompressed_format_select *hfi = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_UNCOMPRESSED_FORMAT_SELECT;
+               hfi->buffer_type = in->buffer_type;
+               hfi->format = in->format;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*hfi);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_FRAME_SIZE: {
+               struct hfi_framesize *in = pdata, *fsize = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_FRAME_SIZE;
+               fsize->buffer_type = in->buffer_type;
+               fsize->height = in->height;
+               fsize->width = in->width;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*fsize);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_REALTIME: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_REALTIME;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) * 2;
+               break;
+       }
+       case HFI_PROPERTY_PARAM_BUFFER_COUNT_ACTUAL: {
+               struct hfi_buffer_count_actual *in = pdata, *count = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_BUFFER_COUNT_ACTUAL;
+               count->count_actual = in->count_actual;
+               count->type = in->type;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*count);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_BUFFER_SIZE_ACTUAL: {
+               struct hfi_buffer_size_actual *in = pdata, *sz = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_BUFFER_SIZE_ACTUAL;
+               sz->size = in->size;
+               sz->type = in->type;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*sz);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_BUFFER_DISPLAY_HOLD_COUNT_ACTUAL: {
+               struct hfi_buffer_display_hold_count_actual *in = pdata;
+               struct hfi_buffer_display_hold_count_actual *count = prop_data;
+
+               pkt->data[0] =
+                       HFI_PROPERTY_PARAM_BUFFER_DISPLAY_HOLD_COUNT_ACTUAL;
+               count->hold_count = in->hold_count;
+               count->type = in->type;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*count);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_NAL_STREAM_FORMAT_SELECT: {
+               struct hfi_nal_stream_format_select *in = pdata;
+               struct hfi_nal_stream_format_select *fmt = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_NAL_STREAM_FORMAT_SELECT;
+               fmt->format = in->format;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*fmt);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_OUTPUT_ORDER: {
+               u32 *in = pdata;
+
+               switch (*in) {
+               case HFI_OUTPUT_ORDER_DECODE:
+               case HFI_OUTPUT_ORDER_DISPLAY:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_OUTPUT_ORDER;
+               pkt->data[1] = *in;
+               pkt->shdr.hdr.size += sizeof(u32) * 2;
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_PICTURE_TYPE_DECODE: {
+               struct hfi_enable_picture *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_PICTURE_TYPE_DECODE;
+               en->picture_type = in->picture_type;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_OUTPUT2_KEEP_ASPECT_RATIO: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] =
+                       HFI_PROPERTY_PARAM_VDEC_OUTPUT2_KEEP_ASPECT_RATIO;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VDEC_POST_LOOP_DEBLOCKER: {
+               struct hfi_enable *in = pdata;
+               struct hfi_enable *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VDEC_POST_LOOP_DEBLOCKER;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_MULTI_STREAM: {
+               struct hfi_multi_stream *in = pdata, *multi = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_MULTI_STREAM;
+               multi->buffer_type = in->buffer_type;
+               multi->enable = in->enable;
+               multi->width = in->width;
+               multi->height = in->height;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*multi);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_DISPLAY_PICTURE_BUFFER_COUNT: {
+               struct hfi_display_picture_buffer_count *in = pdata;
+               struct hfi_display_picture_buffer_count *count = prop_data;
+
+               pkt->data[0] =
+                       HFI_PROPERTY_PARAM_VDEC_DISPLAY_PICTURE_BUFFER_COUNT;
+               count->count = in->count;
+               count->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*count);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_DIVX_FORMAT: {
+               u32 *in = pdata;
+
+               switch (*in) {
+               case HFI_DIVX_FORMAT_4:
+               case HFI_DIVX_FORMAT_5:
+               case HFI_DIVX_FORMAT_6:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_DIVX_FORMAT;
+               pkt->data[1] = *in;
+               pkt->shdr.hdr.size += sizeof(u32) * 2;
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VDEC_MB_ERROR_MAP_REPORTING: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VDEC_MB_ERROR_MAP_REPORTING;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_CONTINUE_DATA_TRANSFER: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_CONTINUE_DATA_TRANSFER;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_THUMBNAIL_MODE: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_THUMBNAIL_MODE;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_SYNC_FRAME_SEQUENCE_HEADER: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] =
+                       HFI_PROPERTY_CONFIG_VENC_SYNC_FRAME_SEQUENCE_HEADER;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_REQUEST_SYNC_FRAME:
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_REQUEST_SYNC_FRAME;
+               pkt->shdr.hdr.size += sizeof(u32);
+               break;
+       case HFI_PROPERTY_PARAM_VENC_MPEG4_SHORT_HEADER:
+               break;
+       case HFI_PROPERTY_PARAM_VENC_MPEG4_AC_PREDICTION:
+               break;
+       case HFI_PROPERTY_CONFIG_VENC_TARGET_BITRATE: {
+               struct hfi_bitrate *in = pdata, *brate = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_TARGET_BITRATE;
+               brate->bitrate = in->bitrate;
+               brate->layer_id = in->layer_id;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*brate);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_MAX_BITRATE: {
+               struct hfi_bitrate *in = pdata, *hfi = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_MAX_BITRATE;
+               hfi->bitrate = in->bitrate;
+               hfi->layer_id = in->layer_id;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*hfi);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_PROFILE_LEVEL_CURRENT: {
+               struct hfi_profile_level *in = pdata, *pl = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_PROFILE_LEVEL_CURRENT;
+               pl->level = in->level;
+               pl->profile = in->profile;
+               if (pl->profile <= 0)
+                       /* Profile not supported, falling back to high */
+                       pl->profile = HFI_H264_PROFILE_HIGH;
+
+               if (!pl->level)
+                       /* Level not supported, falling back to 1 */
+                       pl->level = 1;
+
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*pl);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_H264_ENTROPY_CONTROL: {
+               struct hfi_h264_entropy_control *in = pdata, *hfi = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_H264_ENTROPY_CONTROL;
+               hfi->entropy_mode = in->entropy_mode;
+               if (hfi->entropy_mode == HFI_H264_ENTROPY_CABAC)
+                       hfi->cabac_model = in->cabac_model;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*hfi);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_RATE_CONTROL: {
+               u32 *in = pdata;
+
+               switch (*in) {
+               case HFI_RATE_CONTROL_OFF:
+               case HFI_RATE_CONTROL_CBR_CFR:
+               case HFI_RATE_CONTROL_CBR_VFR:
+               case HFI_RATE_CONTROL_VBR_CFR:
+               case HFI_RATE_CONTROL_VBR_VFR:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_RATE_CONTROL;
+               pkt->data[1] = *in;
+               pkt->shdr.hdr.size += sizeof(u32) * 2;
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_MPEG4_TIME_RESOLUTION: {
+               struct hfi_mpeg4_time_resolution *in = pdata, *res = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_MPEG4_TIME_RESOLUTION;
+               res->time_increment_resolution = in->time_increment_resolution;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*res);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_MPEG4_HEADER_EXTENSION: {
+               struct hfi_mpeg4_header_extension *in = pdata, *ext = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_MPEG4_HEADER_EXTENSION;
+               ext->header_extension = in->header_extension;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*ext);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_H264_DEBLOCK_CONTROL: {
+               struct hfi_h264_db_control *in = pdata, *db = prop_data;
+
+               switch (in->mode) {
+               case HFI_H264_DB_MODE_DISABLE:
+               case HFI_H264_DB_MODE_SKIP_SLICE_BOUNDARY:
+               case HFI_H264_DB_MODE_ALL_BOUNDARY:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_H264_DEBLOCK_CONTROL;
+               db->mode = in->mode;
+               db->slice_alpha_offset = in->slice_alpha_offset;
+               db->slice_beta_offset = in->slice_beta_offset;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*db);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_SESSION_QP: {
+               struct hfi_quantization *in = pdata, *quant = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_SESSION_QP;
+               quant->qp_i = in->qp_i;
+               quant->qp_p = in->qp_p;
+               quant->qp_b = in->qp_b;
+               quant->layer_id = in->layer_id;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*quant);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_SESSION_QP_RANGE: {
+               struct hfi_quantization_range *in = pdata, *range = prop_data;
+               u32 min_qp, max_qp;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_SESSION_QP_RANGE;
+               min_qp = in->min_qp;
+               max_qp = in->max_qp;
+
+               /* We'll be packing in the qp, so make sure we
+                * won't be losing data when masking
+                */
+               if (min_qp > 0xff || max_qp > 0xff) {
+                       ret = -ERANGE;
+                       break;
+               }
+
+               /* When creating the packet, pack the qp value as
+                * 0xiippbb, where ii = qp range for I-frames,
+                * pp = qp range for P-frames, etc.
+                */
+               range->min_qp = min_qp | min_qp << 8 | min_qp << 16;
+               range->max_qp = max_qp | max_qp << 8 | max_qp << 16;
+               range->layer_id = in->layer_id;
+
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*range);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_VC1_PERF_CFG: {
+               struct hfi_vc1e_perf_cfg_type *in = pdata, *perf = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_VC1_PERF_CFG;
+
+               memcpy(perf->search_range_x_subsampled,
+                      in->search_range_x_subsampled,
+                      sizeof(perf->search_range_x_subsampled));
+               memcpy(perf->search_range_y_subsampled,
+                      in->search_range_y_subsampled,
+                      sizeof(perf->search_range_y_subsampled));
+
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*perf);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_MAX_NUM_B_FRAMES: {
+               struct hfi_max_num_b_frames *bframes = prop_data;
+               u32 *in = pdata;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_MAX_NUM_B_FRAMES;
+               bframes->max_num_b_frames = *in;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*bframes);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_INTRA_PERIOD: {
+               struct hfi_intra_period *in = pdata, *intra = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_INTRA_PERIOD;
+               intra->pframes = in->pframes;
+               intra->bframes = in->bframes;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*intra);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_IDR_PERIOD: {
+               struct hfi_idr_period *in = pdata, *idr = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_IDR_PERIOD;
+               idr->idr_period = in->idr_period;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*idr);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_CONCEAL_COLOR: {
+               struct hfi_conceal_color *color = prop_data;
+               u32 *in = pdata;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_CONCEAL_COLOR;
+               color->conceal_color = *in;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*color);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VPE_OPERATIONS: {
+               struct hfi_operations_type *in = pdata, *ops = prop_data;
+
+               switch (in->rotation) {
+               case HFI_ROTATE_NONE:
+               case HFI_ROTATE_90:
+               case HFI_ROTATE_180:
+               case HFI_ROTATE_270:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               switch (in->flip) {
+               case HFI_FLIP_NONE:
+               case HFI_FLIP_HORIZONTAL:
+               case HFI_FLIP_VERTICAL:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VPE_OPERATIONS;
+               ops->rotation = in->rotation;
+               ops->flip = in->flip;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*ops);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_INTRA_REFRESH: {
+               struct hfi_intra_refresh *in = pdata, *intra = prop_data;
+
+               switch (in->mode) {
+               case HFI_INTRA_REFRESH_NONE:
+               case HFI_INTRA_REFRESH_ADAPTIVE:
+               case HFI_INTRA_REFRESH_CYCLIC:
+               case HFI_INTRA_REFRESH_CYCLIC_ADAPTIVE:
+               case HFI_INTRA_REFRESH_RANDOM:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_INTRA_REFRESH;
+               intra->mode = in->mode;
+               intra->air_mbs = in->air_mbs;
+               intra->air_ref = in->air_ref;
+               intra->cir_mbs = in->cir_mbs;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*intra);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_MULTI_SLICE_CONTROL: {
+               struct hfi_multi_slice_control *in = pdata, *multi = prop_data;
+
+               switch (in->multi_slice) {
+               case HFI_MULTI_SLICE_OFF:
+               case HFI_MULTI_SLICE_GOB:
+               case HFI_MULTI_SLICE_BY_MB_COUNT:
+               case HFI_MULTI_SLICE_BY_BYTE_COUNT:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_MULTI_SLICE_CONTROL;
+               multi->multi_slice = in->multi_slice;
+               multi->slice_size = in->slice_size;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*multi);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_SLICE_DELIVERY_MODE: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_SLICE_DELIVERY_MODE;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_H264_VUI_TIMING_INFO: {
+               struct hfi_h264_vui_timing_info *in = pdata, *vui = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_H264_VUI_TIMING_INFO;
+               vui->enable = in->enable;
+               vui->fixed_framerate = in->fixed_framerate;
+               vui->time_scale = in->time_scale;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*vui);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VPE_DEINTERLACE: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VPE_DEINTERLACE;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_H264_GENERATE_AUDNAL: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_H264_GENERATE_AUDNAL;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_BUFFER_ALLOC_MODE: {
+               struct hfi_buffer_alloc_mode *in = pdata, *mode = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_BUFFER_ALLOC_MODE;
+               mode->type = in->type;
+               mode->mode = in->mode;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*mode);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_FRAME_ASSEMBLY: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_FRAME_ASSEMBLY;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_H264_VUI_BITSTREAM_RESTRC: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] =
+                       HFI_PROPERTY_PARAM_VENC_H264_VUI_BITSTREAM_RESTRC;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_PRESERVE_TEXT_QUALITY: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_PRESERVE_TEXT_QUALITY;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_SCS_THRESHOLD: {
+               struct hfi_scs_threshold *thres = prop_data;
+               u32 *in = pdata;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_SCS_THRESHOLD;
+               thres->threshold_value = *in;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*thres);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_MVC_BUFFER_LAYOUT: {
+               struct hfi_mvc_buffer_layout_descp_type *in = pdata;
+               struct hfi_mvc_buffer_layout_descp_type *mvc = prop_data;
+
+               switch (in->layout_type) {
+               case HFI_MVC_BUFFER_LAYOUT_TOP_BOTTOM:
+               case HFI_MVC_BUFFER_LAYOUT_SEQ:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_MVC_BUFFER_LAYOUT;
+               mvc->layout_type = in->layout_type;
+               mvc->bright_view_first = in->bright_view_first;
+               mvc->ngap = in->ngap;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*mvc);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_LTRMODE: {
+               struct hfi_ltr_mode *in = pdata, *ltr = prop_data;
+
+               switch (in->ltr_mode) {
+               case HFI_LTR_MODE_DISABLE:
+               case HFI_LTR_MODE_MANUAL:
+               case HFI_LTR_MODE_PERIODIC:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_LTRMODE;
+               ltr->ltr_mode = in->ltr_mode;
+               ltr->ltr_count = in->ltr_count;
+               ltr->trust_mode = in->trust_mode;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*ltr);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_USELTRFRAME: {
+               struct hfi_ltr_use *in = pdata, *ltr_use = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_USELTRFRAME;
+               ltr_use->frames = in->frames;
+               ltr_use->ref_ltr = in->ref_ltr;
+               ltr_use->use_constrnt = in->use_constrnt;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*ltr_use);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_MARKLTRFRAME: {
+               struct hfi_ltr_mark *in = pdata, *ltr_mark = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_MARKLTRFRAME;
+               ltr_mark->mark_frame = in->mark_frame;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*ltr_mark);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_HIER_P_MAX_NUM_ENH_LAYER: {
+               u32 *in = pdata;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_HIER_P_MAX_NUM_ENH_LAYER;
+               pkt->data[1] = *in;
+               pkt->shdr.hdr.size += sizeof(u32) * 2;
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_HIER_P_ENH_LAYER: {
+               u32 *in = pdata;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_HIER_P_ENH_LAYER;
+               pkt->data[1] = *in;
+               pkt->shdr.hdr.size += sizeof(u32) * 2;
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_DISABLE_RC_TIMESTAMP: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_DISABLE_RC_TIMESTAMP;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_INITIAL_QP: {
+               struct hfi_initial_quantization *in = pdata, *quant = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_INITIAL_QP;
+               quant->init_qp_enable = in->init_qp_enable;
+               quant->qp_i = in->qp_i;
+               quant->qp_p = in->qp_p;
+               quant->qp_b = in->qp_b;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*quant);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VPE_COLOR_SPACE_CONVERSION: {
+               struct hfi_vpe_color_space_conversion *in = pdata;
+               struct hfi_vpe_color_space_conversion *csc = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VPE_COLOR_SPACE_CONVERSION;
+               memcpy(csc->csc_matrix, in->csc_matrix,
+                       sizeof(csc->csc_matrix));
+               memcpy(csc->csc_bias, in->csc_bias, sizeof(csc->csc_bias));
+               memcpy(csc->csc_limit, in->csc_limit, sizeof(csc->csc_limit));
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*csc);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_VPX_ERROR_RESILIENCE_MODE: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] =
+                       HFI_PROPERTY_PARAM_VENC_VPX_ERROR_RESILIENCE_MODE;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_H264_NAL_SVC_EXT: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_H264_NAL_SVC_EXT;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_CONFIG_VENC_PERF_MODE: {
+               u32 *in = pdata;
+
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VENC_PERF_MODE;
+               pkt->data[1] = *in;
+               pkt->shdr.hdr.size += sizeof(u32) * 2;
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_HIER_B_MAX_NUM_ENH_LAYER: {
+               u32 *in = pdata;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_HIER_B_MAX_NUM_ENH_LAYER;
+               pkt->data[1] = *in;
+               pkt->shdr.hdr.size += sizeof(u32) * 2;
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_NONCP_OUTPUT2: {
+               struct hfi_enable *in = pdata, *en = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_NONCP_OUTPUT2;
+               en->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*en);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_HIER_P_HYBRID_MODE: {
+               struct hfi_hybrid_hierp *in = pdata, *hierp = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_HIER_P_HYBRID_MODE;
+               hierp->layers = in->layers;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*hierp);
+               break;
+       }
+
+       /* FOLLOWING PROPERTIES ARE NOT IMPLEMENTED IN CORE YET */
+       case HFI_PROPERTY_CONFIG_BUFFER_REQUIREMENTS:
+       case HFI_PROPERTY_CONFIG_PRIORITY:
+       case HFI_PROPERTY_CONFIG_BATCH_INFO:
+       case HFI_PROPERTY_SYS_IDLE_INDICATOR:
+       case HFI_PROPERTY_PARAM_UNCOMPRESSED_FORMAT_SUPPORTED:
+       case HFI_PROPERTY_PARAM_INTERLACE_FORMAT_SUPPORTED:
+       case HFI_PROPERTY_PARAM_CHROMA_SITE:
+       case HFI_PROPERTY_PARAM_PROPERTIES_SUPPORTED:
+       case HFI_PROPERTY_PARAM_PROFILE_LEVEL_SUPPORTED:
+       case HFI_PROPERTY_PARAM_CAPABILITY_SUPPORTED:
+       case HFI_PROPERTY_PARAM_NAL_STREAM_FORMAT_SUPPORTED:
+       case HFI_PROPERTY_PARAM_MULTI_VIEW_FORMAT:
+       case HFI_PROPERTY_PARAM_MAX_SEQUENCE_HEADER_SIZE:
+       case HFI_PROPERTY_PARAM_CODEC_SUPPORTED:
+       case HFI_PROPERTY_PARAM_VDEC_MULTI_VIEW_SELECT:
+       case HFI_PROPERTY_PARAM_VDEC_MB_QUANTIZATION:
+       case HFI_PROPERTY_PARAM_VDEC_NUM_CONCEALED_MB:
+       case HFI_PROPERTY_PARAM_VDEC_H264_ENTROPY_SWITCHING:
+       case HFI_PROPERTY_PARAM_VENC_MULTI_SLICE_INFO:
+       default:
+               return -EINVAL;
+       }
+
+       return ret;
+}
+
+static int
+pkt_session_get_property_3xx(struct hfi_session_get_property_pkt *pkt,
+                            void *cookie, u32 ptype)
+{
+       int ret = 0;
+
+       if (!pkt || !cookie)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(struct hfi_session_get_property_pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_GET_PROPERTY;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->num_properties = 1;
+
+       switch (ptype) {
+       case HFI_PROPERTY_CONFIG_VDEC_ENTROPY:
+               pkt->data[0] = HFI_PROPERTY_CONFIG_VDEC_ENTROPY;
+               break;
+       default:
+               ret = pkt_session_get_property_1x(pkt, cookie, ptype);
+               break;
+       }
+
+       return ret;
+}
+
+static int
+pkt_session_set_property_3xx(struct hfi_session_set_property_pkt *pkt,
+                            void *cookie, u32 ptype, void *pdata)
+{
+       void *prop_data = &pkt->data[1];
+       int ret = 0;
+
+       if (!pkt || !cookie || !pdata)
+               return -EINVAL;
+
+       pkt->shdr.hdr.size = sizeof(*pkt);
+       pkt->shdr.hdr.pkt_type = HFI_CMD_SESSION_SET_PROPERTY;
+       pkt->shdr.session_id = hash32_ptr(cookie);
+       pkt->num_properties = 1;
+
+       /*
+        * Any session set property which is different in 3XX packetization
+        * should be added as a new case below. All unchanged session set
+        * properties will be handled in the default case.
+        */
+       switch (ptype) {
+       case HFI_PROPERTY_PARAM_VDEC_MULTI_STREAM: {
+               struct hfi_multi_stream *in = pdata;
+               struct hfi_multi_stream_3x *multi = prop_data;
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VDEC_MULTI_STREAM;
+               multi->buffer_type = in->buffer_type;
+               multi->enable = in->enable;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*multi);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VENC_INTRA_REFRESH: {
+               struct hfi_intra_refresh *in = pdata;
+               struct hfi_intra_refresh_3x *intra = prop_data;
+
+               switch (in->mode) {
+               case HFI_INTRA_REFRESH_NONE:
+               case HFI_INTRA_REFRESH_ADAPTIVE:
+               case HFI_INTRA_REFRESH_CYCLIC:
+               case HFI_INTRA_REFRESH_CYCLIC_ADAPTIVE:
+               case HFI_INTRA_REFRESH_RANDOM:
+                       break;
+               default:
+                       ret = -EINVAL;
+                       break;
+               }
+
+               pkt->data[0] = HFI_PROPERTY_PARAM_VENC_INTRA_REFRESH;
+               intra->mode = in->mode;
+               intra->mbs = in->cir_mbs;
+               pkt->shdr.hdr.size += sizeof(u32) + sizeof(*intra);
+               break;
+       }
+       case HFI_PROPERTY_PARAM_VDEC_CONTINUE_DATA_TRANSFER:
+               /* for 3xx fw version session_continue is used */
+               break;
+       default:
+               ret = pkt_session_set_property_1x(pkt, cookie, ptype, pdata);
+               break;
+       }
+
+       return ret;
+}
+
+int pkt_session_get_property(struct hfi_session_get_property_pkt *pkt,
+                            void *cookie, u32 ptype)
+{
+       if (hfi_ver == HFI_VERSION_LEGACY)
+               return pkt_session_get_property_1x(pkt, cookie, ptype);
+
+       return pkt_session_get_property_3xx(pkt, cookie, ptype);
+}
+
+int pkt_session_set_property(struct hfi_session_set_property_pkt *pkt,
+                            void *cookie, u32 ptype, void *pdata)
+{
+       if (hfi_ver == HFI_VERSION_LEGACY)
+               return pkt_session_set_property_1x(pkt, cookie, ptype, pdata);
+
+       return pkt_session_set_property_3xx(pkt, cookie, ptype, pdata);
+}
+
+void pkt_set_version(enum hfi_version version)
+{
+       hfi_ver = version;
+}
diff --git a/drivers/media/platform/qcom/venus/hfi_cmds.h 
b/drivers/media/platform/qcom/venus/hfi_cmds.h
new file mode 100644
index 000000000000..f7617cf59914
--- /dev/null
+++ b/drivers/media/platform/qcom/venus/hfi_cmds.h
@@ -0,0 +1,304 @@
+/*
+ * Copyright (c) 2012-2016, The Linux Foundation. All rights reserved.
+ * Copyright (C) 2017 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+#ifndef __VENUS_HFI_CMDS_H__
+#define __VENUS_HFI_CMDS_H__
+
+#include "hfi.h"
+
+/* commands */
+#define HFI_CMD_SYS_INIT                       0x10001
+#define HFI_CMD_SYS_PC_PREP                    0x10002
+#define HFI_CMD_SYS_SET_RESOURCE               0x10003
+#define HFI_CMD_SYS_RELEASE_RESOURCE           0x10004
+#define HFI_CMD_SYS_SET_PROPERTY               0x10005
+#define HFI_CMD_SYS_GET_PROPERTY               0x10006
+#define HFI_CMD_SYS_SESSION_INIT               0x10007
+#define HFI_CMD_SYS_SESSION_END                        0x10008
+#define HFI_CMD_SYS_SET_BUFFERS                        0x10009
+#define HFI_CMD_SYS_TEST_SSR                   0x10101
+
+#define HFI_CMD_SESSION_SET_PROPERTY           0x11001
+#define HFI_CMD_SESSION_SET_BUFFERS            0x11002
+#define HFI_CMD_SESSION_GET_SEQUENCE_HEADER    0x11003
+
+#define HFI_CMD_SYS_SESSION_ABORT              0x210001
+#define HFI_CMD_SYS_PING                       0x210002
+
+#define HFI_CMD_SESSION_LOAD_RESOURCES         0x211001
+#define HFI_CMD_SESSION_START                  0x211002
+#define HFI_CMD_SESSION_STOP                   0x211003
+#define HFI_CMD_SESSION_EMPTY_BUFFER           0x211004
+#define HFI_CMD_SESSION_FILL_BUFFER            0x211005
+#define HFI_CMD_SESSION_SUSPEND                        0x211006
+#define HFI_CMD_SESSION_RESUME                 0x211007
+#define HFI_CMD_SESSION_FLUSH                  0x211008
+#define HFI_CMD_SESSION_GET_PROPERTY           0x211009
+#define HFI_CMD_SESSION_PARSE_SEQUENCE_HEADER  0x21100a
+#define HFI_CMD_SESSION_RELEASE_BUFFERS                0x21100b
+#define HFI_CMD_SESSION_RELEASE_RESOURCES      0x21100c
+#define HFI_CMD_SESSION_CONTINUE               0x21100d
+#define HFI_CMD_SESSION_SYNC                   0x21100e
+
+/* command packets */
+struct hfi_sys_init_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 arch_type;
+};
+
+struct hfi_sys_pc_prep_pkt {
+       struct hfi_pkt_hdr hdr;
+};
+
+struct hfi_sys_set_resource_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 resource_handle;
+       u32 resource_type;
+       u32 resource_data[1];
+};
+
+struct hfi_sys_release_resource_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 resource_type;
+       u32 resource_handle;
+};
+
+struct hfi_sys_set_property_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 num_properties;
+       u32 data[1];
+};
+
+struct hfi_sys_get_property_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 num_properties;
+       u32 data[1];
+};
+
+struct hfi_sys_set_buffers_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 buffer_type;
+       u32 buffer_size;
+       u32 num_buffers;
+       u32 buffer_addr[1];
+};
+
+struct hfi_sys_ping_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 client_data;
+};
+
+struct hfi_session_init_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 session_domain;
+       u32 session_codec;
+};
+
+struct hfi_session_end_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+struct hfi_session_abort_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+struct hfi_session_set_property_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 num_properties;
+       u32 data[0];
+};
+
+struct hfi_session_set_buffers_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 buffer_type;
+       u32 buffer_size;
+       u32 extradata_size;
+       u32 min_buffer_size;
+       u32 num_buffers;
+       u32 buffer_info[1];
+};
+
+struct hfi_session_get_sequence_header_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 buffer_len;
+       u32 packet_buffer;
+};
+
+struct hfi_session_load_resources_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+struct hfi_session_start_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+struct hfi_session_stop_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+struct hfi_session_empty_buffer_compressed_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 time_stamp_hi;
+       u32 time_stamp_lo;
+       u32 flags;
+       u32 mark_target;
+       u32 mark_data;
+       u32 offset;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 input_tag;
+       u32 packet_buffer;
+       u32 extradata_buffer;
+       u32 data[1];
+};
+
+struct hfi_session_empty_buffer_uncompressed_plane0_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 view_id;
+       u32 time_stamp_hi;
+       u32 time_stamp_lo;
+       u32 flags;
+       u32 mark_target;
+       u32 mark_data;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 offset;
+       u32 input_tag;
+       u32 packet_buffer;
+       u32 extradata_buffer;
+       u32 data[1];
+};
+
+struct hfi_session_empty_buffer_uncompressed_plane1_pkt {
+       u32 flags;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 offset;
+       u32 packet_buffer2;
+       u32 data[1];
+};
+
+struct hfi_session_empty_buffer_uncompressed_plane2_pkt {
+       u32 flags;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 offset;
+       u32 packet_buffer3;
+       u32 data[1];
+};
+
+struct hfi_session_fill_buffer_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 stream_id;
+       u32 offset;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 output_tag;
+       u32 packet_buffer;
+       u32 extradata_buffer;
+       u32 data[1];
+};
+
+struct hfi_session_flush_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 flush_type;
+};
+
+struct hfi_session_suspend_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+struct hfi_session_resume_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+struct hfi_session_get_property_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 num_properties;
+       u32 data[1];
+};
+
+struct hfi_session_release_buffer_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 buffer_type;
+       u32 buffer_size;
+       u32 extradata_size;
+       u32 response_req;
+       u32 num_buffers;
+       u32 buffer_info[1];
+};
+
+struct hfi_session_release_resources_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+struct hfi_session_parse_sequence_header_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 header_len;
+       u32 packet_buffer;
+};
+
+struct hfi_sfr {
+       u32 buf_size;
+       u8 data[1];
+};
+
+struct hfi_sys_test_ssr_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 trigger_type;
+};
+
+void pkt_set_version(enum hfi_version version);
+
+void pkt_sys_init(struct hfi_sys_init_pkt *pkt, u32 arch_type);
+void pkt_sys_pc_prep(struct hfi_sys_pc_prep_pkt *pkt);
+void pkt_sys_idle_indicator(struct hfi_sys_set_property_pkt *pkt, u32 enable);
+void pkt_sys_power_control(struct hfi_sys_set_property_pkt *pkt, u32 enable);
+int pkt_sys_set_resource(struct hfi_sys_set_resource_pkt *pkt, u32 id, u32 
size,
+                        u32 addr, void *cookie);
+int pkt_sys_unset_resource(struct hfi_sys_release_resource_pkt *pkt, u32 id,
+                          u32 size, void *cookie);
+void pkt_sys_debug_config(struct hfi_sys_set_property_pkt *pkt, u32 mode,
+                         u32 config);
+void pkt_sys_coverage_config(struct hfi_sys_set_property_pkt *pkt, u32 mode);
+void pkt_sys_ping(struct hfi_sys_ping_pkt *pkt, u32 cookie);
+void pkt_sys_image_version(struct hfi_sys_get_property_pkt *pkt);
+int pkt_sys_ssr_cmd(struct hfi_sys_test_ssr_pkt *pkt, u32 trigger_type);
+int pkt_session_init(struct hfi_session_init_pkt *pkt, void *cookie,
+                    u32 session_type, u32 codec);
+void pkt_session_cmd(struct hfi_session_pkt *pkt, u32 pkt_type, void *cookie);
+int pkt_session_set_buffers(struct hfi_session_set_buffers_pkt *pkt,
+                           void *cookie, struct hfi_buffer_desc *bd);
+int pkt_session_unset_buffers(struct hfi_session_release_buffer_pkt *pkt,
+                             void *cookie, struct hfi_buffer_desc *bd);
+int pkt_session_etb_decoder(struct hfi_session_empty_buffer_compressed_pkt 
*pkt,
+                           void *cookie, struct hfi_frame_data *input_frame);
+int pkt_session_etb_encoder(
+               struct hfi_session_empty_buffer_uncompressed_plane0_pkt *pkt,
+               void *cookie, struct hfi_frame_data *input_frame);
+int pkt_session_ftb(struct hfi_session_fill_buffer_pkt *pkt,
+                   void *cookie, struct hfi_frame_data *output_frame);
+int pkt_session_parse_seq_header(
+               struct hfi_session_parse_sequence_header_pkt *pkt,
+               void *cookie, u32 seq_hdr, u32 seq_hdr_len);
+int pkt_session_get_seq_hdr(struct hfi_session_get_sequence_header_pkt *pkt,
+                           void *cookie, u32 seq_hdr, u32 seq_hdr_len);
+int pkt_session_flush(struct hfi_session_flush_pkt *pkt, void *cookie,
+                     u32 flush_mode);
+int pkt_session_get_property(struct hfi_session_get_property_pkt *pkt,
+                            void *cookie, u32 ptype);
+int pkt_session_set_property(struct hfi_session_set_property_pkt *pkt,
+                            void *cookie, u32 ptype, void *pdata);
+
+#endif
diff --git a/drivers/media/platform/qcom/venus/hfi_helper.h 
b/drivers/media/platform/qcom/venus/hfi_helper.h
new file mode 100644
index 000000000000..b9083e681b01
--- /dev/null
+++ b/drivers/media/platform/qcom/venus/hfi_helper.h
@@ -0,0 +1,1045 @@
+/*
+ * Copyright (c) 2012-2016, The Linux Foundation. All rights reserved.
+ * Copyright (C) 2017 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+#ifndef __VENUS_HFI_HELPER_H__
+#define __VENUS_HFI_HELPER_H__
+
+#define HFI_DOMAIN_BASE_COMMON                         0
+
+#define HFI_DOMAIN_BASE_VDEC                           0x1000000
+#define HFI_DOMAIN_BASE_VENC                           0x2000000
+#define HFI_DOMAIN_BASE_VPE                            0x3000000
+
+#define HFI_VIDEO_ARCH_OX                              0x1
+
+#define HFI_ARCH_COMMON_OFFSET                         0
+#define HFI_ARCH_OX_OFFSET                             0x200000
+
+#define HFI_OX_BASE                                    0x1000000
+
+#define HFI_CMD_START_OFFSET                           0x10000
+#define HFI_MSG_START_OFFSET                           0x20000
+
+#define HFI_ERR_NONE                                   0x0
+#define HFI_ERR_SYS_FATAL                              0x1
+#define HFI_ERR_SYS_INVALID_PARAMETER                  0x2
+#define HFI_ERR_SYS_VERSION_MISMATCH                   0x3
+#define HFI_ERR_SYS_INSUFFICIENT_RESOURCES             0x4
+#define HFI_ERR_SYS_MAX_SESSIONS_REACHED               0x5
+#define HFI_ERR_SYS_UNSUPPORTED_CODEC                  0x6
+#define HFI_ERR_SYS_SESSION_IN_USE                     0x7
+#define HFI_ERR_SYS_SESSION_ID_OUT_OF_RANGE            0x8
+#define HFI_ERR_SYS_UNSUPPORTED_DOMAIN                 0x9
+
+#define HFI_ERR_SESSION_FATAL                          0x1001
+#define HFI_ERR_SESSION_INVALID_PARAMETER              0x1002
+#define HFI_ERR_SESSION_BAD_POINTER                    0x1003
+#define HFI_ERR_SESSION_INVALID_SESSION_ID             0x1004
+#define HFI_ERR_SESSION_INVALID_STREAM_ID              0x1005
+#define HFI_ERR_SESSION_INCORRECT_STATE_OPERATION      0x1006
+#define HFI_ERR_SESSION_UNSUPPORTED_PROPERTY           0x1007
+#define HFI_ERR_SESSION_UNSUPPORTED_SETTING            0x1008
+#define HFI_ERR_SESSION_INSUFFICIENT_RESOURCES         0x1009
+#define HFI_ERR_SESSION_STREAM_CORRUPT_OUTPUT_STALLED  0x100a
+#define HFI_ERR_SESSION_STREAM_CORRUPT                 0x100b
+#define HFI_ERR_SESSION_ENC_OVERFLOW                   0x100c
+#define HFI_ERR_SESSION_UNSUPPORTED_STREAM             0x100d
+#define HFI_ERR_SESSION_CMDSIZE                                0x100e
+#define HFI_ERR_SESSION_UNSUPPORT_CMD                  0x100f
+#define HFI_ERR_SESSION_UNSUPPORT_BUFFERTYPE           0x1010
+#define HFI_ERR_SESSION_BUFFERCOUNT_TOOSMALL           0x1011
+#define HFI_ERR_SESSION_INVALID_SCALE_FACTOR           0x1012
+#define HFI_ERR_SESSION_UPSCALE_NOT_SUPPORTED          0x1013
+
+#define HFI_EVENT_SYS_ERROR                            0x1
+#define HFI_EVENT_SESSION_ERROR                                0x2
+
+#define HFI_EVENT_DATA_SEQUENCE_CHANGED_SUFFICIENT_BUF_RESOURCES   0x1000001
+#define HFI_EVENT_DATA_SEQUENCE_CHANGED_INSUFFICIENT_BUF_RESOURCES 0x1000002
+#define HFI_EVENT_SESSION_SEQUENCE_CHANGED                        0x1000003
+#define HFI_EVENT_SESSION_PROPERTY_CHANGED                        0x1000004
+#define HFI_EVENT_SESSION_LTRUSE_FAILED                                   
0x1000005
+#define HFI_EVENT_RELEASE_BUFFER_REFERENCE                        0x1000006
+
+#define HFI_BUFFERFLAG_EOS                             0x00000001
+#define HFI_BUFFERFLAG_STARTTIME                       0x00000002
+#define HFI_BUFFERFLAG_DECODEONLY                      0x00000004
+#define HFI_BUFFERFLAG_DATACORRUPT                     0x00000008
+#define HFI_BUFFERFLAG_ENDOFFRAME                      0x00000010
+#define HFI_BUFFERFLAG_SYNCFRAME                       0x00000020
+#define HFI_BUFFERFLAG_EXTRADATA                       0x00000040
+#define HFI_BUFFERFLAG_CODECCONFIG                     0x00000080
+#define HFI_BUFFERFLAG_TIMESTAMPINVALID                        0x00000100
+#define HFI_BUFFERFLAG_READONLY                                0x00000200
+#define HFI_BUFFERFLAG_ENDOFSUBFRAME                   0x00000400
+#define HFI_BUFFERFLAG_EOSEQ                           0x00200000
+#define HFI_BUFFERFLAG_MBAFF                           0x08000000
+#define HFI_BUFFERFLAG_VPE_YUV_601_709_CSC_CLAMP       0x10000000
+#define HFI_BUFFERFLAG_DROP_FRAME                      0x20000000
+#define HFI_BUFFERFLAG_TEI                             0x40000000
+#define HFI_BUFFERFLAG_DISCONTINUITY                   0x80000000
+
+#define HFI_ERR_SESSION_EMPTY_BUFFER_DONE_OUTPUT_PENDING       0x1001001
+#define HFI_ERR_SESSION_SAME_STATE_OPERATION                   0x1001002
+#define HFI_ERR_SESSION_SYNC_FRAME_NOT_DETECTED                        
0x1001003
+#define HFI_ERR_SESSION_START_CODE_NOT_FOUND                   0x1001004
+
+#define HFI_FLUSH_INPUT                                        0x1001001
+#define HFI_FLUSH_OUTPUT                               0x1001002
+#define HFI_FLUSH_OUTPUT2                              0x1001003
+#define HFI_FLUSH_ALL                                  0x1001004
+
+#define HFI_EXTRADATA_NONE                             0x00000000
+#define HFI_EXTRADATA_MB_QUANTIZATION                  0x00000001
+#define HFI_EXTRADATA_INTERLACE_VIDEO                  0x00000002
+#define HFI_EXTRADATA_VC1_FRAMEDISP                    0x00000003
+#define HFI_EXTRADATA_VC1_SEQDISP                      0x00000004
+#define HFI_EXTRADATA_TIMESTAMP                                0x00000005
+#define HFI_EXTRADATA_S3D_FRAME_PACKING                        0x00000006
+#define HFI_EXTRADATA_FRAME_RATE                       0x00000007
+#define HFI_EXTRADATA_PANSCAN_WINDOW                   0x00000008
+#define HFI_EXTRADATA_RECOVERY_POINT_SEI               0x00000009
+#define HFI_EXTRADATA_MPEG2_SEQDISP                    0x0000000d
+#define HFI_EXTRADATA_STREAM_USERDATA                  0x0000000e
+#define HFI_EXTRADATA_FRAME_QP                         0x0000000f
+#define HFI_EXTRADATA_FRAME_BITS_INFO                  0x00000010
+#define HFI_EXTRADATA_MULTISLICE_INFO                  0x7f100000
+#define HFI_EXTRADATA_NUM_CONCEALED_MB                 0x7f100001
+#define HFI_EXTRADATA_INDEX                            0x7f100002
+#define HFI_EXTRADATA_METADATA_LTR                     0x7f100004
+#define HFI_EXTRADATA_METADATA_FILLER                  0x7fe00002
+
+#define HFI_INDEX_EXTRADATA_INPUT_CROP                 0x0700000e
+#define HFI_INDEX_EXTRADATA_DIGITAL_ZOOM               0x07000010
+#define HFI_INDEX_EXTRADATA_ASPECT_RATIO               0x7f100003
+
+#define HFI_INTERLACE_FRAME_PROGRESSIVE                        0x01
+#define HFI_INTERLACE_INTERLEAVE_FRAME_TOPFIELDFIRST   0x02
+#define HFI_INTERLACE_INTERLEAVE_FRAME_BOTTOMFIELDFIRST        0x04
+#define HFI_INTERLACE_FRAME_TOPFIELDFIRST              0x08
+#define HFI_INTERLACE_FRAME_BOTTOMFIELDFIRST           0x10
+
+/*
+ * HFI_PROPERTY_PARAM_OX_START
+ * HFI_DOMAIN_BASE_COMMON + HFI_ARCH_OX_OFFSET + 0x1000
+ */
+#define HFI_PROPERTY_PARAM_BUFFER_COUNT_ACTUAL                         0x201001
+#define HFI_PROPERTY_PARAM_UNCOMPRESSED_PLANE_ACTUAL_CONSTRAINTS_INFO  0x201002
+#define HFI_PROPERTY_PARAM_INTERLACE_FORMAT_SUPPORTED                  0x201003
+#define HFI_PROPERTY_PARAM_CHROMA_SITE                                 0x201004
+#define HFI_PROPERTY_PARAM_EXTRA_DATA_HEADER_CONFIG                    0x201005
+#define HFI_PROPERTY_PARAM_INDEX_EXTRADATA                             0x201006
+#define HFI_PROPERTY_PARAM_DIVX_FORMAT                                 0x201007
+#define HFI_PROPERTY_PARAM_BUFFER_ALLOC_MODE                           0x201008
+#define HFI_PROPERTY_PARAM_S3D_FRAME_PACKING_EXTRADATA                 0x201009
+#define HFI_PROPERTY_PARAM_ERR_DETECTION_CODE_EXTRADATA                        
0x20100a
+#define HFI_PROPERTY_PARAM_BUFFER_ALLOC_MODE_SUPPORTED                 0x20100b
+#define HFI_PROPERTY_PARAM_BUFFER_SIZE_ACTUAL                          0x20100c
+#define HFI_PROPERTY_PARAM_BUFFER_DISPLAY_HOLD_COUNT_ACTUAL            0x20100d
+
+/*
+ * HFI_PROPERTY_CONFIG_OX_START
+ * HFI_DOMAIN_BASE_COMMON + HFI_ARCH_OX_OFFSET + 0x2000
+ */
+#define HFI_PROPERTY_CONFIG_BUFFER_REQUIREMENTS                0x202001
+#define HFI_PROPERTY_CONFIG_REALTIME                   0x202002
+#define HFI_PROPERTY_CONFIG_PRIORITY                   0x202003
+#define HFI_PROPERTY_CONFIG_BATCH_INFO                 0x202004
+
+/*
+ * HFI_PROPERTY_PARAM_VDEC_OX_START    \
+ * HFI_DOMAIN_BASE_VDEC + HFI_ARCH_OX_OFFSET + 0x3000
+ */
+#define HFI_PROPERTY_PARAM_VDEC_CONTINUE_DATA_TRANSFER         0x1203001
+#define HFI_PROPERTY_PARAM_VDEC_DISPLAY_PICTURE_BUFFER_COUNT   0x1203002
+#define HFI_PROPERTY_PARAM_VDEC_MULTI_VIEW_SELECT              0x1203003
+#define HFI_PROPERTY_PARAM_VDEC_PICTURE_TYPE_DECODE            0x1203004
+#define HFI_PROPERTY_PARAM_VDEC_OUTPUT_ORDER                   0x1203005
+#define HFI_PROPERTY_PARAM_VDEC_MB_QUANTIZATION                        
0x1203006
+#define HFI_PROPERTY_PARAM_VDEC_NUM_CONCEALED_MB               0x1203007
+#define HFI_PROPERTY_PARAM_VDEC_H264_ENTROPY_SWITCHING         0x1203008
+#define HFI_PROPERTY_PARAM_VDEC_OUTPUT2_KEEP_ASPECT_RATIO      0x1203009
+#define HFI_PROPERTY_PARAM_VDEC_FRAME_RATE_EXTRADATA           0x120300a
+#define HFI_PROPERTY_PARAM_VDEC_PANSCAN_WNDW_EXTRADATA         0x120300b
+#define HFI_PROPERTY_PARAM_VDEC_RECOVERY_POINT_SEI_EXTRADATA   0x120300c
+#define HFI_PROPERTY_PARAM_VDEC_THUMBNAIL_MODE                 0x120300d
+#define HFI_PROPERTY_PARAM_VDEC_FRAME_ASSEMBLY                 0x120300e
+#define HFI_PROPERTY_PARAM_VDEC_VC1_FRAMEDISP_EXTRADATA                
0x1203011
+#define HFI_PROPERTY_PARAM_VDEC_VC1_SEQDISP_EXTRADATA          0x1203012
+#define HFI_PROPERTY_PARAM_VDEC_TIMESTAMP_EXTRADATA            0x1203013
+#define HFI_PROPERTY_PARAM_VDEC_INTERLACE_VIDEO_EXTRADATA      0x1203014
+#define HFI_PROPERTY_PARAM_VDEC_AVC_SESSION_SELECT             0x1203015
+#define HFI_PROPERTY_PARAM_VDEC_MPEG2_SEQDISP_EXTRADATA                
0x1203016
+#define HFI_PROPERTY_PARAM_VDEC_STREAM_USERDATA_EXTRADATA      0x1203017
+#define HFI_PROPERTY_PARAM_VDEC_FRAME_QP_EXTRADATA             0x1203018
+#define HFI_PROPERTY_PARAM_VDEC_FRAME_BITS_INFO_EXTRADATA      0x1203019
+#define HFI_PROPERTY_PARAM_VDEC_SCS_THRESHOLD                  0x120301a
+
+/*
+ * HFI_PROPERTY_CONFIG_VDEC_OX_START
+ * HFI_DOMAIN_BASE_VDEC + HFI_ARCH_OX_OFFSET + 0x0000
+ */
+#define HFI_PROPERTY_CONFIG_VDEC_POST_LOOP_DEBLOCKER           0x1200001
+#define HFI_PROPERTY_CONFIG_VDEC_MB_ERROR_MAP_REPORTING                
0x1200002
+#define HFI_PROPERTY_CONFIG_VDEC_MB_ERROR_MAP                  0x1200003
+
+#define HFI_PROPERTY_CONFIG_VDEC_ENTROPY                       0x1204004
+
+/*
+ * HFI_PROPERTY_PARAM_VENC_OX_START
+ * HFI_DOMAIN_BASE_VENC + HFI_ARCH_OX_OFFSET + 0x5000
+ */
+#define  HFI_PROPERTY_PARAM_VENC_MULTI_SLICE_INFO              0x2205001
+#define  HFI_PROPERTY_PARAM_VENC_H264_IDR_S3D_FRAME_PACKING_NAL        
0x2205002
+#define  HFI_PROPERTY_PARAM_VENC_LTR_INFO                      0x2205003
+#define  HFI_PROPERTY_PARAM_VENC_MBI_DUMPING                   0x2205005
+
+/*
+ * HFI_PROPERTY_CONFIG_VENC_OX_START
+ * HFI_DOMAIN_BASE_VENC + HFI_ARCH_OX_OFFSET + 0x6000
+ */
+#define HFI_PROPERTY_CONFIG_VENC_FRAME_QP                      0x2206001
+
+/*
+ * HFI_PROPERTY_PARAM_VPE_OX_START
+ * HFI_DOMAIN_BASE_VPE + HFI_ARCH_OX_OFFSET + 0x7000
+ */
+#define HFI_PROPERTY_PARAM_VPE_COLOR_SPACE_CONVERSION          0x3207001
+
+#define HFI_PROPERTY_CONFIG_VPE_OX_START       \
+       (HFI_DOMAIN_BASE_VPE + HFI_ARCH_OX_OFFSET + 0x8000)
+
+#define HFI_CHROMA_SITE_0                      0x1000001
+#define HFI_CHROMA_SITE_1                      0x1000002
+#define HFI_CHROMA_SITE_2                      0x1000003
+#define HFI_CHROMA_SITE_3                      0x1000004
+#define HFI_CHROMA_SITE_4                      0x1000005
+#define HFI_CHROMA_SITE_5                      0x1000006
+
+#define HFI_PRIORITY_LOW                       10
+#define HFI_PRIOIRTY_MEDIUM                    20
+#define HFI_PRIORITY_HIGH                      30
+
+#define HFI_OUTPUT_ORDER_DISPLAY               0x1000001
+#define HFI_OUTPUT_ORDER_DECODE                        0x1000002
+
+#define HFI_RATE_CONTROL_OFF                   0x1000001
+#define HFI_RATE_CONTROL_VBR_VFR               0x1000002
+#define HFI_RATE_CONTROL_VBR_CFR               0x1000003
+#define HFI_RATE_CONTROL_CBR_VFR               0x1000004
+#define HFI_RATE_CONTROL_CBR_CFR               0x1000005
+
+#define HFI_VIDEO_CODEC_H264                   0x00000002
+#define HFI_VIDEO_CODEC_H263                   0x00000004
+#define HFI_VIDEO_CODEC_MPEG1                  0x00000008
+#define HFI_VIDEO_CODEC_MPEG2                  0x00000010
+#define HFI_VIDEO_CODEC_MPEG4                  0x00000020
+#define HFI_VIDEO_CODEC_DIVX_311               0x00000040
+#define HFI_VIDEO_CODEC_DIVX                   0x00000080
+#define HFI_VIDEO_CODEC_VC1                    0x00000100
+#define HFI_VIDEO_CODEC_SPARK                  0x00000200
+#define HFI_VIDEO_CODEC_VP8                    0x00001000
+#define HFI_VIDEO_CODEC_HEVC                   0x00002000
+#define HFI_VIDEO_CODEC_HEVC_HYBRID            0x00004000
+
+#define HFI_H264_PROFILE_BASELINE              0x00000001
+#define HFI_H264_PROFILE_MAIN                  0x00000002
+#define HFI_H264_PROFILE_HIGH                  0x00000004
+#define HFI_H264_PROFILE_STEREO_HIGH           0x00000008
+#define HFI_H264_PROFILE_MULTIVIEW_HIGH                0x00000010
+#define HFI_H264_PROFILE_CONSTRAINED_BASE      0x00000020
+#define HFI_H264_PROFILE_CONSTRAINED_HIGH      0x00000040
+
+#define HFI_H264_LEVEL_1                       0x00000001
+#define HFI_H264_LEVEL_1b                      0x00000002
+#define HFI_H264_LEVEL_11                      0x00000004
+#define HFI_H264_LEVEL_12                      0x00000008
+#define HFI_H264_LEVEL_13                      0x00000010
+#define HFI_H264_LEVEL_2                       0x00000020
+#define HFI_H264_LEVEL_21                      0x00000040
+#define HFI_H264_LEVEL_22                      0x00000080
+#define HFI_H264_LEVEL_3                       0x00000100
+#define HFI_H264_LEVEL_31                      0x00000200
+#define HFI_H264_LEVEL_32                      0x00000400
+#define HFI_H264_LEVEL_4                       0x00000800
+#define HFI_H264_LEVEL_41                      0x00001000
+#define HFI_H264_LEVEL_42                      0x00002000
+#define HFI_H264_LEVEL_5                       0x00004000
+#define HFI_H264_LEVEL_51                      0x00008000
+#define HFI_H264_LEVEL_52                      0x00010000
+
+#define HFI_H263_PROFILE_BASELINE              0x00000001
+
+#define HFI_H263_LEVEL_10                      0x00000001
+#define HFI_H263_LEVEL_20                      0x00000002
+#define HFI_H263_LEVEL_30                      0x00000004
+#define HFI_H263_LEVEL_40                      0x00000008
+#define HFI_H263_LEVEL_45                      0x00000010
+#define HFI_H263_LEVEL_50                      0x00000020
+#define HFI_H263_LEVEL_60                      0x00000040
+#define HFI_H263_LEVEL_70                      0x00000080
+
+#define HFI_MPEG2_PROFILE_SIMPLE               0x00000001
+#define HFI_MPEG2_PROFILE_MAIN                 0x00000002
+#define HFI_MPEG2_PROFILE_422                  0x00000004
+#define HFI_MPEG2_PROFILE_SNR                  0x00000008
+#define HFI_MPEG2_PROFILE_SPATIAL              0x00000010
+#define HFI_MPEG2_PROFILE_HIGH                 0x00000020
+
+#define HFI_MPEG2_LEVEL_LL                     0x00000001
+#define HFI_MPEG2_LEVEL_ML                     0x00000002
+#define HFI_MPEG2_LEVEL_H14                    0x00000004
+#define HFI_MPEG2_LEVEL_HL                     0x00000008
+
+#define HFI_MPEG4_PROFILE_SIMPLE               0x00000001
+#define HFI_MPEG4_PROFILE_ADVANCEDSIMPLE       0x00000002
+
+#define HFI_MPEG4_LEVEL_0                      0x00000001
+#define HFI_MPEG4_LEVEL_0b                     0x00000002
+#define HFI_MPEG4_LEVEL_1                      0x00000004
+#define HFI_MPEG4_LEVEL_2                      0x00000008
+#define HFI_MPEG4_LEVEL_3                      0x00000010
+#define HFI_MPEG4_LEVEL_4                      0x00000020
+#define HFI_MPEG4_LEVEL_4a                     0x00000040
+#define HFI_MPEG4_LEVEL_5                      0x00000080
+#define HFI_MPEG4_LEVEL_6                      0x00000100
+#define HFI_MPEG4_LEVEL_7                      0x00000200
+#define HFI_MPEG4_LEVEL_8                      0x00000400
+#define HFI_MPEG4_LEVEL_9                      0x00000800
+#define HFI_MPEG4_LEVEL_3b                     0x00001000
+
+#define HFI_VC1_PROFILE_SIMPLE                 0x00000001
+#define HFI_VC1_PROFILE_MAIN                   0x00000002
+#define HFI_VC1_PROFILE_ADVANCED               0x00000004
+
+#define HFI_VC1_LEVEL_LOW                      0x00000001
+#define HFI_VC1_LEVEL_MEDIUM                   0x00000002
+#define HFI_VC1_LEVEL_HIGH                     0x00000004
+#define HFI_VC1_LEVEL_0                                0x00000008
+#define HFI_VC1_LEVEL_1                                0x00000010
+#define HFI_VC1_LEVEL_2                                0x00000020
+#define HFI_VC1_LEVEL_3                                0x00000040
+#define HFI_VC1_LEVEL_4                                0x00000080
+
+#define HFI_VPX_PROFILE_SIMPLE                 0x00000001
+#define HFI_VPX_PROFILE_ADVANCED               0x00000002
+#define HFI_VPX_PROFILE_VERSION_0              0x00000004
+#define HFI_VPX_PROFILE_VERSION_1              0x00000008
+#define HFI_VPX_PROFILE_VERSION_2              0x00000010
+#define HFI_VPX_PROFILE_VERSION_3              0x00000020
+
+#define HFI_DIVX_FORMAT_4                      0x1
+#define HFI_DIVX_FORMAT_5                      0x2
+#define HFI_DIVX_FORMAT_6                      0x3
+
+#define HFI_DIVX_PROFILE_QMOBILE               0x00000001
+#define HFI_DIVX_PROFILE_MOBILE                        0x00000002
+#define HFI_DIVX_PROFILE_MT                    0x00000004
+#define HFI_DIVX_PROFILE_HT                    0x00000008
+#define HFI_DIVX_PROFILE_HD                    0x00000010
+
+#define HFI_HEVC_PROFILE_MAIN                  0x00000001
+#define HFI_HEVC_PROFILE_MAIN10                        0x00000002
+#define HFI_HEVC_PROFILE_MAIN_STILL_PIC                0x00000004
+
+#define HFI_HEVC_LEVEL_1                       0x00000001
+#define HFI_HEVC_LEVEL_2                       0x00000002
+#define HFI_HEVC_LEVEL_21                      0x00000004
+#define HFI_HEVC_LEVEL_3                       0x00000008
+#define HFI_HEVC_LEVEL_31                      0x00000010
+#define HFI_HEVC_LEVEL_4                       0x00000020
+#define HFI_HEVC_LEVEL_41                      0x00000040
+#define HFI_HEVC_LEVEL_5                       0x00000080
+#define HFI_HEVC_LEVEL_51                      0x00000100
+#define HFI_HEVC_LEVEL_52                      0x00000200
+#define HFI_HEVC_LEVEL_6                       0x00000400
+#define HFI_HEVC_LEVEL_61                      0x00000800
+#define HFI_HEVC_LEVEL_62                      0x00001000
+
+#define HFI_HEVC_TIER_MAIN                     0x1
+#define HFI_HEVC_TIER_HIGH0                    0x2
+
+#define HFI_BUFFER_INPUT                       0x1
+#define HFI_BUFFER_OUTPUT                      0x2
+#define HFI_BUFFER_OUTPUT2                     0x3
+#define HFI_BUFFER_INTERNAL_PERSIST            0x4
+#define HFI_BUFFER_INTERNAL_PERSIST_1          0x5
+#define HFI_BUFFER_INTERNAL_SCRATCH            0x1000001
+#define HFI_BUFFER_EXTRADATA_INPUT             0x1000002
+#define HFI_BUFFER_EXTRADATA_OUTPUT            0x1000003
+#define HFI_BUFFER_EXTRADATA_OUTPUT2           0x1000004
+#define HFI_BUFFER_INTERNAL_SCRATCH_1          0x1000005
+#define HFI_BUFFER_INTERNAL_SCRATCH_2          0x1000006
+
+#define HFI_BUFFER_TYPE_MAX                    11
+
+#define HFI_BUFFER_MODE_STATIC                 0x1000001
+#define HFI_BUFFER_MODE_RING                   0x1000002
+#define HFI_BUFFER_MODE_DYNAMIC                        0x1000003
+
+#define HFI_VENC_PERFMODE_MAX_QUALITY          0x1
+#define HFI_VENC_PERFMODE_POWER_SAVE           0x2
+
+/*
+ * HFI_PROPERTY_SYS_COMMON_START
+ * HFI_DOMAIN_BASE_COMMON + HFI_ARCH_COMMON_OFFSET + 0x0000
+ */
+#define HFI_PROPERTY_SYS_DEBUG_CONFIG                          0x1
+#define HFI_PROPERTY_SYS_RESOURCE_OCMEM_REQUIREMENT_INFO       0x2
+#define HFI_PROPERTY_SYS_CONFIG_VCODEC_CLKFREQ                 0x3
+#define HFI_PROPERTY_SYS_IDLE_INDICATOR                                0x4
+#define HFI_PROPERTY_SYS_CODEC_POWER_PLANE_CTRL                        0x5
+#define HFI_PROPERTY_SYS_IMAGE_VERSION                         0x6
+#define HFI_PROPERTY_SYS_CONFIG_COVERAGE                       0x7
+
+/*
+ * HFI_PROPERTY_PARAM_COMMON_START
+ * HFI_DOMAIN_BASE_COMMON + HFI_ARCH_COMMON_OFFSET + 0x1000
+ */
+#define HFI_PROPERTY_PARAM_FRAME_SIZE                          0x1001
+#define HFI_PROPERTY_PARAM_UNCOMPRESSED_PLANE_ACTUAL_INFO      0x1002
+#define HFI_PROPERTY_PARAM_UNCOMPRESSED_FORMAT_SELECT          0x1003
+#define HFI_PROPERTY_PARAM_UNCOMPRESSED_FORMAT_SUPPORTED       0x1004
+#define HFI_PROPERTY_PARAM_PROFILE_LEVEL_CURRENT               0x1005
+#define HFI_PROPERTY_PARAM_PROFILE_LEVEL_SUPPORTED             0x1006
+#define HFI_PROPERTY_PARAM_CAPABILITY_SUPPORTED                        0x1007
+#define HFI_PROPERTY_PARAM_PROPERTIES_SUPPORTED                        0x1008
+#define HFI_PROPERTY_PARAM_CODEC_SUPPORTED                     0x1009
+#define HFI_PROPERTY_PARAM_NAL_STREAM_FORMAT_SUPPORTED         0x100a
+#define HFI_PROPERTY_PARAM_NAL_STREAM_FORMAT_SELECT            0x100b
+#define HFI_PROPERTY_PARAM_MULTI_VIEW_FORMAT                   0x100c
+#define HFI_PROPERTY_PARAM_MAX_SEQUENCE_HEADER_SIZE            0x100d
+#define HFI_PROPERTY_PARAM_CODEC_MASK_SUPPORTED                        0x100e
+#define HFI_PROPERTY_PARAM_MVC_BUFFER_LAYOUT                   0x100f
+#define HFI_PROPERTY_PARAM_MAX_SESSIONS_SUPPORTED              0x1010
+
+/*
+ * HFI_PROPERTY_CONFIG_COMMON_START
+ * HFI_DOMAIN_BASE_COMMON + HFI_ARCH_COMMON_OFFSET + 0x2000
+ */
+#define HFI_PROPERTY_CONFIG_FRAME_RATE                         0x2001
+
+/*
+ * HFI_PROPERTY_PARAM_VDEC_COMMON_START
+ * HFI_DOMAIN_BASE_VDEC + HFI_ARCH_COMMON_OFFSET + 0x3000
+ */
+#define HFI_PROPERTY_PARAM_VDEC_MULTI_STREAM                   0x1003001
+#define HFI_PROPERTY_PARAM_VDEC_CONCEAL_COLOR                  0x1003002
+#define HFI_PROPERTY_PARAM_VDEC_NONCP_OUTPUT2                  0x1003003
+
+/*
+ * HFI_PROPERTY_CONFIG_VDEC_COMMON_START
+ * HFI_DOMAIN_BASE_VDEC + HFI_ARCH_COMMON_OFFSET + 0x4000
+ */
+
+/*
+ * HFI_PROPERTY_PARAM_VENC_COMMON_START
+ * HFI_DOMAIN_BASE_VENC + HFI_ARCH_COMMON_OFFSET + 0x5000
+ */
+#define HFI_PROPERTY_PARAM_VENC_SLICE_DELIVERY_MODE            0x2005001
+#define HFI_PROPERTY_PARAM_VENC_H264_ENTROPY_CONTROL           0x2005002
+#define HFI_PROPERTY_PARAM_VENC_H264_DEBLOCK_CONTROL           0x2005003
+#define HFI_PROPERTY_PARAM_VENC_RATE_CONTROL                   0x2005004
+#define HFI_PROPERTY_PARAM_VENC_H264_PICORDER_CNT_TYPE         0x2005005
+#define HFI_PROPERTY_PARAM_VENC_SESSION_QP                     0x2005006
+#define HFI_PROPERTY_PARAM_VENC_MPEG4_AC_PREDICTION            0x2005007
+#define HFI_PROPERTY_PARAM_VENC_SESSION_QP_RANGE               0x2005008
+#define HFI_PROPERTY_PARAM_VENC_MPEG4_TIME_RESOLUTION          0x2005009
+#define HFI_PROPERTY_PARAM_VENC_MPEG4_SHORT_HEADER             0x200500a
+#define HFI_PROPERTY_PARAM_VENC_MPEG4_HEADER_EXTENSION         0x200500b
+#define HFI_PROPERTY_PARAM_VENC_OPEN_GOP                       0x200500c
+#define HFI_PROPERTY_PARAM_VENC_INTRA_REFRESH                  0x200500d
+#define HFI_PROPERTY_PARAM_VENC_MULTI_SLICE_CONTROL            0x200500e
+#define HFI_PROPERTY_PARAM_VENC_VBV_HRD_BUF_SIZE               0x200500f
+#define HFI_PROPERTY_PARAM_VENC_QUALITY_VS_SPEED               0x2005010
+#define HFI_PROPERTY_PARAM_VENC_ADVANCED                       0x2005012
+#define HFI_PROPERTY_PARAM_VENC_H264_SPS_ID                    0x2005014
+#define HFI_PROPERTY_PARAM_VENC_H264_PPS_ID                    0x2005015
+#define HFI_PROPERTY_PARAM_VENC_H264_GENERATE_AUDNAL           0x2005016
+#define HFI_PROPERTY_PARAM_VENC_ASPECT_RATIO                   0x2005017
+#define HFI_PROPERTY_PARAM_VENC_NUMREF                         0x2005018
+#define HFI_PROPERTY_PARAM_VENC_MULTIREF_P                     0x2005019
+#define HFI_PROPERTY_PARAM_VENC_H264_NAL_SVC_EXT               0x200501b
+#define HFI_PROPERTY_PARAM_VENC_LTRMODE                                
0x200501c
+#define HFI_PROPERTY_PARAM_VENC_VIDEO_FULL_RANGE               0x200501d
+#define HFI_PROPERTY_PARAM_VENC_H264_VUI_TIMING_INFO           0x200501e
+#define HFI_PROPERTY_PARAM_VENC_VC1_PERF_CFG                   0x200501f
+#define HFI_PROPERTY_PARAM_VENC_MAX_NUM_B_FRAMES               0x2005020
+#define HFI_PROPERTY_PARAM_VENC_H264_VUI_BITSTREAM_RESTRC      0x2005021
+#define HFI_PROPERTY_PARAM_VENC_PRESERVE_TEXT_QUALITY          0x2005023
+#define HFI_PROPERTY_PARAM_VENC_HIER_P_MAX_NUM_ENH_LAYER       0x2005026
+#define HFI_PROPERTY_PARAM_VENC_DISABLE_RC_TIMESTAMP           0x2005027
+#define HFI_PROPERTY_PARAM_VENC_INITIAL_QP                     0x2005028
+#define HFI_PROPERTY_PARAM_VENC_VPX_ERROR_RESILIENCE_MODE      0x2005029
+#define HFI_PROPERTY_PARAM_VENC_HIER_B_MAX_NUM_ENH_LAYER       0x200502c
+#define HFI_PROPERTY_PARAM_VENC_HIER_P_HYBRID_MODE             0x200502f
+
+/*
+ * HFI_PROPERTY_CONFIG_VENC_COMMON_START
+ * HFI_DOMAIN_BASE_VENC + HFI_ARCH_COMMON_OFFSET + 0x6000
+ */
+#define HFI_PROPERTY_CONFIG_VENC_TARGET_BITRATE                        
0x2006001
+#define HFI_PROPERTY_CONFIG_VENC_IDR_PERIOD                    0x2006002
+#define HFI_PROPERTY_CONFIG_VENC_INTRA_PERIOD                  0x2006003
+#define HFI_PROPERTY_CONFIG_VENC_REQUEST_SYNC_FRAME            0x2006004
+#define HFI_PROPERTY_CONFIG_VENC_SLICE_SIZE                    0x2006005
+#define HFI_PROPERTY_CONFIG_VENC_MAX_BITRATE                   0x2006007
+#define HFI_PROPERTY_CONFIG_VENC_SYNC_FRAME_SEQUENCE_HEADER    0x2006008
+#define HFI_PROPERTY_CONFIG_VENC_MARKLTRFRAME                  0x2006009
+#define HFI_PROPERTY_CONFIG_VENC_USELTRFRAME                   0x200600a
+#define HFI_PROPERTY_CONFIG_VENC_HIER_P_ENH_LAYER              0x200600b
+#define HFI_PROPERTY_CONFIG_VENC_LTRPERIOD                     0x200600c
+#define HFI_PROPERTY_CONFIG_VENC_PERF_MODE                     0x200600e
+
+/*
+ * HFI_PROPERTY_PARAM_VPE_COMMON_START
+ * HFI_DOMAIN_BASE_VPE + HFI_ARCH_COMMON_OFFSET + 0x7000
+ */
+
+/*
+ * HFI_PROPERTY_CONFIG_VPE_COMMON_START
+ * HFI_DOMAIN_BASE_VPE + HFI_ARCH_COMMON_OFFSET + 0x8000
+ */
+#define HFI_PROPERTY_CONFIG_VPE_DEINTERLACE                    0x3008001
+#define HFI_PROPERTY_CONFIG_VPE_OPERATIONS                     0x3008002
+
+enum hfi_version {
+       HFI_VERSION_LEGACY,
+       HFI_VERSION_3XX,
+};
+
+struct hfi_buffer_info {
+       u32 buffer_addr;
+       u32 extradata_addr;
+};
+
+struct hfi_bitrate {
+       u32 bitrate;
+       u32 layer_id;
+};
+
+#define HFI_CAPABILITY_FRAME_WIDTH                     0x01
+#define HFI_CAPABILITY_FRAME_HEIGHT                    0x02
+#define HFI_CAPABILITY_MBS_PER_FRAME                   0x03
+#define HFI_CAPABILITY_MBS_PER_SECOND                  0x04
+#define HFI_CAPABILITY_FRAMERATE                       0x05
+#define HFI_CAPABILITY_SCALE_X                         0x06
+#define HFI_CAPABILITY_SCALE_Y                         0x07
+#define HFI_CAPABILITY_BITRATE                         0x08
+#define HFI_CAPABILITY_BFRAME                          0x09
+#define HFI_CAPABILITY_HIER_P_NUM_ENH_LAYERS           0x10
+#define HFI_CAPABILITY_ENC_LTR_COUNT                   0x11
+#define HFI_CAPABILITY_CP_OUTPUT2_THRESH               0x12
+#define HFI_CAPABILITY_HIER_P_HYBRID_NUM_ENH_LAYERS    0x15
+
+struct hfi_capability {
+       u32 capability_type;
+       u32 min;
+       u32 max;
+       u32 step_size;
+};
+
+struct hfi_capabilities {
+       u32 num_capabilities;
+       struct hfi_capability data[1];
+};
+
+#define HFI_DEBUG_MSG_LOW      0x01
+#define HFI_DEBUG_MSG_MEDIUM   0x02
+#define HFI_DEBUG_MSG_HIGH     0x04
+#define HFI_DEBUG_MSG_ERROR    0x08
+#define HFI_DEBUG_MSG_FATAL    0x10
+#define HFI_DEBUG_MSG_PERF     0x20
+
+#define HFI_DEBUG_MODE_QUEUE   0x01
+#define HFI_DEBUG_MODE_QDSS    0x02
+
+struct hfi_debug_config {
+       u32 config;
+       u32 mode;
+};
+
+struct hfi_enable {
+       u32 enable;
+};
+
+#define HFI_H264_DB_MODE_DISABLE               0x1
+#define HFI_H264_DB_MODE_SKIP_SLICE_BOUNDARY   0x2
+#define HFI_H264_DB_MODE_ALL_BOUNDARY          0x3
+
+struct hfi_h264_db_control {
+       u32 mode;
+       u32 slice_alpha_offset;
+       u32 slice_beta_offset;
+};
+
+#define HFI_H264_ENTROPY_CAVLC                 0x1
+#define HFI_H264_ENTROPY_CABAC                 0x2
+
+#define HFI_H264_CABAC_MODEL_0                 0x1
+#define HFI_H264_CABAC_MODEL_1                 0x2
+#define HFI_H264_CABAC_MODEL_2                 0x3
+
+struct hfi_h264_entropy_control {
+       u32 entropy_mode;
+       u32 cabac_model;
+};
+
+struct hfi_framerate {
+       u32 buffer_type;
+       u32 framerate;
+};
+
+#define HFI_INTRA_REFRESH_NONE                 0x1
+#define HFI_INTRA_REFRESH_CYCLIC               0x2
+#define HFI_INTRA_REFRESH_ADAPTIVE             0x3
+#define HFI_INTRA_REFRESH_CYCLIC_ADAPTIVE      0x4
+#define HFI_INTRA_REFRESH_RANDOM               0x5
+
+struct hfi_intra_refresh {
+       u32 mode;
+       u32 air_mbs;
+       u32 air_ref;
+       u32 cir_mbs;
+};
+
+struct hfi_intra_refresh_3x {
+       u32 mode;
+       u32 mbs;
+};
+
+struct hfi_idr_period {
+       u32 idr_period;
+};
+
+struct hfi_operations_type {
+       u32 rotation;
+       u32 flip;
+};
+
+struct hfi_max_num_b_frames {
+       u32 max_num_b_frames;
+};
+
+struct hfi_vc1e_perf_cfg_type {
+       u32 search_range_x_subsampled[3];
+       u32 search_range_y_subsampled[3];
+};
+
+struct hfi_conceal_color {
+       u32 conceal_color;
+};
+
+struct hfi_intra_period {
+       u32 pframes;
+       u32 bframes;
+};
+
+struct hfi_mpeg4_header_extension {
+       u32 header_extension;
+};
+
+struct hfi_mpeg4_time_resolution {
+       u32 time_increment_resolution;
+};
+
+struct hfi_multi_stream {
+       u32 buffer_type;
+       u32 enable;
+       u32 width;
+       u32 height;
+};
+
+struct hfi_multi_stream_3x {
+       u32 buffer_type;
+       u32 enable;
+};
+
+struct hfi_multi_view_format {
+       u32 views;
+       u32 view_order[1];
+};
+
+#define HFI_MULTI_SLICE_OFF                    0x1
+#define HFI_MULTI_SLICE_BY_MB_COUNT            0x2
+#define HFI_MULTI_SLICE_BY_BYTE_COUNT          0x3
+#define HFI_MULTI_SLICE_GOB                    0x4
+
+struct hfi_multi_slice_control {
+       u32 multi_slice;
+       u32 slice_size;
+};
+
+#define HFI_NAL_FORMAT_STARTCODES              0x01
+#define HFI_NAL_FORMAT_ONE_NAL_PER_BUFFER      0x02
+#define HFI_NAL_FORMAT_ONE_BYTE_LENGTH         0x04
+#define HFI_NAL_FORMAT_TWO_BYTE_LENGTH         0x08
+#define HFI_NAL_FORMAT_FOUR_BYTE_LENGTH                0x10
+
+struct hfi_nal_stream_format {
+       u32 format;
+};
+
+struct hfi_nal_stream_format_select {
+       u32 format;
+};
+
+#define HFI_PICTURE_TYPE_I                     0x01
+#define HFI_PICTURE_TYPE_P                     0x02
+#define HFI_PICTURE_TYPE_B                     0x04
+#define HFI_PICTURE_TYPE_IDR                   0x08
+
+struct hfi_profile_level {
+       u32 profile;
+       u32 level;
+};
+
+#define HFI_MAX_PROFILE_COUNT                  16
+
+struct hfi_profile_level_supported {
+       u32 profile_count;
+       struct hfi_profile_level profile_level[1];
+};
+
+struct hfi_quality_vs_speed {
+       u32 quality_vs_speed;
+};
+
+struct hfi_quantization {
+       u32 qp_i;
+       u32 qp_p;
+       u32 qp_b;
+       u32 layer_id;
+};
+
+struct hfi_initial_quantization {
+       u32 qp_i;
+       u32 qp_p;
+       u32 qp_b;
+       u32 init_qp_enable;
+};
+
+struct hfi_quantization_range {
+       u32 min_qp;
+       u32 max_qp;
+       u32 layer_id;
+};
+
+#define HFI_LTR_MODE_DISABLE   0x0
+#define HFI_LTR_MODE_MANUAL    0x1
+#define HFI_LTR_MODE_PERIODIC  0x2
+
+struct hfi_ltr_mode {
+       u32 ltr_mode;
+       u32 ltr_count;
+       u32 trust_mode;
+};
+
+struct hfi_ltr_use {
+       u32 ref_ltr;
+       u32 use_constrnt;
+       u32 frames;
+};
+
+struct hfi_ltr_mark {
+       u32 mark_frame;
+};
+
+struct hfi_framesize {
+       u32 buffer_type;
+       u32 width;
+       u32 height;
+};
+
+struct hfi_h264_vui_timing_info {
+       u32 enable;
+       u32 fixed_framerate;
+       u32 time_scale;
+};
+
+#define HFI_COLOR_FORMAT_MONOCHROME            0x01
+#define HFI_COLOR_FORMAT_NV12                  0x02
+#define HFI_COLOR_FORMAT_NV21                  0x03
+#define HFI_COLOR_FORMAT_NV12_4x4TILE          0x04
+#define HFI_COLOR_FORMAT_NV21_4x4TILE          0x05
+#define HFI_COLOR_FORMAT_YUYV                  0x06
+#define HFI_COLOR_FORMAT_YVYU                  0x07
+#define HFI_COLOR_FORMAT_UYVY                  0x08
+#define HFI_COLOR_FORMAT_VYUY                  0x09
+#define HFI_COLOR_FORMAT_RGB565                        0x0a
+#define HFI_COLOR_FORMAT_BGR565                        0x0b
+#define HFI_COLOR_FORMAT_RGB888                        0x0c
+#define HFI_COLOR_FORMAT_BGR888                        0x0d
+#define HFI_COLOR_FORMAT_YUV444                        0x0e
+#define HFI_COLOR_FORMAT_RGBA8888              0x10
+
+#define HFI_COLOR_FORMAT_UBWC_BASE             0x8000
+#define HFI_COLOR_FORMAT_10_BIT_BASE           0x4000
+
+#define HFI_COLOR_FORMAT_YUV420_TP10           0x4002
+#define HFI_COLOR_FORMAT_NV12_UBWC             0x8002
+#define HFI_COLOR_FORMAT_YUV420_TP10_UBWC      0xc002
+#define HFI_COLOR_FORMAT_RGBA8888_UBWC         0x8010
+
+struct hfi_uncompressed_format_select {
+       u32 buffer_type;
+       u32 format;
+};
+
+struct hfi_uncompressed_format_supported {
+       u32 buffer_type;
+       u32 format_entries;
+       u32 format_info[1];
+};
+
+struct hfi_uncompressed_plane_actual {
+       int actual_stride;
+       u32 actual_plane_buffer_height;
+};
+
+struct hfi_uncompressed_plane_actual_info {
+       u32 buffer_type;
+       u32 num_planes;
+       struct hfi_uncompressed_plane_actual plane_format[1];
+};
+
+struct hfi_uncompressed_plane_constraints {
+       u32 stride_multiples;
+       u32 max_stride;
+       u32 min_plane_buffer_height_multiple;
+       u32 buffer_alignment;
+};
+
+struct hfi_uncompressed_plane_info {
+       u32 format;
+       u32 num_planes;
+       struct hfi_uncompressed_plane_constraints plane_format[1];
+};
+
+struct hfi_uncompressed_plane_actual_constraints_info {
+       u32 buffer_type;
+       u32 num_planes;
+       struct hfi_uncompressed_plane_constraints plane_format[1];
+};
+
+struct hfi_codec_supported {
+       u32 dec_codecs;
+       u32 enc_codecs;
+};
+
+struct hfi_properties_supported {
+       u32 num_properties;
+       u32 properties[1];
+};
+
+struct hfi_max_sessions_supported {
+       u32 max_sessions;
+};
+
+#define HFI_MAX_MATRIX_COEFFS  9
+#define HFI_MAX_BIAS_COEFFS    3
+#define HFI_MAX_LIMIT_COEFFS   6
+
+struct hfi_vpe_color_space_conversion {
+       u32 csc_matrix[HFI_MAX_MATRIX_COEFFS];
+       u32 csc_bias[HFI_MAX_BIAS_COEFFS];
+       u32 csc_limit[HFI_MAX_LIMIT_COEFFS];
+};
+
+#define HFI_ROTATE_NONE                0x1
+#define HFI_ROTATE_90          0x2
+#define HFI_ROTATE_180         0x3
+#define HFI_ROTATE_270         0x4
+
+#define HFI_FLIP_NONE          0x1
+#define HFI_FLIP_HORIZONTAL    0x2
+#define HFI_FLIP_VERTICAL      0x3
+
+struct hfi_operations {
+       u32 rotate;
+       u32 flip;
+};
+
+#define HFI_RESOURCE_OCMEM     0x1
+
+struct hfi_resource_ocmem {
+       u32 size;
+       u32 mem;
+};
+
+struct hfi_resource_ocmem_requirement {
+       u32 session_domain;
+       u32 width;
+       u32 height;
+       u32 size;
+};
+
+struct hfi_resource_ocmem_requirement_info {
+       u32 num_entries;
+       struct hfi_resource_ocmem_requirement requirements[1];
+};
+
+struct hfi_property_sys_image_version_info_type {
+       u32 string_size;
+       u8  str_image_version[1];
+};
+
+struct hfi_codec_mask_supported {
+       u32 codecs;
+       u32 video_domains;
+};
+
+struct hfi_seq_header_info {
+       u32 max_hader_len;
+};
+
+struct hfi_aspect_ratio {
+       u32 aspect_width;
+       u32 aspect_height;
+};
+
+#define HFI_MVC_BUFFER_LAYOUT_TOP_BOTTOM       0
+#define HFI_MVC_BUFFER_LAYOUT_SIDEBYSIDE       1
+#define HFI_MVC_BUFFER_LAYOUT_SEQ              2
+
+struct hfi_mvc_buffer_layout_descp_type {
+       u32 layout_type;
+       u32 bright_view_first;
+       u32 ngap;
+};
+
+struct hfi_scs_threshold {
+       u32 threshold_value;
+};
+
+#define HFI_TEST_SSR_SW_ERR_FATAL      0x1
+#define HFI_TEST_SSR_SW_DIV_BY_ZERO    0x2
+#define HFI_TEST_SSR_HW_WDOG_IRQ       0x3
+
+struct hfi_buffer_alloc_mode {
+       u32 type;
+       u32 mode;
+};
+
+struct hfi_index_extradata_config {
+       u32 enable;
+       u32 index_extra_data_id;
+};
+
+struct hfi_extradata_header {
+       u32 size;
+       u32 version;
+       u32 port_index;
+       u32 type;
+       u32 data_size;
+       u8 data[1];
+};
+
+struct hfi_batch_info {
+       u32 input_batch_count;
+       u32 output_batch_count;
+};
+
+struct hfi_buffer_count_actual {
+       u32 type;
+       u32 count_actual;
+};
+
+struct hfi_buffer_size_actual {
+       u32 type;
+       u32 size;
+};
+
+struct hfi_buffer_display_hold_count_actual {
+       u32 type;
+       u32 hold_count;
+};
+
+struct hfi_buffer_requirements {
+       u32 type;
+       u32 size;
+       u32 region_size;
+       u32 hold_count;
+       u32 count_min;
+       u32 count_actual;
+       u32 contiguous;
+       u32 alignment;
+};
+
+struct hfi_data_payload {
+       u32 size;
+       u8 data[1];
+};
+
+struct hfi_enable_picture {
+       u32 picture_type;
+};
+
+struct hfi_display_picture_buffer_count {
+       int enable;
+       u32 count;
+};
+
+struct hfi_extra_data_header_config {
+       u32 type;
+       u32 buffer_type;
+       u32 version;
+       u32 port_index;
+       u32 client_extra_data_id;
+};
+
+struct hfi_interlace_format_supported {
+       u32 buffer_type;
+       u32 format;
+};
+
+struct hfi_buffer_alloc_mode_supported {
+       u32 buffer_type;
+       u32 num_entries;
+       u32 data[1];
+};
+
+struct hfi_mb_error_map {
+       u32 error_map_size;
+       u8 error_map[1];
+};
+
+struct hfi_metadata_pass_through {
+       int enable;
+       u32 size;
+};
+
+struct hfi_multi_view_select {
+       u32 view_index;
+};
+
+struct hfi_hybrid_hierp {
+       u32 layers;
+};
+
+struct hfi_pkt_hdr {
+       u32 size;
+       u32 pkt_type;
+};
+
+struct hfi_session_hdr_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 session_id;
+};
+
+struct hfi_session_pkt {
+       struct hfi_session_hdr_pkt shdr;
+};
+
+#endif
diff --git a/drivers/media/platform/qcom/venus/hfi_msgs.c 
b/drivers/media/platform/qcom/venus/hfi_msgs.c
new file mode 100644
index 000000000000..2fdb6a15438c
--- /dev/null
+++ b/drivers/media/platform/qcom/venus/hfi_msgs.c
@@ -0,0 +1,1057 @@
+/*
+ * Copyright (c) 2012-2016, The Linux Foundation. All rights reserved.
+ * Copyright (C) 2017 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+#include <linux/hash.h>
+#include <linux/list.h>
+#include <linux/slab.h>
+#include <media/videobuf2-v4l2.h>
+
+#include "core.h"
+#include "hfi.h"
+#include "hfi_helper.h"
+#include "hfi_msgs.h"
+
+static void event_seq_changed(struct venus_core *core, struct venus_inst *inst,
+                             struct hfi_msg_event_notify_pkt *pkt)
+{
+       struct device *dev = core->dev;
+       struct hfi_event_data event = {0};
+       int num_properties_changed;
+       struct hfi_framesize *frame_sz;
+       struct hfi_profile_level *profile_level;
+       u8 *data_ptr;
+       u32 ptype;
+
+       inst->error = HFI_ERR_NONE;
+
+       switch (pkt->event_data1) {
+       case HFI_EVENT_DATA_SEQUENCE_CHANGED_SUFFICIENT_BUF_RESOURCES:
+       case HFI_EVENT_DATA_SEQUENCE_CHANGED_INSUFFICIENT_BUF_RESOURCES:
+               break;
+       default:
+               inst->error = HFI_ERR_SESSION_INVALID_PARAMETER;
+               goto done;
+       }
+
+       event.event_type = pkt->event_data1;
+
+       num_properties_changed = pkt->event_data2;
+       if (!num_properties_changed) {
+               inst->error = HFI_ERR_SESSION_INSUFFICIENT_RESOURCES;
+               goto done;
+       }
+
+       data_ptr = (u8 *) &pkt->ext_event_data[0];
+       do {
+               ptype = *((u32 *)data_ptr);
+               switch (ptype) {
+               case HFI_PROPERTY_PARAM_FRAME_SIZE:
+                       data_ptr += sizeof(u32);
+                       frame_sz = (struct hfi_framesize *) data_ptr;
+                       event.width = frame_sz->width;
+                       event.height = frame_sz->height;
+                       data_ptr += sizeof(frame_sz);
+                       dev_dbg(dev, "%s cmd: frame size: %ux%u\n",
+                               __func__, event.width, event.height);
+                       break;
+               case HFI_PROPERTY_PARAM_PROFILE_LEVEL_CURRENT:
+                       data_ptr += sizeof(u32);
+                       profile_level = (struct hfi_profile_level *) data_ptr;
+                       event.profile = profile_level->profile;
+                       event.level = profile_level->level;
+                       data_ptr += sizeof(profile_level);
+                       dev_dbg(dev, "%s cmd: profile-level: %u - %u\n",
+                               __func__, event.profile, event.level);
+                       break;
+               default:
+                       dev_dbg(dev, "%s cmd: %#x not supported\n",
+                               __func__, ptype);
+                       break;
+               }
+               num_properties_changed--;
+       } while (num_properties_changed > 0);
+
+done:
+       inst->ops->event_notify(inst, EVT_SYS_EVENT_CHANGE, &event);
+}
+
+static void event_release_buffer_ref(struct venus_core *core,
+                                    struct venus_inst *inst,
+                                    struct hfi_msg_event_notify_pkt *pkt)
+{
+       struct hfi_event_data event = {0};
+       struct hfi_msg_event_release_buffer_ref_pkt *data;
+
+       data = (struct hfi_msg_event_release_buffer_ref_pkt *)
+               pkt->ext_event_data;
+
+       event.event_type = HFI_EVENT_RELEASE_BUFFER_REFERENCE;
+       event.packet_buffer = data->packet_buffer;
+       event.extradata_buffer = data->extradata_buffer;
+
+       inst->error = HFI_ERR_NONE;
+       inst->ops->event_notify(inst, EVT_SYS_EVENT_CHANGE, &event);
+}
+
+static void event_sys_error(struct venus_core *core, u32 event)
+{
+       core->core_ops->event_notify(core, event);
+}
+
+static void
+event_session_error(struct venus_core *core, struct venus_inst *inst,
+                   struct hfi_msg_event_notify_pkt *pkt)
+{
+       struct device *dev = core->dev;
+
+       dev_dbg(dev, "session error: event id:%x, session id:%x\n",
+               pkt->event_data1, pkt->shdr.session_id);
+
+       if (!inst)
+               return;
+
+       switch (pkt->event_data1) {
+       /* non fatal session errors */
+       case HFI_ERR_SESSION_INVALID_SCALE_FACTOR:
+       case HFI_ERR_SESSION_UNSUPPORT_BUFFERTYPE:
+       case HFI_ERR_SESSION_UNSUPPORTED_SETTING:
+       case HFI_ERR_SESSION_UPSCALE_NOT_SUPPORTED:
+               inst->error = HFI_ERR_NONE;
+               break;
+       default:
+               dev_err(dev, "session error: event id:%x, session id:%x\n",
+                       pkt->event_data1, pkt->shdr.session_id);
+
+               inst->error = pkt->event_data1;
+               inst->ops->event_notify(inst, EVT_SESSION_ERROR, NULL);
+               break;
+       }
+}
+
+static void hfi_event_notify(struct venus_core *core, struct venus_inst *inst,
+                            void *packet)
+{
+       struct hfi_msg_event_notify_pkt *pkt = packet;
+
+       if (!packet) {
+               dev_err(core->dev, "invalid packet\n");
+               return;
+       }
+
+       switch (pkt->event_id) {
+       case HFI_EVENT_SYS_ERROR:
+               event_sys_error(core, EVT_SYS_ERROR);
+               break;
+       case HFI_EVENT_SESSION_ERROR:
+               event_session_error(core, inst, pkt);
+               break;
+       case HFI_EVENT_SESSION_SEQUENCE_CHANGED:
+               event_seq_changed(core, inst, pkt);
+               break;
+       case HFI_EVENT_RELEASE_BUFFER_REFERENCE:
+               event_release_buffer_ref(core, inst, pkt);
+               break;
+       case HFI_EVENT_SESSION_PROPERTY_CHANGED:
+               break;
+       default:
+               break;
+       }
+}
+
+static void hfi_sys_init_done(struct venus_core *core, struct venus_inst *inst,
+                             void *packet)
+{
+       struct hfi_msg_sys_init_done_pkt *pkt = packet;
+       u32 rem_bytes, read_bytes = 0, num_properties;
+       u32 error, ptype;
+       u8 *data;
+
+       error = pkt->error_type;
+       if (error != HFI_ERR_NONE)
+               goto err_no_prop;
+
+       num_properties = pkt->num_properties;
+
+       if (!num_properties) {
+               error = HFI_ERR_SYS_INVALID_PARAMETER;
+               goto err_no_prop;
+       }
+
+       rem_bytes = pkt->hdr.size - sizeof(*pkt) + sizeof(u32);
+
+       if (!rem_bytes) {
+               /* missing property data */
+               error = HFI_ERR_SYS_INSUFFICIENT_RESOURCES;
+               goto err_no_prop;
+       }
+
+       data = (u8 *)&pkt->data[0];
+
+       if (core->res->hfi_version == HFI_VERSION_3XX)
+               goto err_no_prop;
+
+       while (num_properties && rem_bytes >= sizeof(u32)) {
+               ptype = *((u32 *)data);
+               data += sizeof(u32);
+
+               switch (ptype) {
+               case HFI_PROPERTY_PARAM_CODEC_SUPPORTED: {
+                       struct hfi_codec_supported *prop;
+
+                       prop = (struct hfi_codec_supported *)data;
+
+                       if (rem_bytes < sizeof(*prop)) {
+                               error = HFI_ERR_SYS_INSUFFICIENT_RESOURCES;
+                               break;
+                       }
+
+                       read_bytes += sizeof(*prop) + sizeof(u32);
+                       core->dec_codecs = prop->dec_codecs;
+                       core->enc_codecs = prop->enc_codecs;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_MAX_SESSIONS_SUPPORTED: {
+                       struct hfi_max_sessions_supported *prop;
+
+                       if (rem_bytes < sizeof(*prop)) {
+                               error = HFI_ERR_SYS_INSUFFICIENT_RESOURCES;
+                               break;
+                       }
+
+                       prop = (struct hfi_max_sessions_supported *)data;
+                       read_bytes += sizeof(*prop) + sizeof(u32);
+                       core->max_sessions_supported = prop->max_sessions;
+                       break;
+               }
+               default:
+                       error = HFI_ERR_SYS_INVALID_PARAMETER;
+                       break;
+               }
+
+               if (!error) {
+                       rem_bytes -= read_bytes;
+                       data += read_bytes;
+                       num_properties--;
+               }
+       }
+
+err_no_prop:
+       core->error = error;
+       complete(&core->done);
+}
+
+static void
+sys_get_prop_image_version(struct device *dev,
+                          struct hfi_msg_sys_property_info_pkt *pkt)
+{
+       int req_bytes;
+
+       req_bytes = pkt->hdr.size - sizeof(*pkt);
+
+       if (req_bytes < 128 || !pkt->data[1] || pkt->num_properties > 1)
+               /* bad packet */
+               return;
+
+       dev_dbg(dev, "F/W version: %s\n", (u8 *)&pkt->data[1]);
+}
+
+static void hfi_sys_property_info(struct venus_core *core,
+                                 struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_sys_property_info_pkt *pkt = packet;
+       struct device *dev = core->dev;
+
+       if (!pkt->num_properties) {
+               dev_dbg(dev, "%s: no properties\n", __func__);
+               return;
+       }
+
+       switch (pkt->data[0]) {
+       case HFI_PROPERTY_SYS_IMAGE_VERSION:
+               sys_get_prop_image_version(dev, pkt);
+               break;
+       default:
+               dev_dbg(dev, "%s: unknown property data\n", __func__);
+               break;
+       }
+}
+
+static void hfi_sys_rel_resource_done(struct venus_core *core,
+                                     struct venus_inst *inst,
+                                     void *packet)
+{
+       struct hfi_msg_sys_release_resource_done_pkt *pkt = packet;
+
+       core->error = pkt->error_type;
+       complete(&core->done);
+}
+
+static void hfi_sys_ping_done(struct venus_core *core, struct venus_inst *inst,
+                             void *packet)
+{
+       struct hfi_msg_sys_ping_ack_pkt *pkt = packet;
+
+       core->error = HFI_ERR_NONE;
+
+       if (pkt->client_data != 0xbeef)
+               core->error = HFI_ERR_SYS_FATAL;
+
+       complete(&core->done);
+}
+
+static void hfi_sys_idle_done(struct venus_core *core, struct venus_inst *inst,
+                             void *packet)
+{
+       dev_dbg(core->dev, "sys idle\n");
+}
+
+static void hfi_sys_pc_prepare_done(struct venus_core *core,
+                                   struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_sys_pc_prep_done_pkt *pkt = packet;
+
+       dev_dbg(core->dev, "pc prepare done (error %x)\n", pkt->error_type);
+}
+
+static void
+hfi_copy_cap_prop(struct hfi_capability *in, struct venus_inst *inst)
+{
+       if (!in || !inst)
+               return;
+
+       switch (in->capability_type) {
+       case HFI_CAPABILITY_FRAME_WIDTH:
+               inst->cap_width = *in;
+               break;
+       case HFI_CAPABILITY_FRAME_HEIGHT:
+               inst->cap_height = *in;
+               break;
+       case HFI_CAPABILITY_MBS_PER_FRAME:
+               inst->cap_mbs_per_frame = *in;
+               break;
+       case HFI_CAPABILITY_MBS_PER_SECOND:
+               inst->cap_mbs_per_sec = *in;
+               break;
+       case HFI_CAPABILITY_FRAMERATE:
+               inst->cap_framerate = *in;
+               break;
+       case HFI_CAPABILITY_SCALE_X:
+               inst->cap_scale_x = *in;
+               break;
+       case HFI_CAPABILITY_SCALE_Y:
+               inst->cap_scale_y = *in;
+               break;
+       case HFI_CAPABILITY_BITRATE:
+               inst->cap_bitrate = *in;
+               break;
+       case HFI_CAPABILITY_HIER_P_NUM_ENH_LAYERS:
+               inst->cap_hier_p = *in;
+               break;
+       case HFI_CAPABILITY_ENC_LTR_COUNT:
+               inst->cap_ltr_count = *in;
+               break;
+       case HFI_CAPABILITY_CP_OUTPUT2_THRESH:
+               inst->cap_secure_output2_threshold = *in;
+               break;
+       default:
+               break;
+       }
+}
+
+static unsigned int
+session_get_prop_profile_level(struct hfi_msg_session_property_info_pkt *pkt,
+                              struct hfi_profile_level *profile_level)
+{
+       struct hfi_profile_level *hfi;
+       u32 req_bytes;
+
+       req_bytes = pkt->shdr.hdr.size - sizeof(*pkt);
+
+       if (!req_bytes || req_bytes % sizeof(struct hfi_profile_level))
+               /* bad packet */
+               return HFI_ERR_SESSION_INVALID_PARAMETER;
+
+       hfi = (struct hfi_profile_level *)&pkt->data[1];
+       profile_level->profile = hfi->profile;
+       profile_level->level = hfi->level;
+
+       return HFI_ERR_NONE;
+}
+
+static unsigned int
+session_get_prop_buf_req(struct hfi_msg_session_property_info_pkt *pkt,
+                        struct hfi_buffer_requirements *bufreq)
+{
+       struct hfi_buffer_requirements *buf_req;
+       u32 req_bytes;
+       unsigned int idx = 0;
+
+       req_bytes = pkt->shdr.hdr.size - sizeof(*pkt);
+
+       if (!req_bytes || req_bytes % sizeof(*buf_req) || !pkt->data[1])
+               /* bad packet */
+               return HFI_ERR_SESSION_INVALID_PARAMETER;
+
+       buf_req = (struct hfi_buffer_requirements *)&pkt->data[1];
+       if (!buf_req)
+               return HFI_ERR_SESSION_INVALID_PARAMETER;
+
+       while (req_bytes) {
+               memcpy(&bufreq[idx], buf_req, sizeof(*bufreq));
+               idx++;
+
+               if (idx > HFI_BUFFER_TYPE_MAX)
+                       return HFI_ERR_SESSION_INVALID_PARAMETER;
+
+               req_bytes -= sizeof(struct hfi_buffer_requirements);
+               buf_req++;
+       }
+
+       return HFI_ERR_NONE;
+}
+
+static void hfi_session_prop_info(struct venus_core *core,
+                                 struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_property_info_pkt *pkt = packet;
+       struct device *dev = core->dev;
+       union hfi_get_property *hprop = &inst->hprop;
+       unsigned int error = HFI_ERR_NONE;
+
+       if (!pkt->num_properties) {
+               error = HFI_ERR_SESSION_INVALID_PARAMETER;
+               dev_err(dev, "%s: no properties\n", __func__);
+               goto done;
+       }
+
+       switch (pkt->data[0]) {
+       case HFI_PROPERTY_CONFIG_BUFFER_REQUIREMENTS:
+               memset(hprop->bufreq, 0, sizeof(hprop->bufreq));
+               error = session_get_prop_buf_req(pkt, hprop->bufreq);
+               break;
+       case HFI_PROPERTY_PARAM_PROFILE_LEVEL_CURRENT:
+               memset(&hprop->profile_level, 0, sizeof(hprop->profile_level));
+               error = session_get_prop_profile_level(pkt,
+                                                      &hprop->profile_level);
+               break;
+       case HFI_PROPERTY_CONFIG_VDEC_ENTROPY:
+               break;
+       default:
+               dev_dbg(dev, "%s: unknown property id:%x\n", __func__,
+                       pkt->data[0]);
+               return;
+       }
+
+done:
+       inst->error = error;
+       complete(&inst->done);
+}
+
+static u32 init_done_read_prop(struct venus_core *core, struct venus_inst 
*inst,
+                              struct hfi_msg_session_init_done_pkt *pkt)
+{
+       struct device *dev = core->dev;
+       u32 rem_bytes, num_props, codecs = 0, domain = 0;
+       u32 ptype, next_offset = 0;
+       u32 err;
+       u8 *data;
+
+       rem_bytes = pkt->shdr.hdr.size - sizeof(*pkt) + sizeof(u32);
+       if (!rem_bytes) {
+               dev_err(dev, "%s: missing property info\n", __func__);
+               return HFI_ERR_SESSION_INSUFFICIENT_RESOURCES;
+       }
+
+       err = pkt->error_type;
+       if (err)
+               return err;
+
+       data = (u8 *) &pkt->data[0];
+       num_props = pkt->num_properties;
+
+       while (err == HFI_ERR_NONE && num_props && rem_bytes >= sizeof(u32)) {
+               ptype = *((u32 *)data);
+               next_offset = sizeof(u32);
+
+               switch (ptype) {
+               case HFI_PROPERTY_PARAM_CODEC_MASK_SUPPORTED: {
+                       struct hfi_codec_mask_supported *masks =
+                               (struct hfi_codec_mask_supported *)
+                               (data + next_offset);
+
+                       codecs = masks->codecs;
+                       domain = masks->video_domains;
+                       next_offset += sizeof(*masks);
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_CAPABILITY_SUPPORTED: {
+                       struct hfi_capabilities *caps;
+                       struct hfi_capability *cap;
+                       u32 num_caps;
+
+                       if ((rem_bytes - next_offset) < sizeof(*cap)) {
+                               err = HFI_ERR_SESSION_INVALID_PARAMETER;
+                               break;
+                       }
+
+                       caps = (struct hfi_capabilities *)(data + next_offset);
+
+                       num_caps = caps->num_capabilities;
+                       cap = &caps->data[0];
+                       next_offset += sizeof(u32);
+
+                       while (num_caps &&
+                             (rem_bytes - next_offset) >= sizeof(u32)) {
+                               hfi_copy_cap_prop(cap, inst);
+                               cap++;
+                               next_offset += sizeof(*cap);
+                               num_caps--;
+                       }
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_UNCOMPRESSED_FORMAT_SUPPORTED: {
+                       struct hfi_uncompressed_format_supported *prop =
+                               (struct hfi_uncompressed_format_supported *)
+                               (data + next_offset);
+                       u32 num_fmt_entries;
+                       u8 *fmt;
+                       struct hfi_uncompressed_plane_info *inf;
+
+                       if ((rem_bytes - next_offset) < sizeof(*prop)) {
+                               err = HFI_ERR_SESSION_INVALID_PARAMETER;
+                               break;
+                       }
+
+                       num_fmt_entries = prop->format_entries;
+                       next_offset = sizeof(*prop) - sizeof(u32);
+                       fmt = (u8 *)&prop->format_info[0];
+
+                       dev_dbg(dev, "uncomm format support num entries:%u\n",
+                               num_fmt_entries);
+
+                       while (num_fmt_entries) {
+                               struct hfi_uncompressed_plane_constraints *cnts;
+                               u32 bytes_to_skip;
+
+                               inf = (struct hfi_uncompressed_plane_info *)fmt;
+
+                               if ((rem_bytes - next_offset) < sizeof(*inf)) {
+                                       err = HFI_ERR_SESSION_INVALID_PARAMETER;
+                                       break;
+                               }
+
+                               dev_dbg(dev, "plane info: fmt:%x, planes:%x\n",
+                                       inf->format, inf->num_planes);
+
+                               cnts = &inf->plane_format[0];
+                               dev_dbg(dev, "%u %u %u %u\n",
+                                       cnts->stride_multiples,
+                                       cnts->max_stride,
+                                       cnts->min_plane_buffer_height_multiple,
+                                       cnts->buffer_alignment);
+
+                               bytes_to_skip = sizeof(*inf) - sizeof(*cnts) +
+                                               inf->num_planes * sizeof(*cnts);
+
+                               fmt += bytes_to_skip;
+                               next_offset += bytes_to_skip;
+                               num_fmt_entries--;
+                       }
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_PROPERTIES_SUPPORTED: {
+                       struct hfi_properties_supported *prop =
+                               (struct hfi_properties_supported *)
+                               (data + next_offset);
+
+                       next_offset += sizeof(*prop) - sizeof(u32)
+                                       + prop->num_properties * sizeof(u32);
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_PROFILE_LEVEL_SUPPORTED: {
+                       struct hfi_profile_level_supported *prop =
+                               (struct hfi_profile_level_supported *)
+                               (data + next_offset);
+                       struct hfi_profile_level *pl;
+                       unsigned int prop_count = 0;
+                       unsigned int count = 0;
+                       u8 *ptr;
+
+                       ptr = (u8 *)&prop->profile_level[0];
+                       prop_count = prop->profile_count;
+
+                       if (prop_count > HFI_MAX_PROFILE_COUNT)
+                               prop_count = HFI_MAX_PROFILE_COUNT;
+
+                       while (prop_count) {
+                               ptr++;
+                               pl = (struct hfi_profile_level *)ptr;
+
+                               inst->pl[count].profile = pl->profile;
+                               inst->pl[count].level = pl->level;
+                               prop_count--;
+                               count++;
+                               ptr += sizeof(*pl) / sizeof(u32);
+                       }
+
+                       inst->pl_count = count;
+                       next_offset += sizeof(*prop) - sizeof(*pl) +
+                                      prop->profile_count * sizeof(*pl);
+
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_INTERLACE_FORMAT_SUPPORTED: {
+                       next_offset +=
+                               sizeof(struct hfi_interlace_format_supported);
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_NAL_STREAM_FORMAT_SUPPORTED: {
+                       struct hfi_nal_stream_format *nal =
+                               (struct hfi_nal_stream_format *)
+                               (data + next_offset);
+                       dev_dbg(dev, "NAL format: %x\n", nal->format);
+                       next_offset += sizeof(*nal);
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_NAL_STREAM_FORMAT_SELECT: {
+                       next_offset += sizeof(u32);
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_MAX_SEQUENCE_HEADER_SIZE: {
+                       u32 *max_seq_sz = (u32 *)(data + next_offset);
+
+                       dev_dbg(dev, "max seq header sz: %x\n", *max_seq_sz);
+                       next_offset += sizeof(u32);
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_VENC_INTRA_REFRESH: {
+                       next_offset += sizeof(struct hfi_intra_refresh);
+                       num_props--;
+                       break;
+               }
+               case HFI_PROPERTY_PARAM_BUFFER_ALLOC_MODE_SUPPORTED: {
+                       struct hfi_buffer_alloc_mode_supported *prop =
+                               (struct hfi_buffer_alloc_mode_supported *)
+                               (data + next_offset);
+                       int i;
+
+                       if (prop->buffer_type == HFI_BUFFER_OUTPUT ||
+                           prop->buffer_type == HFI_BUFFER_OUTPUT2) {
+                               for (i = 0; i < prop->num_entries; i++) {
+                                       switch (prop->data[i]) {
+                                       case HFI_BUFFER_MODE_STATIC:
+                                               inst->cap_bufs_mode_static = 1;
+                                               break;
+                                       case HFI_BUFFER_MODE_DYNAMIC:
+                                               inst->cap_bufs_mode_dynamic = 1;
+                                               break;
+                                       }
+                               }
+                       }
+                       next_offset += sizeof(*prop) -
+                               sizeof(u32) + prop->num_entries * sizeof(u32);
+                       num_props--;
+                       break;
+               }
+               default:
+                       dev_dbg(dev, "%s: default case %#x\n", __func__, ptype);
+                       break;
+               }
+
+               rem_bytes -= next_offset;
+               data += next_offset;
+       }
+
+       return err;
+}
+
+static void hfi_session_init_done(struct venus_core *core,
+                                 struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_init_done_pkt *pkt = packet;
+       unsigned int error;
+
+       error = pkt->error_type;
+       if (error != HFI_ERR_NONE)
+               goto done;
+
+       if (core->res->hfi_version != HFI_VERSION_LEGACY)
+               goto done;
+
+       error = init_done_read_prop(core, inst, pkt);
+
+done:
+       inst->error = error;
+       complete(&inst->done);
+}
+
+static void hfi_session_load_res_done(struct venus_core *core,
+                                     struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_load_resources_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+static void hfi_session_flush_done(struct venus_core *core,
+                                  struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_flush_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+static void hfi_session_etb_done(struct venus_core *core,
+                                struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_empty_buffer_done_pkt *pkt = packet;
+       u32 flags = 0;
+
+       inst->error = pkt->error_type;
+
+       inst->ops->buf_done(inst, HFI_BUFFER_INPUT, pkt->input_tag,
+                           pkt->filled_len, pkt->offset, flags, 0);
+}
+
+static void hfi_session_ftb_done(struct venus_core *core,
+                                struct venus_inst *inst, void *packet)
+{
+       u32 session_type = inst->session_type;
+       u64 timestamp_us = 0;
+       u32 timestamp_hi = 0, timestamp_lo = 0;
+       unsigned int error;
+       u32 flags = 0, hfi_flags = 0, offset = 0, filled_len = 0;
+       u32 pic_type = 0, packet_buffer, buffer_type = 0, output_tag = -1;
+
+       if (session_type == VIDC_SESSION_TYPE_ENC) {
+               struct hfi_msg_session_fbd_compressed_pkt *pkt = packet;
+
+               timestamp_hi = pkt->time_stamp_hi;
+               timestamp_lo = pkt->time_stamp_lo;
+               hfi_flags = pkt->flags;
+               offset = pkt->offset;
+               filled_len = pkt->filled_len;
+               pic_type = pkt->picture_type;
+               packet_buffer = pkt->packet_buffer;
+               output_tag = pkt->output_tag;
+               buffer_type = HFI_BUFFER_OUTPUT;
+
+               error = pkt->error_type;
+       } else if (session_type == VIDC_SESSION_TYPE_DEC) {
+               struct hfi_msg_session_fbd_uncompressed_plane0_pkt *pkt =
+                       packet;
+
+               timestamp_hi = pkt->time_stamp_hi;
+               timestamp_lo = pkt->time_stamp_lo;
+               hfi_flags = pkt->flags;
+               offset = pkt->offset;
+               filled_len = pkt->filled_len;
+               pic_type = pkt->picture_type;
+               packet_buffer = pkt->packet_buffer;
+               output_tag = pkt->output_tag;
+
+               if (pkt->stream_id == 0)
+                       buffer_type = HFI_BUFFER_OUTPUT;
+               else if (pkt->stream_id == 1)
+                       buffer_type = HFI_BUFFER_OUTPUT2;
+
+               error = pkt->error_type;
+       } else {
+               error = HFI_ERR_SESSION_INVALID_PARAMETER;
+       }
+
+       if (buffer_type != HFI_BUFFER_OUTPUT)
+               goto done;
+
+       if (hfi_flags & HFI_BUFFERFLAG_EOS)
+               flags |= V4L2_BUF_FLAG_LAST;
+
+       switch (pic_type) {
+       case HFI_PICTURE_IDR:
+               flags |= V4L2_BUF_FLAG_KEYFRAME;
+               break;
+       case HFI_PICTURE_I:
+               flags |= V4L2_BUF_FLAG_KEYFRAME;
+               break;
+       case HFI_PICTURE_P:
+               flags |= V4L2_BUF_FLAG_PFRAME;
+               break;
+       case HFI_PICTURE_B:
+               flags |= V4L2_BUF_FLAG_BFRAME;
+               break;
+       case HFI_FRAME_NOTCODED:
+       case HFI_UNUSED_PICT:
+       case HFI_FRAME_YUV:
+       default:
+               break;
+       }
+
+       if (!(hfi_flags & HFI_BUFFERFLAG_TIMESTAMPINVALID) && filled_len) {
+               timestamp_us = timestamp_hi;
+               timestamp_us = (timestamp_us << 32) | timestamp_lo;
+       }
+
+done:
+       inst->error = error;
+       inst->ops->buf_done(inst, buffer_type, output_tag, filled_len,
+                           offset, flags, timestamp_us);
+}
+
+static void hfi_session_start_done(struct venus_core *core,
+                                  struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_start_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+static void hfi_session_stop_done(struct venus_core *core,
+                                 struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_stop_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+static void hfi_session_rel_res_done(struct venus_core *core,
+                                    struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_release_resources_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+static void hfi_session_rel_buf_done(struct venus_core *core,
+                                    struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_release_buffers_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+static void hfi_session_end_done(struct venus_core *core,
+                                struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_end_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+static void hfi_session_abort_done(struct venus_core *core,
+                                  struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_sys_session_abort_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+static void hfi_session_get_seq_hdr_done(struct venus_core *core,
+                                        struct venus_inst *inst, void *packet)
+{
+       struct hfi_msg_session_get_sequence_hdr_done_pkt *pkt = packet;
+
+       inst->error = pkt->error_type;
+       complete(&inst->done);
+}
+
+struct hfi_done_handler {
+       u32 pkt;
+       u32 pkt_sz;
+       u32 pkt_sz2;
+       void (*done)(struct venus_core *, struct venus_inst *, void *);
+       bool is_sys_pkt;
+};
+
+static const struct hfi_done_handler handlers[] = {
+       {.pkt = HFI_MSG_EVENT_NOTIFY,
+        .pkt_sz = sizeof(struct hfi_msg_event_notify_pkt),
+        .done = hfi_event_notify,
+       },
+       {.pkt = HFI_MSG_SYS_INIT,
+        .pkt_sz = sizeof(struct hfi_msg_sys_init_done_pkt),
+        .done = hfi_sys_init_done,
+        .is_sys_pkt = true,
+       },
+       {.pkt = HFI_MSG_SYS_PROPERTY_INFO,
+        .pkt_sz = sizeof(struct hfi_msg_sys_property_info_pkt),
+        .done = hfi_sys_property_info,
+        .is_sys_pkt = true,
+       },
+       {.pkt = HFI_MSG_SYS_RELEASE_RESOURCE,
+        .pkt_sz = sizeof(struct hfi_msg_sys_release_resource_done_pkt),
+        .done = hfi_sys_rel_resource_done,
+        .is_sys_pkt = true,
+       },
+       {.pkt = HFI_MSG_SYS_PING_ACK,
+        .pkt_sz = sizeof(struct hfi_msg_sys_ping_ack_pkt),
+        .done = hfi_sys_ping_done,
+        .is_sys_pkt = true,
+       },
+       {.pkt = HFI_MSG_SYS_IDLE,
+        .pkt_sz = sizeof(struct hfi_msg_sys_idle_pkt),
+        .done = hfi_sys_idle_done,
+        .is_sys_pkt = true,
+       },
+       {.pkt = HFI_MSG_SYS_PC_PREP,
+        .pkt_sz = sizeof(struct hfi_msg_sys_pc_prep_done_pkt),
+        .done = hfi_sys_pc_prepare_done,
+        .is_sys_pkt = true,
+       },
+       {.pkt = HFI_MSG_SYS_SESSION_INIT,
+        .pkt_sz = sizeof(struct hfi_msg_session_init_done_pkt),
+        .done = hfi_session_init_done,
+       },
+       {.pkt = HFI_MSG_SYS_SESSION_END,
+        .pkt_sz = sizeof(struct hfi_msg_session_end_done_pkt),
+        .done = hfi_session_end_done,
+       },
+       {.pkt = HFI_MSG_SESSION_LOAD_RESOURCES,
+        .pkt_sz = sizeof(struct hfi_msg_session_load_resources_done_pkt),
+        .done = hfi_session_load_res_done,
+       },
+       {.pkt = HFI_MSG_SESSION_START,
+        .pkt_sz = sizeof(struct hfi_msg_session_start_done_pkt),
+        .done = hfi_session_start_done,
+       },
+       {.pkt = HFI_MSG_SESSION_STOP,
+        .pkt_sz = sizeof(struct hfi_msg_session_stop_done_pkt),
+        .done = hfi_session_stop_done,
+       },
+       {.pkt = HFI_MSG_SYS_SESSION_ABORT,
+        .pkt_sz = sizeof(struct hfi_msg_sys_session_abort_done_pkt),
+        .done = hfi_session_abort_done,
+       },
+       {.pkt = HFI_MSG_SESSION_EMPTY_BUFFER,
+        .pkt_sz = sizeof(struct hfi_msg_session_empty_buffer_done_pkt),
+        .done = hfi_session_etb_done,
+       },
+       {.pkt = HFI_MSG_SESSION_FILL_BUFFER,
+        .pkt_sz = sizeof(struct hfi_msg_session_fbd_uncompressed_plane0_pkt),
+        .pkt_sz2 = sizeof(struct hfi_msg_session_fbd_compressed_pkt),
+        .done = hfi_session_ftb_done,
+       },
+       {.pkt = HFI_MSG_SESSION_FLUSH,
+        .pkt_sz = sizeof(struct hfi_msg_session_flush_done_pkt),
+        .done = hfi_session_flush_done,
+       },
+       {.pkt = HFI_MSG_SESSION_PROPERTY_INFO,
+        .pkt_sz = sizeof(struct hfi_msg_session_property_info_pkt),
+        .done = hfi_session_prop_info,
+       },
+       {.pkt = HFI_MSG_SESSION_RELEASE_RESOURCES,
+        .pkt_sz = sizeof(struct hfi_msg_session_release_resources_done_pkt),
+        .done = hfi_session_rel_res_done,
+       },
+       {.pkt = HFI_MSG_SESSION_GET_SEQUENCE_HEADER,
+        .pkt_sz = sizeof(struct hfi_msg_session_get_sequence_hdr_done_pkt),
+        .done = hfi_session_get_seq_hdr_done,
+       },
+       {.pkt = HFI_MSG_SESSION_RELEASE_BUFFERS,
+        .pkt_sz = sizeof(struct hfi_msg_session_release_buffers_done_pkt),
+        .done = hfi_session_rel_buf_done,
+       },
+};
+
+void hfi_process_watchdog_timeout(struct venus_core *core)
+{
+       event_sys_error(core, EVT_SYS_WATCHDOG_TIMEOUT);
+}
+
+static struct venus_inst *to_instance(struct venus_core *core, u32 session_id)
+{
+       struct venus_inst *inst;
+
+       mutex_lock(&core->lock);
+       list_for_each_entry(inst, &core->instances, list)
+               if (hash32_ptr(inst) == session_id) {
+                       mutex_unlock(&core->lock);
+                       return inst;
+               }
+       mutex_unlock(&core->lock);
+
+       return NULL;
+}
+
+u32 hfi_process_msg_packet(struct venus_core *core, struct hfi_pkt_hdr *hdr)
+{
+       const struct hfi_done_handler *handler;
+       struct device *dev = core->dev;
+       struct venus_inst *inst;
+       bool found = false;
+       unsigned int i;
+
+       for (i = 0; i < ARRAY_SIZE(handlers); i++) {
+               handler = &handlers[i];
+               if (handler->pkt != hdr->pkt_type)
+                       continue;
+               found = true;
+               break;
+       }
+
+       if (found == false)
+               return hdr->pkt_type;
+
+       if (hdr->size && hdr->size < handler->pkt_sz &&
+           hdr->size < handler->pkt_sz2) {
+               dev_err(dev, "bad packet size (%d should be %d, pkt type:%x)\n",
+                       hdr->size, handler->pkt_sz, hdr->pkt_type);
+
+               return hdr->pkt_type;
+       }
+
+       if (handler->is_sys_pkt) {
+               inst = NULL;
+       } else {
+               struct hfi_session_pkt *pkt;
+
+               pkt = (struct hfi_session_pkt *)hdr;
+               inst = to_instance(core, pkt->shdr.session_id);
+
+               if (!inst)
+                       dev_warn(dev, "no valid instance(pkt session_id:%x)\n",
+                                pkt->shdr.session_id);
+
+               /*
+                * Event of type HFI_EVENT_SYS_ERROR will not have any session
+                * associated with it
+                */
+               if (!inst && hdr->pkt_type != HFI_MSG_EVENT_NOTIFY) {
+                       dev_err(dev, "got invalid session id:%x\n",
+                               pkt->shdr.session_id);
+                       goto invalid_session;
+               }
+       }
+
+       handler->done(core, inst, hdr);
+
+invalid_session:
+       return hdr->pkt_type;
+}
diff --git a/drivers/media/platform/qcom/venus/hfi_msgs.h 
b/drivers/media/platform/qcom/venus/hfi_msgs.h
new file mode 100644
index 000000000000..14d9a3979b14
--- /dev/null
+++ b/drivers/media/platform/qcom/venus/hfi_msgs.h
@@ -0,0 +1,283 @@
+/*
+ * Copyright (c) 2012-2016, The Linux Foundation. All rights reserved.
+ * Copyright (C) 2017 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+#ifndef __VENUS_HFI_MSGS_H__
+#define __VENUS_HFI_MSGS_H__
+
+/* message calls */
+#define HFI_MSG_SYS_INIT                       0x20001
+#define HFI_MSG_SYS_PC_PREP                    0x20002
+#define HFI_MSG_SYS_RELEASE_RESOURCE           0x20003
+#define HFI_MSG_SYS_DEBUG                      0x20004
+#define HFI_MSG_SYS_SESSION_INIT               0x20006
+#define HFI_MSG_SYS_SESSION_END                        0x20007
+#define HFI_MSG_SYS_IDLE                       0x20008
+#define HFI_MSG_SYS_COV                                0x20009
+#define HFI_MSG_SYS_PROPERTY_INFO              0x2000a
+
+#define HFI_MSG_EVENT_NOTIFY                   0x21001
+#define HFI_MSG_SESSION_GET_SEQUENCE_HEADER    0x21002
+
+#define HFI_MSG_SYS_PING_ACK                   0x220002
+#define HFI_MSG_SYS_SESSION_ABORT              0x220004
+
+#define HFI_MSG_SESSION_LOAD_RESOURCES         0x221001
+#define HFI_MSG_SESSION_START                  0x221002
+#define HFI_MSG_SESSION_STOP                   0x221003
+#define HFI_MSG_SESSION_SUSPEND                        0x221004
+#define HFI_MSG_SESSION_RESUME                 0x221005
+#define HFI_MSG_SESSION_FLUSH                  0x221006
+#define HFI_MSG_SESSION_EMPTY_BUFFER           0x221007
+#define HFI_MSG_SESSION_FILL_BUFFER            0x221008
+#define HFI_MSG_SESSION_PROPERTY_INFO          0x221009
+#define HFI_MSG_SESSION_RELEASE_RESOURCES      0x22100a
+#define HFI_MSG_SESSION_PARSE_SEQUENCE_HEADER  0x22100b
+#define HFI_MSG_SESSION_RELEASE_BUFFERS                0x22100c
+
+#define HFI_PICTURE_I                          0x00000001
+#define HFI_PICTURE_P                          0x00000002
+#define HFI_PICTURE_B                          0x00000004
+#define HFI_PICTURE_IDR                                0x00000008
+#define HFI_FRAME_NOTCODED                     0x7f002000
+#define HFI_FRAME_YUV                          0x7f004000
+#define HFI_UNUSED_PICT                                0x10000000
+
+/* message packets */
+struct hfi_msg_event_notify_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 event_id;
+       u32 event_data1;
+       u32 event_data2;
+       u32 ext_event_data[1];
+};
+
+struct hfi_msg_event_release_buffer_ref_pkt {
+       u32 packet_buffer;
+       u32 extradata_buffer;
+       u32 output_tag;
+};
+
+struct hfi_msg_sys_init_done_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 error_type;
+       u32 num_properties;
+       u32 data[1];
+};
+
+struct hfi_msg_sys_pc_prep_done_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 error_type;
+};
+
+struct hfi_msg_sys_release_resource_done_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 resource_handle;
+       u32 error_type;
+};
+
+struct hfi_msg_session_init_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+       u32 num_properties;
+       u32 data[1];
+};
+
+struct hfi_msg_session_end_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+};
+
+struct hfi_msg_session_get_sequence_hdr_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+       u32 header_len;
+       u32 sequence_header;
+};
+
+struct hfi_msg_sys_session_abort_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+};
+
+struct hfi_msg_sys_idle_pkt {
+       struct hfi_pkt_hdr hdr;
+};
+
+struct hfi_msg_sys_ping_ack_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 client_data;
+};
+
+struct hfi_msg_sys_property_info_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 num_properties;
+       u32 data[1];
+};
+
+struct hfi_msg_session_load_resources_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+};
+
+struct hfi_msg_session_start_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+};
+
+struct hfi_msg_session_stop_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+};
+
+struct hfi_msg_session_suspend_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+};
+
+struct hfi_msg_session_resume_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+};
+
+struct hfi_msg_session_flush_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+       u32 flush_type;
+};
+
+struct hfi_msg_session_empty_buffer_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+       u32 offset;
+       u32 filled_len;
+       u32 input_tag;
+       u32 packet_buffer;
+       u32 extradata_buffer;
+       u32 data[0];
+};
+
+struct hfi_msg_session_fbd_compressed_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 time_stamp_hi;
+       u32 time_stamp_lo;
+       u32 error_type;
+       u32 flags;
+       u32 mark_target;
+       u32 mark_data;
+       u32 stats;
+       u32 offset;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 input_tag;
+       u32 output_tag;
+       u32 picture_type;
+       u32 packet_buffer;
+       u32 extradata_buffer;
+       u32 data[0];
+};
+
+struct hfi_msg_session_fbd_uncompressed_plane0_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 stream_id;
+       u32 view_id;
+       u32 error_type;
+       u32 time_stamp_hi;
+       u32 time_stamp_lo;
+       u32 flags;
+       u32 mark_target;
+       u32 mark_data;
+       u32 stats;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 offset;
+       u32 frame_width;
+       u32 frame_height;
+       u32 start_x_coord;
+       u32 start_y_coord;
+       u32 input_tag;
+       u32 input_tag2;
+       u32 output_tag;
+       u32 picture_type;
+       u32 packet_buffer;
+       u32 extradata_buffer;
+       u32 data[0];
+};
+
+struct hfi_msg_session_fbd_uncompressed_plane1_pkt {
+       u32 flags;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 offset;
+       u32 packet_buffer2;
+       u32 data[0];
+};
+
+struct hfi_msg_session_fbd_uncompressed_plane2_pkt {
+       u32 flags;
+       u32 alloc_len;
+       u32 filled_len;
+       u32 offset;
+       u32 packet_buffer3;
+       u32 data[0];
+};
+
+struct hfi_msg_session_parse_sequence_header_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+       u32 num_properties;
+       u32 data[1];
+};
+
+struct hfi_msg_session_property_info_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 num_properties;
+       u32 data[1];
+};
+
+struct hfi_msg_session_release_resources_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+};
+
+struct hfi_msg_session_release_buffers_done_pkt {
+       struct hfi_session_hdr_pkt shdr;
+       u32 error_type;
+       u32 num_buffers;
+       u32 buffer_info[1];
+};
+
+struct hfi_msg_sys_debug_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 msg_type;
+       u32 msg_size;
+       u32 time_stamp_hi;
+       u32 time_stamp_lo;
+       u8 msg_data[1];
+};
+
+struct hfi_msg_sys_coverage_pkt {
+       struct hfi_pkt_hdr hdr;
+       u32 msg_size;
+       u32 time_stamp_hi;
+       u32 time_stamp_lo;
+       u8 msg_data[1];
+};
+
+struct venus_core;
+struct hfi_pkt_hdr;
+
+void hfi_process_watchdog_timeout(struct venus_core *core);
+u32 hfi_process_msg_packet(struct venus_core *core, struct hfi_pkt_hdr *hdr);
+
+#endif
-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to