From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ferruh Yigit <ferruh.yigit@intel.com>
Subject: Re: [PATCH v9 1/3] ethdev: add Rx HW timestamp capability
Date: Wed, 11 Oct 2017 02:30:43 +0100
Message-ID: <ed90c18a-ad36-a2ad-c4f7-dddf7fb41331@intel.com>
References: <1507028429-23944-3-git-send-email-rasland@mellanox.com>
 <1507646227-12338-1-git-send-email-rasland@mellanox.com>
 <0a2ca47a-30be-34cf-0807-932f5131ea8d@intel.com>
 <63E777F1-7A36-48F5-8643-3681CC682668@mellanox.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Cc: Raslan Darawsheh <rasland@mellanox.com>, "dev@dpdk.org" <dev@dpdk.org>,
 Shahaf Shuler <shahafs@mellanox.com>
To: Yongseok Koh <yskoh@mellanox.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mga14.intel.com (mga14.intel.com [192.55.52.115])
 by dpdk.org (Postfix) with ESMTP id 736B31B1AB
 for <dev@dpdk.org>; Wed, 11 Oct 2017 03:30:44 +0200 (CEST)
In-Reply-To: <63E777F1-7A36-48F5-8643-3681CC682668@mellanox.com>
Content-Language: en-US
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On 10/11/2017 2:22 AM, Yongseok Koh wrote:
> 
>> On Oct 10, 2017, at 6:14 PM, Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>>
>> On 10/10/2017 3:37 PM, Raslan Darawsheh wrote:
>>> Add a new offload capability flag for Rx HW
>>> timestamp and enabling/disabling this via rte_eth_rxmode.
>>>
>>> Signed-off-by: Raslan Darawsheh <rasland@mellanox.com>
>>> Reviewed-by: Shahaf Shuler <shahafs@mellanox.com>
>>> Acked-by: Yongseok Koh <yskoh@mellanox.com>
>>
>> Series applied to dpdk-next-net/master, thanks.
>>
>> (fixed testpmd 32bits build error while applying, please confirm the commit)
> 
> There seems to be one line of change in "app/testpmd: add Rx HW timestamp"
> 
> < +                     printf(" - timestamp %lu ", mb->timestamp);
> ---
>> +                     printf(" - timestamp %"PRIu64" ", mb->timestamp);
> 

Yes, only this one.

> And we are okay with it.

Thanks.

> 
> Thanks,
> Yongseok
>