2021-03-17 19:22:54

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH net-next v1] misc: Add Renesas Synchronization Management Unit (SMU) support

On Wed, Mar 17, 2021 at 02:59:56PM -0400, [email protected] wrote:
> From: Min Li <[email protected]>
>
> This driver is developed for the IDT ClockMatrix(TM) and 82P33xxx families
> of timing and synchronization devices.It will be used by Renesas PTP Clock
> Manager for Linux (pcm4l) software to provide support to GNSS assisted
> partial timing support (APTS) and other networking timing functions.
>
> Current version provides kernel API's to support the following functions
> -set combomode to enable SYNCE clock support
> -read dpll's state to determine if the dpll is locked to the GNSS channel
> -read dpll's ffo (fractional frequency offset)
>
> Signed-off-by: Min Li <[email protected]>

Why the "net-next" in the subject line?

This is not for the network tree (given that you didn't even cc:
netdev...) or am I confused?

thanks,

gre gk-h


2021-03-17 19:37:47

by Min Li

[permalink] [raw]
Subject: RE: [PATCH net-next v1] misc: Add Renesas Synchronization Management Unit (SMU) support

Hi Greg

You are right, this is not for the network tree.

Can you point me to the right tree to base this change for?

Thanks

Min

> -----Original Message-----
> From: Greg KH <[email protected]>
> Sent: March 17, 2021 3:20 PM
> To: Min Li <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]
> Subject: Re: [PATCH net-next v1] misc: Add Renesas Synchronization
> Management Unit (SMU) support
>
> On Wed, Mar 17, 2021 at 02:59:56PM -0400, [email protected] wrote:
> > From: Min Li <[email protected]>
> >
> > This driver is developed for the IDT ClockMatrix(TM) and 82P33xxx
> > families of timing and synchronization devices.It will be used by
> > Renesas PTP Clock Manager for Linux (pcm4l) software to provide
> > support to GNSS assisted partial timing support (APTS) and other
> networking timing functions.
> >
> > Current version provides kernel API's to support the following
> > functions -set combomode to enable SYNCE clock support -read dpll's
> > state to determine if the dpll is locked to the GNSS channel -read
> > dpll's ffo (fractional frequency offset)
> >
> > Signed-off-by: Min Li <[email protected]>
>
> Why the "net-next" in the subject line?
>
> This is not for the network tree (given that you didn't even cc:
> netdev...) or am I confused?
>
> thanks,
>
> gre gk-h

2021-03-17 19:46:50

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH net-next v1] misc: Add Renesas Synchronization Management Unit (SMU) support

A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?

A: No.
Q: Should I include quotations after my reply?

http://daringfireball.net/2007/07/on_top

On Wed, Mar 17, 2021 at 07:34:44PM +0000, Min Li wrote:
> Hi Greg
>
> You are right, this is not for the network tree.
>
> Can you point me to the right tree to base this change for?

MAINTAINERS and scripts/get_maintainers.pl is your friend, please look
there.

thanks,

greg k-h