[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b40848ed739c466683239d317870350a@BY2PR03MB441.namprd03.prod.outlook.com>
Date: Wed, 8 Oct 2014 03:30:24 +0000
From: "luwei.zhou@...escale.com" <luwei.zhou@...escale.com>
To: Richard Cochran <richardcochran@...il.com>
CC: "davem@...emloft.net" <davem@...emloft.net>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"shawn.guo@...aro.org" <shawn.guo@...aro.org>,
"bhutchings@...arflare.com" <bhutchings@...arflare.com>,
"Fabio.Estevam@...escale.com" <Fabio.Estevam@...escale.com>,
"fugang.duan@...escale.com" <fugang.duan@...escale.com>,
"Frank.Li@...escale.com" <Frank.Li@...escale.com>,
"stephen@...workplumber.org" <stephen@...workplumber.org>
Subject: RE: [PATCH v1 0/4] Enable FEC pps ouput
On Wed, Oct 3, 2014 at 04:24:00PM, Richard Cochran wrote:
> -----Original Message-----
> From: Richard Cochran [mailto:richardcochran@...il.com]
> Sent: Friday, October 03, 2014 4:24 PM
> To: Zhou Luwei-B45643
> Cc: davem@...emloft.net; netdev@...r.kernel.org; shawn.guo@...aro.org;
> bhutchings@...arflare.com; Estevam Fabio-R49496; Duan Fugang-B38611; Li
> Frank-B20596; stephen@...workplumber.org
> Subject: Re: [PATCH v1 0/4] Enable FEC pps ouput
>
> On Thu, Sep 25, 2014 at 04:10:17PM +0800, Luwei Zhou wrote:
> > This patch does:
> > - Replace 32-bit free-running PTP timer with 31-bit.
> > - Implement hardware PTP timestamp adjustment.
> > - Enable PPS output based on hardware adjustment.
> >
> >
> > Luwei Zhou (4):
> > net: fec: ptp: Use the 31-bit ptp timer.
> > net: fec: ptp: Use hardware algorithm to adjust PTP counter.
> > net: fec: ptp: Enalbe PPS ouput based on ptp clock
> ^^^^^^ ^^^^^
> Enable output
>
> Also, it looks like you have implemented the wrong feature.
>
> The "pps" capability means that the clock provides a PPS event
> (interrupt) to the kernel.
>
> IOW, if .pps==1, then you must call ptp_clock_event() with event type
> PTP_CLOCK_PPS.
>
> Your patch set seems to be creating a periodic output and not a PPS
> kernel callback.
>
> You should clarify what you are trying to do, and then implement the
> appropriate interface in your driver.
>
> Thanks,
> Richard
Hum..The FEC IP does not support a specified channel for PPS function. So I have to use
a ptp timer output channel to implement the PPS function. That is why the software flow
looks like the periodic output but the function is PPS function. We only want the event to happen
on the per second. So I think it should be clarified to be PPS event.
Thanks
Luwei
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists