[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL_JsqLu+KxXdZseQiRFPr5MG0hSnwnQJpBLg0M5tgO-ap4F=g@mail.gmail.com>
Date: Wed, 25 May 2016 09:38:54 -0500
From: Rob Herring <robh+dt@...nel.org>
To: Jaewon Kim <jaewon31.kim@...sung.com>
Cc: r64343@...escale.com, Marek Szyprowski <m.szyprowski@...sung.com>,
Grant Likely <grant.likely@...aro.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-mm@...ck.org" <linux-mm@...ck.org>, jaewon31.kim@...il.com
Subject: Re: [RESEND][PATCH] drivers: of: of_reserved_mem: fixup the CMA
alignment not to affect dma-coherent
On Tue, May 24, 2016 at 11:29 PM, Jaewon Kim <jaewon31.kim@...sung.com> wrote:
> From: Jaewon <jaewon31.kim@...sung.com>
>
> There was an alignment mismatch issue for CMA and it was fixed by
> commit 1cc8e3458b51 ("drivers: of: of_reserved_mem: fixup the alignment with CMA setup").
> However the way of the commit considers not only dma-contiguous(CMA) but also
> dma-coherent which has no that requirement.
>
> This patch checks more to distinguish dma-contiguous(CMA) from dma-coherent.
>
> Signed-off-by: Jaewon Kim <jaewon31.kim@...sung.com>
I suppose this needs to go to stable? If so, adding the stable tag and
kernel version would be nice so I don't have to.
> ---
> drivers/of/of_reserved_mem.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
I'm looking for an ack from Marek on this.
Rob
Powered by blists - more mailing lists