Re: [PATCH V7 6/6] of: unittest: Statically apply overlays using fdtoverlay

[Viresh Kumar <viresh.kumar@linaro.org>]

On 09-02-21, 15:40, Viresh Kumar wrote:
> And after decent amount of effort understanding how to do this, I
> finally did it in a not so efficient way, I am sure you can help
> improving it :)

Ping!

Also, where do we send patches for dt-schema ? Which list ?

> Author: Viresh Kumar <viresh.kumar@linaro.org>
> Date:   Tue Feb 9 12:19:50 2021 +0530
> 
>     dt-validate: Skip "required property" checks for overlays
>     
>     The overlays may not carry the required properties and would depend on
>     the base dtb to carry those, there is no point raising those errors
>     here.
>     
>     Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
> ---
>  tools/dt-validate | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/tools/dt-validate b/tools/dt-validate
> index 410b0538ef47..c6117504f1d1 100755
> --- a/tools/dt-validate
> +++ b/tools/dt-validate
> @@ -80,6 +80,23 @@ show_unmatched = False
>                                    (filename, line, col, fullname, node['compatible']), file=sys.stderr)
>                              continue
>  
> +                        if nodename == '/':
> +                            is_fragment = False
> +                            for name in node.items():
> +                                if name[0] == 'fragment@0':
> +                                    is_fragment = True
> +                                    break;
> +
> +                            if is_fragment == True:
> +                                if 'required property' in error.message:
> +                                    continue
> +                                elif error.context:
> +                                    for e in error.context:
> +                                        if not 'required property' in e.message:
> +                                            break
> +                                    else:
> +                                        continue
> +
>                          print(dtschema.format_error(filename, error, nodename=nodename, verbose=verbose) +
>                              '\n\tFrom schema: ' + schema['$filename'],
>                              file=sys.stderr)

-- 
viresh