Re: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-28 Thread Chris Ball
Hi Aaron, On Mon, Jul 11 2011, Aaron Lu wrote: > Currently, the retuning timer for retuning mode 1 will be deleted in > function sdhci_tasklet_finish after a mmc request done, which will make > retuning timing never trigger again. This patch fixed this problem. > > Signed-off-by: Aaron Lu > --- >

RE: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-27 Thread Philip Rakity
Reviewed-by: Philip Rakity From: Aaron Lu [aaron...@amd.com] Sent: Wednesday, July 27, 2011 2:15 AM To: zhangfei gao; Philip Rakity Cc: Nath, Arindam; Chris Ball; linux-mmc@vger.kernel.org Subject: Re: [PATCH] mmc: sdhci: fix retuning timer wrongly

Re: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-27 Thread Aaron Lu
On Fri, Jul 22, 2011 at 06:21:11PM +0800, zhangfei gao wrote: > > Thanks for explanation, looks good to me. > Hi Philip & Zhangfei, Can I have your reviewed-by tag in the patch? Thanks, Aaron > >> >> > > >> >> > Signed-off-by: Aaron Lu > >> >> > --- > >> >> >  drivers/mmc/host/sdhci.c |    3

Re: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-22 Thread zhangfei gao
On Thu, Jul 21, 2011 at 6:03 PM, Aaron Lu wrote: > On Thu, Jul 21, 2011 at 05:35:02PM +0800, zhangfei gao wrote: >> >> Does the execute_tuning is called again? >> del_timer is not delete timer really, but deactivate the timer, which >> could be re-activated by mod_timer. >> So if execute_tuning is

Re: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-21 Thread Aaron Lu
Hi Philip, On Thu, Jul 21, 2011 at 01:27:11PM -0700, Philip Rakity wrote: > > Aaron, > > Code is fine. Thanks. > Do you want to also fix the problem with suspend or should I do the patch ? > When we are suspending we should kill the tuning timer. Yes, that is a problem. I've another patch to a

Re: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-21 Thread Philip Rakity
Aaron, Code is fine. Do you want to also fix the problem with suspend or should I do the patch ? When we are suspending we should kill the tuning timer. diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 4da6a4d..88c25e8 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/

Re: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-21 Thread Aaron Lu
On Thu, Jul 21, 2011 at 05:35:02PM +0800, zhangfei gao wrote: > > Does the execute_tuning is called again? > del_timer is not delete timer really, but deactivate the timer, which > could be re-activated by mod_timer. > So if execute_tuning is called, the mod_timer will tigger the tuning > timer ag

Re: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-21 Thread zhangfei gao
ron Lu [mailto:aaron...@amd.com] >> > Sent: Monday, July 11, 2011 10:57 AM >> > To: Chris Ball >> > Cc: Nath, Arindam; linux-mmc@vger.kernel.org; Lu, Aaron >> > Subject: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in >> > sdhci_tasklet_finish >&

Re: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-20 Thread Aaron Lu
Monday, July 11, 2011 10:57 AM > > To: Chris Ball > > Cc: Nath, Arindam; linux-mmc@vger.kernel.org; Lu, Aaron > > Subject: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in > > sdhci_tasklet_finish > > > > Currently, the retuning timer for retuning mode

RE: [PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-14 Thread Nath, Arindam
[PATCH] mmc: sdhci: fix retuning timer wrongly deleted in > sdhci_tasklet_finish > > Currently, the retuning timer for retuning mode 1 will be deleted in > function sdhci_tasklet_finish after a mmc request done, which will make > retuning timing never trigger again. This patch fixed

[PATCH] mmc: sdhci: fix retuning timer wrongly deleted in sdhci_tasklet_finish

2011-07-10 Thread Aaron Lu
Currently, the retuning timer for retuning mode 1 will be deleted in function sdhci_tasklet_finish after a mmc request done, which will make retuning timing never trigger again. This patch fixed this problem. Signed-off-by: Aaron Lu --- drivers/mmc/host/sdhci.c |3 --- 1 files changed, 0 ins