[PATCH v4 1/9] CONFIG_NR_DRAM_BANKS: Remove unreferenced code as its always defined

Stefan Roese sr at denx.de
Thu Aug 13 13:28:29 CEST 2020


On 13.08.20 13:14, Andy Shevchenko wrote:
> On Thu, Aug 13, 2020 at 07:47:52AM +0200, Stefan Roese wrote:
>> Since commit 86cf1c82850f ("configs: Migrate CONFIG_NR_DRAM_BANKS") &
>> commit 999a772d9f24 ("Kconfig: Migrate CONFIG_NR_DRAM_BANKS"),
>> CONFIG_NR_DRAM_BANKS is always defined with a value (4 is default).
>> It makes no sense to still carry code that is guarded with
>> "#ifndef CONFIG_NR_DRAM_BANKS" (and similar). This patch removes
>> all these unreferenced code paths.
> 
> ...
> 
>>   	int pci_addr_cells, addr_cells, size_cells;
>> +	struct bd_info *bd = gd->bd;
>>   	int cells_per_record;
>>   	const u32 *prop;
>>   	int len;
> 
>>   	/* Add a region for our local memory */
> 
>> -#ifdef CONFIG_NR_DRAM_BANKS
>> -	struct bd_info *bd = gd->bd;
>> -
> 
> Can we leave assignment here?

Sure. Just curious why? Is it recommended not to make the assignment
with the declaration?

Thanks,
Stefan


More information about the U-Boot mailing list