Re: [PATCH] can-calc-bit-timing: add support for rcar_can

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

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

On 02/29/2016 11:18 AM, Ramesh Shanmugasundaram wrote:
> Signed-off-by: Ramesh Shanmugasundaram <ramesh.shanmugasundaram@bp.renesas.com>
> ---
> Hi Mark,
> 
>    This patch is generated on top of can-utils master
> 
>    commit:
>    cda6117 can: avoid using timeval for uapi
> 
>    in response to https://www.mail-archive.com/netdev@vger.kernel.org/msg99549.html
> 
>  can-calc-bit-timing.c | 35 +++++++++++++++++++++++++++++++++++
>  1 file changed, 35 insertions(+)
> 
> diff --git a/can-calc-bit-timing.c b/can-calc-bit-timing.c
> index 9031a78..b6236f0 100644
> --- a/can-calc-bit-timing.c
> +++ b/can-calc-bit-timing.c
> @@ -219,6 +219,26 @@ static void printf_btr_ti_hecc(struct can_bittiming *bt, int hdr)
>  	}
>  }
>  
> +static void printf_btr_rcar_can(struct can_bittiming *bt, int hdr)
> +{
> +	if (hdr) {
> +		printf("%10s", "CiBCR");
> +	} else {
> +		uint32_t bcr;
> +#define RCAR_CAN_BCR_TSEG1(x)   (((x) & 0x0f) << 20)
> +#define RCAR_CAN_BCR_BPR(x)     (((x) & 0x3ff) << 8)
> +#define RCAR_CAN_BCR_SJW(x)     (((x) & 0x3) << 4)
> +#define RCAR_CAN_BCR_TSEG2(x)   ((x) & 0x07)
> +
> +		bcr = RCAR_CAN_BCR_TSEG1(bt->phase_seg1 + bt->prop_seg - 1) |
> +			RCAR_CAN_BCR_BPR(bt->brp - 1) |
> +			RCAR_CAN_BCR_SJW(bt->sjw - 1) |
> +			RCAR_CAN_BCR_TSEG2(bt->phase_seg2 - 1);
> +
> +		printf("0x%08x", bcr<<8);
> +	}
> +}
> +
>  static struct can_bittiming_const can_calc_consts[] = {
>  	{
>  		.name = "sja1000",
> @@ -472,6 +492,21 @@ static struct can_bittiming_const can_calc_consts[] = {
>  
>  		.ref_clk = 13000000,
>  		.printf_btr = printf_btr_ti_hecc,
> +	},
> +	{
> +		.name = "rcar_can",
> +		.tseg1_min = 4,
> +		.tseg1_max = 16,
> +		.tseg2_min = 2,
> +		.tseg2_max = 8,
> +		.sjw_max = 4,
> +		.brp_min = 1,
> +		.brp_max = 1024,
> +		.brp_inc = 1,
> +
> +		.ref_clk = 65000000,
> +		.printf_btr = printf_btr_rcar_can,
> +	},
>  	}
       ^^^

This doesn't look right, fixed while applying.

thanks,
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: 455 bytes --]