[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1386797711.1516.279.camel@bwh-desktop.uk.level5networks.com>
Date: Wed, 11 Dec 2013 21:35:11 +0000
From: Ben Hutchings <bhutchings@...arflare.com>
To: Shawn Bohrer <shawn.bohrer@...il.com>
CC: Or Gerlitz <ogerlitz@...lanox.com>,
Amir Vadai <amirv@...lanox.com>,
Richard Cochran <richardcochran@...il.com>,
<netdev@...r.kernel.org>, <tomk@...advisors.com>,
Shawn Bohrer <sbohrer@...advisors.com>
Subject: Re: [PATCH RFC] mlx4_en: Add PTP hardware clock
On Wed, 2013-12-11 at 12:24 -0600, Shawn Bohrer wrote:
[...]
> --- a/drivers/net/ethernet/mellanox/mlx4/en_clock.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/en_clock.c
[...]
> -void mlx4_en_ptp_overflow_check(struct mlx4_en_dev *mdev)
> -{
> - bool timeout = time_is_before_jiffies(mdev->last_overflow_check +
> - mdev->overflow_period);
> + /* Configure the PHC */
> + mdev->ptp_clock_info = mlx4_en_ptp_clock_info;
> + mlx4_foreach_port(i, dev, MLX4_PORT_TYPE_ETH) {
> + snprintf(mdev->ptp_clock_info.name,
> + sizeof(mdev->ptp_clock_info.name), "%pm",
[...]
Name should be "mlx4", I think.
Does each function have an independent clock or is this shared between
functions?
Ben.
--
Ben Hutchings, Staff Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists