[U-Boot] [PATCH] disk: part: align buffer so it can be used with DMA enabled drivers

[U-Boot] [PATCH] disk: part: align buffer so it can be used with DMA enabled drivers

[Stefan Agner]

From: Stefan Agner <stefan.agner@toradex.com>

When using ISO partitions with a DMA enabled block device driver
reading the ISO partition leads to unaligned DMA operations:
  CACHE: Misaligned operation at range [bffb7da8, bffb85a8]

Align the buffer to make sure we pass a buffer which works for
DMA operations.

Signed-off-by: Stefan Agner <stefan.agner@toradex.com>
---

 disk/part_iso.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/disk/part_iso.c b/disk/part_iso.c
index bb8ed658f2..8aef251f4e 100644
--- a/disk/part_iso.c
+++ b/disk/part_iso.c
@@ -24,7 +24,7 @@
 #undef CHECK_FOR_POWERPC_PLATTFORM
 #define CD_SECTSIZE 2048
 
-static unsigned char tmpbuf[CD_SECTSIZE];
+static unsigned char tmpbuf[CD_SECTSIZE] __aligned(ARCH_DMA_MINALIGN);
 
 unsigned long iso_dread(struct blk_desc *block_dev, lbaint_t start,
                         lbaint_t blkcnt, void *buffer)
-- 
2.14.1

[U-Boot] [PATCH] disk: part: align buffer so it can be used with DMA enabled drivers

[Bin Meng]

On Thu, Aug 24, 2017 at 12:46 AM, Stefan Agner <stefan@agner.ch> wrote:
> From: Stefan Agner <stefan.agner@toradex.com>
>
> When using ISO partitions with a DMA enabled block device driver
> reading the ISO partition leads to unaligned DMA operations:
>   CACHE: Misaligned operation at range [bffb7da8, bffb85a8]
>
> Align the buffer to make sure we pass a buffer which works for
> DMA operations.
>
> Signed-off-by: Stefan Agner <stefan.agner@toradex.com>
> ---
>
>  disk/part_iso.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

[U-Boot] disk: part: align buffer so it can be used with DMA enabled drivers

[Tom Rini]

On Wed, Aug 23, 2017 at 09:46:17AM -0700, Stefan Agner wrote:

> From: Stefan Agner <stefan.agner@toradex.com>
> 
> When using ISO partitions with a DMA enabled block device driver
> reading the ISO partition leads to unaligned DMA operations:
>   CACHE: Misaligned operation at range [bffb7da8, bffb85a8]
> 
> Align the buffer to make sure we pass a buffer which works for
> DMA operations.
> 
> Signed-off-by: Stefan Agner <stefan.agner@toradex.com>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170826/36d515db/attachment.sig>