Commit b19f8ef5 authored by Hans de Goede's avatar Hans de Goede Committed by Mauro Carvalho Chehab

media: atomisp: Remove no longer used atomisp_css_flush()

Remove the no longer used atomisp_css_flush() function and merge
atomisp_assert_recovery_work() and __atomisp_css_recover() into
a single function.
Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@kernel.org>
parent 0f072c1e
...@@ -935,18 +935,20 @@ void atomisp_buf_done(struct atomisp_sub_device *asd, int error, ...@@ -935,18 +935,20 @@ void atomisp_buf_done(struct atomisp_sub_device *asd, int error,
atomisp_qbuffers_to_css(asd); atomisp_qbuffers_to_css(asd);
} }
static void __atomisp_css_recover(struct atomisp_device *isp) void atomisp_assert_recovery_work(struct work_struct *work)
{ {
struct atomisp_device *isp = container_of(work, struct atomisp_device,
assert_recovery_work);
struct pci_dev *pdev = to_pci_dev(isp->dev); struct pci_dev *pdev = to_pci_dev(isp->dev);
enum ia_css_pipe_id css_pipe_id; enum ia_css_pipe_id css_pipe_id;
bool stream_restart = false; bool stream_restart = false;
unsigned long flags; unsigned long flags;
int ret; int ret;
lockdep_assert_held(&isp->mutex); mutex_lock(&isp->mutex);
if (!atomisp_streaming_count(isp)) if (!atomisp_streaming_count(isp))
return; goto out_unlock;
atomisp_css_irq_enable(isp, IA_CSS_IRQ_INFO_CSS_RECEIVER_SOF, false); atomisp_css_irq_enable(isp, IA_CSS_IRQ_INFO_CSS_RECEIVER_SOF, false);
...@@ -1028,26 +1030,11 @@ static void __atomisp_css_recover(struct atomisp_device *isp) ...@@ -1028,26 +1030,11 @@ static void __atomisp_css_recover(struct atomisp_device *isp)
dev_warn(isp->dev, dev_warn(isp->dev,
"can't start streaming on sensor!\n"); "can't start streaming on sensor!\n");
} }
}
void atomisp_assert_recovery_work(struct work_struct *work) out_unlock:
{
struct atomisp_device *isp = container_of(work, struct atomisp_device,
assert_recovery_work);
mutex_lock(&isp->mutex);
__atomisp_css_recover(isp);
mutex_unlock(&isp->mutex); mutex_unlock(&isp->mutex);
} }
void atomisp_css_flush(struct atomisp_device *isp)
{
/* Start recover */
__atomisp_css_recover(isp);
dev_dbg(isp->dev, "atomisp css flush done\n");
}
void atomisp_setup_flash(struct atomisp_sub_device *asd) void atomisp_setup_flash(struct atomisp_sub_device *asd)
{ {
struct atomisp_device *isp = asd->isp; struct atomisp_device *isp = asd->isp;
......
...@@ -282,8 +282,6 @@ void atomisp_buf_done(struct atomisp_sub_device *asd, int error, ...@@ -282,8 +282,6 @@ void atomisp_buf_done(struct atomisp_sub_device *asd, int error,
enum ia_css_pipe_id css_pipe_id, enum ia_css_pipe_id css_pipe_id,
bool q_buffers, enum atomisp_input_stream_id stream_id); bool q_buffers, enum atomisp_input_stream_id stream_id);
void atomisp_css_flush(struct atomisp_device *isp);
/* Events. Only one event has to be exported for now. */ /* Events. Only one event has to be exported for now. */
void atomisp_eof_event(struct atomisp_sub_device *asd, uint8_t exp_id); void atomisp_eof_event(struct atomisp_sub_device *asd, uint8_t exp_id);
......
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