From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754725Ab0LIGLy (ORCPT ); Thu, 9 Dec 2010 01:11:54 -0500 Received: from mga02.intel.com ([134.134.136.20]:54911 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754661Ab0LIGLw convert rfc822-to-8bit (ORCPT ); Thu, 9 Dec 2010 01:11:52 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.59,319,1288594800"; d="scan'208";a="582008660" From: "Dong, Chuanxiao" To: Kyungmin Park CC: "linux-mmc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "cjb@laptop.org" , "arjan@linux.intel.com" , "alan@linux.intel.com" , "akpm@linux-foundation.org" Date: Thu, 9 Dec 2010 14:10:53 +0800 Subject: RE: [PATCH v4 1/4]enable HW reset capbility Thread-Topic: [PATCH v4 1/4]enable HW reset capbility Thread-Index: AcuXZxP1leeUKS60SpOINpZCTu8h/AAAEXhQ Message-ID: <5D8008F58939784290FAB48F549751983263F995EC@shsmsx502.ccr.corp.intel.com> References: <20101201124900.GB5421@intel.com> <5D8008F58939784290FAB48F549751983263F995AB@shsmsx502.ccr.corp.intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: kyungmin78@gmail.com [mailto:kyungmin78@gmail.com] On Behalf Of > Kyungmin Park > Sent: Thursday, December 09, 2010 2:05 PM > To: Dong, Chuanxiao > Cc: linux-mmc@vger.kernel.org; linux-kernel@vger.kernel.org; cjb@laptop.org; > arjan@linux.intel.com; alan@linux.intel.com; akpm@linux-foundation.org > Subject: Re: [PATCH v4 1/4]enable HW reset capbility > > On Thu, Dec 9, 2010 at 2:40 PM, Dong, Chuanxiao > wrote: > > > > > >> -----Original Message----- > >> From: kyungmin78@gmail.com [mailto:kyungmin78@gmail.com] On Behalf Of > >> Kyungmin Park > >> Sent: Thursday, December 09, 2010 1:24 PM > >> To: Dong, Chuanxiao > >> Cc: linux-mmc@vger.kernel.org; linux-kernel@vger.kernel.org; cjb@laptop.org; > >> arjan@linux.intel.com; alan@linux.intel.com; akpm@linux-foundation.org > >> Subject: Re: [PATCH v4 1/4]enable HW reset capbility > >> > >> On Wed, Dec 1, 2010 at 9:49 PM, Chuanxiao Dong > >> wrote: > >> > From d3be0b4fe8e8a40294830dfd8d0543d1e957fd29 Mon Sep 17 00:00:00 > 2001 > >> > From: Chuanxiao Dong > >> > Date: Wed, 1 Dec 2010 19:14:02 +0800 > >> > Subject: [PATCH 1/4] enable HW reset caps of MMC card if card supports > >> > > >> > HW reset capbility enable bit is byte 162 in card EXT_CSD > >> > register, only version4.4 card or later can support to enable > >> > this feature. > >> > > >> > HW reset feature can be used to reset eMMC card when occures > >> > timeout errors during read/write/erase. > >> > > >> > Signed-off-by: Chuanxiao Dong > >> > --- > >> >  drivers/mmc/core/mmc.c   |   29 +++++++++++++++++++++++++++++ > >> >  include/linux/mmc/card.h |    1 + > >> >  include/linux/mmc/mmc.h  |    1 + > >> >  3 files changed, 31 insertions(+), 0 deletions(-) > >> > > >> > diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c > >> > index 77f93c3..85cc7f6 100644 > >> > --- a/drivers/mmc/core/mmc.c > >> > +++ b/drivers/mmc/core/mmc.c > >> > @@ -310,6 +310,10 @@ static int mmc_read_ext_csd(struct mmc_card > *card) > >> >                        ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT]; > >> >                card->ext_csd.trim_timeout = 300 * > >> >                        ext_csd[EXT_CSD_TRIM_MULT]; > >> > +               /* > >> > +                * check Hardware reset cap > >> Check hardware. > >> > +                */ > >> > +               card->ext_csd.rst = ext_csd[EXT_CSD_RST]; > >> >        } > >> > > >> >        if (ext_csd[EXT_CSD_ERASED_MEM_CONT]) > >> > @@ -484,6 +488,31 @@ static int mmc_init_card(struct mmc_host *host, > u32 > >> ocr, > >> >        } > >> > > >> >        /* > >> > +        * eMMC4.4 version card has HW reset capbility. > >> capability. > >> > +        * Enable this feature here: > >> > +        * RST_N_FUNCTION register is W/R, one time programmable > >> Use RST_n_FUNCTION as spec. > >> > +        * or readable. > >> > +        * So need to enable this register only once after power on > >> > +        */ > >> > +       if (card->csd.mmca_vsn >= CSD_SPEC_VER_4 && > >> > +                       card->ext_csd.rev >= 4 && > >> it should be card->ext_csd.rev >= 5 since 4 is obsolete. > >> > >> Thank you, > >> Kyungmin Park > > > > Thanks, Park. I will fix this. > > And by the way, I noticed mmc_read_ext_csd function also used > card->ext_csd.rev >=4. Is there some other reason for this kind of using? Or it is > just because at that time the value 4 has not be obsolete yet....? > > Maybe need to fix it also. > > And remaining patches also need to fix 'capability' typo. Thanks. Will pay more attention on this. And will resubmit new version for review. Thanks > >> > +                       card->ext_csd.rst == 0) { > >> > +               err = mmc_switch(card, EXT_CSD_CMD_SET_NORMAL, > >> > +                               EXT_CSD_RST, 1); > >> > + > >> > +               if (err && err != -EBADMSG) > >> > +                       goto free_card; > >> > + > >> > +               if (err) { > >> > +                       printk(KERN_WARNING "%s: switch to rst > enable > >> " > >> > +                                  "failed %d\n", > >> > +                                  mmc_hostname(card->host), > err); > >> > +                       err = 0; > >> > +               } else > >> > +                       card->ext_csd.rst = 1; > >> > +       } > >> > + > >> > +       /* > >> >         * Activate high speed (if supported) > >> >         */ > >> >        if ((card->ext_csd.hs_max_dtr != 0) && > >> > diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h > >> > index 8ce0827..ebee676 100644 > >> > --- a/include/linux/mmc/card.h > >> > +++ b/include/linux/mmc/card.h > >> > @@ -54,6 +54,7 @@ struct mmc_ext_csd { > >> >        unsigned int            sec_trim_mult;  /* Secure trim > >> multiplier  */ > >> >        unsigned int            sec_erase_mult; /* Secure erase > multiplier > >> */ > >> >        unsigned int            trim_timeout;           /* In > >> milliseconds */ > >> > +       unsigned int            rst; /* hardware reset enable bit */ > >> >  }; > >> > > >> >  struct sd_scr { > >> > diff --git a/include/linux/mmc/mmc.h b/include/linux/mmc/mmc.h > >> > index 956fbd8..b7ab0da 100644 > >> > --- a/include/linux/mmc/mmc.h > >> > +++ b/include/linux/mmc/mmc.h > >> > @@ -251,6 +251,7 @@ struct _mmc_csd { > >> >  * EXT_CSD fields > >> >  */ > >> > > >> > +#define EXT_CSD_RST            162     /* onetime programmable > R/W > >> */ > >> >  #define EXT_CSD_ERASE_GROUP_DEF                175     /* > R/W */ > >> >  #define EXT_CSD_ERASED_MEM_CONT                181     /* > RO */ > >> >  #define EXT_CSD_BUS_WIDTH              183     /* R/W */ > >> > -- > >> > 1.6.6.1 > >> > > >> > -- > >> > 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 > >> > > > -- > > 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 > >