public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: shawn.guo@linaro.org (Shawn Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 2/6] Document: dt: fsl: snvs: change support syscon
Date: Mon, 25 May 2015 14:26:33 +0800	[thread overview]
Message-ID: <20150525062632.GE3264@dragon> (raw)
In-Reply-To: <1432305399-30571-3-git-send-email-Frank.Li@freescale.com>

On Fri, May 22, 2015 at 10:36:35PM +0800, Frank.Li at freescale.com wrote:
> From: Frank Li <Frank.Li@freescale.com>
> 
> snvs actually is multi fucntion driver.
> Change to use syscon to access register.

The rationale of 'interrupts' change did not get covered in the commit
log.

Shawn

> 
> Signed-off-by: Frank Li <Frank.Li@freescale.com>
> ---
>  .../devicetree/bindings/crypto/fsl-sec4.txt        | 28 ++++++++++++----------
>  1 file changed, 15 insertions(+), 13 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/crypto/fsl-sec4.txt b/Documentation/devicetree/bindings/crypto/fsl-sec4.txt
> index e402277..87f5921 100644
> --- a/Documentation/devicetree/bindings/crypto/fsl-sec4.txt
> +++ b/Documentation/devicetree/bindings/crypto/fsl-sec4.txt
> @@ -288,12 +288,13 @@ Secure Non-Volatile Storage (SNVS) Node
>      Node defines address range and the associated
>      interrupt for the SNVS function.  This function
>      monitors security state information & reports
> -    security violations.
> +    security violations. This also included rtc,
> +    system power off and ON/OFF key.
>  
>    - compatible
>        Usage: required
>        Value type: <string>
> -      Definition: Must include "fsl,sec-v4.0-mon".
> +      Definition: Must include "fsl,sec-v4.0-mon" and "syscon".
>  
>    - reg
>        Usage: required
> @@ -324,7 +325,7 @@ Secure Non-Volatile Storage (SNVS) Node
>             the child address, parent address, & length.
>  
>     - interrupts
> -      Usage: required
> +      Usage: optional
>        Value type: <prop_encoded-array>
>        Definition:  Specifies the interrupts generated by this
>             device.  The value of the interrupts property
> @@ -341,7 +342,7 @@ Secure Non-Volatile Storage (SNVS) Node
>  
>  EXAMPLE
>  	sec_mon at 314000 {
> -		compatible = "fsl,sec-v4.0-mon";
> +		compatible = "fsl,sec-v4.0-mon", "syscon";
>  		reg = <0x314000 0x1000>;
>  		ranges = <0 0x314000 0x1000>;
>  		interrupt-parent = <&mpic>;
> @@ -358,16 +359,19 @@ Secure Non-Volatile Storage (SNVS) Low Power (LP) RTC Node
>        Value type: <string>
>        Definition: Must include "fsl,sec-v4.0-mon-rtc-lp".
>  
> -  - reg
> +  - interrupts
>        Usage: required
> -      Value type: <prop-encoded-array>
> -      Definition: A standard property.  Specifies the physical
> -          address and length of the SNVS LP configuration registers.
> +      Value type: <prop_encoded-array>
> +      Definition: Specifies the interrupts generated by this
> +	   device.  The value of the interrupts property
> +	   consists of one interrupt specifier. The format
> +	   of the specifier is defined by the binding document
> +	   describing the node's interrupt parent.
>  
>  EXAMPLE
> -	sec_mon_rtc_lp at 314000 {
> +	sec_mon_rtc_lp at 1 {
>  		compatible = "fsl,sec-v4.0-mon-rtc-lp";
> -		reg = <0x34 0x58>;
> +		interrupts = <93 2>;
>  	};
>  
>  =====================================================================
> @@ -443,12 +447,10 @@ FULL EXAMPLE
>  		compatible = "fsl,sec-v4.0-mon";
>  		reg = <0x314000 0x1000>;
>  		ranges = <0 0x314000 0x1000>;
> -		interrupt-parent = <&mpic>;
> -		interrupts = <93 2>;
>  
>  		sec_mon_rtc_lp at 34 {
>  			compatible = "fsl,sec-v4.0-mon-rtc-lp";
> -			reg = <0x34 0x58>;
> +			interrupts = <93 2>;
>  		};
>  	};
>  
> -- 
> 1.9.1
> 

  reply	other threads:[~2015-05-25  6:26 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-22 14:36 [PATCH v4 0/6] Change snvs rtc and poweroff to use syscon add pwrkey driver Frank.Li at freescale.com
2015-05-22 14:36 ` [PATCH v4 1/6] rtc: arm: imx: snvs: change use syscon to access register Frank.Li at freescale.com
2015-05-25  6:18   ` Shawn Guo
2015-05-25  6:44   ` Shawn Guo
2015-05-22 14:36 ` [PATCH v4 2/6] Document: dt: fsl: snvs: change support syscon Frank.Li at freescale.com
2015-05-25  6:26   ` Shawn Guo [this message]
2015-05-22 14:36 ` [PATCH v4 3/6] arm: dts: imx: update snvs to use syscon access register Frank.Li at freescale.com
2015-05-25  6:39   ` Shawn Guo
2015-05-22 14:36 ` [PATCH v4 4/6] input: keyboard: imx: add snvs power key driver Frank.Li at freescale.com
2015-05-25  7:35   ` Shawn Guo
2015-05-22 14:36 ` [PATCH v4 5/6] document: devicetree: input: imx: i.mx snvs power device tree bindings Frank.Li at freescale.com
2015-05-22 14:36 ` [PATCH v4 6/6] arm: dts: imx6sx: enable snvs power key Frank.Li at freescale.com
2015-05-25  7:39   ` Shawn Guo
2015-05-25  6:47 ` [PATCH v4 0/6] Change snvs rtc and poweroff to use syscon add pwrkey driver Shawn Guo

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=20150525062632.GE3264@dragon \
    --to=shawn.guo@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox