Messages in this thread | | | Subject | Re: [PATCH v2 1/2] ARM: debug: Add Iproc UART3 debug addresses | From | Ray Jui <> | Date | Fri, 1 Jun 2018 10:54:32 -0700 |
| |
Sorry Florian, I had a couple issues with my email client yesterday and I'm not sure if you have received my reply.
Please see my reply inline.
On 5/31/2018 3:16 PM, Florian Fainelli wrote: > > > On 05/31/2018 10:24 AM, Ray Jui wrote: >> >> >> On 5/31/2018 1:22 AM, Clément Péron wrote: >>> Hi Rai, >>> >>> On Wed, 30 May 2018 at 19:25, Ray Jui <ray.jui@broadcom.com> wrote: >>>> >>>> Hi Clément, >>>> >>>> Correct me if I'm wrong, but I thought the trend is to move to use >>>> earlycon that can be activated from kernel command line for early print >>>> before the serial driver is loaded. >>>> >>>> Have you tried earlcon? >>> No, only tested this method. >>> >>> Thanks, >>> Clement >>> >> >> If I remember it correctly, I think the trend is to use earlycon. There >> are obvious shortcomings by making this configuration compile time based. > > This is true, though on ARM 32-bit kernels DEBUG_LL gets used by the > kernel self-decompressor and also before earlycon has a chance to run, > this is useful to debugging memory issues where your memory > configuration is incorrect typically. >
This is true.
> Either way is fine with me (accepting or dropping) the patch, though > there is probably minimal impact in just accepting such a change. >
Will leave the decision to you then.
Thanks,
Ray
| |