Hello Rayagond ! On 10/27/2016 12:25 PM, Rayagond Kokatanur wrote:
+static int dwmac4_wrback_get_rx_timestamp_status(void *desc, u32 ats) > { > struct dma_desc *p = (struct dma_desc *)desc; > + int ret = -EINVAL; > + > + /* Get the status from normal w/b descriptor */ > + if (likely(p->des3 & TDES3_RS1V)) { > + if (likely(p->des1 & RDES1_TIMESTAMP_AVAILABLE)) { > + int i = 0; > + > + /* Check if timestamp is OK from context descriptor */ > + do { > + ret = dwmac4_rx_check_timestamp(desc);Here, "desc" is not pointing to next descriptor (ie context descriptor). Driver should check the context descriptor.
you are right and this is done by the caller: stmmac_get_rx_hwtstamp Cheers peppe
