Hi Erez,

On Wed, 21 Jun, 2023 00:33:28 +0200 Erez <erezge...@gmail.com> wrote:
> Hi,
>
> You already submitted the patch seria.
> Has it changed?

Yes, I took feedback from the RFC (request for comments) I sent out and
applied it in this submission. In the git notes field in the first patch
of the series, I document the change made since the RFC. Since the first
submission was an RFC, I did not treat it as a formal submission, so I
did not consider it a v1 but rather a draft. I submitted as an RFC due
to pending kernel side changes.

> If so, please mark it with version 2. "git format-patch -v 2".

I agree with this if a formal v1 was submitted. However since my
previous submission was an RFC, is the practice still to increment the
official submission as v2?

> If not, why do you send it again?

Ignoring the fact that I applied changes based on feedback from the RFC
submission, isn't the normal practice to submit an RFC as a normal
submission to the mailing list to indicate readiness for applying the
patches onto the target?

> I think Richard wanted to close version 4 first.

Sure, that makes sense. I am hoping RFC patches are not considered for
merging into releases or the default branch of the project. At the time,
the needed kernel side changes were not merged into net-next of the
linux netdev tree.

Apologize in advance if we are implementing the practice of closing
submissions during release windows (similar to what net-next does in the
linux netdev tree). I did not see a mention of that on the mailing list.
Might have missed it.

Thanks,

Rahul Rameshbabu

>
> Erez
>
> On Tue, 20 Jun 2023 at 19:39, Rahul Rameshbabu via Linuxptp-devel 
> <linuxptp-devel@lists.sourceforge.net> wrote:
>
>  The main focus of this submission is adding support for testing ADJ_OFFSET 
> with
>  phc_ctl and querying the maximum supported ADJ_OFFSET adjustment that a 
> device
>  is capable of. Some other minor cleanups are also included in the submission.
>
>  That patch the introduces support for querying the maximum offset supported 
> by
>  ADJ_OFFSET depends on a kernel patch series (linked below) that is targeted 
> for
>  kernel 6.5. Previously, sent this series out as an RFC to inquire feedback 
> early
>  on. Have incorporated that feedback into this submission.
>
>  Link: https://sourceforge.net/p/linuxptp/mailman/message/37854603/
>  Link: 
> https://lore.kernel.org/netdev/20230612211500.309075-1-rrameshb...@nvidia.com/
>
>  Rahul Rameshbabu (5):
>    Rename NSEC2SEC as NSEC_PER_SEC and refactor to util.h
>    phc_ctl: Add phase command to support ADJ_OFFSET
>    phc_ctl: Add maximum offset capability
>    phc_ctl: Use pr_notice instead of pr_err for displaying adjusted
>      frequency
>    phc_ctl: Handle errors returned by various clockadj helpers
>
>   missing.h      |  9 +++---
>   phc_ctl.8      |  4 +++
>   phc_ctl.c      | 77 ++++++++++++++++++++++++++++++++++++++------------
>   port.c         | 14 ++++-----
>   port_private.h |  3 +-
>   servo.c        |  3 +-
>   tc.c           |  6 ++--
>   util.h         |  2 ++
>   8 files changed, 82 insertions(+), 36 deletions(-)
>
>  -- 
>  2.40.1
>
>  _______________________________________________
>  Linuxptp-devel mailing list
>  Linuxptp-devel@lists.sourceforge.net
>  https://lists.sourceforge.net/lists/listinfo/linuxptp-devel


_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to