[U-Boot] [PATCH 03/12 v2] serial: atmel_usart: Use fixed clock value in SPL version with DM_SERIAL

Stefan Roese sr at denx.de
Fri Mar 29 15:31:55 UTC 2019


On 29.03.19 10:54, Eugen.Hristev at microchip.com wrote:
> 
> 
> On 26.03.2019 14:16, Stefan Roese wrote:
> 
>> This patch adds an alterative SPL version of atmel_serial_enable_clk().
>> This enables the usage of this driver without full clock support (in
>> drivers and DT nodes). This saves some space in the SPL image.
>>
>> Please note that this fixed clock support is only added to the SPL code
>> in the DM_SERIAL part of this file. All boards not using SPL & DM_SERIAL
>> should not be affected.
> 
> Hello Stefan,
> 
> For example, sama5d2_xplained_mmc_defconfig has DM_SERIAL and SPL.

Ah, I was not aware of that. It also has CONFIG_SPL_CLK enabled,
that's why I didn't spot it. I should make sure that in such cases
the original atmel_serial_enable_clk() function is used.

BTW: I did not enable CONFIG_SPL_CLK for size reasons of the SPL.

> This means it's affected ? If so, then it's not OK to take it as-is.

Sure, this is not good, as mentioned above. I'll re-work this
path to only enable the new fixed clock value for !SPL_CLK.
  
> And another question, is the rest of your patch series OK if this patch
> is missing ? Or it's any dependency on this ? the SPL for Gardena will
> not fit in the SRAM without this patch ?

Without this patch, building the gardena board will break (IIRC),
most likely because the CLK stuff is missing. And enabling SPL_CLK
will result in a too big image.

Thanks,
Stefan


More information about the U-Boot mailing list