All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jaehoon Chung <jh80.chung@samsung.com>
To: 장민우 <huskey27@gmail.com>, linux-mmc@vger.kernel.org
Subject: Re: use default speed of the eMMC
Date: Tue, 24 May 2016 18:41:34 +0900	[thread overview]
Message-ID: <574421CE.4080503@samsung.com> (raw)
In-Reply-To: <CA+pahgdUP3Zam-cSeW8fJR9AmiUo2_Ok_uDHqWTgwvzCGC58+g@mail.gmail.com>

Hi,

On 05/24/2016 06:07 PM, 장민우 wrote:
> Dear linux-mmc.
> 
> Hello, I'm Minwoo Jang.
> 
> I have a question about using default speed of the eMMC.
> 
> When default speed is used, mmc_select_bus_width() is never called.
> 
> So, eMMC can not be set 4 bit or 8 bit bus width, I think.
> 
> Please, give me your opinions on the following diff codes.

In mmc_select_timing(), timing should be set to MMC_TIMING_HS or others.
Then mmc_select_bus_width() should be entered.

Best Regards,
Jaehoon Chung

> 
> Thank you.
> 
> =====================================================
> 
> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
> index 94b4462..3b1cc4d 100644
> --- a/drivers/mmc/core/mmc.c
> +++ b/drivers/mmc/core/mmc.c
> @@ -952,6 +952,8 @@ static int mmc_select_bus_width(struct mmc_card *card)
> 
>                 if (!err) {
>                         err = bus_width;
> +                       pr_warn("%s: switch to bus width %d\n",
> +                               mmc_hostname(host), (1 << bus_width));
>                         break;
>                 } else {
>                         pr_warn("%s: switch to bus width %d failed\n",
> @@ -1500,6 +1502,14 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
>                         if (err)
>                                 goto err;
>                 }
> +       } else {
> +               /* Select the bus width for normal speed mode */
> +               err = mmc_select_bus_width(card);
> +               if (IS_ERR_VALUE(err)) {
> +                       pr_warn("%s: Selecting bus width failed\n",
> +                               mmc_hostname(card->host));
> +                       goto err;
> +               }
>         }
> 
> Best regards,
> 
> MW Jang.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
> 


  reply	other threads:[~2016-05-24  9:41 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20160524090753epcas1p125c06a036988b476840c79048cb45428@epcas1p1.samsung.com>
2016-05-24  9:07 ` use default speed of the eMMC 장민우
2016-05-24  9:41   ` Jaehoon Chung [this message]
2016-05-26  5:16     ` 장민우
2016-05-26  5:49       ` Jaehoon Chung
2016-05-26  6:34         ` 장민우
2016-05-26  8:30           ` Jaehoon Chung
2016-05-26  9:04             ` 장민우
2016-05-26  9:45               ` Jaehoon Chung
2016-05-24  9:05 장민우

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=574421CE.4080503@samsung.com \
    --to=jh80.chung@samsung.com \
    --cc=huskey27@gmail.com \
    --cc=linux-mmc@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.