media: platform: mtk-mdp3: add support second sets of MUTEX

After setting up the second set of MMSYS (VPPSYS1), it is necessary
to have a corresponding second set of MUTEX (MUTEX2) to assist in
handling SOF/EOF.

Signed-off-by: Moudy Ho <moudy.ho@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Sebastian Fricke <sebastian.fricke@collabora.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
This commit is contained in:
Moudy Ho 2023-12-20 11:18:28 +01:00 committed by Mauro Carvalho Chehab
parent e280d1a0eb
commit ee0d0dbb80
3 changed files with 74 additions and 38 deletions

View File

@ -44,15 +44,21 @@ static bool is_output_disabled(int p_id, const struct img_compparam *param, u32
return (count < num) ? (dis_output || dis_tile) : true;
}
static struct mtk_mutex *__get_mutex(const struct mdp_dev *mdp_dev,
const struct mdp_pipe_info *p)
{
return mdp_dev->mm_subsys[p->sub_id].mdp_mutex[p->mutex_id];
}
static int mdp_path_subfrm_require(const struct mdp_path *path,
struct mdp_cmdq_cmd *cmd,
s32 *mutex_id, u32 count)
struct mdp_pipe_info *p, u32 count)
{
const int p_id = path->mdp_dev->mdp_data->mdp_plat_id;
const struct mdp_comp_ctx *ctx;
const struct mtk_mdp_driver_data *data = path->mdp_dev->mdp_data;
struct device *dev = &path->mdp_dev->pdev->dev;
struct mtk_mutex **mutex = path->mdp_dev->mdp_mutex;
struct mtk_mutex *mutex;
int id, index;
u32 num_comp = 0;
@ -77,7 +83,8 @@ static int mdp_path_subfrm_require(const struct mdp_path *path,
dev_err(dev, "Unknown pipeline and no mutex is assigned");
return -EINVAL;
}
*mutex_id = data->pipe_info[index].mutex_id;
memcpy(p, &data->pipe_info[index], sizeof(struct mdp_pipe_info));
mutex = __get_mutex(path->mdp_dev, p);
/* Set mutex mod */
for (index = 0; index < num_comp; index++) {
@ -85,29 +92,28 @@ static int mdp_path_subfrm_require(const struct mdp_path *path,
if (is_output_disabled(p_id, ctx->param, count))
continue;
id = ctx->comp->public_id;
mtk_mutex_write_mod(mutex[*mutex_id],
data->mdp_mutex_table_idx[id], false);
mtk_mutex_write_mod(mutex, data->mdp_mutex_table_idx[id], false);
}
mtk_mutex_write_sof(mutex[*mutex_id],
MUTEX_SOF_IDX_SINGLE_MODE);
mtk_mutex_write_sof(mutex, MUTEX_SOF_IDX_SINGLE_MODE);
return 0;
}
static int mdp_path_subfrm_run(const struct mdp_path *path,
struct mdp_cmdq_cmd *cmd,
s32 *mutex_id, u32 count)
struct mdp_pipe_info *p, u32 count)
{
const int p_id = path->mdp_dev->mdp_data->mdp_plat_id;
const struct mdp_comp_ctx *ctx;
struct device *dev = &path->mdp_dev->pdev->dev;
struct mtk_mutex **mutex = path->mdp_dev->mdp_mutex;
struct mtk_mutex *mutex;
int index;
u32 num_comp = 0;
s32 event;
if (-1 == *mutex_id) {
if (-1 == p->mutex_id) {
dev_err(dev, "Incorrect mutex id");
return -EINVAL;
}
@ -127,7 +133,8 @@ static int mdp_path_subfrm_run(const struct mdp_path *path,
}
/* Enable the mutex */
mtk_mutex_enable_by_cmdq(mutex[*mutex_id], (void *)&cmd->pkt);
mutex = __get_mutex(path->mdp_dev, p);
mtk_mutex_enable_by_cmdq(mutex, (void *)&cmd->pkt);
/* Wait SOF events and clear mutex modules (optional) */
for (index = 0; index < num_comp; index++) {
@ -174,7 +181,7 @@ static int mdp_path_config_subfrm(struct mdp_cmdq_cmd *cmd,
const struct img_mmsys_ctrl *ctrl = NULL;
const struct img_mux *set;
struct mdp_comp_ctx *ctx;
s32 mutex_id;
struct mdp_pipe_info pipe;
int index, ret;
u32 num_comp = 0;
@ -185,7 +192,7 @@ static int mdp_path_config_subfrm(struct mdp_cmdq_cmd *cmd,
ctrl = CFG_ADDR(MT8183, path->config, ctrls[count]);
/* Acquire components */
ret = mdp_path_subfrm_require(path, cmd, &mutex_id, count);
ret = mdp_path_subfrm_require(path, cmd, &pipe, count);
if (ret)
return ret;
/* Enable mux settings */
@ -204,7 +211,7 @@ static int mdp_path_config_subfrm(struct mdp_cmdq_cmd *cmd,
return ret;
}
/* Run components */
ret = mdp_path_subfrm_run(path, cmd, &mutex_id, count);
ret = mdp_path_subfrm_run(path, cmd, &pipe, count);
if (ret)
return ret;
/* Wait components done */
@ -328,13 +335,13 @@ static void mdp_auto_release_work(struct work_struct *work)
{
struct mdp_cmdq_cmd *cmd;
struct mdp_dev *mdp;
int id;
struct mtk_mutex *mutex;
cmd = container_of(work, struct mdp_cmdq_cmd, auto_release_work);
mdp = cmd->mdp;
id = mdp->mdp_data->pipe_info[MDP_PIPE_RDMA0].mutex_id;
mtk_mutex_unprepare(mdp->mdp_mutex[id]);
mutex = __get_mutex(mdp, &mdp->mdp_data->pipe_info[MDP_PIPE_RDMA0]);
mtk_mutex_unprepare(mutex);
mdp_comp_clocks_off(&mdp->pdev->dev, cmd->comps,
cmd->num_comps);
@ -354,7 +361,6 @@ static void mdp_handle_cmdq_callback(struct mbox_client *cl, void *mssg)
struct cmdq_cb_data *data;
struct mdp_dev *mdp;
struct device *dev;
int id;
if (!mssg) {
pr_info("%s:no callback data\n", __func__);
@ -379,9 +385,11 @@ static void mdp_handle_cmdq_callback(struct mbox_client *cl, void *mssg)
INIT_WORK(&cmd->auto_release_work, mdp_auto_release_work);
if (!queue_work(mdp->clock_wq, &cmd->auto_release_work)) {
struct mtk_mutex *mutex;
dev_err(dev, "%s:queue_work fail!\n", __func__);
id = mdp->mdp_data->pipe_info[MDP_PIPE_RDMA0].mutex_id;
mtk_mutex_unprepare(mdp->mdp_mutex[id]);
mutex = __get_mutex(mdp, &mdp->mdp_data->pipe_info[MDP_PIPE_RDMA0]);
mtk_mutex_unprepare(mutex);
mdp_comp_clocks_off(&mdp->pdev->dev, cmd->comps,
cmd->num_comps);
@ -403,6 +411,7 @@ int mdp_cmdq_send(struct mdp_dev *mdp, struct mdp_cmdq_param *param)
struct mdp_comp *comps = NULL;
struct device *dev = &mdp->pdev->dev;
const int p_id = mdp->mdp_data->mdp_plat_id;
struct mtk_mutex *mutex = NULL;
int i, ret;
u32 num_comp = 0;
@ -440,8 +449,8 @@ int mdp_cmdq_send(struct mdp_dev *mdp, struct mdp_cmdq_param *param)
goto err_free_comps;
}
i = mdp->mdp_data->pipe_info[MDP_PIPE_RDMA0].mutex_id;
ret = mtk_mutex_prepare(mdp->mdp_mutex[i]);
mutex = __get_mutex(mdp, &mdp->mdp_data->pipe_info[MDP_PIPE_RDMA0]);
ret = mtk_mutex_prepare(mutex);
if (ret) {
dev_err(dev, "Fail to enable mutex clk\n");
goto err_free_path;
@ -506,8 +515,8 @@ err_clock_off:
mdp_comp_clocks_off(&mdp->pdev->dev, cmd->comps,
cmd->num_comps);
err_free_path:
i = mdp->mdp_data->pipe_info[MDP_PIPE_RDMA0].mutex_id;
mtk_mutex_unprepare(mdp->mdp_mutex[i]);
if (mutex)
mtk_mutex_unprepare(mutex);
kfree(path);
err_free_comps:
kfree(comps);

View File

@ -146,8 +146,17 @@ void mdp_video_device_release(struct video_device *vdev)
vb2_dma_contig_clear_max_seg_size(&mdp->pdev->dev);
mdp_comp_destroy(mdp);
for (i = 0; i < MDP_PIPE_MAX; i++)
mtk_mutex_put(mdp->mdp_mutex[i]);
for (i = 0; i < mdp->mdp_data->pipe_info_len; i++) {
enum mdp_mm_subsys_id idx;
struct mtk_mutex *m;
u32 m_id;
idx = mdp->mdp_data->pipe_info[i].sub_id;
m_id = mdp->mdp_data->pipe_info[i].mutex_id;
m = mdp->mm_subsys[idx].mdp_mutex[m_id];
if (!IS_ERR_OR_NULL(m))
mtk_mutex_put(m);
}
mdp_vpu_shared_mem_free(&mdp->vpu);
v4l2_m2m_release(mdp->m2m_dev);
@ -171,6 +180,9 @@ static int mdp_mm_subsys_deploy(struct mdp_dev *mdp, enum mdp_infra_id id)
case MDP_INFRA_MMSYS:
dev = &mdp->mm_subsys[i].mmsys;
break;
case MDP_INFRA_MUTEX:
dev = &mdp->mm_subsys[i].mutex;
break;
default:
dev_err(&mdp->pdev->dev, "Unknown infra id %d", id);
return -EINVAL;
@ -214,18 +226,24 @@ static int mdp_probe(struct platform_device *pdev)
if (ret)
goto err_destroy_device;
mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_MUTEX);
if (WARN_ON(!mm_pdev)) {
ret = -ENODEV;
ret = mdp_mm_subsys_deploy(mdp, MDP_INFRA_MUTEX);
if (ret)
goto err_destroy_device;
}
for (i = 0; i < mdp->mdp_data->pipe_info_len; i++) {
enum mdp_mm_subsys_id idx;
struct mtk_mutex **m;
idx = mdp->mdp_data->pipe_info[i].sub_id;
mutex_id = mdp->mdp_data->pipe_info[i].mutex_id;
if (!IS_ERR_OR_NULL(mdp->mdp_mutex[mutex_id]))
m = &mdp->mm_subsys[idx].mdp_mutex[mutex_id];
if (!IS_ERR_OR_NULL(*m))
continue;
mdp->mdp_mutex[mutex_id] = mtk_mutex_get(&mm_pdev->dev);
if (IS_ERR(mdp->mdp_mutex[mutex_id])) {
ret = PTR_ERR(mdp->mdp_mutex[mutex_id]);
*m = mtk_mutex_get(mdp->mm_subsys[idx].mutex);
if (IS_ERR(*m)) {
ret = PTR_ERR(*m);
goto err_free_mutex;
}
}
@ -309,9 +327,16 @@ err_destroy_job_wq:
err_deinit_comp:
mdp_comp_destroy(mdp);
err_free_mutex:
for (i = 0; i < mdp->mdp_data->pipe_info_len; i++)
if (!IS_ERR_OR_NULL(mdp->mdp_mutex[i]))
mtk_mutex_put(mdp->mdp_mutex[i]);
for (i = 0; i < mdp->mdp_data->pipe_info_len; i++) {
enum mdp_mm_subsys_id idx;
struct mtk_mutex *m;
idx = mdp->mdp_data->pipe_info[i].sub_id;
mutex_id = mdp->mdp_data->pipe_info[i].mutex_id;
m = mdp->mm_subsys[idx].mdp_mutex[mutex_id];
if (!IS_ERR_OR_NULL(m))
mtk_mutex_put(m);
}
err_destroy_device:
kfree(mdp);
err_return:

View File

@ -26,6 +26,7 @@ enum mdp_infra_id {
MDP_INFRA_MMSYS,
MDP_INFRA_MMSYS2,
MDP_INFRA_MUTEX,
MDP_INFRA_MUTEX2,
MDP_INFRA_SCP,
MDP_INFRA_MAX
};
@ -78,12 +79,13 @@ struct mtk_mdp_driver_data {
struct mdp_mm_subsys {
struct device *mmsys;
struct device *mutex;
struct mtk_mutex *mdp_mutex[MDP_PIPE_MAX];
};
struct mdp_dev {
struct platform_device *pdev;
struct mdp_mm_subsys mm_subsys[MDP_MM_SUBSYS_MAX];
struct mtk_mutex *mdp_mutex[MDP_PIPE_MAX];
struct mdp_comp *comp[MDP_MAX_COMP_COUNT];
const struct mtk_mdp_driver_data *mdp_data;