public inbox for linux-mmc@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH RFT] mmc: wmt-sdmmc: Fix setting BM_EIGHTBIT_MODE bit in wmt_mci_set_ios()
@ 2013-10-22  8:54 Axel Lin
  2013-10-23  4:51 ` Tony Prisk
  0 siblings, 1 reply; 4+ messages in thread
From: Axel Lin @ 2013-10-22  8:54 UTC (permalink / raw)
  To: Chris Ball; +Cc: Tony Prisk, linux-mmc

For MMC_BUS_WIDTH_8 case, current code missed setting BM_EIGHTBIT_MODE bit.
Also has a small refactor to make the code looks better in readability.

So the bit settings witch below logic:

SDMMC_BUSMODE register:
Set EIGHTBIT_MODE bit for 8 bit mode, Set FOURBIT_MODE bit for 4 bit mode.
Clear both EIGHTBIT_MODE and FOURBIT_MODE bits for 1 bit mode.

SDMMC_EXTCTRL register:
Set EXT_EIGHTBIT bit for 8 bit mode, Clear EXT_EIGHTBIT bit for 1/4 bit mode.

Add define for EXT_EIGHTBIT to avoid using magic number.
BM_ONEBIT_MASK is no longer used, thus remove it.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
Hi Tony,
After checking wm8505+sdmmc+controller+suppliment.pdf, I'm wondering if this
dirver works for MMC_BUS_WIDTH_8 case without setting BM_EIGHTBIT_MODE bit.
I don't have this h/w to test.
I'd appreciate if you can review and test if this patch works.
Thanks,
Axel
 drivers/mmc/host/wmt-sdmmc.c | 31 +++++++++++++++----------------
 1 file changed, 15 insertions(+), 16 deletions(-)

diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c
index 8a48a50..6ed18f8 100644
--- a/drivers/mmc/host/wmt-sdmmc.c
+++ b/drivers/mmc/host/wmt-sdmmc.c
@@ -72,7 +72,6 @@
 #define BM_SPI_CS			0x20
 #define BM_SD_POWER			0x40
 #define BM_SOFT_RESET			0x80
-#define BM_ONEBIT_MASK			0xFD
 
 /* SDMMC_BLKLEN bit fields */
 #define BLKL_CRCERR_ABORT		0x0800
@@ -120,6 +119,8 @@
 #define STS2_DATARSP_BUSY		0x20
 #define STS2_DIS_FORCECLK		0x80
 
+/* SDMMC_EXTCTRL bit fields */
+#define EXT_EIGHTBIT			0x04
 
 /* MMC/SD DMA Controller Registers */
 #define SDDMA_GCR			0x100
@@ -672,7 +673,7 @@ static void wmt_mci_request(struct mmc_host *mmc, struct mmc_request *req)
 static void wmt_mci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
 {
 	struct wmt_mci_priv *priv;
-	u32 reg_tmp;
+	u32 busmode, extctrl;
 
 	priv = mmc_priv(mmc);
 
@@ -687,28 +688,26 @@ static void wmt_mci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
 	if (ios->clock != 0)
 		clk_set_rate(priv->clk_sdmmc, ios->clock);
 
+	busmode = readb(priv->sdmmc_base + SDMMC_BUSMODE);
+	extctrl = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
+
+	busmode &= ~(BM_EIGHTBIT_MODE | BM_FOURBIT_MODE);
+	extctrl &= ~EXT_EIGHTBIT;
+
 	switch (ios->bus_width) {
 	case MMC_BUS_WIDTH_8:
-		reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
-		writeb(reg_tmp | 0x04, priv->sdmmc_base + SDMMC_EXTCTRL);
+		busmode |= BM_EIGHTBIT_MODE;
+		extctrl |= EXT_EIGHTBIT;
 		break;
 	case MMC_BUS_WIDTH_4:
-		reg_tmp = readb(priv->sdmmc_base + SDMMC_BUSMODE);
-		writeb(reg_tmp | BM_FOURBIT_MODE, priv->sdmmc_base +
-		       SDMMC_BUSMODE);
-
-		reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
-		writeb(reg_tmp & 0xFB, priv->sdmmc_base + SDMMC_EXTCTRL);
+		busmode |= BM_FOURBIT_MODE;
 		break;
 	case MMC_BUS_WIDTH_1:
-		reg_tmp = readb(priv->sdmmc_base + SDMMC_BUSMODE);
-		writeb(reg_tmp & BM_ONEBIT_MASK, priv->sdmmc_base +
-		       SDMMC_BUSMODE);
-
-		reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
-		writeb(reg_tmp & 0xFB, priv->sdmmc_base + SDMMC_EXTCTRL);
 		break;
 	}
+
+	writeb(busmode, priv->sdmmc_base + SDMMC_BUSMODE);
+	writeb(extctrl, priv->sdmmc_base + SDMMC_EXTCTRL);
 }
 
 static int wmt_mci_get_ro(struct mmc_host *mmc)
-- 
1.8.1.2




^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH RFT] mmc: wmt-sdmmc: Fix setting BM_EIGHTBIT_MODE bit in wmt_mci_set_ios()
  2013-10-22  8:54 [PATCH RFT] mmc: wmt-sdmmc: Fix setting BM_EIGHTBIT_MODE bit in wmt_mci_set_ios() Axel Lin
@ 2013-10-23  4:51 ` Tony Prisk
  2013-10-23  6:48   ` Axel Lin
  0 siblings, 1 reply; 4+ messages in thread
From: Tony Prisk @ 2013-10-23  4:51 UTC (permalink / raw)
  To: Axel Lin, Chris Ball; +Cc: linux-mmc

On 22/10/13 21:54, Axel Lin wrote:
> For MMC_BUS_WIDTH_8 case, current code missed setting BM_EIGHTBIT_MODE bit.
> Also has a small refactor to make the code looks better in readability.
>
> So the bit settings witch below logic:
>
> SDMMC_BUSMODE register:
> Set EIGHTBIT_MODE bit for 8 bit mode, Set FOURBIT_MODE bit for 4 bit mode.
> Clear both EIGHTBIT_MODE and FOURBIT_MODE bits for 1 bit mode.
>
> SDMMC_EXTCTRL register:
> Set EXT_EIGHTBIT bit for 8 bit mode, Clear EXT_EIGHTBIT bit for 1/4 bit mode.
>
> Add define for EXT_EIGHTBIT to avoid using magic number.
> BM_ONEBIT_MASK is no longer used, thus remove it.
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---
> Hi Tony,
> After checking wm8505+sdmmc+controller+suppliment.pdf, I'm wondering if this
> dirver works for MMC_BUS_WIDTH_8 case without setting BM_EIGHTBIT_MODE bit.
> I don't have this h/w to test.
> I'd appreciate if you can review and test if this patch works.
> Thanks,
> Axel
>   drivers/mmc/host/wmt-sdmmc.c | 31 +++++++++++++++----------------
>   1 file changed, 15 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c
> index 8a48a50..6ed18f8 100644
> --- a/drivers/mmc/host/wmt-sdmmc.c
> +++ b/drivers/mmc/host/wmt-sdmmc.c
> @@ -72,7 +72,6 @@
>   #define BM_SPI_CS			0x20
>   #define BM_SD_POWER			0x40
>   #define BM_SOFT_RESET			0x80
> -#define BM_ONEBIT_MASK			0xFD
>   
>   /* SDMMC_BLKLEN bit fields */
>   #define BLKL_CRCERR_ABORT		0x0800
> @@ -120,6 +119,8 @@
>   #define STS2_DATARSP_BUSY		0x20
>   #define STS2_DIS_FORCECLK		0x80
>   
> +/* SDMMC_EXTCTRL bit fields */
> +#define EXT_EIGHTBIT			0x04
>   
>   /* MMC/SD DMA Controller Registers */
>   #define SDDMA_GCR			0x100
> @@ -672,7 +673,7 @@ static void wmt_mci_request(struct mmc_host *mmc, struct mmc_request *req)
>   static void wmt_mci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
>   {
>   	struct wmt_mci_priv *priv;
> -	u32 reg_tmp;
> +	u32 busmode, extctrl;
>   
>   	priv = mmc_priv(mmc);
>   
> @@ -687,28 +688,26 @@ static void wmt_mci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
>   	if (ios->clock != 0)
>   		clk_set_rate(priv->clk_sdmmc, ios->clock);
>   
> +	busmode = readb(priv->sdmmc_base + SDMMC_BUSMODE);
> +	extctrl = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
> +
> +	busmode &= ~(BM_EIGHTBIT_MODE | BM_FOURBIT_MODE);
> +	extctrl &= ~EXT_EIGHTBIT;
> +
>   	switch (ios->bus_width) {
>   	case MMC_BUS_WIDTH_8:
> -		reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
> -		writeb(reg_tmp | 0x04, priv->sdmmc_base + SDMMC_EXTCTRL);
> +		busmode |= BM_EIGHTBIT_MODE;
> +		extctrl |= EXT_EIGHTBIT;
>   		break;
>   	case MMC_BUS_WIDTH_4:
> -		reg_tmp = readb(priv->sdmmc_base + SDMMC_BUSMODE);
> -		writeb(reg_tmp | BM_FOURBIT_MODE, priv->sdmmc_base +
> -		       SDMMC_BUSMODE);
> -
> -		reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
> -		writeb(reg_tmp & 0xFB, priv->sdmmc_base + SDMMC_EXTCTRL);
> +		busmode |= BM_FOURBIT_MODE;
>   		break;
>   	case MMC_BUS_WIDTH_1:
> -		reg_tmp = readb(priv->sdmmc_base + SDMMC_BUSMODE);
> -		writeb(reg_tmp & BM_ONEBIT_MASK, priv->sdmmc_base +
> -		       SDMMC_BUSMODE);
> -
> -		reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
> -		writeb(reg_tmp & 0xFB, priv->sdmmc_base + SDMMC_EXTCTRL);
>   		break;
>   	}
> +
> +	writeb(busmode, priv->sdmmc_base + SDMMC_BUSMODE);
> +	writeb(extctrl, priv->sdmmc_base + SDMMC_EXTCTRL);
>   }
>   
>   static int wmt_mci_get_ro(struct mmc_host *mmc)
I don't have any means of testing this - All the vendor hardware I have 
uses 4-bit mode.

Regards
Tony Prisk

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH RFT] mmc: wmt-sdmmc: Fix setting BM_EIGHTBIT_MODE bit in wmt_mci_set_ios()
  2013-10-23  4:51 ` Tony Prisk
@ 2013-10-23  6:48   ` Axel Lin
  2013-10-23 18:10     ` Tony Prisk
  0 siblings, 1 reply; 4+ messages in thread
From: Axel Lin @ 2013-10-23  6:48 UTC (permalink / raw)
  To: Tony Prisk; +Cc: Chris Ball, linux-mmc

2013/10/23 Tony Prisk <linux@prisktech.co.nz>:
> On 22/10/13 21:54, Axel Lin wrote:
>>
>> For MMC_BUS_WIDTH_8 case, current code missed setting BM_EIGHTBIT_MODE
>> bit.
>> Also has a small refactor to make the code looks better in readability.
>>
>> So the bit settings witch below logic:
>>
>> SDMMC_BUSMODE register:
>> Set EIGHTBIT_MODE bit for 8 bit mode, Set FOURBIT_MODE bit for 4 bit mode.
>> Clear both EIGHTBIT_MODE and FOURBIT_MODE bits for 1 bit mode.
>>
>> SDMMC_EXTCTRL register:
>> Set EXT_EIGHTBIT bit for 8 bit mode, Clear EXT_EIGHTBIT bit for 1/4 bit
>> mode.
>>
>> Add define for EXT_EIGHTBIT to avoid using magic number.
>> BM_ONEBIT_MASK is no longer used, thus remove it.
>>
>> Signed-off-by: Axel Lin <axel.lin@ingics.com>
>> ---
>> Hi Tony,
>> After checking wm8505+sdmmc+controller+suppliment.pdf, I'm wondering if
>> this
>> dirver works for MMC_BUS_WIDTH_8 case without setting BM_EIGHTBIT_MODE
>> bit.
>> I don't have this h/w to test.
>> I'd appreciate if you can review and test if this patch works.
>> Thanks,
>> Axel
>>   drivers/mmc/host/wmt-sdmmc.c | 31 +++++++++++++++----------------
>>   1 file changed, 15 insertions(+), 16 deletions(-)
>>
>> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c
>> index 8a48a50..6ed18f8 100644
>> --- a/drivers/mmc/host/wmt-sdmmc.c
>> +++ b/drivers/mmc/host/wmt-sdmmc.c
>> @@ -72,7 +72,6 @@
>>   #define BM_SPI_CS                     0x20
>>   #define BM_SD_POWER                   0x40
>>   #define BM_SOFT_RESET                 0x80
>> -#define BM_ONEBIT_MASK                 0xFD
>>     /* SDMMC_BLKLEN bit fields */
>>   #define BLKL_CRCERR_ABORT             0x0800
>> @@ -120,6 +119,8 @@
>>   #define STS2_DATARSP_BUSY             0x20
>>   #define STS2_DIS_FORCECLK             0x80
>>   +/* SDMMC_EXTCTRL bit fields */
>> +#define EXT_EIGHTBIT                   0x04
>>     /* MMC/SD DMA Controller Registers */
>>   #define SDDMA_GCR                     0x100
>> @@ -672,7 +673,7 @@ static void wmt_mci_request(struct mmc_host *mmc,
>> struct mmc_request *req)
>>   static void wmt_mci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
>>   {
>>         struct wmt_mci_priv *priv;
>> -       u32 reg_tmp;
>> +       u32 busmode, extctrl;
>>         priv = mmc_priv(mmc);
>>   @@ -687,28 +688,26 @@ static void wmt_mci_set_ios(struct mmc_host *mmc,
>> struct mmc_ios *ios)
>>         if (ios->clock != 0)
>>                 clk_set_rate(priv->clk_sdmmc, ios->clock);
>>   +     busmode = readb(priv->sdmmc_base + SDMMC_BUSMODE);
>> +       extctrl = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
>> +
>> +       busmode &= ~(BM_EIGHTBIT_MODE | BM_FOURBIT_MODE);
>> +       extctrl &= ~EXT_EIGHTBIT;
>> +
>>         switch (ios->bus_width) {
>>         case MMC_BUS_WIDTH_8:
>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
>> -               writeb(reg_tmp | 0x04, priv->sdmmc_base + SDMMC_EXTCTRL);
>> +               busmode |= BM_EIGHTBIT_MODE;
>> +               extctrl |= EXT_EIGHTBIT;
>>                 break;
>>         case MMC_BUS_WIDTH_4:
>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_BUSMODE);
>> -               writeb(reg_tmp | BM_FOURBIT_MODE, priv->sdmmc_base +
>> -                      SDMMC_BUSMODE);
>> -
>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
>> -               writeb(reg_tmp & 0xFB, priv->sdmmc_base + SDMMC_EXTCTRL);
>> +               busmode |= BM_FOURBIT_MODE;
>>                 break;
>>         case MMC_BUS_WIDTH_1:
>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_BUSMODE);
>> -               writeb(reg_tmp & BM_ONEBIT_MASK, priv->sdmmc_base +
>> -                      SDMMC_BUSMODE);
>> -
>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
>> -               writeb(reg_tmp & 0xFB, priv->sdmmc_base + SDMMC_EXTCTRL);
>>                 break;
>>         }
>> +
>> +       writeb(busmode, priv->sdmmc_base + SDMMC_BUSMODE);
>> +       writeb(extctrl, priv->sdmmc_base + SDMMC_EXTCTRL);
>>   }
>>     static int wmt_mci_get_ro(struct mmc_host *mmc)
>
> I don't have any means of testing this - All the vendor hardware I have uses
> 4-bit mode.
Oh, well.
But how do you think about this patch?
Since the code is there, and it looks wrong because
wm8505+sdmmc+controller+suppliment.pdf says

SDMMC Ext Control (Offset 0x0034)
BIT2 RW
        0: One Bit / Four bit mode
        1: Eight bit mode

Probably still good to make the code matches the document.

Regards,
Axel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH RFT] mmc: wmt-sdmmc: Fix setting BM_EIGHTBIT_MODE bit in wmt_mci_set_ios()
  2013-10-23  6:48   ` Axel Lin
@ 2013-10-23 18:10     ` Tony Prisk
  0 siblings, 0 replies; 4+ messages in thread
From: Tony Prisk @ 2013-10-23 18:10 UTC (permalink / raw)
  To: Axel Lin; +Cc: Chris Ball, linux-mmc

On 23/10/13 19:48, Axel Lin wrote:
> 2013/10/23 Tony Prisk <linux@prisktech.co.nz>:
>> On 22/10/13 21:54, Axel Lin wrote:
>>> For MMC_BUS_WIDTH_8 case, current code missed setting BM_EIGHTBIT_MODE
>>> bit.
>>> Also has a small refactor to make the code looks better in readability.
>>>
>>> So the bit settings witch below logic:
>>>
>>> SDMMC_BUSMODE register:
>>> Set EIGHTBIT_MODE bit for 8 bit mode, Set FOURBIT_MODE bit for 4 bit mode.
>>> Clear both EIGHTBIT_MODE and FOURBIT_MODE bits for 1 bit mode.
>>>
>>> SDMMC_EXTCTRL register:
>>> Set EXT_EIGHTBIT bit for 8 bit mode, Clear EXT_EIGHTBIT bit for 1/4 bit
>>> mode.
>>>
>>> Add define for EXT_EIGHTBIT to avoid using magic number.
>>> BM_ONEBIT_MASK is no longer used, thus remove it.
>>>
>>> Signed-off-by: Axel Lin <axel.lin@ingics.com>
>>> ---
>>> Hi Tony,
>>> After checking wm8505+sdmmc+controller+suppliment.pdf, I'm wondering if
>>> this
>>> dirver works for MMC_BUS_WIDTH_8 case without setting BM_EIGHTBIT_MODE
>>> bit.
>>> I don't have this h/w to test.
>>> I'd appreciate if you can review and test if this patch works.
>>> Thanks,
>>> Axel
>>>    drivers/mmc/host/wmt-sdmmc.c | 31 +++++++++++++++----------------
>>>    1 file changed, 15 insertions(+), 16 deletions(-)
>>>
>>> diff --git a/drivers/mmc/host/wmt-sdmmc.c b/drivers/mmc/host/wmt-sdmmc.c
>>> index 8a48a50..6ed18f8 100644
>>> --- a/drivers/mmc/host/wmt-sdmmc.c
>>> +++ b/drivers/mmc/host/wmt-sdmmc.c
>>> @@ -72,7 +72,6 @@
>>>    #define BM_SPI_CS                     0x20
>>>    #define BM_SD_POWER                   0x40
>>>    #define BM_SOFT_RESET                 0x80
>>> -#define BM_ONEBIT_MASK                 0xFD
>>>      /* SDMMC_BLKLEN bit fields */
>>>    #define BLKL_CRCERR_ABORT             0x0800
>>> @@ -120,6 +119,8 @@
>>>    #define STS2_DATARSP_BUSY             0x20
>>>    #define STS2_DIS_FORCECLK             0x80
>>>    +/* SDMMC_EXTCTRL bit fields */
>>> +#define EXT_EIGHTBIT                   0x04
>>>      /* MMC/SD DMA Controller Registers */
>>>    #define SDDMA_GCR                     0x100
>>> @@ -672,7 +673,7 @@ static void wmt_mci_request(struct mmc_host *mmc,
>>> struct mmc_request *req)
>>>    static void wmt_mci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
>>>    {
>>>          struct wmt_mci_priv *priv;
>>> -       u32 reg_tmp;
>>> +       u32 busmode, extctrl;
>>>          priv = mmc_priv(mmc);
>>>    @@ -687,28 +688,26 @@ static void wmt_mci_set_ios(struct mmc_host *mmc,
>>> struct mmc_ios *ios)
>>>          if (ios->clock != 0)
>>>                  clk_set_rate(priv->clk_sdmmc, ios->clock);
>>>    +     busmode = readb(priv->sdmmc_base + SDMMC_BUSMODE);
>>> +       extctrl = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
>>> +
>>> +       busmode &= ~(BM_EIGHTBIT_MODE | BM_FOURBIT_MODE);
>>> +       extctrl &= ~EXT_EIGHTBIT;
>>> +
>>>          switch (ios->bus_width) {
>>>          case MMC_BUS_WIDTH_8:
>>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
>>> -               writeb(reg_tmp | 0x04, priv->sdmmc_base + SDMMC_EXTCTRL);
>>> +               busmode |= BM_EIGHTBIT_MODE;
>>> +               extctrl |= EXT_EIGHTBIT;
>>>                  break;
>>>          case MMC_BUS_WIDTH_4:
>>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_BUSMODE);
>>> -               writeb(reg_tmp | BM_FOURBIT_MODE, priv->sdmmc_base +
>>> -                      SDMMC_BUSMODE);
>>> -
>>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
>>> -               writeb(reg_tmp & 0xFB, priv->sdmmc_base + SDMMC_EXTCTRL);
>>> +               busmode |= BM_FOURBIT_MODE;
>>>                  break;
>>>          case MMC_BUS_WIDTH_1:
>>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_BUSMODE);
>>> -               writeb(reg_tmp & BM_ONEBIT_MASK, priv->sdmmc_base +
>>> -                      SDMMC_BUSMODE);
>>> -
>>> -               reg_tmp = readb(priv->sdmmc_base + SDMMC_EXTCTRL);
>>> -               writeb(reg_tmp & 0xFB, priv->sdmmc_base + SDMMC_EXTCTRL);
>>>                  break;
>>>          }
>>> +
>>> +       writeb(busmode, priv->sdmmc_base + SDMMC_BUSMODE);
>>> +       writeb(extctrl, priv->sdmmc_base + SDMMC_EXTCTRL);
>>>    }
>>>      static int wmt_mci_get_ro(struct mmc_host *mmc)
>> I don't have any means of testing this - All the vendor hardware I have uses
>> 4-bit mode.
> Oh, well.
> But how do you think about this patch?
> Since the code is there, and it looks wrong because
> wm8505+sdmmc+controller+suppliment.pdf says
>
> SDMMC Ext Control (Offset 0x0034)
> BIT2 RW
>          0: One Bit / Four bit mode
>          1: Eight bit mode
>
> Probably still good to make the code matches the document.
>
> Regards,
> Axel
I have no issue with the patch - it seems to make sense in relation to 
the documentation.
I would however suggest that given it is untested, that you perhaps add 
a note to the header to indicate it as such so no one gets confused down 
the track if/when there is 8-bit hardware.

Regards
Tony Prisk

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-10-23 18:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-22  8:54 [PATCH RFT] mmc: wmt-sdmmc: Fix setting BM_EIGHTBIT_MODE bit in wmt_mci_set_ios() Axel Lin
2013-10-23  4:51 ` Tony Prisk
2013-10-23  6:48   ` Axel Lin
2013-10-23 18:10     ` Tony Prisk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox