Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <3e7ad9b4-9962-2eea-d6f2-c8f0cb7c121d@intel.com>
Date:   Mon, 19 Dec 2022 14:08:32 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: [Patch net-next v4 01/13] net: dsa: microchip: ptp: add the posix
 clock support
Content-Language: en-US
To:     Arun Ramadoss <arun.ramadoss@microchip.com>,
        <linux-kernel@vger.kernel.org>, <netdev@vger.kernel.org>
CC:     <woojung.huh@microchip.com>, <UNGLinuxDriver@microchip.com>,
        <andrew@lunn.ch>, <vivien.didelot@gmail.com>,
        <f.fainelli@gmail.com>, <olteanv@gmail.com>, <davem@davemloft.net>,
        <edumazet@google.com>, <kuba@kernel.org>, <pabeni@redhat.com>,
        <linux@armlinux.org.uk>, <Tristram.Ha@microchip.com>,
        <richardcochran@gmail.com>, <ceggers@arri.de>
References: <20221212102639.24415-1-arun.ramadoss@microchip.com>
 <20221212102639.24415-2-arun.ramadoss@microchip.com>
From:   Jacob Keller <jacob.e.keller@intel.com>
In-Reply-To: <20221212102639.24415-2-arun.ramadoss@microchip.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Y0ZMWFJYV1FMZ2VYckxXbmdtVDBKNGhPenJ1azJBMGpDM0F2Y3FMdWxLUnMw?=
 =?utf-8?B?RXNpcHZ2MytLTW9hTzh4RHE4eitnUTlsRklXZmZLZ2ZmV3NBejlLeTkvTGZl?=
 =?utf-8?B?M2t0S2pDbDNBSWFJS0xvblNaME1rdTVzTDIyQWdVOU5QYlNQVkNuTFBqYWFr?=
 =?utf-8?B?b2xBRlJ2Y1IvV3Z1Nnl0UjhNRGFTMTdhMEZISXpSVXlwM0lVTlNHaWNGRVNZ?=
 =?utf-8?B?WmdkL2VML3dzdkdzdnI4NUJZQTBJRGJxZXFvYkhSMG53aG1HYWZ2a29IUWJn?=
 =?utf-8?B?Z3Z6VUxPTGU5aTUrSUhNZGNLMURwZ1JYQUxKcHlTdVhGU3pLcmlLYXBJaWxr?=
 =?utf-8?B?TWFCdWJtdUFHWXRxV3R5bWJmNitZWm1oekUyS0g1Y0tQWFNaNXhHdEg1VFVv?=
 =?utf-8?B?ZDFveGdVWVd5NGxTUEJ3c2MralJyQUlUcFZxNTBqMWQ2V0d2a3UyUkxHSmlo?=
 =?utf-8?B?Z1Q5ZURSYmdkVi9FTHN3L0wvbXQ0dnRiOGlLbHZTNlF2Y3YvRnhxUzhyelhi?=
 =?utf-8?B?RXFpc3JDeS9VMThtdzBBSk5aTEdibzlNQ1Y5TmdJbmlmbEpWQnVSMVBKbXFR?=
 =?utf-8?B?ajFnK1Z3U0xCVEU4UWY2RUw1UGJhbitMZ0RRU0JJQWphQnNkRUdwNnNGZVV3?=
 =?utf-8?B?OXRLbm5pT0pROVJYRkY4UE55MkhscUZJbGFOckN2VmdiNHFrY3djTENSMjhC?=
 =?utf-8?B?M0h6STNwYzRmR1NxdkFhZWt5SGNGa0MrcXc5V293V1ZCVnhRUFMxWGROR3Vp?=
 =?utf-8?B?bjNIdGxnR0hyQ25ISVl3T05vSlNSRDhwMDJBekM2STR5UG5LZ0RDekl6TXRT?=
 =?utf-8?B?eGlXZUh4UEJDeGk3WGtwM2pwUGg0cVZid2UrNVJCUjRmR2FjMHN2Wnl6QUFp?=
 =?utf-8?B?OVIwdUJTTzZ0V2cyazN6RDZNRmp1Ulh5d29vdEw4eXBpeml4SndTYzVOSHNi?=
 =?utf-8?B?RTdzL04yeVpBL2JGdFNpaUdEVE9yR1hZYXV5M0FhbmxXMGhpUWVkNTYyZ1U2?=
 =?utf-8?B?Q1JzYjA2SEFlWDk2emJkYmFLYnRBdkorQkZDUVpXd0krQjNMeFRDOUtNVmly?=
 =?utf-8?B?cGh0NnhBTTN4UitIOWhtZ3JkNzdiWGZSN2VOYlNTTmVtUkxXQzFpQkVLUDlh?=
 =?utf-8?B?RkFoRDUxS2Vac3o1ajNISEhjOXRCdkJFNzJ1VHdvZDZzVW9FWHlsRGRCNklZ?=
 =?utf-8?B?TFR4eFVEdFZ6S2hiczdVSXVYcnVRSkJCMW50OHZqU2hCL3JwWW9pUmdzeDZ0?=
 =?utf-8?B?ZkF5TzNjZnBYQk1qcUtZbEMrbjFVRGI4Y20xaWs2WTE4a3VwVWlPTUp3R25H?=
 =?utf-8?B?TWhWclFVdHdLelFVdU5uSjk3YkRQZkRXdG5QZzZnZFBDbGY4aFovazlzV0Qv?=
 =?utf-8?B?RVV0YjBrMnp2T3VIajZxbTM3R3Nmd2dlOTBhSm1XY1YxL0pSVjdOU3BSa0Vz?=
 =?utf-8?B?WVp3eEh5bi91N3NJeGhWYXBlUW9hUXBBMmFQbndDeHp1WklWcUMwelFvUkFU?=
 =?utf-8?B?aUJpZWl4bVJWM2JLUDFKdCtlc21taFVBdjZFYys4akgzTDd2dCsrNjFYT1Rx?=
 =?utf-8?B?c0EzZUc0M3pCREtkVU5EcVVmaXRHdU5QdERVSExROHpIOWdaUndLUGY1Rm1W?=
 =?utf-8?B?cGg5VUlaN0JDeVYvalFrLzYzWlRva1l6bjZQdndtWVlIQ3VvWThlVW1xOGdD?=
 =?utf-8?B?ZnBlS2p5Q21wa2xBMzZBOXJnbG96Sy9QSUllSmE0VnFISGlIdkFNMFA1SXU4?=
 =?utf-8?B?WjBYY1RiTDFON1NiQ2xPS2d0bk1uNllHZkwxYjdyTEZ6cnJCL2FEYUhEZU5j?=
 =?utf-8?B?ZFJQc3RMSlE0OU5Ed2xIVVJNV3VFbWYrODdWNDJtaVowRG16MDBrdjRQZnZa?=
 =?utf-8?B?NzNONGlocWUwdVFKa1Q4RXBDTzc3bzhvcUFIM1JCbUJvbG9abDM2UzJVeUMy?=
 =?utf-8?B?aTFiajN0RUk3d20yNDQxNG1ybWlucU53SGFnNUpIVHJsV0hDL1FDY3RoSm1X?=
 =?utf-8?B?aVVHN0Z1V1h1UWJ5MzRwYWVuZ2J1SmVYc2F1TzdjMXBSaHVSUTNXZG9uY2ho?=
 =?utf-8?B?NGRhbHdBMitZUHZmWGZDc29tVC9Ucmhhdk1jZFFVOWN6VzdoaEt4M2hUYlJr?=
 =?utf-8?B?V1BYN2JoTUVBYnZaaGVUTjJjQkVyZThXdzY3QzFuaW1RUkhmSzM2OWRvOExn?=
 =?utf-8?B?aVE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 52cf48dd-c46e-46ba-de86-08dae20d92d7
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB5089.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Dec 2022 22:08:35.5856
 (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: Om2Giw5Ivob/pUuyHJvDlhH/K+NfSl9x15vUTe2PsYHYnNH32GgJxolEqSzXrQgL/wEAPS1XNs1rIk7Nn3OIoY/KI/7IKruESIRxTqZCMpk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR11MB6992
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_NONE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 12/12/2022 2:26 AM, Arun Ramadoss wrote:
> +static int ksz_ptp_adjfine(struct ptp_clock_info *ptp, long scaled_ppm)
> +{
> +	struct ksz_ptp_data *ptp_data = ptp_caps_to_data(ptp);
> +	struct ksz_device *dev = ptp_data_to_ksz_dev(ptp_data);
> +	int ret;
> +
> +	mutex_lock(&ptp_data->lock);
> +
> +	if (scaled_ppm) {
> +		s64 ppb, adj;
> +		u32 data32;
> +
> +		/* see scaled_ppm_to_ppb() in ptp_clock.c for details */
> +		ppb = 1 + scaled_ppm;
> +		ppb *= 125;
> +		ppb *= KSZ_PTP_INC_NS;
> +		ppb <<= KSZ_PTP_SUBNS_BITS - 13;
> +		adj = div_s64(ppb, NSEC_PER_SEC);
> +
> +		data32 = abs(adj);
> +		data32 &= PTP_SUBNANOSEC_M;
> +		if (adj >= 0)
> +			data32 |= PTP_RATE_DIR;
> +

Can you use adjust_by_scaled_ppm or diff_by_scalled_ppm? These work by
defining the base increment for your device to achieve nominal
nanoseconds, and then perform the multiple+divide to calculate the
modified adjustment based on scaled_ppm. The diff_by_scaled_ppm looks
like what you want as it takes a base, the scaled adjustment factor and
then exports the diff as the 3rd argument. It returns true if the
difference is negative so ou can use that to determine if you need to
add the PTP_RATE_DIR flag.

If for some reason diff_by_scaled_ppm isn't sufficient for your
hardware, at least use scaled_ppm_to_ppb to get the ppb value instead of
open coding the conversion.

Thanks,
Jake