| Message ID | 20230603200243.243878-4-varshini.rajendran@microchip.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add support for sam9x7 SoC family | expand |
Hey Varshini, On Sun, Jun 04, 2023 at 01:32:25AM +0530, Varshini Rajendran wrote: > Document the property clock-names in the schema. > > It fixes the dtbs_warning, s/dtbs_warning/dtbs_check warning/? > 'clock-names' does not match any of the regexes: 'pinctrl-[0-9]+' Does this fix a warning currently in the tree, or fix a warning introduced by some patches in this series? (Or both?) Cheers, Conor. > > Signed-off-by: Varshini Rajendran <varshini.rajendran@microchip.com> > --- > Documentation/devicetree/bindings/usb/generic-ehci.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > index 7e486cc6cfb8..542ac26960fc 100644 > --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > @@ -102,6 +102,10 @@ properties: > - if a USB DRD channel: first clock should be host and second > one should be peripheral > > + clock-names: > + minItems: 1 > + maxItems: 4 > + > power-domains: > maxItems: 1 > > -- > 2.25.1 >
On 03/06/2023 22:02, Varshini Rajendran wrote: > Document the property clock-names in the schema. > > It fixes the dtbs_warning, > 'clock-names' does not match any of the regexes: 'pinctrl-[0-9]+' You cut too much from the warning. Which target/board? > > Signed-off-by: Varshini Rajendran <varshini.rajendran@microchip.com> > --- > Documentation/devicetree/bindings/usb/generic-ehci.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > index 7e486cc6cfb8..542ac26960fc 100644 > --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > @@ -102,6 +102,10 @@ properties: > - if a USB DRD channel: first clock should be host and second > one should be peripheral > > + clock-names: > + minItems: 1 > + maxItems: 4 Not really, because we want them to be fixed, so you need to list the items. But it seems this is not needed at all... which boards and drivers use names? Best regards, Krzysztof
On 03/06/2023 at 23:15, Conor Dooley wrote: > Hey Varshini, > > On Sun, Jun 04, 2023 at 01:32:25AM +0530, Varshini Rajendran wrote: >> Document the property clock-names in the schema. >> >> It fixes the dtbs_warning, > s/dtbs_warning/dtbs_check warning/? > >> 'clock-names' does not match any of the regexes: 'pinctrl-[0-9]+' > Does this fix a warning currently in the tree, or fix a warning > introduced by some patches in this series? (Or both?) Our USB DT pattern is the same on all our newer SoC, to it mustn't be introduced by the addition of this one. Best regards, Nicolas >> Signed-off-by: Varshini Rajendran<varshini.rajendran@microchip.com> >> --- >> Documentation/devicetree/bindings/usb/generic-ehci.yaml | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml >> index 7e486cc6cfb8..542ac26960fc 100644 >> --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml >> +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml >> @@ -102,6 +102,10 @@ properties: >> - if a USB DRD channel: first clock should be host and second >> one should be peripheral >> >> + clock-names: >> + minItems: 1 >> + maxItems: 4 >> + >> power-domains: >> maxItems: 1 >> >> -- >> 2.25.1
diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml index 7e486cc6cfb8..542ac26960fc 100644 --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml @@ -102,6 +102,10 @@ properties: - if a USB DRD channel: first clock should be host and second one should be peripheral + clock-names: + minItems: 1 + maxItems: 4 + power-domains: maxItems: 1
Document the property clock-names in the schema. It fixes the dtbs_warning, 'clock-names' does not match any of the regexes: 'pinctrl-[0-9]+' Signed-off-by: Varshini Rajendran <varshini.rajendran@microchip.com> --- Documentation/devicetree/bindings/usb/generic-ehci.yaml | 4 ++++ 1 file changed, 4 insertions(+)