Please only post plain text emails. On Fri, Jul 7, 2017 at 2:03 PM, Al Cooper <al.cooper@xxxxxxxxxxxx> wrote: > > > On Thu, Jul 6, 2017 at 10:18 AM, Rob Herring <robh@xxxxxxxxxx> wrote: >> >> On Tue, Jun 27, 2017 at 02:23:20PM -0400, Al Cooper wrote: >> > Add Device Tree binding document for Broadcom USB Device >> > Controller (BDC). >> > >> > Signed-off-by: Al Cooper <alcooperx@xxxxxxxxx> >> > --- >> > .../devicetree/bindings/usb/brcm,bdc-udc.txt | 28 ++++++++++++++++++++++ >> > 1 file changed, 28 insertions(+) >> > create mode 100644 Documentation/devicetree/bindings/usb/brcm,bdc-udc.txt >> > >> > diff --git a/Documentation/devicetree/bindings/usb/brcm,bdc-udc.txt b/Documentation/devicetree/bindings/usb/brcm,bdc-udc.txt >> > new file mode 100644 >> > index 0000000..4eeaddb >> > --- /dev/null >> > +++ b/Documentation/devicetree/bindings/usb/brcm,bdc-udc.txt >> > @@ -0,0 +1,28 @@ >> > +Broadcom USB Device Controller (BDC) >> > +==================================== >> > + >> > +Required properties: >> > + >> > +- compatible: must be one of: >> > + "brcm,bdc-udc-v0.16" >> > + "brcm,bdc-udc" >> >> Isn't "bdc-udc" redundant? > > > I'll change the base name to "brcm-udc" Well, it should be "brcm,<something>" with the vendor prefix followed by a comma. If the block is known as the BDC, then I'd use that. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html