All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] mmc: MMC_NUM_PHY_PARTITION is not right.
@ 2014-08-13  5:34 Yi Sun
  2014-08-13 15:01 ` Ulf Hansson
  0 siblings, 1 reply; 2+ messages in thread
From: Yi Sun @ 2014-08-13  5:34 UTC (permalink / raw)
  To: linux-mmc, ulf.hansson; +Cc: yi.y.sun

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: Yi Sun <yi.y.sun@intel.com>
---
 include/linux/mmc/card.h |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
index d424b9d..bde5147 100644
--- a/include/linux/mmc/card.h
+++ b/include/linux/mmc/card.h
@@ -214,11 +214,12 @@ enum mmc_blk_status {
 };
 
 /* The number of MMC physical partitions.  These consist of:
- * boot partitions (2), general purpose partitions (4) in MMC v4.4.
+ * boot partitions (2), general purpose partitions (4) and
+ * RPMB partition (1) in MMC v4.4.
  */
 #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


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

* Re: [PATCH v2] mmc: MMC_NUM_PHY_PARTITION is not right.
  2014-08-13  5:34 [PATCH v2] mmc: MMC_NUM_PHY_PARTITION is not right Yi Sun
@ 2014-08-13 15:01 ` Ulf Hansson
  0 siblings, 0 replies; 2+ messages in thread
From: Ulf Hansson @ 2014-08-13 15:01 UTC (permalink / raw)
  To: Yi Sun; +Cc: linux-mmc

On 13 August 2014 07:34, Yi Sun <yi.y.sun@intel.com> wrote:
> 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: Yi Sun <yi.y.sun@intel.com>

Thanks! I queued this one for 3.18. I took the liberty to update the
author name and made some minor clarifications to the commit message.
You should probably update your .gitconfig for future patches, to get
the correct name.

Kind regards
Uffe

> ---
>  include/linux/mmc/card.h |    5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
> index d424b9d..bde5147 100644
> --- a/include/linux/mmc/card.h
> +++ b/include/linux/mmc/card.h
> @@ -214,11 +214,12 @@ enum mmc_blk_status {
>  };
>
>  /* The number of MMC physical partitions.  These consist of:
> - * boot partitions (2), general purpose partitions (4) in MMC v4.4.
> + * boot partitions (2), general purpose partitions (4) and
> + * RPMB partition (1) in MMC v4.4.
>   */
>  #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
>

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

end of thread, other threads:[~2014-08-13 15:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-13  5:34 [PATCH v2] mmc: MMC_NUM_PHY_PARTITION is not right Yi Sun
2014-08-13 15:01 ` Ulf Hansson

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.