On Thu, Jan 13, 2022 at 11:58 AM Robert Hancock <robert.hancock@xxxxxxxxxx> wrote: > > On Wed, 2022-01-12 at 13:46 -0600, Rob Herring wrote: > > On Wed, Jan 12, 2022 at 10:54 AM Robert Hancock > > <robert.hancock@xxxxxxxxxx> wrote: > > > On Tue, 2022-01-11 at 09:14 -0600, Rob Herring wrote: > > > > On Mon, 10 Jan 2022 14:19:34 -0600, Robert Hancock wrote: > > > > > Document the new snps,ref-clock-fladj property which can be used to set > > > > > the fractional portion of the reference clock period. > > > > > > > > > > Signed-off-by: Robert Hancock <robert.hancock@xxxxxxxxxx> > > > > > --- > > > > > Documentation/devicetree/bindings/usb/snps,dwc3.yaml | 12 ++++++++++++ > > > > > 1 file changed, 12 insertions(+) > > > > > > > > > > > > > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' > > > > on your patch (DT_CHECKER_FLAGS is new in v5.13): > > > > > > > > yamllint warnings/errors: > > > > > > > > dtschema/dtc warnings/errors: > > > > schemas/usb/snps,dwc3.yaml: ignoring, error in schema: properties: > > > > snps,ref- > > > > clock-fladj > > > > > > I'm assuming this schema file needs to be updated, but I'm not sure where > > > it > > > lives? I don't see such a file in the devicetree-org/dt-schema repo? > > > > Try Documentation/devicetree/bindings/usb/snps,dwc3.yaml > > That's the one I'm patching, but it seems like it is comparing that to another > schema file that I can't find, and is unhappy because the new property I'm > adding isn't there? The way it's defined in the bindings file itself seems no > different from the others: The problem is in Documentation/devicetree/bindings/usb/snps,dwc3.yaml with the property you added. It is as simple as that. It's failing on the meta-schema vendor-props.yaml which is in the dtschema repo (and also on your computer where dtschema is installed). Just run: dt-doc-validate -u Documentation/devicetree/bindings/ Documentation/devicetree/bindings/usb/snps,dwc3.yaml > > dtschema/dtc warnings/errors: > schemas/usb/snps,dwc3.yaml: ignoring, error in schema: properties: snps,ref- > clock-fladj > /builds/robherring/linux-dt- > review/Documentation/devicetree/bindings/usb/snps,dwc3.yaml: > properties:snps,ref-clock-fladj: 'oneOf' conditional failed, one must be fixed: The important parts are what came after the above. Read the 'hints'. Rob
