Re: [PATCH] m32r: add simple dma

From: Andrew Morton
Date: Thu Oct 20 2016 - 23:29:27 EST


On Sat, 8 Oct 2016 23:23:18 +0530 Sudip Mukherjee <sudipm.mukherjee@xxxxxxxxx> wrote:

> Some builds of m32r were failing as it tried to build few drivers which
> needed dma but m32r is not having dma support. Objections were raised
> when it was tried to make those drivers depend on HAS_DMA.

Huh. What were these objections? That sounds like the appropriate
fix. And I suggest that a summary of those objections be captured in
this patch's changelog.

> So the next
> best thing is to add dma support to m32r.
> dma_noop is a very simple dma with 1:1 memory mapping.
>
> Signed-off-by: Sudip Mukherjee <sudip.mukherjee@xxxxxxxxxxxxxxx>
> ---
>
> Hi Andrew,
> Just to let you know that this was not tested on any board. I think I
> have told you earlier that inspite of all my efforts I could not find
> any source of information to procure a board of m32r.

It is a worry. We're saying "m32r linux now supports these drivers",
only we don't know if that is true.

> Hi Fengguang,
> If Andrew accepts this patch then it will open pandora's box as lots of
> other drivers which were never build for m32r will be built now and you
> are going to gets lots and lots of new build warnings. But nothing
> breaks.