On Wed, 21 Jun 2023 at 01:13, Rahul Rameshbabu <rrameshb...@nvidia.com>
wrote:

> 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?
>

Rahul, all patches are for review.
Do not make our life too much sophisticated.
Just add a version tag, please. Whether you add an RFC or not.
If you really must , you can add a note in the cover letter for Richard.

Linux works with several groups.
Linuxptp has only one developer group, it is not that big :-)

Thanks
  Erez


>
> > 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