Received-SPF: pass (google.com: domain of linux-kernel+bounces-65943-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <da0050e0-f038-4d14-875c-158f4bef4981@intel.com>
Date: Wed, 14 Feb 2024 12:37:04 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH net v1 1/1] igb: extend PTP timestamp adjustments to i211
Content-Language: en-US
To: Oleksij Rempel <o.rempel@pengutronix.de>, Jesse Brandeburg
	<jesse.brandeburg@intel.com>, Tony Nguyen <anthony.l.nguyen@intel.com>,
	"David S. Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>, "Richard
 Cochran" <richardcochran@gmail.com>
CC: <kernel@pengutronix.de>, <intel-wired-lan@lists.osuosl.org>,
	<netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>
References: <20240213115410.415573-1-o.rempel@pengutronix.de>
From: Jacob Keller <jacob.e.keller@intel.com>
In-Reply-To: <20240213115410.415573-1-o.rempel@pengutronix.de>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SDE0NTdxM2dFeG96Z05xQi9uOXlMMVlLRDVuUlREb1Vzcm9xNUZVaS9qamUw?=
 =?utf-8?B?ZTZoOGNyZHd6UWtCWlkxaEZmTllkSFo0c2xNK2pQVlZmNVFPbkJYbUF6WWk1?=
 =?utf-8?B?d0p4OFpBb2lTSFc2a2ViVEd3OE1TMWk1bTAwbXVqZGVXRjBPZ3dnMXpoek9Q?=
 =?utf-8?B?QXRDVTdjSWZGREJXeEQxUUlHMlhWa0FDTFdtMk9KZmVLSms1eGRNT3M4SWJo?=
 =?utf-8?B?SENJMHJ4L2M2aE9Jd05IajE3ODNQYXJPK1cvQmNaNmlzNmhXVjkxK2h0R0xl?=
 =?utf-8?B?ZHJnL2NkRHlHT2c2cHhEUUk4RnRTbXlHRXl2aE1DVFBRTWR2WG9WQzNWVmpT?=
 =?utf-8?B?OUNSb2d3bTg0cCtTaGFJSXhHYkdiK0tLVGpKcXpIcW5TRG00M3ArZ0RXbmcr?=
 =?utf-8?B?U3Q2MXcxYnVlL1lKTlpSUm53MlN5T2wydkhEcGxHM3duQ2E0VUkzS2Q4bVFj?=
 =?utf-8?B?MFFXTUZLL1NKb0pqNFhaTjhjOTZXYlhBOXBSUkZvWmVkTmhyaFRxdEtab1VX?=
 =?utf-8?B?Q3ZkMm9lbzB4RWxUM0kzaURZQnB2cGNrcnNvK3BRdDlQTGJXR3AydlRrcmdv?=
 =?utf-8?B?elFhdVV6Rjl2dG1OaFBNYU1LQ2dib3hBbEZoV2RLajhKdzZlY0szL05XS0JF?=
 =?utf-8?B?KytiZTdDYU5LWE5yZUVrVUx5OXl1QVVqSGxrdjRXaG9BY3NGdFVXY3BQc1dR?=
 =?utf-8?B?V2Jkd25mQzcxMVhJbWJscFRoT25vNVRjbkk1V2JFK1JvUkNWdSt4R0RhdkdQ?=
 =?utf-8?B?Q0dxNmpWS2RXUmpkR0Nka1Y0bS9iTitZZmlVZE5FT2pMTTZ4c1g0cnBFZFJF?=
 =?utf-8?B?bXBja1NLV28vT2Y1M2dwMzgvd3p4WVUwOEl1Q1ByM29HUzhqTmtCV1Z5V2sy?=
 =?utf-8?B?UVIwVXBpTlJscWxRMk5yMGhpZ1ZhbXVGczBMT21GcFZ2KzRXR29VZVIrekR5?=
 =?utf-8?B?cW9yMzY3WXBVNm5zQnFET09TMkpvbnBKeXQ5SktwRmhBZnJvYXFiRkluYmNB?=
 =?utf-8?B?RFBtNHVXcEw3b3Z2M2E2QUVESG9pMUkvYWlMaFpnN2tpTHJXSHU0YzBHa2NW?=
 =?utf-8?B?ajNXSHBydmZGbHV6blREVmU4N2tBWTgrYitiaFVjM0xmNCt0djNOb1craThm?=
 =?utf-8?B?SXN4aTlzTmJ6U081b3lMU1doM3h1ZFVsbjZIS1JxTTdTZ1RsWXFuRC9rTTJQ?=
 =?utf-8?B?cnJjUE1LQzVha3lDcUtnSnJLbk1qNG93SzFHZWNLLy9WUndSMlBGS1dhVU1F?=
 =?utf-8?B?UGtzZ3BqSUNzbERnaXRhcGhXZnpIV2RqSXk2Y3ZXS0RiYXFWV0hLT2RrRHRt?=
 =?utf-8?B?SEhQTXc1dzEza1VqbmpmeUdud3pBaVd5czdWWVBmZnNiMEMxOEU1Y1RUTVln?=
 =?utf-8?B?V29ydXErU3FObnE4cmJmTCt4bGo0M043L2hUekl2Q1lWVXk0WmRscnNETHVP?=
 =?utf-8?B?WWZ3TzYrc2VqSklUYy9XZTJuaUQrNFRjVTBuYzQwQkw0WjBoYnhHRm14ZnZU?=
 =?utf-8?B?dXhrakhiS3RjaEdsOFVSZ1dNVUh3TTRDdnNFeDNTeDdqV2NQeUZxaUJyU2hw?=
 =?utf-8?B?MzI0dStEcjRhNXJvUVZiR2FTWUJVNlQrTENrQzhuRWtqdzZWSCtGL3VXVGJw?=
 =?utf-8?B?ODFDRTVoRDl5Q1Y1UFg1OUNhamplSUgzQVMyYUF3K0JmQm41ZFhsZmtUZ0l2?=
 =?utf-8?B?TGpJOVU0R3drSHlvNW9HaUV6czArbk5wMFdKYWE4TS9BTE5WWEJCOC9kcVlE?=
 =?utf-8?B?d01rcnJocUEvUDJqalNhNmJNaW9yK0lqeHdyME1QaFp5Y2lLcGxIbVhzdWlx?=
 =?utf-8?B?ekdYMGdjclA4VDJPZHZGa1ZYS29wVkZrRjQ3LzdsT2gxM2F4Qm8yZkQ2VTIv?=
 =?utf-8?B?UDNIMW42cnY5QzhrWkp3QmFCU3pHeThSMmU3Q0Z1a3IrTUJlSytjOWVZeHoz?=
 =?utf-8?B?SVRFbnJhTXRQQUJnaUQya29nS2QvYmVBQytpMTJqdEUwYUpxU3RydnB2K2pS?=
 =?utf-8?B?Mms1bXpCa0E5YTgwTFZjQmFubFptTDUvcWxLWUIrczBUeEFBZnBEYm41eEpL?=
 =?utf-8?B?bWVQbnViZEp0clBhaGxnNTBGdU5CRVYwYkpFNEIzMWpEMnhNdlRmcW9tc3A0?=
 =?utf-8?B?RURLdDFwOWkxYmRubVdINjBISU81M3pkV1NzR3ZNVDJHTS9LeWRtT1VXMTRN?=
 =?utf-8?B?K0E9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: b589239e-3501-4de0-3600-08dc2d9cb579
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB5089.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Feb 2024 20:37:06.3511
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: sO3f9dlMhYff1nfYKZIxIUnW10k1X0HDit6EUsXmUMnVdjxiTQNVQnNCc84mH61Fn0H5VaYYacQDN8WKfBdCcMfIb6QJCiD5hlZxl8+/0Lg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB7589
X-OriginatorOrg: intel.com


On 2/13/2024 3:54 AM, Oleksij Rempel wrote:
> The i211 requires the same PTP timestamp adjustments as the i210,
> according to its datasheet. To ensure consistent timestamping across
> different platforms, this change extends the existing adjustments to
> include the i211.
> 
> The adjustment result are tested and comparable for i210 and i211 based
> systems.
> 
> Fixes: 3f544d2a4d5c ("igb: adjust PTP timestamps for Tx/Rx latency")
> Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
> ---

Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>