[U-Boot] [PATCH 1/6] mtd: cfi_flash: Use CONFIG_SYS_MONITOR_BASE only when defined

Vignesh Raghavendra vigneshr at ti.com
Wed Oct 9 09:41:27 UTC 2019


Hi Stefan,

On 09/10/19 12:46 PM, Stefan Roese wrote:
> Hi Vignesh,
> 
> sorry for the late review.
> 
> On 10.09.19 07:10, Vignesh Raghavendra wrote:
>> Make use of CONFIG_SYS_MONITOR_BASE only when available to avoid build
>> error when CONFIG_SYS_MONITOR_BASE is not defined.
>>
>> Signed-off-by: Vignesh Raghavendra <vigneshr at ti.com>
>> ---
>>   drivers/mtd/cfi_flash.c | 5 ++++-
>>   1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
>> index c59254c76e3e..fea15fb523d8 100644
>> --- a/drivers/mtd/cfi_flash.c
>> +++ b/drivers/mtd/cfi_flash.c
>> @@ -178,7 +178,8 @@ __maybe_weak u64 flash_read64(void *addr)
>>  
>> /*-----------------------------------------------------------------------
>>    */
>>   #if defined(CONFIG_ENV_IS_IN_FLASH) ||
>> defined(CONFIG_ENV_ADDR_REDUND) || \
>> -    (CONFIG_SYS_MONITOR_BASE >= CONFIG_SYS_FLASH_BASE)
>> +    (defined(CONFIG_SYS_MONITOR_BASE) && \
>> +    (CONFIG_SYS_MONITOR_BASE >= CONFIG_SYS_FLASH_BASE))
> 
> Here you extend the exisiting #ifdef ...
> 
>>   static flash_info_t *flash_get_info(ulong base)
>>   {
>>       int i;
>> @@ -2329,12 +2330,14 @@ static void flash_protect_default(void)
>>   #endif
>>         /* Monitor protection ON by default */
>> +#ifdef CONFIG_SYS_MONITOR_BASE
>>   #if (CONFIG_SYS_MONITOR_BASE >= CONFIG_SYS_FLASH_BASE) && \
>>       (!defined(CONFIG_MONITOR_IS_IN_RAM))
>>       flash_protect(FLAG_PROTECT_SET,
>>                 CONFIG_SYS_MONITOR_BASE,
>>                 CONFIG_SYS_MONITOR_BASE + monitor_flash_len  - 1,
>>                 flash_get_info(CONFIG_SYS_MONITOR_BASE));
>> +#endif
>>   #endif
> 
> ... and here you add a 2nd #ifdef. Why don't you extend the existing
> one here as well?
> 

Ah, yes, Will fix it in next version. Thanks!

> Thanks,
> Stefan

-- 
Regards
Vignesh


More information about the U-Boot mailing list