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: 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: -- Robert Hancock Senior Hardware Designer, Calian Advanced Technologies www.calian.com
