[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140218213428.GM1706@sonymobile.com>
Date: Tue, 18 Feb 2014 13:34:29 -0800
From: Courtney Cavin <courtney.cavin@...ymobile.com>
To: Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
CC: "Ivan T. Ivanov" <iivanov@...sol.com>,
Peter Chen <Peter.Chen@...escale.com>,
Grant Likely <grant.likely@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-arm-msm@...r.kernel.org" <linux-arm-msm@...r.kernel.org>
Subject: Re: [PATCH v2 2/3] usb: chipidea: msm: Add device tree support
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@...sol.com>
>
> >>> Allows controller to be specified via device tree.
> >>> Pass PHY phandle specified in DT to core driver.
>
> >>> Signed-off-by: Ivan T. Ivanov <iivanov@...sol.com>
> >>> ---
> >>> 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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists