Messages in this thread | | | Subject | Re: [PATCH v2 1/2] ARM: debug: Add Iproc UART3 debug addresses | From | Florian Fainelli <> | Date | Thu, 31 May 2018 15:16:20 -0700 |
| |
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.
Either way is fine with me (accepting or dropping) the patch, though there is probably minimal impact in just accepting such a change.
-- Florian
| |