Suggestion: Revert commit c7fad78ec0ee ("Convert CONFIG_SYS_BR0_PRELIM et al to Kconfig")

[Tom Rini]

On Tue, Apr 26, 2022 at 08:17:40PM +0200, Pali Rohár wrote:

> Hello! I would suggest to revert commit c7fad78ec0ee ("Convert
> CONFIG_SYS_BR0_PRELIM et al to Kconfig").
> 
> The reason is that this commit made configuration, understanding,
> maintenance and debugging of the powerpc/mpc85xx Local Bus Controller
> hard, mainly impossible.
> 
> This commit completely removed usage of named macros, to easily check
> address and size of the LBC memory banks. Removal was done also for
> explaining comments of configuration options.
> 
> It is just a mess what this commit introduced and took me really long
> time to debug and understand what is happening here... until I reverted
> this commit manually in my tree.
> 
> Any opinions?
> 
> Btw, current values are wrong.

AFAICT, the current values match what was in use prior.  But, these
should probably not be in CONFIG namespace at all and pulled from either
device tree or some other non-board.h header file.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220426/10839bbe/attachment.sig>