From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1260233E2 for ; Tue, 22 Aug 2023 07:33:17 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0C650C433C8; Tue, 22 Aug 2023 07:33:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1692689597; bh=An+zgJpx0EFTgVuD4rgsolfBepRSMglZzX8LoYa3aCg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=qR/R8nAI6PhTWuSLHULHtkfEP8aq0Y3GPayOzjHrkfkcIzScABc1IpCjJ5LA2dC7+ sbB6S0hWf3IM6UzFZAFn+74ruTgKGkrm+uIZ1azIs5467//5notCGQ4V7E2Enn47VM TWmPba+q1i9p8h5IVuAV9MVheypL/MRFVBQ03eYXXJQI3F8AeqsPRo1laxNMwwOMGk bWmwYrUxWaAfq7Vmz7qiaIL16vXWeU9Ab0YRTexJYdHM5xjDhC176KqwHceB5UeIHy mF0/Gja+zcYq0btKLB+0wYhjjU06JwwSZGHkisIFpHS8Q0pmEkoz8nWpNNo7fjiD+8 Lwag/+KB1tHGg== Date: Tue, 22 Aug 2023 09:33:13 +0200 From: Simon Horman To: Tony Nguyen Cc: davem@davemloft.net, kuba@kernel.org, pabeni@redhat.com, edumazet@google.com, netdev@vger.kernel.org, Sasha Neftin , Naama Meir Subject: Re: [PATCH net] igc: Fix the typo in the PTM Control macro Message-ID: <20230822073313.GP2711035@kernel.org> References: <20230821171721.2203572-1-anthony.l.nguyen@intel.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230821171721.2203572-1-anthony.l.nguyen@intel.com> On Mon, Aug 21, 2023 at 10:17:21AM -0700, Tony Nguyen wrote: > From: Sasha Neftin > > The IGC_PTM_CTRL_SHRT_CYC defines the time between two consecutive PTM > requests. The bit resolution of this field is six bits. That bit five was > missing in the mask. This patch comes to correct the typo in the > IGC_PTM_CTRL_SHRT_CYC macro. > > Fixes: a90ec8483732 ("igc: Add support for PTP getcrosststamp()") > Signed-off-by: Sasha Neftin > Tested-by: Naama Meir > Signed-off-by: Tony Nguyen Reviewed-by: Simon Horman