Re: [PATCH 1/4] usb: chipidea: msm: Add device tree bindinginformation
From: Ivan T. Ivanov
Date: Wed Dec 04 2013 - 03:35:40 EST
Hi Peter,
Thank you for reviewing this patch.
On Wed, 2013-12-04 at 13:40 +0800, Peter Chen wrote:
> On Mon, Nov 11, 2013 at 03:35:34PM +0200, Ivan T. Ivanov wrote:
> > From: "Ivan T. Ivanov" <iivanov@xxxxxxxxxx>
> >
>
> Please add something in commit log
>
> > Signed-off-by: Ivan T. Ivanov <iivanov@xxxxxxxxxx>
> > Cc: devicetree@xxxxxxxxxxxxxxx
> > ---
> > .../devicetree/bindings/usb/msm-hsusb.txt | 16 ++++++++++++++++
> > 1 file changed, 16 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/msm-hsusb.txt b/Documentation/devicetree/bindings/usb/msm-hsusb.txt
> > index 5ea26c6..0a85eba 100644
> > --- a/Documentation/devicetree/bindings/usb/msm-hsusb.txt
> > +++ b/Documentation/devicetree/bindings/usb/msm-hsusb.txt
> > @@ -15,3 +15,19 @@ Example EHCI controller device node:
> > usb-phy = <&usb_otg>;
> > };
> >
> > +CI13xxx (Chipidea) USB controllers
> > +
>
> We have already renamed ci13xxx to ci_hdrc.
Yes, but the chip is still CI13XXX. Freescale documentations is also
referring to actual chip family.
>
> > +Required properties:
> > +- compatible: should contain "qcom,ci-hdrc"
> > +- reg: offset and length of the register set in the memory map
> > +- interrupts: interrupt-specifier for the controller interrupt.
> > +- usb-phy: phandle for the PHY device
> > +- dr_mode: Sould be "peripheral"
> > +
>
> Please keep alignment for "reg"
>
It looks aligned in editor with smart-tab support.
Regards,
Ivan
> Peter
>
> > + gadget@f9a55000 {
> > + compatible = "qcom,ci-hdrc";
> > + reg = <0xf9a55000 0x400>;
> > + dr_mode = "peripheral";
> > + interrupts = <0 134 0>;
> > + usb-phy = <&usb_otg>;
> > + };
> > \ No newline at end of file
> > --
> > 1.7.9.5
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-usb" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/