From: Courtney Cavin <courtney.cavin@sonymobile.com>
To: "Ivan T. Ivanov" <iivanov@mm-sol.com>
Cc: Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>, Rob Landley <rob@landley.net>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
David Brown <davidb@codeaurora.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-msm@vger.kernel.org" <linux-arm-msm@vger.kernel.org>
Subject: Re: [PATCH v2 1/3] usb: chipidea: msm: Add device tree binding information
Date: Wed, 19 Feb 2014 09:46:56 -0800 [thread overview]
Message-ID: <20140219174655.GN1706@sonymobile.com> (raw)
In-Reply-To: <1392824602.17130.68.camel@iivanov-dev>
On Wed, Feb 19, 2014 at 04:43:22PM +0100, Ivan T. Ivanov wrote:
>
> Hi,
>
> On Tue, 2014-02-18 at 13:26 -0800, Courtney Cavin wrote:
> > On Tue, Feb 18, 2014 at 02:21:19PM +0100, Ivan T. Ivanov wrote:
> > > From: "Ivan T. Ivanov" <iivanov@mm-sol.com>
> > >
> > > Document device tree binding information as required by
> > > the Qualcomm USB controller.
> > >
> > > Signed-off-by: Ivan T. Ivanov <iivanov@mm-sol.com>
> > > ---
> > > .../devicetree/bindings/usb/msm-hsusb.txt | 17 +++++++++++++++++
> >
> > Although you mentioned to Josh that this is intended for "non-standard"
> > Chipidea properties, I don't see any other than requiring that 'dr_mode'
> > must be "peripheral". It would seem that this should all be integrated
> > into a ci3xxx.txt.
>
> Hm, there is no ci3xxx.txt. The closest match is ci-hdrc-imx.txt.
> So it could be ci-hdrc-qcom.txt or my preferred name qcom,ci-hdrc.txt?
Sorry, I was referring to ci13xxx-imx.txt, which was apparently moved to
ci-hdrc-imx.txt. I was recommending to merge the two into one
'ci13xxx.txt', as this binding seems to be a new compatible for the same
basic chip. Now perhaps 'ci-hdrc.txt'.
Although I agree with Josh that this name should be changed, and I think
either of your two suggestions would be acceptable, I would like to at
least discuss the possibility of actually merging the two in this
series.
Comments?
-Courtney
next prev parent reply other threads:[~2014-02-19 17:46 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-18 13:21 [PATCH v2 0/3] usb: chipidea: msm: Clean and fix glue layer driver Ivan T. Ivanov
2014-02-18 13:21 ` Ivan T. Ivanov
2014-02-18 13:21 ` [PATCH v2 1/3] usb: chipidea: msm: Add device tree binding information Ivan T. Ivanov
2014-02-18 16:13 ` Josh Cartwright
2014-02-18 17:27 ` Ivan T. Ivanov
2014-02-18 19:07 ` Josh Cartwright
2014-02-18 21:26 ` Courtney Cavin
2014-02-19 15:43 ` Ivan T. Ivanov
2014-02-19 17:46 ` Courtney Cavin [this message]
2014-02-24 10:18 ` Ivan T. Ivanov
2014-02-25 1:57 ` Peter Chen
2014-02-25 1:57 ` Peter Chen
[not found] ` <1392729681-21022-1-git-send-email-iivanov-NEYub+7Iv8PQT0dZR+AlfA@public.gmane.org>
2014-02-18 13:21 ` [PATCH v2 2/3] usb: chipidea: msm: Add device tree support Ivan T. Ivanov
2014-02-18 13:21 ` Ivan T. Ivanov
2014-02-18 14:08 ` Josh Cartwright
2014-02-18 17:14 ` Ivan T. Ivanov
[not found] ` <1392729681-21022-3-git-send-email-iivanov-NEYub+7Iv8PQT0dZR+AlfA@public.gmane.org>
2014-02-18 17:53 ` Sergei Shtylyov
2014-02-18 17:53 ` Sergei Shtylyov
[not found] ` <53039E2E.2030503-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
2014-02-18 17:14 ` Ivan T. Ivanov
2014-02-18 17:14 ` Ivan T. Ivanov
2014-02-18 18:31 ` Sergei Shtylyov
[not found] ` <5303A71B.7060208-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
2014-02-18 21:34 ` Courtney Cavin
2014-02-18 21:34 ` Courtney Cavin
2014-02-18 23:32 ` Sergei Shtylyov
2014-02-19 15:48 ` Ivan T. Ivanov
2014-02-19 15:48 ` Ivan T. Ivanov
2014-02-19 20:24 ` Sergei Shtylyov
2014-02-19 20:24 ` Sergei Shtylyov
2014-02-20 9:53 ` [PATCH RESEND v2 1/3] usb: chipidea: msm: Add device tree binding information Ivan T. Ivanov
2014-02-24 8:15 ` Peter Chen
2014-02-24 10:09 ` Ivan T. Ivanov
2014-02-24 10:19 ` Peter Chen
2014-02-18 13:21 ` [PATCH v2 3/3] usb: chipidea: msm: Initialize offset of the capability registers Ivan T. Ivanov
2014-02-19 6:21 ` [PATCH v2 0/3] usb: chipidea: msm: Clean and fix glue layer driver Tim Bird
2014-02-19 10:32 ` Ivan T. Ivanov
2014-02-19 14:58 ` Ivan T. Ivanov
2014-02-20 4:07 ` Tim Bird
2014-02-20 4:07 ` Tim Bird
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140219174655.GN1706@sonymobile.com \
--to=courtney.cavin@sonymobile.com \
--cc=davidb@codeaurora.org \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=gregkh@linuxfoundation.org \
--cc=iivanov@mm-sol.com \
--cc=ijc+devicetree@hellion.org.uk \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=rob@landley.net \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.