Re: [PATCHv2] serial: 8250: omap: Fix idling of clocks for unused uarts

2018-05-04 Thread Tony Lindgren
* Sebastian Andrzej Siewior [180504 17:50]: > On 2018-05-04 10:39:31 [-0700], Tony Lindgren wrote: > > Uhh sorry I managed to commit also a patch I was testing while > > updating patch comments.. Will send out v3 shortly, this can be > > ignored. > > I assumed you were

Re: [PATCHv2] serial: 8250: omap: Fix idling of clocks for unused uarts

2018-05-04 Thread Tony Lindgren
* Sebastian Andrzej Siewior [180504 17:50]: > On 2018-05-04 10:39:31 [-0700], Tony Lindgren wrote: > > Uhh sorry I managed to commit also a patch I was testing while > > updating patch comments.. Will send out v3 shortly, this can be > > ignored. > > I assumed you were helping out to sneak a

Re: [PATCHv2] serial: 8250: omap: Fix idling of clocks for unused uarts

2018-05-04 Thread Sebastian Andrzej Siewior
On 2018-05-04 10:39:31 [-0700], Tony Lindgren wrote: > Uhh sorry I managed to commit also a patch I was testing while > updating patch comments.. Will send out v3 shortly, this can be > ignored. I assumed you were helping out to sneak a patch in. > Regards, > > Tony Sebastian

Re: [PATCHv2] serial: 8250: omap: Fix idling of clocks for unused uarts

2018-05-04 Thread Sebastian Andrzej Siewior
On 2018-05-04 10:39:31 [-0700], Tony Lindgren wrote: > Uhh sorry I managed to commit also a patch I was testing while > updating patch comments.. Will send out v3 shortly, this can be > ignored. I assumed you were helping out to sneak a patch in. > Regards, > > Tony Sebastian

Re: [PATCHv2] serial: 8250: omap: Fix idling of clocks for unused uarts

2018-05-04 Thread Tony Lindgren
* Tony Lindgren [180504 17:33]: > I noticed that unused UARTs won't necessarily idle properly always > unless at least one byte tx transfer is done first. > --- > arch/arm/mach-actions/platsmp.c | 6 +++--- > arch/arm/mach-exynos/platsmp.c | 12 ++-- >

Re: [PATCHv2] serial: 8250: omap: Fix idling of clocks for unused uarts

2018-05-04 Thread Tony Lindgren
* Tony Lindgren [180504 17:33]: > I noticed that unused UARTs won't necessarily idle properly always > unless at least one byte tx transfer is done first. > --- > arch/arm/mach-actions/platsmp.c | 6 +++--- > arch/arm/mach-exynos/platsmp.c | 12 ++-- >

[PATCHv2] serial: 8250: omap: Fix idling of clocks for unused uarts

2018-05-04 Thread Tony Lindgren
I noticed that unused UARTs won't necessarily idle properly always unless at least one byte tx transfer is done first. After some debugging I narrowed down the problem to the scr register dma configuration bits that need to be set before softreset for the clocks to idle. Unless we do this, the

[PATCHv2] serial: 8250: omap: Fix idling of clocks for unused uarts

2018-05-04 Thread Tony Lindgren
I noticed that unused UARTs won't necessarily idle properly always unless at least one byte tx transfer is done first. After some debugging I narrowed down the problem to the scr register dma configuration bits that need to be set before softreset for the clocks to idle. Unless we do this, the