RE: Cache Control during mmc_suspend_host

2011-12-16 Thread Seungwon Jeon
Saugata Das wrote: > On 14 December 2011 10:04, Seungwon Jeon wrote: > > Saugata Das wrote: > >> On 13 December 2011 11:57, Seungwon Jeon wrote: > >> > Hi Saugata, > >> > > >> > Saugata Das wrote: > >> >> Hi Seungwon Jeon > >> >> > >> >> I see a small issue with the implementation mmc_suspend_hos

Re: Cache Control during mmc_suspend_host

2011-12-15 Thread Saugata Das
On 14 December 2011 10:04, Seungwon Jeon wrote: > Saugata Das wrote: >> On 13 December 2011 11:57, Seungwon Jeon wrote: >> > Hi Saugata, >> > >> > Saugata Das wrote: >> >> Hi Seungwon Jeon >> >> >> >> I see a small issue with the implementation mmc_suspend_host, >> >> >> >> int mmc_suspend_host(s

RE: Cache Control during mmc_suspend_host

2011-12-13 Thread Seungwon Jeon
Saugata Das wrote: > On 13 December 2011 11:57, Seungwon Jeon wrote: > > Hi Saugata, > > > > Saugata Das wrote: > >> Hi Seungwon Jeon > >> > >> I see a small issue with the implementation mmc_suspend_host, > >> > >> int mmc_suspend_host(struct mmc_host *host) > >> { > >> . > >>       err = mmc

Re: Cache Control during mmc_suspend_host

2011-12-13 Thread Saugata Das
On 13 December 2011 11:57, Seungwon Jeon wrote: > Hi Saugata, > > Saugata Das wrote: >> Hi Seungwon Jeon >> >> I see a small issue with the implementation mmc_suspend_host, >> >> int mmc_suspend_host(struct mmc_host *host) >> { >> . >>       err = mmc_cache_ctrl(host, 0); >> } >> >> So, within

RE: Cache Control during mmc_suspend_host

2011-12-12 Thread Seungwon Jeon
Hi Saugata, Saugata Das wrote: > Hi Seungwon Jeon > > I see a small issue with the implementation mmc_suspend_host, > > int mmc_suspend_host(struct mmc_host *host) > { > . > err = mmc_cache_ctrl(host, 0); > } > > So, within mmc_suspend_host we are disabling the cache. But I do not > s

Cache Control during mmc_suspend_host

2011-12-12 Thread Saugata Das
Hi Seungwon Jeon I see a small issue with the implementation mmc_suspend_host, int mmc_suspend_host(struct mmc_host *host) { . err = mmc_cache_ctrl(host, 0); } So, within mmc_suspend_host we are disabling the cache. But I do not see a corresponding enabling of cache within mmc_resume