Re: [PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-09 Thread H. Nikolaus Schaller
Hi Russell, > Am 09.11.2017 um 18:35 schrieb Russell King - ARM Linux > : > > Hi Nikkylos, > > On Thu, Nov 09, 2017 at 06:16:29PM +0100, H. Nikolaus Schaller wrote: >> Hi Russel, >> >>> Am 09.11.2017 um 17:58 schrieb Russell King - ARM Linux >>> : >>> >>> On Thu, Nov 09, 2017 at 06:44:49AM +

Re: [PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-09 Thread Russell King - ARM Linux
Hi Nikkylos, On Thu, Nov 09, 2017 at 06:16:29PM +0100, H. Nikolaus Schaller wrote: > Hi Russel, > > > Am 09.11.2017 um 17:58 schrieb Russell King - ARM Linux > > : > > > > On Thu, Nov 09, 2017 at 06:44:49AM +0100, H. Nikolaus Schaller wrote: > >> Hi Russel, > > > > Nikolus, > > > >>> Am 08.11

Re: [PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-09 Thread H. Nikolaus Schaller
Hi Russel, > Am 09.11.2017 um 17:58 schrieb Russell King - ARM Linux > : > > On Thu, Nov 09, 2017 at 06:44:49AM +0100, H. Nikolaus Schaller wrote: >> Hi Russel, > > Nikolus, > >>> Am 08.11.2017 um 23:38 schrieb Russell King - ARM Linux >>> : >>> >>> On Wed, Nov 08, 2017 at 10:36:04PM +,

Re: [PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-09 Thread Russell King - ARM Linux
On Thu, Nov 09, 2017 at 06:44:49AM +0100, H. Nikolaus Schaller wrote: > Hi Russel, Nikolus, > > Am 08.11.2017 um 23:38 schrieb Russell King - ARM Linux > > : > > > > On Wed, Nov 08, 2017 at 10:36:04PM +, Russell King - ARM Linux wrote: > >> We don't need a compiler warning there, we probabl

Re: [PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-08 Thread H. Nikolaus Schaller
Hi Russel, > Am 08.11.2017 um 23:38 schrieb Russell King - ARM Linux > : > > On Wed, Nov 08, 2017 at 10:36:04PM +, Russell King - ARM Linux wrote: >> We don't need a compiler warning there, we probably need better help >> text against DEBUG_LL and against EARLY_PRINTK. > > Actually, this is

Re: [PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-08 Thread Russell King - ARM Linux
On Wed, Nov 08, 2017 at 10:36:04PM +, Russell King - ARM Linux wrote: > We don't need a compiler warning there, we probably need better help > text against DEBUG_LL and against EARLY_PRINTK. Actually, this is already clearly stated against DEBUG_LL: Note that selecting this option w

Re: [PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-08 Thread Russell King - ARM Linux
On Wed, Nov 08, 2017 at 02:27:55PM -0800, Tony Lindgren wrote: > * H. Nikolaus Schaller [171108 21:32]: > > commit d2b310b0234c ("ARM: debug: Use generic 8250 debug_ll for omap2 and > > omap3/4/5 common uarts") > > commit fc23beb8a577 ("ARM: debug: Use generic 8250 debug_ll for omap3/4/5") > > >

Re: [PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-08 Thread Tony Lindgren
* H. Nikolaus Schaller [171108 21:32]: > commit d2b310b0234c ("ARM: debug: Use generic 8250 debug_ll for omap2 and > omap3/4/5 common uarts") > commit fc23beb8a577 ("ARM: debug: Use generic 8250 debug_ll for omap3/4/5") > > switched to generic 8250 debug_ll code which seems to be incompatible >

[PATCH] Kernel debugging: omap: print warning if CONFIG_DEBUG_LL is enabled

2017-11-08 Thread H. Nikolaus Schaller
commit d2b310b0234c ("ARM: debug: Use generic 8250 debug_ll for omap2 and omap3/4/5 common uarts") commit fc23beb8a577 ("ARM: debug: Use generic 8250 debug_ll for omap3/4/5") switched to generic 8250 debug_ll code which seems to be incompatible with at least OMAP5 boards (OMAP5EVM, Pyra) if CONFI