[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6b1256b73b7a4f978573a77ec3e36831@BLUPR03MB486.namprd03.prod.outlook.com>
Date: Tue, 14 Oct 2014 18:43:51 +0000
From: "Frank.Li@...escale.com" <Frank.Li@...escale.com>
To: Richard Cochran <richardcochran@...il.com>
CC: David Miller <davem@...emloft.net>,
"fugang.duan@...escale.com" <fugang.duan@...escale.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"bhutchings@...arflare.com" <bhutchings@...arflare.com>
Subject: RE: [PATCH] net: fec: ptp: fix convergence issue to support LinuxPTP
stack
> -----Original Message-----
> From: Richard Cochran [mailto:richardcochran@...il.com]
> Sent: Tuesday, October 14, 2014 1:40 PM
> To: Li Frank-B20596
> Cc: David Miller; Duan Fugang-B38611; netdev@...r.kernel.org;
> bhutchings@...arflare.com
> Subject: Re: [PATCH] net: fec: ptp: fix convergence issue to support LinuxPTP
> stack
>
> Frank,
>
> On Tue, Oct 14, 2014 at 06:27:28PM +0000, Frank.Li@...escale.com wrote:
> > Fugang's patch fix the problem existed in MX6SX platform.
>
> You say that Fugang's patch fixes a quirk in the SX device.
>
> But he said that the user's manual tells us to wait for some status bits to
> clear. (See the third line, with the 'while' key word).
>
> > > > On Tue, Oct 14, 2014 at 01:39:47PM +0800, Fugang Duan wrote:
> > > >> IEEE 1588 module has one hw issue in capturing the ATVR register.
> > > >> According to the user manual it is:
> > > >> ENET0->ATCR |= ENET_ATCR_CAPTURE_MASK;
> > > >> while(ENET0->ATCR & ENET_ATCR_CAPTURE_MASK);
> > > >> ts_counter_ns = ENET0->ATVR;
>
> That applies to all IMX devices, doesn't it?
> In any case, Fugang's change log is not clear.
Only MX6 SX. Only MX6 SX added FEC_QUIRK_BUG_CAPTURE.
.driver_data = FEC_QUIRK_ENET_MAC | FEC_QUIRK_HAS_GBIT |
FEC_QUIRK_HAS_BUFDESC_EX | FEC_QUIRK_HAS_CSUM |
FEC_QUIRK_HAS_VLAN | FEC_QUIRK_HAS_AVB |
- FEC_QUIRK_ERR007885,
+ FEC_QUIRK_ERR007885 | FEC_QUIRK_BUG_CAPTURE,
>
> Confused,
> Richard
--
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