Hi Sachin, Thank you for your patch, fixing this bug and testing suspend/resume. Best wishes, -- Kamil Debski Linux Platform Group Samsung Poland R&D Center > -----Original Message----- > From: Sachin Kamat [mailto:sachin.kamat@xxxxxxxxxx] > Sent: 28 September 2012 09:02 > To: linux-media@xxxxxxxxxxxxxxx > Cc: k.debski@xxxxxxxxxxx; s.nawrocki@xxxxxxxxxxx; sachin.kamat@xxxxxxxxxx; > patches@xxxxxxxxxx > Subject: [PATCH] [media] s5p-mfc: Fix misplaced return statement in > s5p_mfc_suspend() > > return statement was wrongly placed before a code that needs to be > executed. Moved the return statement to the end of the function. > > Tested suspend/resume on SMDK4412 board using 3.5-rc6 kernel. > > Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> Acked-by: Kamil Debski <k.debski@xxxxxxxxxxx> > --- > drivers/media/platform/s5p-mfc/s5p_mfc.c | 5 +++-- > 1 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc.c > b/drivers/media/platform/s5p-mfc/s5p_mfc.c > index e3e616d..e913a1c 100644 > --- a/drivers/media/platform/s5p-mfc/s5p_mfc.c > +++ b/drivers/media/platform/s5p-mfc/s5p_mfc.c > @@ -1148,7 +1148,7 @@ static int s5p_mfc_suspend(struct device *dev) > > if (m_dev->num_inst == 0) > return 0; > - return s5p_mfc_sleep(m_dev); > + > if (test_and_set_bit(0, &m_dev->enter_suspend) != 0) { > mfc_err("Error: going to suspend for a second time\n"); > return -EIO; > @@ -1167,7 +1167,8 @@ static int s5p_mfc_suspend(struct device *dev) > return -EIO; > } > } > - return 0; > + > + return s5p_mfc_sleep(m_dev); > } > > static int s5p_mfc_resume(struct device *dev) > -- > 1.7.4.1 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
