Re: [PATCH v4 00/10] ARM: dts: TI: Add CAN support

[Marc Kleine-Budde <mkl@pengutronix.de>]

[-- Attachment #1: Type: text/plain, Size: 1405 bytes --]

On 11/13/2014 01:22 PM, Roger Quadros wrote:
> Hi Tony,
> 
> These patches add CAN support for am33xx, am43xx and dra7 platforms.
> They can go in independent of [1] but depend on [1] for functionality.
> 
> Tested on am335x-evm, am437x-gp-evm and dra7-evm.
> 
> NOTE:
> - for DCAN to work on DRA7 we need this patch to keep the CAN PHY powered.
> https://lkml.org/lkml/2014/10/21/484
> 
> - on DRA7 there is another issue which causes undesired behaviour if WKUPAON and
> L4CFG clock domains are allowed to HW_AUTO. Undesired behaviour is in the form
> of DCAN1 module being stuck-in-transition when disabled and RAMINIT-done bit
> not getting set as part of RAMINIT mechanism. To address these issues there is a patch
> to prevent these clockdomains from HW_AUTO.
> https://github.com/rogerq/linux/commit/31d6da939dbbaf8ea2ab96ca5680b81cd5607e51
> Even without this second patch DCAN seems to work.

FWIW:
Acked-by: Marc Kleine-Budde <mkl@pengutronix.de>

I've send a pull request with the CAN driver changes to David Miller
yesterday, the code is supposed to hit mainline in v3.19.

regards,
Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]