From: Jaehoon Chung <jh80.chung@samsung.com>
To: Philip Rakity <prakity@marvell.com>
Cc: "linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
Chris Ball <cjb@laptop.org>,
Jaehoon Chung <jh80.chung@samsung.com>
Subject: Re: [PATCH V2] mmc: Bus width testing needs to handle suspend/resume
Date: Fri, 08 Jul 2011 10:18:37 +0900 [thread overview]
Message-ID: <4E165AED.6090805@samsung.com> (raw)
In-Reply-To: <0281C9EE-5CAE-41B7-9A8D-4BE2F125EF2F@marvell.com>
Hi Philip.
It's working well on my board.
Tested-by: Jaehoon Chung <jh80.chung@samsung.com>
Best regards,
Jaehoon Chung
Philip Rakity wrote:
> V2
> ==
> Save the ext_csd information needed for bus width
> compare when we read the ext_csd the first time (in 1 bit mode).
>
> On every pass we make re-reading the ext_csd compare
> the data against the saved ext_csd data.
>
> note: unsubmitted diff (with printks) tested by
> Jaehoon Chung <jh80.chung@samsung.com>
>
> V1
> ==
> We need to read the ext_csd when doing bus width testing for
> old cards to ensure test is run correctly.
>
> Signed-off-by: Philip Rakity <prakity@marvell.com>
> ---
> drivers/mmc/core/mmc.c | 75 ++++++++++++++++++++++++++++++----------------
> include/linux/mmc/card.h | 13 ++++++++
> 2 files changed, 62 insertions(+), 26 deletions(-)
>
> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
> index 2a7e43b..afabdc3 100644
> --- a/drivers/mmc/core/mmc.c
> +++ b/drivers/mmc/core/mmc.c
> @@ -247,12 +247,12 @@ static int mmc_read_ext_csd(struct mmc_card *card, u8 *ext_csd)
> return 0;
>
> /* Version is coded in the CSD_STRUCTURE byte in the EXT_CSD register */
> + card->ext_csd.raw_ext_csd_structure = ext_csd[EXT_CSD_STRUCTURE];
> if (card->csd.structure == 3) {
> - int ext_csd_struct = ext_csd[EXT_CSD_STRUCTURE];
> - if (ext_csd_struct > 2) {
> + if (card->ext_csd.raw_ext_csd_structure > 2) {
> printk(KERN_ERR "%s: unrecognised EXT_CSD structure "
> "version %d\n", mmc_hostname(card->host),
> - ext_csd_struct);
> + card->ext_csd.raw_ext_csd_structure);
> err = -EINVAL;
> goto out;
> }
> @@ -266,6 +266,10 @@ static int mmc_read_ext_csd(struct mmc_card *card, u8 *ext_csd)
> goto out;
> }
>
> + card->ext_csd.raw_sectors[0] = ext_csd[EXT_CSD_SEC_CNT + 0];
> + card->ext_csd.raw_sectors[1] = ext_csd[EXT_CSD_SEC_CNT + 1];
> + card->ext_csd.raw_sectors[2] = ext_csd[EXT_CSD_SEC_CNT + 2];
> + card->ext_csd.raw_sectors[3] = ext_csd[EXT_CSD_SEC_CNT + 3];
> if (card->ext_csd.rev >= 2) {
> card->ext_csd.sectors =
> ext_csd[EXT_CSD_SEC_CNT + 0] << 0 |
> @@ -277,7 +281,7 @@ static int mmc_read_ext_csd(struct mmc_card *card, u8 *ext_csd)
> if (card->ext_csd.sectors > (2u * 1024 * 1024 * 1024) / 512)
> mmc_card_set_blockaddr(card);
> }
> -
> + card->ext_csd.raw_card_type = ext_csd[EXT_CSD_CARD_TYPE];
> switch (ext_csd[EXT_CSD_CARD_TYPE] & EXT_CSD_CARD_TYPE_MASK) {
> case EXT_CSD_CARD_TYPE_DDR_52 | EXT_CSD_CARD_TYPE_52 |
> EXT_CSD_CARD_TYPE_26:
> @@ -307,6 +311,11 @@ static int mmc_read_ext_csd(struct mmc_card *card, u8 *ext_csd)
> mmc_hostname(card->host));
> }
>
> + card->ext_csd.raw_s_a_timeout = ext_csd[EXT_CSD_S_A_TIMEOUT];
> + card->ext_csd.raw_erase_timeout_mult =
> + ext_csd[EXT_CSD_ERASE_TIMEOUT_MULT];
> + card->ext_csd.raw_hc_erase_grp_size =
> + ext_csd[EXT_CSD_HC_ERASE_GRP_SIZE];
> if (card->ext_csd.rev >= 3) {
> u8 sa_shift = ext_csd[EXT_CSD_S_A_TIMEOUT];
> card->ext_csd.part_config = ext_csd[EXT_CSD_PART_CONFIG];
> @@ -334,6 +343,16 @@ static int mmc_read_ext_csd(struct mmc_card *card, u8 *ext_csd)
> card->ext_csd.boot_size = ext_csd[EXT_CSD_BOOT_MULT] << 17;
> }
>
> + card->ext_csd.raw_hc_erase_gap_size =
> + ext_csd[EXT_CSD_PARTITION_ATTRIBUTE];
> + card->ext_csd.raw_sec_trim_mult =
> + ext_csd[EXT_CSD_SEC_TRIM_MULT];
> + card->ext_csd.raw_sec_erase_mult =
> + ext_csd[EXT_CSD_SEC_ERASE_MULT];
> + card->ext_csd.raw_sec_feature_support =
> + ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT];
> + card->ext_csd.raw_trim_mult =
> + ext_csd[EXT_CSD_TRIM_MULT];
> if (card->ext_csd.rev >= 4) {
> /*
> * Enhanced area feature support -- check whether the eMMC
> @@ -401,17 +420,17 @@ static inline void mmc_free_ext_csd(u8 *ext_csd)
> }
>
>
> -static int mmc_compare_ext_csds(struct mmc_card *card, u8 *ext_csd,
> - unsigned bus_width)
> +static int mmc_compare_ext_csds(struct mmc_card *card, unsigned bus_width)
> {
> u8 *bw_ext_csd;
> int err;
>
> + if (bus_width == MMC_BUS_WIDTH_1)
> + return 0;
> +
> err = mmc_get_ext_csd(card, &bw_ext_csd);
> - if (err)
> - return err;
>
> - if ((ext_csd == NULL || bw_ext_csd == NULL)) {
> + if (err || bw_ext_csd == NULL) {
> if (bus_width != MMC_BUS_WIDTH_1)
> err = -EINVAL;
> goto out;
> @@ -421,35 +440,40 @@ static int mmc_compare_ext_csds(struct mmc_card *card, u8 *ext_csd,
> goto out;
>
> /* only compare read only fields */
> - err = (!(ext_csd[EXT_CSD_PARTITION_SUPPORT] ==
> + err = (!(card->ext_csd.raw_partition_support ==
> bw_ext_csd[EXT_CSD_PARTITION_SUPPORT]) &&
> - (ext_csd[EXT_CSD_ERASED_MEM_CONT] ==
> + (card->ext_csd.raw_erased_mem_count ==
> bw_ext_csd[EXT_CSD_ERASED_MEM_CONT]) &&
> - (ext_csd[EXT_CSD_REV] ==
> + (card->ext_csd.rev ==
> bw_ext_csd[EXT_CSD_REV]) &&
> - (ext_csd[EXT_CSD_STRUCTURE] ==
> + (card->ext_csd.raw_ext_csd_structure ==
> bw_ext_csd[EXT_CSD_STRUCTURE]) &&
> - (ext_csd[EXT_CSD_CARD_TYPE] ==
> + (card->ext_csd.raw_card_type ==
> bw_ext_csd[EXT_CSD_CARD_TYPE]) &&
> - (ext_csd[EXT_CSD_S_A_TIMEOUT] ==
> + (card->ext_csd.raw_s_a_timeout ==
> bw_ext_csd[EXT_CSD_S_A_TIMEOUT]) &&
> - (ext_csd[EXT_CSD_HC_WP_GRP_SIZE] ==
> + (card->ext_csd.raw_hc_erase_gap_size ==
> bw_ext_csd[EXT_CSD_HC_WP_GRP_SIZE]) &&
> - (ext_csd[EXT_CSD_ERASE_TIMEOUT_MULT] ==
> + (card->ext_csd.raw_erase_timeout_mult ==
> bw_ext_csd[EXT_CSD_ERASE_TIMEOUT_MULT]) &&
> - (ext_csd[EXT_CSD_HC_ERASE_GRP_SIZE] ==
> + (card->ext_csd.raw_hc_erase_grp_size ==
> bw_ext_csd[EXT_CSD_HC_ERASE_GRP_SIZE]) &&
> - (ext_csd[EXT_CSD_SEC_TRIM_MULT] ==
> + (card->ext_csd.raw_sec_trim_mult ==
> bw_ext_csd[EXT_CSD_SEC_TRIM_MULT]) &&
> - (ext_csd[EXT_CSD_SEC_ERASE_MULT] ==
> + (card->ext_csd.raw_sec_erase_mult ==
> bw_ext_csd[EXT_CSD_SEC_ERASE_MULT]) &&
> - (ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT] ==
> + (card->ext_csd.raw_sec_feature_support ==
> bw_ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT]) &&
> - (ext_csd[EXT_CSD_TRIM_MULT] ==
> + (card->ext_csd.raw_trim_mult ==
> bw_ext_csd[EXT_CSD_TRIM_MULT]) &&
> - memcmp(&ext_csd[EXT_CSD_SEC_CNT],
> - &bw_ext_csd[EXT_CSD_SEC_CNT],
> - 4) != 0);
> + (card->ext_csd.raw_sectors[0] ==
> + bw_ext_csd[EXT_CSD_SEC_CNT + 0]) &&
> + (card->ext_csd.raw_sectors[1] ==
> + bw_ext_csd[EXT_CSD_SEC_CNT + 1]) &&
> + (card->ext_csd.raw_sectors[2] ==
> + bw_ext_csd[EXT_CSD_SEC_CNT + 2]) &&
> + (card->ext_csd.raw_sectors[3] ==
> + bw_ext_csd[EXT_CSD_SEC_CNT + 3]));
> if (err)
> err = -EINVAL;
>
> @@ -770,7 +794,6 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
> */
> if (!(host->caps & MMC_CAP_BUS_WIDTH_TEST))
> err = mmc_compare_ext_csds(card,
> - ext_csd,
> bus_width);
> else
> err = mmc_bus_test(card, bus_width);
> diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
> index 7b4fd7b..b460fc2 100644
> --- a/include/linux/mmc/card.h
> +++ b/include/linux/mmc/card.h
> @@ -64,6 +64,19 @@ struct mmc_ext_csd {
> unsigned long long enhanced_area_offset; /* Units: Byte */
> unsigned int enhanced_area_size; /* Units: KB */
> unsigned int boot_size; /* in bytes */
> + u8 raw_partition_support; /* 160 */
> + u8 raw_erased_mem_count; /* 181 */
> + u8 raw_ext_csd_structure; /* 194 */
> + u8 raw_card_type; /* 196 */
> + u8 raw_s_a_timeout; /* 217 */
> + u8 raw_hc_erase_gap_size; /* 221 */
> + u8 raw_erase_timeout_mult; /* 223 */
> + u8 raw_hc_erase_grp_size; /* 224 */
> + u8 raw_sec_trim_mult; /* 229 */
> + u8 raw_sec_erase_mult; /* 230 */
> + u8 raw_sec_feature_support;/* 231 */
> + u8 raw_trim_mult; /* 232 */
> + u8 raw_sectors[4]; /* 212 - 4 bytes */
> };
>
> struct sd_scr {
next prev parent reply other threads:[~2011-07-08 1:18 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-07 16:04 [PATCH V2] mmc: Bus width testing needs to handle suspend/resume Philip Rakity
2011-07-08 1:18 ` Jaehoon Chung [this message]
2011-07-09 22:20 ` Chris Ball
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=4E165AED.6090805@samsung.com \
--to=jh80.chung@samsung.com \
--cc=cjb@laptop.org \
--cc=linux-mmc@vger.kernel.org \
--cc=prakity@marvell.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox