Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-27 Thread Mark Brown
On Fri, Sep 27, 2013 at 11:56:26AM +0200, Ulf Hansson wrote: > To go forward, I suppose our discussion shall not prevent Chris from > merging this patchset? We shall be able to address your input as work > on top if this patchset, right? To be honest I don't think I saw the original patches so I

Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-27 Thread Ulf Hansson
On 17 September 2013 14:50, Ulf Hansson wrote: > On 17 September 2013 13:04, Mark Brown wrote: >> On Tue, Sep 17, 2013 at 11:29:21AM +0200, Ulf Hansson wrote: >> >>> If you want to change voltage level for eMMC, both VCC and VCCQ must >>> be possible to power cycle, unless you have a hw-reset wir

Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-17 Thread Ulf Hansson
On 17 September 2013 13:04, Mark Brown wrote: > On Tue, Sep 17, 2013 at 11:29:21AM +0200, Ulf Hansson wrote: > >> If you want to change voltage level for eMMC, both VCC and VCCQ must >> be possible to power cycle, unless you have a hw-reset wire connected >> to the eMMC which is implemented by a h

Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-17 Thread Mark Brown
On Tue, Sep 17, 2013 at 11:29:21AM +0200, Ulf Hansson wrote: > If you want to change voltage level for eMMC, both VCC and VCCQ must > be possible to power cycle, unless you have a hw-reset wire connected > to the eMMC which is implemented by a host->ops callback. This > complicates things for SD/S

Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-17 Thread Mark Brown
On Tue, Sep 17, 2013 at 10:16:06AM +0200, Guennadi Liakhovetski wrote: *Please* remember to trim context when replying. > On Tue, 17 Sep 2013, Ulf Hansson wrote: > > > In one of my cases I am also using a regulator, but from a board > > configuration point of view I know I am the only user on th

Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-17 Thread Ulf Hansson
On 17 September 2013 10:16, Guennadi Liakhovetski wrote: > (adding Mark to cc) > > On Tue, 17 Sep 2013, Ulf Hansson wrote: > >> On 16 September 2013 22:57, Guennadi Liakhovetski >> wrote: >> > Hi Ulf >> > >> > On Mon, 16 Sep 2013, Ulf Hansson wrote: >> > >> >> According to the eMMC/SD/SDIO specs

Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-17 Thread Guennadi Liakhovetski
(adding Mark to cc) On Tue, 17 Sep 2013, Ulf Hansson wrote: > On 16 September 2013 22:57, Guennadi Liakhovetski > wrote: > > Hi Ulf > > > > On Mon, 16 Sep 2013, Ulf Hansson wrote: > > > >> According to the eMMC/SD/SDIO specs the VDD voltage level must not be > >> changed > >> during the initia

Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-17 Thread Ulf Hansson
On 16 September 2013 22:57, Guennadi Liakhovetski wrote: > Hi Ulf > > On Mon, 16 Sep 2013, Ulf Hansson wrote: > >> According to the eMMC/SD/SDIO specs the VDD voltage level must not be changed >> during the initialization, without a complete power cycle of the card. >> >> Before this patchset, som

Re: [PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-16 Thread Guennadi Liakhovetski
Hi Ulf On Mon, 16 Sep 2013, Ulf Hansson wrote: > According to the eMMC/SD/SDIO specs the VDD voltage level must not be changed > during the initialization, without a complete power cycle of the card. > > Before this patchset, some host drivers were trying to change voltage level > at MMC_POWER_O

[PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-16 Thread Ulf Hansson
According to the eMMC/SD/SDIO specs the VDD voltage level must not be changed during the initialization, without a complete power cycle of the card. Before this patchset, some host drivers were trying to change voltage level at MMC_POWER_ON state, which is also what the protocol layer advised them

[PATCH 0/7] mmc: core: Fixup ocr mask setup to prevent spec violation

2013-09-16 Thread Ulf Hansson
According to the eMMC/SD/SDIO specs the VDD voltage level must not be changed during the initialization, without a complete power cycle of the card. Before this patchset, some host drivers were trying to change voltage level at MMC_POWER_ON state, which is also what the protocol layer advised them