Re: [PATCH v2 2/3] usb: chipidea: msm: Add device tree support

From: Courtney Cavin
Date: Tue Feb 18 2014 - 16:33:47 EST


On Tue, Feb 18, 2014 at 07:31:55PM +0100, Sergei Shtylyov wrote:
> On 02/18/2014 08:14 PM, Ivan T. Ivanov wrote:
>
> >>> From: "Ivan T. Ivanov" <iivanov@xxxxxxxxxx>
>
> >>> Allows controller to be specified via device tree.
> >>> Pass PHY phandle specified in DT to core driver.
>
> >>> Signed-off-by: Ivan T. Ivanov <iivanov@xxxxxxxxxx>
> >>> ---
> >>> drivers/usb/chipidea/ci_hdrc_msm.c | 23 ++++++++++++++++++++++-
> >>> 1 file changed, 22 insertions(+), 1 deletion(-)
>
> >> You also need to describe the binding you're creating in
> >> Documentation/devicetree/bindings/usb/<file>.txt.
>
> > Did you check "[PATCH v2 1/3]"?
>
> Did you send it to 'linux-usb'? I just didn't get the patch.
> (Typically, the bindings are described in the same patch the DT support is
> added to the driver bu YMMV, of course.)

Although I would personally agree that this is the most logical method,
it would appear that the DT guys disagree with us [1]. Lately, they
seem to have either given up or are otherwise preoccupied, so perhaps
you can still slip a few patches by them. ;)

-Courtney

[1] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/submitting-patches.txt
--
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/