[dts] [PATCH] Allow to select nic and target options if default execution templates are to be used

Jianbo Liu jianbo.liu at linaro.org
Wed Jul 5 11:02:40 CEST 2017


On 5 July 2017 at 16:24, Herbert Guan <herbert.guan at arm.com> wrote:
> It's necessary to allow user selecting the nic and target options after reading out the default execution templates.
> Signed-off-by: Herbert Guan <herbert.guan at arm.com>
> ---
>  tools/setup.py | 34 +++++++++++++++++-----------------
>  1 file changed, 17 insertions(+), 17 deletions(-)
>
> diff --git a/tools/setup.py b/tools/setup.py
> index 090dda9..cc51ec5 100755
> --- a/tools/setup.py
> +++ b/tools/setup.py
> @@ -259,23 +259,23 @@ def config_execution():
>          opt = Option(**suites_option)
>          suites = opt.parse_input()
>
> -        nics = ['cfg']
> -        nics += NICS.keys()
> -        nic_option = {'prompt': 'Choose one of nics',
> -                      'type': 'choice',
> -                      'help': 'Choose one of dpdk support NIC',
> -                      'options': nics,
> -                      'default': '0'}
> -        opt = Option(**nic_option)
> -        nic_type = opt.parse_input()
> -
> -        target_option = {'prompt': 'Choose target for execution',
> -                         'type': 'choice',
> -                         'help': 'Choose one of dpdk targets',
> -                         'options': targets,
> -                         'default': '0'}
> -        opt = Option(**target_option)
> -        target = opt.parse_input()
> +    nics = ['cfg']
> +    nics += NICS.keys()
> +    nic_option = {'prompt': 'Choose one of nics',
> +                  'type': 'choice',
> +                  'help': 'Choose one of dpdk support NIC',
> +                  'options': nics,
> +                  'default': '0'}
> +    opt = Option(**nic_option)
> +    nic_type = opt.parse_input()
> +
> +    target_option = {'prompt': 'Choose target for execution',
> +                     'type': 'choice',
> +                     'help': 'Choose one of dpdk targets',
> +                     'options': targets,
> +                     'default': '0'}
> +    opt = Option(**target_option)
> +    target = opt.parse_input()
>
>      driver_option = {'prompt': 'Choose one of them',
>                       'type': 'choice',
> --
> 1.8.3.1
>

Acked-by: Jianbo Liu <jianbo.liu at linaro.org>


More information about the dts mailing list