Re: [REGRESSION next-20170321] acbdad8dd1ab ("serial: 8250_dw: simplify optional reset handling") causes silent kernel.

From: Ralph Sennhauser
Date: Wed Mar 22 2017 - 11:11:40 EST


Hi Philipp

On Wed, 22 Mar 2017 15:33:47 +0100
Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> wrote:

> Hi Ralph,
>
> On Wed, 2017-03-22 at 15:24 +0100, Ralph Sennhauser wrote:
> > Hi Philipp,
> >
> > Commit acbdad8dd1ab ("serial: 8250_dw: simplify optional reset
> > handling") causes the kernel to no longer print anything to the
> > serial console on a Linksys WRT3200ACM (Armada-385, a similar board
> > would be ./arch/arm/boot/dts/armada-385-linksys-cobra.dts). The
> > last message from uboot is "Starting kernel ..." and then silence.
> > The device boots all the way nonetheless.
> >
> > Reverting the above commit gets the console output back.
> >
> > Ralph
>
> Thank you for the report. I assume that you are compiling with the
> RESET_CONTROLLER Kconfig option disabled? Does commit 0ca10b60ceeb
> [1] in
> the arm-soc fixes branch help with this issue?

Yes, RESET_CONTROLLER is not set.

Commit 0ca10b60ceeb ("reset: fix optional reset_control_get stubs to
return NULL") fixes the issue for me.

Thanks
Ralph

>
> [1]
> https://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc.git/commit/?h=fixes&id=0ca10b60ceeb5372da01798ca68c116ae45a6eb6
>
> regards
> Philipp
>