Hi Denis,

On Wed, Mar 12, 2014 at 7:01 AM, Denis Carikli <de...@eukrea.com> wrote:
> From: Fabio Estevam <fabio.este...@freescale.com>
>
> This patch was adapted from the thread named
> "USB Host support for mx25" on linux-usb@vger.kernel.org
>

As you add me in the From field, you also need to add:

Signed-off-by: Fabio Estevam <fabio.este...@freescale.com> above your
Signed-off-by line.

> Signed-off-by: Denis Carikli <de...@eukrea.com>
> +
> +       usbphy {
> +               #address-cells = <1>;
> +               #size-cells = <0>;
> +               compatible = "simple-bus";

I made this comment earlier: why do we place usbphy0/1 under simple-bus?

This is not documented in the the bindings.

> +
> +               usbphy0: usbphy@0 {
> +                       reg = <0>;
> +                       compatible = "usb-nop-xceiv";
> +               };
> +
> +               usbphy1: usbphy@1 {
> +                       reg = <1>;
> +                       compatible = "usb-nop-xceiv";
> +               };
> +       };

Regards,

Fabio Estevam
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to