RE: [PATCH V2 0/3] mmc: block: Fix tuning (by avoiding it) for RPMB

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Hunter, Adrian
> Sent: Wednesday, May 04, 2016 14:38
> To: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
> Cc: linux-mmc <linux-mmc@xxxxxxxxxxxxxxx>; Winkler, Tomas
> <tomas.winkler@xxxxxxxxx>
> Subject: [PATCH V2 0/3] mmc: block: Fix tuning (by avoiding it) for RPMB
> 
> Hi
> 
> The RPMB partition only allows certain commands.  In particular, the tuning
> command (CMD21) is not allowed -  refer JEDEC eMMC standard v5.1 section
> 6.2.2 Command restrictions.
> 
> That means commands will begin failing if re-tuning is needed while switched
> to the RPMB partition.
> 
> As we discussed here:
> 
> 	http://marc.info/?l=linux-mmc&m=146218765801869
> 
> this patch set takes a new approach which is:
> 
> 	Re-tune before switching to the RPMB partition
> 	Don't allow re-tuning while switched to RPMB
> 	Switch back from the RPMB partition immediately
> 
> I gave it a brief test with a HS400 eMMC and using mmc utils to access RPMB.
> Despite all the partition switches and re-tuning in between, all RPMB
> operations were successful.
> 
> 
> Changes in V2:
> 
> 	New approach entirely
> 
> 
> Adrian Hunter (3):
>       mmc: core: Add a facility to "pause" re-tuning
>       mmc: block: Always switch back to main area after RPMB access
>       mmc: block: Pause re-tuning while switched to the RPMB partition
> 
>  drivers/mmc/card/block.c | 19 ++++++++++++++++++-
> drivers/mmc/core/host.c  | 22 ++++++++++++++++++++++
> include/linux/mmc/host.h |  4 ++++
>  3 files changed, 44 insertions(+), 1 deletion(-)
> 

The series looks good, it will be easy to adjust to my RPMB patch.
Thanks
Tomas



--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux