Message ID | 20201214140109.21955-1-a-govindraju@ti.com |
---|---|
State | Superseded |
Headers | show |
Series | [v5] dt-bindings: usb: Add new compatible string for AM64 SoC | expand |
On Mon, Dec 14, 2020 at 07:31:09PM +0530, Aswath Govindraju wrote: > Add compatible string in j721e-usb binding file as the same USB subsystem > is present in AM64. > > Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> > --- > > Changes since v4: > - used oneOf instead of enum, as the schema has to convey that the strings > ti,j721e-usb and ti,am64-usb can be used combined or seperately in the > DT nodes. > > Changes since v3: > - used enum instead of anyOf. > > Changes since v2: > - added changes done over the versions. > > Changes since v1: > - replaced the '\t' at the beginning of the lines with spaces as it was > causing the dt_binding_check to fail. > > Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml > index 388245b91a55..21e51ceca293 100644 > --- a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml > +++ b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml > @@ -11,8 +11,12 @@ maintainers: > > properties: > compatible: > - items: > + oneOf: > - const: ti,j721e-usb > + - const: ti,am64-usb > + - items: > + - ti,j721e-usb > + - ti,am64-usb ^ const needed before the strings > > reg: > description: module registers > -- > 2.17.1 >
Hi Rob, On 15/12/20 2:18 am, Rob Herring wrote: > On Mon, Dec 14, 2020 at 07:31:09PM +0530, Aswath Govindraju wrote: >> Add compatible string in j721e-usb binding file as the same USB subsystem >> is present in AM64. >> >> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> >> --- >> >> Changes since v4: >> - used oneOf instead of enum, as the schema has to convey that the strings >> ti,j721e-usb and ti,am64-usb can be used combined or seperately in the >> DT nodes. >> >> Changes since v3: >> - used enum instead of anyOf. >> >> Changes since v2: >> - added changes done over the versions. >> >> Changes since v1: >> - replaced the '\t' at the beginning of the lines with spaces as it was >> causing the dt_binding_check to fail. >> >> Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml | 6 +++++- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml >> index 388245b91a55..21e51ceca293 100644 >> --- a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml >> +++ b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml >> @@ -11,8 +11,12 @@ maintainers: >> >> properties: >> compatible: >> - items: >> + oneOf: >> - const: ti,j721e-usb >> + - const: ti,am64-usb >> + - items: >> + - ti,j721e-usb >> + - ti,am64-usb > > ^ > > const needed before the strings > Thank you for the comments. I am very sorry for this error, I forgot to add my changes to the commit before creating the patch. I have sent a respin (v6). Thanks, Aswath >> >> reg: >> description: module registers >> -- >> 2.17.1 >>
diff --git a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml index 388245b91a55..21e51ceca293 100644 --- a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml +++ b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml @@ -11,8 +11,12 @@ maintainers: properties: compatible: - items: + oneOf: - const: ti,j721e-usb + - const: ti,am64-usb + - items: + - ti,j721e-usb + - ti,am64-usb reg: description: module registers
Add compatible string in j721e-usb binding file as the same USB subsystem is present in AM64. Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> --- Changes since v4: - used oneOf instead of enum, as the schema has to convey that the strings ti,j721e-usb and ti,am64-usb can be used combined or seperately in the DT nodes. Changes since v3: - used enum instead of anyOf. Changes since v2: - added changes done over the versions. Changes since v1: - replaced the '\t' at the beginning of the lines with spaces as it was causing the dt_binding_check to fail. Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)