On 8/14/19 12:12 AM, Michal Simek wrote:
On 13. 08. 19 23:21, Stephen Warren wrote:
On 8/1/19 10:48 PM, Michal Simek wrote:
Test is using random command which has own Kconfig symbol CMD_RANDOM
which
already depends on CMD_MEMORY. That's why replace cmd_memory by
cmd_random.

This might not always be true; I think it'd be better to keep the
existing dependency list entries and just add CMD_RANDOM:

Can you please tell me when exactly? I didn't see that.

If in the future somebody changes the Kconfig file so that CMD_RANDOM doesn't depend on or select CMD_MEMORY. It may well not be possible right now; I just want to make sure the test doesn't rely on implementation details of CMD_RANDOM/CMD_MEMORY; if it explicitly needs both, it should simply depend on both.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to