Re: [Intel-wired-lan] [PATCH 01/27] timecounter: Make cyclecounter struct part of timecounter struct

2018-01-09 Thread Sagar Arun Kamble

Thanks Aaron.

Regards,
Sagar

On 1/9/2018 3:50 AM, Brown, Aaron F wrote:

From: Intel-wired-lan [mailto:intel-wired-lan-boun...@osuosl.org] On
Behalf Of Sagar Arun Kamble
Sent: Thursday, December 14, 2017 11:38 PM
To: linux-ker...@vger.kernel.org
Cc: alsa-de...@alsa-project.org; linux-r...@vger.kernel.org;
netdev@vger.kernel.org; Richard Cochran ;
Stephen Boyd ; Chris Wilson ; John Stultz ; intel-wired-
l...@lists.osuosl.org; Thomas Gleixner ; Kamble, Sagar A
; kvm...@lists.cs.columbia.edu; linux-arm-
ker...@lists.infradead.org
Subject: [Intel-wired-lan] [PATCH 01/27] timecounter: Make cyclecounter
struct part of timecounter struct

There is no real need for the users of timecounters to define cyclecounter
and timecounter variables separately. Since timecounter will always be
based on cyclecounter, have cyclecounter struct as member of timecounter
struct.

v2: Rebase.

Suggested-by: Chris Wilson 
Signed-off-by: Sagar Arun Kamble 
Cc: Chris Wilson 
Cc: Richard Cochran 
Cc: John Stultz 
Cc: Thomas Gleixner 
Cc: Stephen Boyd 
Cc: linux-ker...@vger.kernel.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: netdev@vger.kernel.org
Cc: intel-wired-...@lists.osuosl.org
Cc: linux-r...@vger.kernel.org
Cc: alsa-de...@alsa-project.org
Cc: kvm...@lists.cs.columbia.edu
Acked-by: Jeff Kirsher  (Intel drivers)
---
  arch/microblaze/kernel/timer.c | 20 ++--
  drivers/clocksource/arm_arch_timer.c   | 19 ++--
  drivers/net/ethernet/amd/xgbe/xgbe-dev.c   |  3 +-
  drivers/net/ethernet/amd/xgbe/xgbe-ptp.c   |  9 +++---
  drivers/net/ethernet/amd/xgbe/xgbe.h   |  1 -
  drivers/net/ethernet/broadcom/bnx2x/bnx2x.h|  1 -
  drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c   | 20 ++--
  drivers/net/ethernet/freescale/fec.h   |  1 -
  drivers/net/ethernet/freescale/fec_ptp.c   | 30 +-
  drivers/net/ethernet/intel/e1000e/e1000.h  |  1 -
  drivers/net/ethernet/intel/e1000e/netdev.c | 27 
  drivers/net/ethernet/intel/e1000e/ptp.c|  2 +-
  drivers/net/ethernet/intel/igb/igb.h   |  1 -
  drivers/net/ethernet/intel/igb/igb_ptp.c   | 25 ---
  drivers/net/ethernet/intel/ixgbe/ixgbe.h   |  1 -
  drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c   | 17 +-
  drivers/net/ethernet/mellanox/mlx4/en_clock.c  | 28 -
  drivers/net/ethernet/mellanox/mlx4/mlx4_en.h   |  1 -
  .../net/ethernet/mellanox/mlx5/core/lib/clock.c| 34 ++--
  drivers/net/ethernet/qlogic/qede/qede_ptp.c| 20 ++--
  drivers/net/ethernet/ti/cpts.c | 36 --
  drivers/net/ethernet/ti/cpts.h |  1 -
  include/linux/mlx5/driver.h|  1 -
  include/linux/timecounter.h|  4 +--
  include/sound/hdaudio.h|  1 -
  kernel/time/timecounter.c  | 28 -
  sound/hda/hdac_stream.c|  7 +++--
  virt/kvm/arm/arch_timer.c  |  6 ++--
  28 files changed, 163 insertions(+), 182 deletions(-)


For Intel e1000e and igb drivers:
Tested-by: Aaron Brown 




RE: [Intel-wired-lan] [PATCH 01/27] timecounter: Make cyclecounter struct part of timecounter struct

2018-01-08 Thread Brown, Aaron F
> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@osuosl.org] On
> Behalf Of Sagar Arun Kamble
> Sent: Thursday, December 14, 2017 11:38 PM
> To: linux-ker...@vger.kernel.org
> Cc: alsa-de...@alsa-project.org; linux-r...@vger.kernel.org;
> netdev@vger.kernel.org; Richard Cochran ;
> Stephen Boyd ; Chris Wilson  wilson.co.uk>; John Stultz ; intel-wired-
> l...@lists.osuosl.org; Thomas Gleixner ; Kamble, Sagar A
> ; kvm...@lists.cs.columbia.edu; linux-arm-
> ker...@lists.infradead.org
> Subject: [Intel-wired-lan] [PATCH 01/27] timecounter: Make cyclecounter
> struct part of timecounter struct
> 
> There is no real need for the users of timecounters to define cyclecounter
> and timecounter variables separately. Since timecounter will always be
> based on cyclecounter, have cyclecounter struct as member of timecounter
> struct.
> 
> v2: Rebase.
> 
> Suggested-by: Chris Wilson 
> Signed-off-by: Sagar Arun Kamble 
> Cc: Chris Wilson 
> Cc: Richard Cochran 
> Cc: John Stultz 
> Cc: Thomas Gleixner 
> Cc: Stephen Boyd 
> Cc: linux-ker...@vger.kernel.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: netdev@vger.kernel.org
> Cc: intel-wired-...@lists.osuosl.org
> Cc: linux-r...@vger.kernel.org
> Cc: alsa-de...@alsa-project.org
> Cc: kvm...@lists.cs.columbia.edu
> Acked-by: Jeff Kirsher  (Intel drivers)
> ---
>  arch/microblaze/kernel/timer.c | 20 ++--
>  drivers/clocksource/arm_arch_timer.c   | 19 ++--
>  drivers/net/ethernet/amd/xgbe/xgbe-dev.c   |  3 +-
>  drivers/net/ethernet/amd/xgbe/xgbe-ptp.c   |  9 +++---
>  drivers/net/ethernet/amd/xgbe/xgbe.h   |  1 -
>  drivers/net/ethernet/broadcom/bnx2x/bnx2x.h|  1 -
>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c   | 20 ++--
>  drivers/net/ethernet/freescale/fec.h   |  1 -
>  drivers/net/ethernet/freescale/fec_ptp.c   | 30 +-
>  drivers/net/ethernet/intel/e1000e/e1000.h  |  1 -
>  drivers/net/ethernet/intel/e1000e/netdev.c | 27 
>  drivers/net/ethernet/intel/e1000e/ptp.c|  2 +-
>  drivers/net/ethernet/intel/igb/igb.h   |  1 -
>  drivers/net/ethernet/intel/igb/igb_ptp.c   | 25 ---
>  drivers/net/ethernet/intel/ixgbe/ixgbe.h   |  1 -
>  drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c   | 17 +-
>  drivers/net/ethernet/mellanox/mlx4/en_clock.c  | 28 -
>  drivers/net/ethernet/mellanox/mlx4/mlx4_en.h   |  1 -
>  .../net/ethernet/mellanox/mlx5/core/lib/clock.c| 34 ++--
>  drivers/net/ethernet/qlogic/qede/qede_ptp.c| 20 ++--
>  drivers/net/ethernet/ti/cpts.c | 36 
> --
>  drivers/net/ethernet/ti/cpts.h |  1 -
>  include/linux/mlx5/driver.h|  1 -
>  include/linux/timecounter.h|  4 +--
>  include/sound/hdaudio.h|  1 -
>  kernel/time/timecounter.c  | 28 -
>  sound/hda/hdac_stream.c|  7 +++--
>  virt/kvm/arm/arch_timer.c  |  6 ++--
>  28 files changed, 163 insertions(+), 182 deletions(-)
> 

For Intel e1000e and igb drivers:
Tested-by: Aaron Brown