Hello Geert,
On Tue, May 14, 2019 at 10:28:34AM +0200, Geert Uytterhoeven wrote:
> Hi George,
>
> On Mon, May 13, 2019 at 5:48 PM George G. Davis <george_da...@mentor.com>
> wrote:
> > As noted in commit 84b40e3b57ee ("serial: 8250: omap: Disable DMA for
> > console UART"), UART console lines use low-level PIO only access functions
> > which will conflict with use of the line when DMA is enabled, e.g. when
> > the console line is also used for systemd messages. So disable DMA
> > support for UART console lines.
> >
> > Fixes: https://patchwork.kernel.org/patch/10929511/
>
> I don't think this is an appropriate reference, as it points to a patch that
> was never applied.
I included it as a link to an upstream problem report similar to other commits
that I previewed. The link provides the extra context that I was perhaps to
lazy to note in the commit header.
> As the problem has basically existed forever,
Agreed
> IMHO no Fixes tag
> is needed.
I've dropped the Fixes line.
> > Reported-by: Michael Rodin <mro...@de.adit-jv.com>
> > Tested-by: Eugeniu Rosca <ero...@de.adit-jv.com>
> > Reviewed-by: Simon Horman <horms+rene...@verge.net.au>
> > Reviewed-by: Wolfram Sang <wsa+rene...@sang-engineering.com>
> > Cc: sta...@vger.kernel.org
> > Signed-off-by: George G. Davis <george_da...@mentor.com>
> > ---
> > v2: Clarify comment regarding DMA support on kernel console,
> > add {Tested,Reviewed}-by:, and Cc: linux-stable lines.
>
> Thanks for the update!
Thanks!
I'll submit v3 later today.
> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 --
> ge...@linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like
> that.
> -- Linus Torvalds
--
Regards,
George
