On Fri, Jul 7, 2017 at 4:15 PM, Rob Herring <robh@kernel.org> wrote:
>
> >> > +====================================
> >> > +
> >> > +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.

Actually I meant to say I'd change it to "brcm,udc". I thought that udc
would be better than bdc because BDC stands for Broadcom Universal Device
Controller, so the "b" seems redundant in "brcm,bdc" and I thought that at
a glance more people would know what a "udc" driver was that what a "bdc"
driver was. I you still think "brcm,bdc" is better, I'll change it to that.

Thanks
Al