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.