From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 34F9FE92FE9 for ; Fri, 6 Oct 2023 06:24:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230175AbjJFGYV (ORCPT ); Fri, 6 Oct 2023 02:24:21 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46804 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229848AbjJFGYT (ORCPT ); Fri, 6 Oct 2023 02:24:19 -0400 X-Greylist: delayed 60 seconds by postgrey-1.37 at lindbergh.monkeyblade.net; Thu, 05 Oct 2023 23:24:16 PDT Received: from smtpcmd04132.aruba.it (smtpcmd04132.aruba.it [62.149.158.132]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5300590 for ; Thu, 5 Oct 2023 23:24:16 -0700 (PDT) Received: from [192.168.1.56] ([79.0.204.227]) by Aruba Outgoing Smtp with ESMTPSA id oeEwqIZBPRbLIoeExqSU1E; Fri, 06 Oct 2023 08:23:12 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=aruba.it; s=a1; t=1696573392; bh=1g4cE2HuG32l4c0jdgyiNG0Vc4f/Jd4BC4OAvpvtJu8=; h=Date:MIME-Version:Subject:To:From:Content-Type; b=m6x/TAAfP6HnbJu44l9yFOq+rN+xFSmM2Cu0xkgvVT3A1PnyRjldZ9sVAo92jitTN vU5poXE2Oh4NZnF9PaCPoLMsojq/gxS/Rme/Bh1M2H7tS82gRSWAqELkyujuRSXCkf pK21CckOXU5H2YONqoVUluYCdRn60U5DkB2+CXFwQxLSGZ0JZUJwBf4OOR7tLb/ZfR QrBhjuJGuxexB7ep+vHSTeJUc4MYioCWhlUFNWt6/WjvhnjIqIuzrs784RsBDPp2VH nIX5kq4X/ng7KXABWJcsZfxi6jiYFFUh4yiji0ouaGjEZIKtCXTLy/j9Dp7rhmYvj9 cbr+4WDKBbIjg== Message-ID: <8ca736c3-7da9-2599-7e55-15e2fc9fedc2@enneenne.com> Date: Fri, 6 Oct 2023 08:23:09 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 Subject: Re: PPS functionality for Intel Timed I/O Content-Language: en-US To: "N, Pandith" Cc: "linux-kernel@vger.kernel.org" , "Sangannavar, Mallikarjunappa" , "D, Lakshmi Sowjanya" , "T R, Thejesh Reddy" , "Hall, Christopher S" , "Gross, Mark" , "andriy.shevchenko@linux.intel.com" References: <1e02cc71-baee-1e75-9160-062d563af795@enneenne.com> From: Rodolfo Giometti In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4xfMAx4tZAxRB7YBxzABY8FxJ4N3jczSvovPedbUKVPywsYjMWLUH6jDo+TwaOhasfJFiG2l8OCieAyiy/DbTSIMEgwdAEYtu/FSLtxznBpXS2ej/HQeKB mxySHZA7U+8rv6mjdrLbM2EwX8mCZKPTqhyfCY+50Chho7SiQjvkgk5qlcn+I+XykDtZvWEhiF6JE09zyd5QNG6toCoIouI3Qo5PzygcjPlmHHg8h6VFXXN0 eYX2HuQmv3i9aN4ua5vgheHAl20XQnIzOgQcpKh7wHuei+FFb3cLvmFQZz1dBotbTXSahLv4glluFtkze766i5h0NxmLS+qa3k/MMnkgDTbA5atIeaMAUpr0 ZVVHcC9S2IbRF/v1pTfimwlKiTKi11/e97P/vHdHOahSIM5rj1jZVJroCqfyNOxQAHxUq/Uzgkd1jPOSURwK9pp8tMa2sgIjvnAKLNbNiZf2zOQNF2jEa1bd xCjD1KhOUoR4yBTCu7S4nvaL8tW+80BylMalfQ== Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/10/23 07:31, N, Pandith wrote: > Hi, Hello. >> -----Original Message----- >> From: Rodolfo Giometti >> Sent: Wednesday, February 15, 2023 1:16 PM >> To: N, Pandith >> Cc: linux-kernel@vger.kernel.org; Sangannavar, Mallikarjunappa >> ; D, Lakshmi Sowjanya >> ; T R, Thejesh Reddy >> ; Hall, Christopher S >> ; Gross, Mark >> Subject: Re: PPS functionality for Intel Timed I/O >> >> On 15/02/23 08:09, N, Pandith wrote: >>> Hi Rodolfo, >> >> Hello. >> >>>> -----Original Message----- >>>> From: Rodolfo Giometti >>>> Sent: Monday, February 6, 2023 4:17 PM >>>> To: N, Pandith >>>> Cc: linux-kernel@vger.kernel.org; Sangannavar, Mallikarjunappa >>>> ; D, Lakshmi Sowjanya >>>> ; T R, Thejesh Reddy >>>> ; Hall, Christopher S >>>> >>>> Subject: Re: PPS functionality for Intel Timed I/O >> >> [snip] >> >>>> Mmm... I'm not sure this is correct since PPS generators should >>>> generate their pulses according to system clock and not according to >>>> their internal clocks even if they are synced with the system clock. >>>> >>> Timed I/O and system time are both driven by the *same* hardware clock. >>> Timed I/O is a high precision device (nanoseconds), able to output pulses. >>> The driver gets the system time and schedules output at target_time in >>> the future. >> >> OK, in this case it would be OK. Please put an appropriate note within the >> generator's code and a detailed one within Documentation/driver-api/pps.rst. >> >> Regarding Documentation/driver-api/pps.rst let me suggest you to prose a >> separate patch to rewrite the Generators section in such a way you easily can add >> your solution at the end. A possible example is attached but feel free to rewrite it >> according to your needs. >> > Ok, we are updating documentation as you have suggested as a separate patch. > Can we use your "Signed-off-by" tag for attached patch. If you just followed my suggestion then it's OK for me. > We are adding Intel timed I/O documentation as a separate patch. OK. Ciao, Rodolfo Giometti -- GNU/Linux Solutions e-mail: giometti@enneenne.com Linux Device Driver giometti@linux.it Embedded Systems phone: +39 349 2432127 UNIX programming skype: rodolfo.giometti