netdev - Re: [PATCH v2 2/3] net: can: c_can: Add syscon/regmap RAMINIT mechanism

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140930132650.GN1325@katana>
Date:	Tue, 30 Sep 2014 15:26:50 +0200
From:	Wolfram Sang <wsa@...-dreams.de>
To:	Roger Quadros <rogerq@...com>
Cc:	wg@...ndegger.com, mkl@...gutronix.de, tony@...mide.com,
	tglx@...utronix.de, mugunthanvnm@...com, george.cherian@...com,
	balbi@...com, nsekhar@...com, nm@...com,
	sergei.shtylyov@...entembedded.com, linux-omap@...r.kernel.org,
	linux-can@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH v2 2/3] net: can: c_can: Add syscon/regmap RAMINIT
 mechanism

On Tue, Sep 09, 2014 at 05:31:09PM +0300, Roger Quadros wrote:
> Some TI SoCs like DRA7 have a RAMINIT register specification
> different from the other AMxx SoCs and as expected by the
> existing driver.
> 
> To add more insanity, this register is shared with other
> IPs like DSS, PCIe and PWM.
> 
> Provides a more generic mechanism to specify the RAMINIT
> register location and START/DONE bit position and use the
> syscon/regmap framework to access the register.
> 
> Signed-off-by: Roger Quadros <rogerq@...com>
> ---
>  .../devicetree/bindings/net/can/c_can.txt          |   7 ++
>  drivers/net/can/c_can/c_can.h                      |  11 ++-
>  drivers/net/can/c_can/c_can_platform.c             | 109 +++++++++++++++------
>  3 files changed, 95 insertions(+), 32 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/can/c_can.txt b/Documentation/devicetree/bindings/net/can/c_can.txt
> index 8f1ae81..e12d1a1 100644
> --- a/Documentation/devicetree/bindings/net/can/c_can.txt
> +++ b/Documentation/devicetree/bindings/net/can/c_can.txt
> @@ -13,6 +13,13 @@ Optional properties:
>  - ti,hwmods		: Must be "d_can<n>" or "c_can<n>", n being the
>  			  instance number
>  
> +- ti,raminit-syscon	: Handle to system control region that contains the
> +			  RAMINIT register. If specified, the second memory resource
> +			  in the reg property must index into the RAMINIT
> +			  register within the syscon region

There seems to be a simple "syscon" property these days.

> +- ti,raminit-start-bit	: Bit posistion of START bit in the RAMINIT register
> +- ti,raminit-done-bit	: Bit position of DONE bit in the RAMINIT register

This should not be encoded in DT! This is not describing hardware setup.
The driver should know where the bits are for the syscon phandle,
depending on which SoC it runs...


Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)