Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 16 Feb 2021 16:09:57 +0530
From:   Viresh Kumar <viresh.kumar@linaro.org>
To:     Rob Herring <robh@kernel.org>
Cc:     Frank Rowand <frowand.list@gmail.com>,
        Pantelis Antoniou <pantelis.antoniou@konsulko.com>,
        Masahiro Yamada <masahiroy@kernel.org>,
        Vincent Guittot <vincent.guittot@linaro.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Anmar Oueja <anmar.oueja@linaro.org>,
        Bill Mills <bill.mills@linaro.org>,
        David Gibson <david@gibson.dropbear.id.au>,
        devicetree@vger.kernel.org, Michal Marek <michal.lkml@markovi.net>
Subject: Re: [PATCH V7 6/6] of: unittest: Statically apply overlays using
 fdtoverlay
Message-ID: <20210216103957.gwikizhtoxvb66le@vireshk-i7>
References: <cover.1611904394.git.viresh.kumar@linaro.org>
 <3683a542d4141cfcf9c2524a40a9ee75b657c1c2.1611904394.git.viresh.kumar@linaro.org>
 <20210204015409.GA3150757@robh.at.kernel.org>
 <20210208111813.oql6jmeaxuq3btz7@vireshk-i7>
 <CAL_JsqK+eM_RrzXnbm2GJGJuVwAjEz1TeTux18Hc727vovbqDA@mail.gmail.com>
 <20210209101058.kmqqjxfjz3alre55@vireshk-i7>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210209101058.kmqqjxfjz3alre55@vireshk-i7>
User-Agent: NeoMutt/20180716-391-311a52
Precedence: bulk

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