RE: [PATCH 2/2] mmc: core: Add poweroff notify handling feature

2011-09-06 Thread Seungwon Jeon
Hi Girish K S, I think short type is proper rather than long type by default. Long type seems to be unacceptable in suspend regarding long timeout. One question, Is this patch considered for system power off besides suspend? Beset regards, Seungwon Jeon. Girish K S wrote: Hi Mr Park, On 5

Re: [PATCH 2/2] mmc: core: Add poweroff notify handling feature

2011-09-06 Thread Girish K S
Hello Mr Jeon, i have modified the patch to handle the poweroff in suspend and normal condition. during suspend it is be short and during normal poweroff it is long. I will update the V2 patch with Mr Parks changes updated. regards Girish K S On 6 September 2011 12:31, Seungwon Jeon

Re: [PATCH 2/2] mmc: core: Add poweroff notify handling feature

2011-09-05 Thread Kyungmin Park
Hi Girish, I think it's still incomplete, does power off short function is called at suspend properly? there are some comments below. Thank you, Kyungmin Park On Mon, Sep 5, 2011 at 8:49 PM, Girish K S girish.shivananja...@linaro.org wrote: This patch adds the handling of the poweroff notify

Re: [PATCH 2/2] mmc: core: Add poweroff notify handling feature

2011-09-05 Thread Girish K S
Hi Mr Park, On 5 September 2011 18:03, Kyungmin Park kmp...@infradead.org wrote: Hi Girish, I think it's still incomplete, does power off short function is called at suspend properly? there are some comments below. Thank you, Kyungmin Park On Mon, Sep 5, 2011 at 8:49 PM, Girish K S