Hi Andy, On Wed, 1 Apr 2020 at 08:45, Andy Shevchenko <andy.shevche...@gmail.com> wrote: > > On Wed, Apr 1, 2020 at 5:32 PM Bin Meng <bmeng...@gmail.com> wrote: > > On Wed, Apr 1, 2020 at 9:58 PM Andy Shevchenko > > <andriy.shevche...@linux.intel.com> wrote: > > > > > > The commit breaks serial console on the Intel Edison. > > > > > > This reverts commit 720f9e1fdb0c92d3fd16e1bfc25bcbd35612675c. > > > > > > Signed-off-by: Andy Shevchenko <andriy.shevche...@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