[PATCH v4 01/10] mtd: spi-nor: fsl-quadspi: dynamically map memory space for AHB read
Brian Norris
computersforpeace at gmail.com
Thu Aug 6 10:09:56 PDT 2015
On Tue, Aug 04, 2015 at 10:25:22AM -0500, Han Xu wrote:
> From: Han Xu <han.xu at freescale.com>
>
> QSPI may failed to map enough memory (256MB) for AHB read in
> previous implementation, especially in 3G/1G memory layout kernel.
> Dynamically map memory to avoid such issue.
>
> This implementation generally map QUADSPI_MAX_IOMAP (default 4MB) memory
> for AHB read, it should be enough for common scenarios, and the side
> effect (0.6% performance drop) is minor.
>
> Previous implementation
>
> root at imx6qdlsolo:~# dd if=/dev/mtd0 of=/dev/null bs=1K count=32K
> 32768+0 records in
> 32768+0 records out
> 33554432 bytes (34 MB) copied, 2.16006 s, 15.5 MB/s
>
> root at imx6qdlsolo:~# dd if=/dev/mtd0 of=/dev/null bs=32M count=1
> 1+0 records in
> 1+0 records out
> 33554432 bytes (34 MB) copied, 1.43149 s, 23.4 MB/s
>
> After applied the patch
>
> root at imx6qdlsolo:~# dd if=/dev/mtd0 of=/dev/null bs=1K count=32K
> 32768+0 records in
> 32768+0 records out
> 33554432 bytes (34 MB) copied, 2.1743 s, 15.4 MB/s
>
> root at imx6qdlsolo:~# dd if=/dev/mtd0 of=/dev/null bs=32M count=1
> 1+0 records in
> 1+0 records out
> 33554432 bytes (34 MB) copied, 1.43158 s, 23.4 MB/s
>
> Signed-off-by: Han Xu <han.xu at freescale.com>
> Signed-off-by: Frank Li <Frank.Li at freescale.com>
> ---
>
> Change from v3 to v4:
> According to Brian's comments,
> - introduce a new macro QUADSPI_MIN_IOMAP instead of SZ_4M
> - reduce the duplicated computation for offset
> - reintroduce the devm_request_mem_region()
>
> Change from v2 to v3:
> Resend whole serial patch
>
> Change from v1 to v2:
> rebase to last l2-mtd
Thanks, pushed the series to l2-mtd.git, with a few whitespace fixups.
Brian
More information about the linux-mtd
mailing list