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

[Greg Ungerer]

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