* [PATCH] mmc: MMC_NUM_PHY_PARTITION is not right.
@ 2014-08-01 7:03 yisun1
2014-08-12 9:29 ` Sun, Yi Y
0 siblings, 1 reply; 3+ messages in thread
From: yisun1 @ 2014-08-01 7:03 UTC (permalink / raw)
To: linux-mmc; +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: yisun1 <yi.y.sun@intel.com>
---
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 {
*/
#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] 3+ messages in thread
* RE: [PATCH] mmc: MMC_NUM_PHY_PARTITION is not right.
2014-08-01 7:03 [PATCH] mmc: MMC_NUM_PHY_PARTITION is not right yisun1
@ 2014-08-12 9:29 ` Sun, Yi Y
2014-08-12 9:51 ` Ulf Hansson
0 siblings, 1 reply; 3+ messages in thread
From: Sun, Yi Y @ 2014-08-12 9:29 UTC (permalink / raw)
To: linux-mmc, Ulf Hansson
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(-)
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
/*
--
1.7.9.5
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] mmc: MMC_NUM_PHY_PARTITION is not right.
2014-08-12 9:29 ` Sun, Yi Y
@ 2014-08-12 9:51 ` Ulf Hansson
0 siblings, 0 replies; 3+ messages in thread
From: Ulf Hansson @ 2014-08-12 9:51 UTC (permalink / raw)
To: Sun, Yi Y; +Cc: linux-mmc
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
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-08-12 9:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-01 7:03 [PATCH] mmc: MMC_NUM_PHY_PARTITION is not right yisun1
2014-08-12 9:29 ` Sun, Yi Y
2014-08-12 9:51 ` 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.