All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Faiz Abbas <faiz_abbas-l0cyMroinI0@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-can-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	catalin.marinas-5wv7dgnIgG8@public.gmane.org,
	mark.rutland-5wv7dgnIgG8@public.gmane.org,
	mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org,
	wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org,
	sriram.dash-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org,
	dmurphy-l0cyMroinI0@public.gmane.org
Subject: Re: [PATCH v2 1/3] dt-bindings: m_can: Add Documentation for transceiver regulator
Date: Wed, 19 Feb 2020 14:35:29 -0600	[thread overview]
Message-ID: <20200219203529.GA21085@bogus> (raw)
In-Reply-To: <20200217142836.23702-2-faiz_abbas-l0cyMroinI0@public.gmane.org>

On Mon, Feb 17, 2020 at 07:58:34PM +0530, Faiz Abbas wrote:
> Some CAN transceivers have a standby line that needs to be asserted
> before they can be used. Model this GPIO lines as an optional
> fixed-regulator node. Document bindings for the same.
> 
> Signed-off-by: Faiz Abbas <faiz_abbas-l0cyMroinI0@public.gmane.org>
> ---
>  Documentation/devicetree/bindings/net/can/m_can.txt | 3 +++
>  1 file changed, 3 insertions(+)

This has moved to DT schema in my tree, so please adjust it and resend.

> diff --git a/Documentation/devicetree/bindings/net/can/m_can.txt b/Documentation/devicetree/bindings/net/can/m_can.txt
> index ed614383af9c..f17e2a5207dc 100644
> --- a/Documentation/devicetree/bindings/net/can/m_can.txt
> +++ b/Documentation/devicetree/bindings/net/can/m_can.txt
> @@ -48,6 +48,9 @@ Optional Subnode:
>  			  that can be used for CAN/CAN-FD modes. See
>  			  Documentation/devicetree/bindings/net/can/can-transceiver.txt
>  			  for details.
> +
> +- xceiver-supply: Regulator that powers the CAN transceiver.

The supply for a transceiver should go in the transceiver node.

> +
>  Example:
>  SoC dtsi:
>  m_can1: can@20e8000 {
> -- 
> 2.19.2
> 

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Faiz Abbas <faiz_abbas@ti.com>
Cc: mark.rutland@arm.com, devicetree@vger.kernel.org,
	lgirdwood@gmail.com, netdev@vger.kernel.org,
	sriram.dash@samsung.com, linux-kernel@vger.kernel.org,
	linux-can@vger.kernel.org, broonie@kernel.org,
	mkl@pengutronix.de, wg@grandegger.com, catalin.marinas@arm.com,
	linux-arm-kernel@lists.infradead.org, dmurphy@ti.com
Subject: Re: [PATCH v2 1/3] dt-bindings: m_can: Add Documentation for transceiver regulator
Date: Wed, 19 Feb 2020 14:35:29 -0600	[thread overview]
Message-ID: <20200219203529.GA21085@bogus> (raw)
In-Reply-To: <20200217142836.23702-2-faiz_abbas@ti.com>

On Mon, Feb 17, 2020 at 07:58:34PM +0530, Faiz Abbas wrote:
> Some CAN transceivers have a standby line that needs to be asserted
> before they can be used. Model this GPIO lines as an optional
> fixed-regulator node. Document bindings for the same.
> 
> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
> ---
>  Documentation/devicetree/bindings/net/can/m_can.txt | 3 +++
>  1 file changed, 3 insertions(+)

This has moved to DT schema in my tree, so please adjust it and resend.

> diff --git a/Documentation/devicetree/bindings/net/can/m_can.txt b/Documentation/devicetree/bindings/net/can/m_can.txt
> index ed614383af9c..f17e2a5207dc 100644
> --- a/Documentation/devicetree/bindings/net/can/m_can.txt
> +++ b/Documentation/devicetree/bindings/net/can/m_can.txt
> @@ -48,6 +48,9 @@ Optional Subnode:
>  			  that can be used for CAN/CAN-FD modes. See
>  			  Documentation/devicetree/bindings/net/can/can-transceiver.txt
>  			  for details.
> +
> +- xceiver-supply: Regulator that powers the CAN transceiver.

The supply for a transceiver should go in the transceiver node.

> +
>  Example:
>  SoC dtsi:
>  m_can1: can@20e8000 {
> -- 
> 2.19.2
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Faiz Abbas <faiz_abbas@ti.com>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	netdev@vger.kernel.org, linux-can@vger.kernel.org,
	broonie@kernel.org, lgirdwood@gmail.com, catalin.marinas@arm.com,
	mark.rutland@arm.com, mkl@pengutronix.de, wg@grandegger.com,
	sriram.dash@samsung.com, dmurphy@ti.com
Subject: Re: [PATCH v2 1/3] dt-bindings: m_can: Add Documentation for transceiver regulator
Date: Wed, 19 Feb 2020 14:35:29 -0600	[thread overview]
Message-ID: <20200219203529.GA21085@bogus> (raw)
In-Reply-To: <20200217142836.23702-2-faiz_abbas@ti.com>

On Mon, Feb 17, 2020 at 07:58:34PM +0530, Faiz Abbas wrote:
> Some CAN transceivers have a standby line that needs to be asserted
> before they can be used. Model this GPIO lines as an optional
> fixed-regulator node. Document bindings for the same.
> 
> Signed-off-by: Faiz Abbas <faiz_abbas@ti.com>
> ---
>  Documentation/devicetree/bindings/net/can/m_can.txt | 3 +++
>  1 file changed, 3 insertions(+)

This has moved to DT schema in my tree, so please adjust it and resend.

> diff --git a/Documentation/devicetree/bindings/net/can/m_can.txt b/Documentation/devicetree/bindings/net/can/m_can.txt
> index ed614383af9c..f17e2a5207dc 100644
> --- a/Documentation/devicetree/bindings/net/can/m_can.txt
> +++ b/Documentation/devicetree/bindings/net/can/m_can.txt
> @@ -48,6 +48,9 @@ Optional Subnode:
>  			  that can be used for CAN/CAN-FD modes. See
>  			  Documentation/devicetree/bindings/net/can/can-transceiver.txt
>  			  for details.
> +
> +- xceiver-supply: Regulator that powers the CAN transceiver.

The supply for a transceiver should go in the transceiver node.

> +
>  Example:
>  SoC dtsi:
>  m_can1: can@20e8000 {
> -- 
> 2.19.2
> 

  parent reply	other threads:[~2020-02-19 20:35 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-17 14:28 [PATCH v2 0/3] Add Support for MCAN in AM654x-idk Faiz Abbas
2020-02-17 14:28 ` Faiz Abbas
2020-02-17 14:28 ` Faiz Abbas
2020-02-17 14:28 ` [PATCH v2 1/3] dt-bindings: m_can: Add Documentation for transceiver regulator Faiz Abbas
2020-02-17 14:28   ` Faiz Abbas
2020-02-17 14:28   ` Faiz Abbas
     [not found]   ` <20200217142836.23702-2-faiz_abbas-l0cyMroinI0@public.gmane.org>
2020-02-19 20:35     ` Rob Herring [this message]
2020-02-19 20:35       ` Rob Herring
2020-02-19 20:35       ` Rob Herring
2020-02-21  8:31       ` Faiz Abbas
2020-02-21  8:31         ` Faiz Abbas
2020-02-21  8:31         ` Faiz Abbas
2020-02-21  8:31         ` Marc Kleine-Budde
2020-02-21  8:31           ` Marc Kleine-Budde
2020-02-21  8:31           ` Marc Kleine-Budde
2020-02-26  9:10           ` Faiz Abbas
2020-02-26  9:10             ` Faiz Abbas
2020-02-26  9:10             ` Faiz Abbas
2020-03-02  8:16             ` Faiz Abbas
2020-03-02  8:16               ` Faiz Abbas
2020-03-02  8:16               ` Faiz Abbas
2020-03-09 14:30               ` Faiz Abbas
2020-03-09 14:30                 ` Faiz Abbas
2020-03-09 14:30                 ` Faiz Abbas
2020-02-17 14:28 ` [PATCH v2 2/3] can: m_can: m_can_platform: Add support for enabling transceiver Faiz Abbas
2020-02-17 14:28   ` Faiz Abbas
2020-02-17 14:28   ` Faiz Abbas
2020-02-17 15:10   ` Dan Murphy
2020-02-17 15:10     ` Dan Murphy
2020-02-17 15:10     ` Dan Murphy
2020-02-21  7:54     ` Faiz Abbas
2020-02-21  7:54       ` Faiz Abbas
2020-02-21  7:54       ` Faiz Abbas
2020-02-21  8:09       ` Marc Kleine-Budde
2020-02-21  8:09         ` Marc Kleine-Budde
2020-02-17 14:28 ` [PATCH v2 3/3] arm64: defconfig: Add Support for Bosch M_CAN controllers Faiz Abbas
2020-02-17 14:28   ` Faiz Abbas
2020-02-17 14:28   ` Faiz Abbas

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=20200219203529.GA21085@bogus \
    --to=robh-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
    --cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=catalin.marinas-5wv7dgnIgG8@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=dmurphy-l0cyMroinI0@public.gmane.org \
    --cc=faiz_abbas-l0cyMroinI0@public.gmane.org \
    --cc=lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-can-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
    --cc=netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=sriram.dash-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.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.