From: Rob Herring <robh@kernel.org>
To: Jun Li <jun.li@nxp.com>
Cc: "heikki.krogerus@linux.intel.com"
<heikki.krogerus@linux.intel.com>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
"hdegoede@redhat.com" <hdegoede@redhat.com>,
"andy.shevchenko@gmail.com" <andy.shevchenko@gmail.com>,
"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
dl-linux-imx <linux-imx@nxp.com>
Subject: [v3,1/2] dt-bindings: usb: add documentation for typec switch via GPIO
Date: Tue, 12 Mar 2019 09:45:27 -0500 [thread overview]
Message-ID: <20190312144527.GA18735@bogus> (raw)
On Mon, Mar 11, 2019 at 10:40:09AM +0000, Jun Li wrote:
> Some typec super speed active channel switch can be controlled via
> a GPIO, this binding can be used to specify the switch node by
> a GPIO and the remote endpoint of its consumer.
>
> Signed-off-by: Li Jun <jun.li@nxp.com>
> ---
> .../devicetree/bindings/usb/typec-switch-gpio.txt | 30 ++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/usb/typec-switch-gpio.txt b/Documentation/devicetree/bindings/usb/typec-switch-gpio.txt
> new file mode 100644
> index 0000000..4ef76cf
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/usb/typec-switch-gpio.txt
> @@ -0,0 +1,30 @@
> +Typec orientation switch via a GPIO
> +-----------------------------------
> +
> +Required properties:
> +- compatible: should be set one of following:
> + - "nxp,ptn36043" for NXP Type-C SuperSpeed active switch.
> +
> +- gpios: the GPIO used to switch the super speed active channel,
Perhaps switch-gpios in case there are other gpios needed.
> + GPIO_ACTIVE_HIGH: GPIO state high for cc1;
> + GPIO_ACTIVE_LOW: GPIO state low for cc1.
> +- orientation-switch: must be present.
Why is this needed? The compatible can't imply this?
> +
> +Required sub-node:
> +- port: specify the remote endpoint of typec switch consumer.
> +
> +Example:
> +
> +ptn36043 {
> + compatible = "nxp,ptn36043";
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_ss_sel>;
> + gpios = <&gpio3 15 GPIO_ACTIVE_HIGH>;
> + orientation-switch;
> +
> + port {
> + usb3_data_ss: endpoint {
> + remote-endpoint = <&typec_con_ss>;
> + };
> + };
> +};
> --
> 2.7.4
>
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Jun Li <jun.li@nxp.com>
Cc: "heikki.krogerus@linux.intel.com"
<heikki.krogerus@linux.intel.com>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
"hdegoede@redhat.com" <hdegoede@redhat.com>,
"andy.shevchenko@gmail.com" <andy.shevchenko@gmail.com>,
"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
dl-linux-imx <linux-imx@nxp.com>
Subject: Re: [PATCH v3 1/2] dt-bindings: usb: add documentation for typec switch via GPIO
Date: Tue, 12 Mar 2019 09:45:27 -0500 [thread overview]
Message-ID: <20190312144527.GA18735@bogus> (raw)
In-Reply-To: <1552299557-6306-1-git-send-email-jun.li@nxp.com>
On Mon, Mar 11, 2019 at 10:40:09AM +0000, Jun Li wrote:
> Some typec super speed active channel switch can be controlled via
> a GPIO, this binding can be used to specify the switch node by
> a GPIO and the remote endpoint of its consumer.
>
> Signed-off-by: Li Jun <jun.li@nxp.com>
> ---
> .../devicetree/bindings/usb/typec-switch-gpio.txt | 30 ++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/usb/typec-switch-gpio.txt b/Documentation/devicetree/bindings/usb/typec-switch-gpio.txt
> new file mode 100644
> index 0000000..4ef76cf
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/usb/typec-switch-gpio.txt
> @@ -0,0 +1,30 @@
> +Typec orientation switch via a GPIO
> +-----------------------------------
> +
> +Required properties:
> +- compatible: should be set one of following:
> + - "nxp,ptn36043" for NXP Type-C SuperSpeed active switch.
> +
> +- gpios: the GPIO used to switch the super speed active channel,
Perhaps switch-gpios in case there are other gpios needed.
> + GPIO_ACTIVE_HIGH: GPIO state high for cc1;
> + GPIO_ACTIVE_LOW: GPIO state low for cc1.
> +- orientation-switch: must be present.
Why is this needed? The compatible can't imply this?
> +
> +Required sub-node:
> +- port: specify the remote endpoint of typec switch consumer.
> +
> +Example:
> +
> +ptn36043 {
> + compatible = "nxp,ptn36043";
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_ss_sel>;
> + gpios = <&gpio3 15 GPIO_ACTIVE_HIGH>;
> + orientation-switch;
> +
> + port {
> + usb3_data_ss: endpoint {
> + remote-endpoint = <&typec_con_ss>;
> + };
> + };
> +};
> --
> 2.7.4
>
next reply other threads:[~2019-03-12 14:45 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-12 14:45 Rob Herring [this message]
2019-03-12 14:45 ` [PATCH v3 1/2] dt-bindings: usb: add documentation for typec switch via GPIO Rob Herring
-- strict thread matches above, loose matches on Subject: below --
2019-03-19 8:18 [v3,1/2] " Heikki Krogerus
2019-03-19 8:18 ` [PATCH v3 1/2] " Heikki Krogerus
2019-03-18 10:59 [v3,1/2] " Jun Li
2019-03-18 10:59 ` [PATCH v3 1/2] " Jun Li
2019-03-18 10:48 [v3,1/2] " Jun Li
2019-03-18 10:48 ` [PATCH v3 1/2] " Jun Li
2019-03-18 10:46 [v3,1/2] " Jun Li
2019-03-18 10:46 ` [PATCH v3 1/2] " Jun Li
2019-03-13 9:35 [v3,1/2] " Heikki Krogerus
2019-03-13 9:35 ` [PATCH v3 1/2] " Heikki Krogerus
2019-03-12 11:46 [v3,1/2] " Hans de Goede
2019-03-12 11:46 ` [PATCH v3 1/2] " Hans de Goede
2019-03-12 10:45 [v3,1/2] " Heikki Krogerus
2019-03-12 10:45 ` [PATCH v3 1/2] " Heikki Krogerus
2019-03-12 10:32 [v3,1/2] " Jun Li
2019-03-12 10:32 ` [PATCH v3 1/2] " Jun Li
2019-03-11 11:12 [v3,1/2] " Hans de Goede
2019-03-11 11:12 ` [PATCH v3 1/2] " Hans de Goede
2019-03-11 11:02 [v3,1/2] " Hans de Goede
2019-03-11 11:02 ` [PATCH v3 1/2] " Hans de Goede
2019-03-11 10:40 [v3,2/2] usb: typec: add typec switch via GPIO control Jun Li
2019-03-11 10:40 ` [PATCH v3 2/2] " Jun Li
2019-03-11 10:40 [v3,1/2] dt-bindings: usb: add documentation for typec switch via GPIO Jun Li
2019-03-11 10:40 ` [PATCH v3 1/2] " Jun Li
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=20190312144527.GA18735@bogus \
--to=robh@kernel.org \
--cc=andy.shevchenko@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=hdegoede@redhat.com \
--cc=heikki.krogerus@linux.intel.com \
--cc=jun.li@nxp.com \
--cc=linux-imx@nxp.com \
--cc=linux-usb@vger.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.