[PATCH v2 1/2] serial: ns16550: Revert "Move PCI access from ofdata_to_platdata() to probe()"

Simon Glass sjg at chromium.org
Wed Apr 1 18:56:26 CEST 2020


Hi Andy,

On Wed, 1 Apr 2020 at 08:45, Andy Shevchenko <andy.shevchenko at gmail.com> wrote:
>
> On Wed, Apr 1, 2020 at 5:32 PM Bin Meng <bmeng.cn at gmail.com> wrote:
> > On Wed, Apr 1, 2020 at 9:58 PM Andy Shevchenko
> > <andriy.shevchenko at linux.intel.com> wrote:
> > >
> > > The commit breaks serial console on the Intel Edison.
> > >
> > > This reverts commit 720f9e1fdb0c92d3fd16e1bfc25bcbd35612675c.
> > >
> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > > ---
> > >  drivers/serial/ns16550.c | 40 ++++++++++++----------------------------
> > >  1 file changed, 12 insertions(+), 28 deletions(-)
> > >
> >
> > Could you please spend some time to investigate why this breaks Intel Edison?
> >
> > Reverting this patch would mean we break other boards too as
> > Wolfgang's patch wanted to fix the breakage in the first place. Much
> > appreciated!
>
> I guess I'm wrong person here. The DM code is a complete black box to me.
> Nevertheless, I may test any provided fix / debug / etc patch by request.
>
> And I think it's fair to investigate by the one who made a regression
> in the first place.

Given that we have conflicting breakages, we need to debug Edison.
Does it enable the debug UART?

I have one but have not powered it on for a while, nor added it to my lab.

Regards,
Simon


More information about the U-Boot mailing list