Commit 4b0ec19e authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Mauro Carvalho Chehab

[media] omap3isp: Add set performance callback in isp platform data

Replace the omap_pm_set_min_bus_tput() with a callback to board code.
This makes the omap3isp driver compilable as a module, and lets board
code manage OMAP PM details depending on board requirements.
Signed-off-by: default avatarStanimir Varbanov <svarbanov@mm-sol.com>
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 7e8970e1
...@@ -199,6 +199,7 @@ struct isp_v4l2_subdevs_group { ...@@ -199,6 +199,7 @@ struct isp_v4l2_subdevs_group {
struct isp_platform_data { struct isp_platform_data {
struct isp_v4l2_subdevs_group *subdevs; struct isp_v4l2_subdevs_group *subdevs;
void (*set_constraints)(struct isp_device *isp, bool enable);
}; };
struct isp_platform_callback { struct isp_platform_callback {
......
...@@ -921,18 +921,8 @@ isp_video_streamon(struct file *file, void *fh, enum v4l2_buf_type type) ...@@ -921,18 +921,8 @@ isp_video_streamon(struct file *file, void *fh, enum v4l2_buf_type type)
pipe->output = far_end; pipe->output = far_end;
} }
/* Make sure the interconnect clock runs fast enough. if (video->isp->pdata->set_constraints)
* video->isp->pdata->set_constraints(video->isp, true);
* Formula from: resource34xx.c set_opp()
* If MPU freq is above 500MHz, make sure the interconnect
* is at 100Mhz or above.
* throughput in KiB/s for 100 Mhz = 100 * 1000 * 4.
*
* We want to be fast enough then set OCP clock to be max as
* possible, in that case 185Mhz then:
* throughput in KiB/s for 185Mhz = 185 * 1000 * 4 = 740000 KiB/s
*/
omap_pm_set_min_bus_tput(video->isp->dev, OCP_INITIATOR_AGENT, 740000);
pipe->l3_ick = clk_get_rate(video->isp->clock[ISP_CLK_L3_ICK]); pipe->l3_ick = clk_get_rate(video->isp->clock[ISP_CLK_L3_ICK]);
/* Validate the pipeline and update its state. */ /* Validate the pipeline and update its state. */
...@@ -978,8 +968,8 @@ isp_video_streamon(struct file *file, void *fh, enum v4l2_buf_type type) ...@@ -978,8 +968,8 @@ isp_video_streamon(struct file *file, void *fh, enum v4l2_buf_type type)
error: error:
if (ret < 0) { if (ret < 0) {
omap3isp_video_queue_streamoff(&vfh->queue); omap3isp_video_queue_streamoff(&vfh->queue);
omap_pm_set_min_bus_tput(video->isp->dev, if (video->isp->pdata->set_constraints)
OCP_INITIATOR_AGENT, 0); video->isp->pdata->set_constraints(video->isp, false);
media_entity_pipeline_stop(&video->video.entity); media_entity_pipeline_stop(&video->video.entity);
video->queue = NULL; video->queue = NULL;
} }
...@@ -1032,7 +1022,8 @@ isp_video_streamoff(struct file *file, void *fh, enum v4l2_buf_type type) ...@@ -1032,7 +1022,8 @@ isp_video_streamoff(struct file *file, void *fh, enum v4l2_buf_type type)
video->queue = NULL; video->queue = NULL;
video->streaming = 0; video->streaming = 0;
omap_pm_set_min_bus_tput(video->isp->dev, OCP_INITIATOR_AGENT, 0); if (video->isp->pdata->set_constraints)
video->isp->pdata->set_constraints(video->isp, false);
media_entity_pipeline_stop(&video->video.entity); media_entity_pipeline_stop(&video->video.entity);
done: done:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment