diff mbox

mmc: MMC_NUM_PHY_PARTITION is not right.

Message ID 525EAED47491124EB5123A51BD2FC791019D95FD@SHSMSX101.ccr.corp.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

yisun1 Aug. 12, 2014, 9:29 a.m. UTC
Hi, Ulf,

Could you please check this fix? Thanks!

BRs,
Sun Yi

-----Original Message-----
From: Sun, Yi Y 
Sent: Friday, August 01, 2014 3:03 PM
To: linux-mmc@vger.kernel.org
Cc: Sun, Yi Y
Subject: [PATCH] mmc: MMC_NUM_PHY_PARTITION is not right.

eMMC card can support up to 7 partitions, including 2 boot,
1 RPMB and 4 GPs. But MMC_NUM_PHY_PARTITION is defined to
6 so we should fix it to 7.

Signed-off-by: yisun1 <yi.y.sun@intel.com>
---
 include/linux/mmc/card.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
1.7.9.5

--
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

Comments

Ulf Hansson Aug. 12, 2014, 9:51 a.m. UTC | #1
On 12 August 2014 11:29, Sun, Yi Y <yi.y.sun@intel.com> wrote:
> Hi, Ulf,
>
> Could you please check this fix? Thanks!
>
> BRs,
> Sun Yi
>
> -----Original Message-----
> From: Sun, Yi Y
> Sent: Friday, August 01, 2014 3:03 PM
> To: linux-mmc@vger.kernel.org
> Cc: Sun, Yi Y
> Subject: [PATCH] mmc: MMC_NUM_PHY_PARTITION is not right.
>
> eMMC card can support up to 7 partitions, including 2 boot,
> 1 RPMB and 4 GPs. But MMC_NUM_PHY_PARTITION is defined to
> 6 so we should fix it to 7.
>
> Signed-off-by: yisun1 <yi.y.sun@intel.com>

Could you please update your name in SOB tag to your proper name.

> ---
>  include/linux/mmc/card.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h index d424b9d..6dadac2 100644
> --- a/include/linux/mmc/card.h
> +++ b/include/linux/mmc/card.h
> @@ -218,7 +218,7 @@ enum mmc_blk_status {
>   */

There are a small comment in the code just prior these defines, could
update it so that it becomes obvious why MMC_NUM_PHY_PARTITION shall
be 7.

>  #define MMC_NUM_BOOT_PARTITION 2
>  #define MMC_NUM_GP_PARTITION   4
> -#define MMC_NUM_PHY_PARTITION  6
> +#define MMC_NUM_PHY_PARTITION  7
>  #define MAX_MMC_PART_NAME_LEN  20
>
>  /*
> --
> 1.7.9.5
>

I think we should handle this a fix and it should go stable as well.

Kind regards
Uffe
--
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
diff mbox

Patch

diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h index d424b9d..6dadac2 100644
--- a/include/linux/mmc/card.h
+++ b/include/linux/mmc/card.h
@@ -218,7 +218,7 @@  enum mmc_blk_status {
  */
 #define MMC_NUM_BOOT_PARTITION	2
 #define MMC_NUM_GP_PARTITION	4
-#define MMC_NUM_PHY_PARTITION	6
+#define MMC_NUM_PHY_PARTITION	7
 #define MAX_MMC_PART_NAME_LEN	20
 
 /*