Re: [RFC PATCH] m68k: set dma and coherent masks for Macintosh SONIC based ethernet

From: Greg Ungerer
Date: Tue May 29 2018 - 20:28:29 EST


Hi Geert,

On 28/05/18 20:15, Geert Uytterhoeven wrote:
On Mon, May 28, 2018 at 7:26 AM, Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx> wrote:
On Mon, 28 May 2018, Michael Schmitz wrote:
Am 27.05.2018 um 17:49 schrieb Finn Thain:
On Sun, 27 May 2018, Michael Schmitz wrote:

That should have fixed the warning already ...

It's still not fixed (hence my "acked-by" for Geunter's patch).


Odd - does link order still matter even though the
arch_setup_dev_archdata() function from the core platform code is
declared as a weak symbol?

I'll see what I can find out on elgar ...


Any one of the numerous patches/rfcs/suggestions that I sent will avoid
the WARN splat.

When I said "it's still not fixed", what I meant to say was, "it's still
not fixed in mainline and no proposed fix was accepted to the best of my
knowledge".

Indeed.

Do we have a consensus on the way forward? The merge window for
v4.18 will open soon.

For whatever it is worth I thought Finn's patch was the best approach
(https://lkml.org/lkml/2018/5/17/333, "m68k: Set default dma mask for
platform device").

We seem to be hitting quite a few places (within m68k) that otherwise
need individual fixes. There is no immediate need to revert existing
changes that have already been applied if we use this now either
(like my FEC fix, commit f61e64310b75 "m68k: set dma and coherent
masks for platform FEC ethernets").

Regards
Greg