Re: [PATCH net-next 00/11] remove open coded ns_to_timespec64 and reverse

["Keller, Jacob E" <jacob.e.keller@intel.com>]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 2121 bytes --]

On Tue, 2015-03-31 at 23:08 +0200, Richard Cochran wrote:
> This patch series is a follow up to the recent timespec64 work for the
> PTP Hardware Clock drivers.  Arnd noticed that drivers are using open
> coded implementations of ns_to_timespec64 and timespec64_to_ns.  This
> series replaces the open coded logic with the helper functions.
> 
> Thanks,
> Richard
> 
> Richard Cochran (11):
>   ptp: blackfin: use helpers for converting ns to timespec.
>   ptp: bnx2x: use helpers for converting ns to timespec.
>   ptp: tg3: use helpers for converting ns to timespec.
>   ptp: fec: use helpers for converting ns to timespec.
>   ptp: gianfar: use helpers for converting ns to timespec.
>   ptp: e1000e: use helpers for converting ns to timespec.
>   ptp: igb: use helpers for converting ns to timespec.
>   ptp: ixgbe: use helpers for converting ns to timespec.
>   ptp: mlx4: use helpers for converting ns to timespec.
>   ptp: stmmac: use helpers for converting ns to timespec.
>   ptp: cpts: use helpers for converting ns to timespec.
> 
>  drivers/net/ethernet/adi/bfin_mac.c              |    8 +++-----
>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c |    7 ++-----
>  drivers/net/ethernet/broadcom/tg3.c              |    4 +---
>  drivers/net/ethernet/freescale/fec_ptp.c         |    7 ++-----
>  drivers/net/ethernet/freescale/gianfar_ptp.c     |    8 +++-----
>  drivers/net/ethernet/intel/e1000e/ptp.c          |    4 +---
>  drivers/net/ethernet/intel/igb/igb_ptp.c         |    7 ++-----
>  drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c     |    7 ++-----
>  drivers/net/ethernet/mellanox/mlx4/en_clock.c    |    4 +---
>  drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c |    4 +---
>  drivers/net/ethernet/ti/cpts.c                   |    7 ++-----
>  11 files changed, 20 insertions(+), 47 deletions(-)
> 

For all the Intel drivers, this looks fine. I'm surprised I never
noticed before.

Acked-by: Jacob Keller <jacob.e.keller@intel.com>
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥