From: Simon Horman <horms@kernel.org>
To: Karol Kolacinski <karol.kolacinski@intel.com>
Cc: netdev@vger.kernel.org, anthony.l.nguyen@intel.com,
intel-wired-lan@lists.osuosl.org, przemyslaw.kitszel@intel.com
Subject: Re: [Intel-wired-lan] [PATCH v4 iwl-next 0/4] ice: Implement PTP support for E830 devices
Date: Fri, 26 Jul 2024 18:28:23 +0100 [thread overview]
Message-ID: <20240726172823.GA1699125@kernel.org> (raw)
In-Reply-To: <20240726113631.200083-6-karol.kolacinski@intel.com>
On Fri, Jul 26, 2024 at 01:34:42PM +0200, Karol Kolacinski wrote:
> Add specific functions and definitions for E830 devices to enable
> PTP support.
> Refactor processing of timestamping interrupt and cross timestamp
> to avoid code redundancy.
>
> Jacob Keller (1):
> ice: combine cross timestamp functions for E82x and E830
>
> Karol Kolacinski (2):
> ice: Process TSYN IRQ in a separate function
> ice: Add timestamp ready bitmap for E830 products
>
> Michal Michalik (1):
> ice: Implement PTP support for E830 devices
>
> drivers/net/ethernet/intel/Kconfig | 10 +-
> drivers/net/ethernet/intel/ice/ice_common.c | 17 +-
> drivers/net/ethernet/intel/ice/ice_common.h | 1 +
> .../net/ethernet/intel/ice/ice_hw_autogen.h | 12 +
> drivers/net/ethernet/intel/ice/ice_main.c | 25 +-
> drivers/net/ethernet/intel/ice/ice_osdep.h | 3 +
> drivers/net/ethernet/intel/ice/ice_ptp.c | 356 ++++++++++++------
> drivers/net/ethernet/intel/ice/ice_ptp.h | 9 +-
> drivers/net/ethernet/intel/ice/ice_ptp_hw.c | 208 +++++++++-
> drivers/net/ethernet/intel/ice/ice_ptp_hw.h | 25 +-
> drivers/net/ethernet/intel/ice/ice_type.h | 1 +
> 11 files changed, 508 insertions(+), 159 deletions(-)
>
> V3 -> V4: Further kdoc fixes in "ice: Implement PTP support for
> E830 devices"
> V2 -> V3: Rebased and fixed kdoc in "ice: Implement PTP support for
> E830 devices"
> V1 -> V2: Fixed compilation issue in "ice: Implement PTP support for
> E830 devices"
As the recent changes are about kdoc, please consider running
kernel-doc -none -Wall and ensuring no new warnings are introduced:
ice_is_e830 needs a short description.
And, perhaps things crossed in flight.
But please address the review by Jacob and Alexander of v3.
Thanks!
WARNING: multiple messages have this Message-ID (diff)
From: Simon Horman <horms@kernel.org>
To: Karol Kolacinski <karol.kolacinski@intel.com>
Cc: intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org,
anthony.l.nguyen@intel.com, przemyslaw.kitszel@intel.com
Subject: Re: [PATCH v4 iwl-next 0/4] ice: Implement PTP support for E830 devices
Date: Fri, 26 Jul 2024 18:28:23 +0100 [thread overview]
Message-ID: <20240726172823.GA1699125@kernel.org> (raw)
In-Reply-To: <20240726113631.200083-6-karol.kolacinski@intel.com>
On Fri, Jul 26, 2024 at 01:34:42PM +0200, Karol Kolacinski wrote:
> Add specific functions and definitions for E830 devices to enable
> PTP support.
> Refactor processing of timestamping interrupt and cross timestamp
> to avoid code redundancy.
>
> Jacob Keller (1):
> ice: combine cross timestamp functions for E82x and E830
>
> Karol Kolacinski (2):
> ice: Process TSYN IRQ in a separate function
> ice: Add timestamp ready bitmap for E830 products
>
> Michal Michalik (1):
> ice: Implement PTP support for E830 devices
>
> drivers/net/ethernet/intel/Kconfig | 10 +-
> drivers/net/ethernet/intel/ice/ice_common.c | 17 +-
> drivers/net/ethernet/intel/ice/ice_common.h | 1 +
> .../net/ethernet/intel/ice/ice_hw_autogen.h | 12 +
> drivers/net/ethernet/intel/ice/ice_main.c | 25 +-
> drivers/net/ethernet/intel/ice/ice_osdep.h | 3 +
> drivers/net/ethernet/intel/ice/ice_ptp.c | 356 ++++++++++++------
> drivers/net/ethernet/intel/ice/ice_ptp.h | 9 +-
> drivers/net/ethernet/intel/ice/ice_ptp_hw.c | 208 +++++++++-
> drivers/net/ethernet/intel/ice/ice_ptp_hw.h | 25 +-
> drivers/net/ethernet/intel/ice/ice_type.h | 1 +
> 11 files changed, 508 insertions(+), 159 deletions(-)
>
> V3 -> V4: Further kdoc fixes in "ice: Implement PTP support for
> E830 devices"
> V2 -> V3: Rebased and fixed kdoc in "ice: Implement PTP support for
> E830 devices"
> V1 -> V2: Fixed compilation issue in "ice: Implement PTP support for
> E830 devices"
As the recent changes are about kdoc, please consider running
kernel-doc -none -Wall and ensuring no new warnings are introduced:
ice_is_e830 needs a short description.
And, perhaps things crossed in flight.
But please address the review by Jacob and Alexander of v3.
Thanks!
next prev parent reply other threads:[~2024-07-26 17:28 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-26 11:34 [Intel-wired-lan] [PATCH v4 iwl-next 0/4] ice: Implement PTP support for E830 devices Karol Kolacinski
2024-07-26 11:34 ` Karol Kolacinski
2024-07-26 11:34 ` [Intel-wired-lan] [PATCH v4 iwl-next 1/4] " Karol Kolacinski
2024-07-26 11:34 ` Karol Kolacinski
2024-07-26 11:34 ` [Intel-wired-lan] [PATCH v4 iwl-next 2/4] ice: Process TSYN IRQ in a separate function Karol Kolacinski
2024-07-26 11:34 ` Karol Kolacinski
2024-07-26 11:34 ` [Intel-wired-lan] [PATCH v4 iwl-next 3/4] ice: Add timestamp ready bitmap for E830 products Karol Kolacinski
2024-07-26 11:34 ` Karol Kolacinski
2024-07-26 11:34 ` [Intel-wired-lan] [PATCH v4 iwl-next 4/4] ice: combine cross timestamp functions for E82x and E830 Karol Kolacinski
2024-07-26 11:34 ` Karol Kolacinski
2024-07-26 17:28 ` Simon Horman [this message]
2024-07-26 17:28 ` [PATCH v4 iwl-next 0/4] ice: Implement PTP support for E830 devices Simon Horman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240726172823.GA1699125@kernel.org \
--to=horms@kernel.org \
--cc=anthony.l.nguyen@intel.com \
--cc=intel-wired-lan@lists.osuosl.org \
--cc=karol.kolacinski@intel.com \
--cc=netdev@vger.kernel.org \
--cc=przemyslaw.kitszel@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.