[PATCH 2/2] serial: a37xx: Do not call get_ref_clk() in _debug_uart_init()

Pali Rohár pali at kernel.org
Wed Aug 11 20:57:51 CEST 2021


On Monday 26 July 2021 17:24:26 Marek Behun wrote:
> On Mon, 26 Jul 2021 16:58:04 +0200
> Pali Rohár <pali at kernel.org> wrote:
> 
> > On Monday 26 July 2021 16:55:22 Marek Behun wrote:
> > > On Mon, 26 Jul 2021 14:58:59 +0200
> > > Pali Rohár <pali at kernel.org> wrote:
> > >   
> > > > Static inline function _debug_uart_init() should avoid calling external
> > > > (non-inline) functions.  
> > > 
> > > Why?  
> > 
> > Function is called in stage when stack is not fully initialized and
> > documentation suggest to avoid stack usage and other functions.
> 
> OK, but maybe we should use the macro names for register constants.
> 
> Could you move (in a separate patch) the corresponding macros from
> arch/arm/mach-mvebu/armada3700/cpu.c to
> arch/arm/mach-mvebu/include/mach/soc.h and then in this patch
> include <asm/arch/soc.h> in the serial driver and use the constant
> names?

Implemented in separate patch as Stefan wanted:
http://patchwork.ozlabs.org/project/uboot/patch/20210811185330.15414-2-pali@kernel.org/

> Thanks.
> 
> Marek


More information about the U-Boot mailing list