Commit e66dccce authored by Grygorii Strashko's avatar Grygorii Strashko Committed by David S. Miller

net: ethernet: ti: cpts: separate hw counter read from timecounter

Now CPTS HW time reading code is implemented in timecounter->cyclecounter
.read() callback and performs following operations:
timecounter_read() ->cc.read() -> cpts_systim_read()
 - request current CPTS HW time CPTS_TS_PUSH.TS_PUSH = 1
 - poll CPTS FIFO for CPTS_EV_PUSH event with current HW timestamp

This approach need to be changed for the future switch to PTP PHC
.gettimex64() callback, which require to separate requesting current CPTS
HW time and processing CPTS FIFO. And for the follow up patch, which
improves .adjfreq() implementation.

This patch moves code accessing CPTS HW out of timecounter code as
following:
- convert HW timestamp of every CPTS event to PTP time (us) and store it as
part struct cpts_event;
- add CPTS context field to store current CPTS HW time (counter) value and
update it on CPTS_EV_PUSH reception;
- move code accessing CPTS HW out of timecounter code and use current CPTS
HW time (counter) from CPTS context instead;
- ensure timecounter->cycle_last is updated on CPTS_EV_PUSH reception.

After this change CPTS timecounter will only perform timekeeper role
without actually accessing CPTS HW.
Signed-off-by: default avatarGrygorii Strashko <grygorii.strashko@ti.com>
Acked-by: default avatarRichard Cochran <richardcochran@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 79d6e755
...@@ -112,10 +112,8 @@ static bool cpts_match_tx_ts(struct cpts *cpts, struct cpts_event *event) ...@@ -112,10 +112,8 @@ static bool cpts_match_tx_ts(struct cpts *cpts, struct cpts_event *event)
(struct cpts_skb_cb_data *)skb->cb; (struct cpts_skb_cb_data *)skb->cb;
if (cpts_match(skb, class, seqid, mtype)) { if (cpts_match(skb, class, seqid, mtype)) {
u64 ns = timecounter_cyc2time(&cpts->tc, event->low);
memset(&ssh, 0, sizeof(ssh)); memset(&ssh, 0, sizeof(ssh));
ssh.hwtstamp = ns_to_ktime(ns); ssh.hwtstamp = ns_to_ktime(event->timestamp);
skb_tstamp_tx(skb, &ssh); skb_tstamp_tx(skb, &ssh);
found = true; found = true;
__skb_unlink(skb, &cpts->txq); __skb_unlink(skb, &cpts->txq);
...@@ -158,8 +156,16 @@ static int cpts_fifo_read(struct cpts *cpts, int match) ...@@ -158,8 +156,16 @@ static int cpts_fifo_read(struct cpts *cpts, int match)
event->tmo = jiffies + 2; event->tmo = jiffies + 2;
event->high = hi; event->high = hi;
event->low = lo; event->low = lo;
event->timestamp = timecounter_cyc2time(&cpts->tc, event->low);
type = event_type(event); type = event_type(event);
dev_dbg(cpts->dev, "CPTS_EV: %d high:%08X low:%08x\n",
type, event->high, event->low);
switch (type) { switch (type) {
case CPTS_EV_PUSH:
WRITE_ONCE(cpts->cur_timestamp, lo);
timecounter_read(&cpts->tc);
break;
case CPTS_EV_TX: case CPTS_EV_TX:
if (cpts_match_tx_ts(cpts, event)) { if (cpts_match_tx_ts(cpts, event)) {
/* if the new event matches an existing skb, /* if the new event matches an existing skb,
...@@ -168,7 +174,6 @@ static int cpts_fifo_read(struct cpts *cpts, int match) ...@@ -168,7 +174,6 @@ static int cpts_fifo_read(struct cpts *cpts, int match)
break; break;
} }
/* fall through */ /* fall through */
case CPTS_EV_PUSH:
case CPTS_EV_RX: case CPTS_EV_RX:
list_del_init(&event->list); list_del_init(&event->list);
list_add_tail(&event->list, &cpts->events); list_add_tail(&event->list, &cpts->events);
...@@ -189,26 +194,17 @@ static int cpts_fifo_read(struct cpts *cpts, int match) ...@@ -189,26 +194,17 @@ static int cpts_fifo_read(struct cpts *cpts, int match)
static u64 cpts_systim_read(const struct cyclecounter *cc) static u64 cpts_systim_read(const struct cyclecounter *cc)
{ {
u64 val = 0;
struct cpts_event *event;
struct list_head *this, *next;
struct cpts *cpts = container_of(cc, struct cpts, cc); struct cpts *cpts = container_of(cc, struct cpts, cc);
cpts_write32(cpts, TS_PUSH, ts_push); return READ_ONCE(cpts->cur_timestamp);
if (cpts_fifo_read(cpts, CPTS_EV_PUSH)) }
dev_err(cpts->dev, "cpts: unable to obtain a time stamp\n");
list_for_each_safe(this, next, &cpts->events) { static void cpts_update_cur_time(struct cpts *cpts, int match)
event = list_entry(this, struct cpts_event, list); {
if (event_type(event) == CPTS_EV_PUSH) { cpts_write32(cpts, TS_PUSH, ts_push);
list_del_init(&event->list);
list_add(&event->list, &cpts->pool);
val = event->low;
break;
}
}
return val; if (cpts_fifo_read(cpts, match) && match != -1)
dev_err(cpts->dev, "cpts: unable to obtain a time stamp\n");
} }
/* PTP clock operations */ /* PTP clock operations */
...@@ -232,7 +228,7 @@ static int cpts_ptp_adjfreq(struct ptp_clock_info *ptp, s32 ppb) ...@@ -232,7 +228,7 @@ static int cpts_ptp_adjfreq(struct ptp_clock_info *ptp, s32 ppb)
spin_lock_irqsave(&cpts->lock, flags); spin_lock_irqsave(&cpts->lock, flags);
timecounter_read(&cpts->tc); cpts_update_cur_time(cpts, CPTS_EV_PUSH);
cpts->cc.mult = neg_adj ? mult - diff : mult + diff; cpts->cc.mult = neg_adj ? mult - diff : mult + diff;
...@@ -260,6 +256,9 @@ static int cpts_ptp_gettime(struct ptp_clock_info *ptp, struct timespec64 *ts) ...@@ -260,6 +256,9 @@ static int cpts_ptp_gettime(struct ptp_clock_info *ptp, struct timespec64 *ts)
struct cpts *cpts = container_of(ptp, struct cpts, info); struct cpts *cpts = container_of(ptp, struct cpts, info);
spin_lock_irqsave(&cpts->lock, flags); spin_lock_irqsave(&cpts->lock, flags);
cpts_update_cur_time(cpts, CPTS_EV_PUSH);
ns = timecounter_read(&cpts->tc); ns = timecounter_read(&cpts->tc);
spin_unlock_irqrestore(&cpts->lock, flags); spin_unlock_irqrestore(&cpts->lock, flags);
...@@ -294,11 +293,14 @@ static long cpts_overflow_check(struct ptp_clock_info *ptp) ...@@ -294,11 +293,14 @@ static long cpts_overflow_check(struct ptp_clock_info *ptp)
{ {
struct cpts *cpts = container_of(ptp, struct cpts, info); struct cpts *cpts = container_of(ptp, struct cpts, info);
unsigned long delay = cpts->ov_check_period; unsigned long delay = cpts->ov_check_period;
struct timespec64 ts;
unsigned long flags; unsigned long flags;
u64 ns;
spin_lock_irqsave(&cpts->lock, flags); spin_lock_irqsave(&cpts->lock, flags);
ts = ns_to_timespec64(timecounter_read(&cpts->tc));
cpts_update_cur_time(cpts, -1);
ns = timecounter_read(&cpts->tc);
if (!skb_queue_empty(&cpts->txq)) { if (!skb_queue_empty(&cpts->txq)) {
cpts_purge_txq(cpts); cpts_purge_txq(cpts);
...@@ -307,8 +309,7 @@ static long cpts_overflow_check(struct ptp_clock_info *ptp) ...@@ -307,8 +309,7 @@ static long cpts_overflow_check(struct ptp_clock_info *ptp)
} }
spin_unlock_irqrestore(&cpts->lock, flags); spin_unlock_irqrestore(&cpts->lock, flags);
dev_dbg(cpts->dev, "cpts overflow check at %lld.%09ld\n", dev_dbg(cpts->dev, "cpts overflow check at %lld\n", ns);
(long long)ts.tv_sec, ts.tv_nsec);
return (long)delay; return (long)delay;
} }
...@@ -390,7 +391,7 @@ static u64 cpts_find_ts(struct cpts *cpts, struct sk_buff *skb, int ev_type) ...@@ -390,7 +391,7 @@ static u64 cpts_find_ts(struct cpts *cpts, struct sk_buff *skb, int ev_type)
seqid = (event->high >> SEQUENCE_ID_SHIFT) & SEQUENCE_ID_MASK; seqid = (event->high >> SEQUENCE_ID_SHIFT) & SEQUENCE_ID_MASK;
if (ev_type == event_type(event) && if (ev_type == event_type(event) &&
cpts_match(skb, class, seqid, mtype)) { cpts_match(skb, class, seqid, mtype)) {
ns = timecounter_cyc2time(&cpts->tc, event->low); ns = event->timestamp;
list_del_init(&event->list); list_del_init(&event->list);
list_add(&event->list, &cpts->pool); list_add(&event->list, &cpts->pool);
break; break;
......
...@@ -94,6 +94,7 @@ struct cpts_event { ...@@ -94,6 +94,7 @@ struct cpts_event {
unsigned long tmo; unsigned long tmo;
u32 high; u32 high;
u32 low; u32 low;
u64 timestamp;
}; };
struct cpts { struct cpts {
...@@ -114,6 +115,7 @@ struct cpts { ...@@ -114,6 +115,7 @@ struct cpts {
struct cpts_event pool_data[CPTS_MAX_EVENTS]; struct cpts_event pool_data[CPTS_MAX_EVENTS];
unsigned long ov_check_period; unsigned long ov_check_period;
struct sk_buff_head txq; struct sk_buff_head txq;
u64 cur_timestamp;
}; };
void cpts_rx_timestamp(struct cpts *cpts, struct sk_buff *skb); void cpts_rx_timestamp(struct cpts *cpts, struct sk_buff *skb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment