diff mbox

[v1,2/5] mmc: card: Do not scan RPMB partitions

Message ID 1341564244-6178-3-git-send-email-loic.pallardy@stericsson.com (mailing list archive)
State New, archived
Headers show

Commit Message

Loic Pallardy July 6, 2012, 8:44 a.m. UTC
Do not scan rpmb partitions for "soft" partitions, since the rpmb
partition contains protected data. Silences the following
message during boot:

 mmcblkXRPMB: unknown partition table

Signed-off-by: Johan Rudholm <johan.rudholm@stericsson.com>
---
 drivers/mmc/card/block.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Linus Walleij July 8, 2012, 7:18 p.m. UTC | #1
On Fri, Jul 6, 2012 at 10:44 AM, Loic Pallardy
<loic.pallardy@stericsson.com> wrote:

> Do not scan rpmb partitions for "soft" partitions, since the rpmb
> partition contains protected data. Silences the following
> message during boot:
>
>  mmcblkXRPMB: unknown partition table
>
> Signed-off-by: Johan Rudholm <johan.rudholm@stericsson.com>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij
--
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
Namjae Jeon July 11, 2012, 12:35 a.m. UTC | #2
>
> diff --git a/drivers/mmc/card/block.c b/drivers/mmc/card/block.c
> index 276d21c..1b202fe 100644
> --- a/drivers/mmc/card/block.c
> +++ b/drivers/mmc/card/block.c
> @@ -1521,6 +1521,9 @@ static struct mmc_blk_data *mmc_blk_alloc_req(struct
> mmc_card *card,
>  	md->disk->queue = md->queue.queue;
>  	md->disk->driverfs_dev = parent;
>  	set_disk_ro(md->disk, md->read_only || default_ro);
> +	md->disk->flags = GENHD_FL_EXT_DEVT;
Currrenlty mmc have been setting minors to CONFIG_MMC_BLOCK_MINORS.
Why did you add GENHD_FL_EXT_DEVT ?

Thanks.
--
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
Loic pallardy July 11, 2012, 12:36 p.m. UTC | #3
>>
>> diff --git a/drivers/mmc/card/block.c b/drivers/mmc/card/block.c
>> index 276d21c..1b202fe 100644
>> --- a/drivers/mmc/card/block.c
>> +++ b/drivers/mmc/card/block.c
>> @@ -1521,6 +1521,9 @@ static struct mmc_blk_data *mmc_blk_alloc_req(struct
>> mmc_card *card,
>>       md->disk->queue = md->queue.queue;
>>       md->disk->driverfs_dev = parent;
>>       set_disk_ro(md->disk, md->read_only || default_ro);
>> +     md->disk->flags = GENHD_FL_EXT_DEVT;
> Currrenlty mmc have been setting minors to CONFIG_MMC_BLOCK_MINORS.
> Why did you add GENHD_FL_EXT_DEVT ?
>
> Thanks.
Yes, my mistake, indeed not related to this patch.
Thanks.
--
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/drivers/mmc/card/block.c b/drivers/mmc/card/block.c
index 276d21c..1b202fe 100644
--- a/drivers/mmc/card/block.c
+++ b/drivers/mmc/card/block.c
@@ -1521,6 +1521,9 @@  static struct mmc_blk_data *mmc_blk_alloc_req(struct mmc_card *card,
 	md->disk->queue = md->queue.queue;
 	md->disk->driverfs_dev = parent;
 	set_disk_ro(md->disk, md->read_only || default_ro);
+	md->disk->flags = GENHD_FL_EXT_DEVT;
+	if (area_type & MMC_BLK_DATA_AREA_RPMB)
+		md->disk->flags |= GENHD_FL_NO_PART_SCAN;
 
 	/*
 	 * As discussed on lkml, GENHD_FL_REMOVABLE should: