Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
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" <pandith.n@intel.com>
Cc:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Sangannavar, Mallikarjunappa" 
        <mallikarjunappa.sangannavar@intel.com>,
        "D, Lakshmi Sowjanya" <lakshmi.sowjanya.d@intel.com>,
        "T R, Thejesh Reddy" <thejesh.reddy.t.r@intel.com>,
        "Hall, Christopher S" <christopher.s.hall@intel.com>,
        "Gross, Mark" <mark.gross@intel.com>,
        "andriy.shevchenko@linux.intel.com" 
        <andriy.shevchenko@linux.intel.com>
References: <BYAPR11MB3240A86B426158623DB9983EE1D39@BYAPR11MB3240.namprd11.prod.outlook.com>
 <fa3f1765-eb59-bf69-7f7b-14621caef6ea@enneenne.com>
 <BYAPR11MB3240801F21598EEFAEA79605E1D39@BYAPR11MB3240.namprd11.prod.outlook.com>
 <1e02cc71-baee-1e75-9160-062d563af795@enneenne.com>
 <BYAPR11MB32408E2D9758BD01EC65FB49E1DA9@BYAPR11MB3240.namprd11.prod.outlook.com>
 <f2788a74-19f8-8992-5b92-427c7b2a27ab@enneenne.com>
 <BYAPR11MB3240C6789B4C04F3BDAE34D5E1A39@BYAPR11MB3240.namprd11.prod.outlook.com>
 <f8a97493-a5ab-565f-825a-dd0a508f2b66@enneenne.com>
 <BYAPR11MB32405694C3C9A1DE083EA673E1C9A@BYAPR11MB3240.namprd11.prod.outlook.com>
From:   Rodolfo Giometti <giometti@enneenne.com>
In-Reply-To: <BYAPR11MB32405694C3C9A1DE083EA673E1C9A@BYAPR11MB3240.namprd11.prod.outlook.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 06/10/23 07:31, N, Pandith wrote:
> Hi,

Hello.

>> -----Original Message-----
>> From: Rodolfo Giometti <giometti@enneenne.com>
>> Sent: Wednesday, February 15, 2023 1:16 PM
>> To: N, Pandith <pandith.n@intel.com>
>> Cc: linux-kernel@vger.kernel.org; Sangannavar, Mallikarjunappa
>> <mallikarjunappa.sangannavar@intel.com>; D, Lakshmi Sowjanya
>> <lakshmi.sowjanya.d@intel.com>; T R, Thejesh Reddy
>> <thejesh.reddy.t.r@intel.com>; Hall, Christopher S
>> <christopher.s.hall@intel.com>; Gross, Mark <mark.gross@intel.com>
>> 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 <giometti@enneenne.com>
>>>> Sent: Monday, February 6, 2023 4:17 PM
>>>> To: N, Pandith <pandith.n@intel.com>
>>>> Cc: linux-kernel@vger.kernel.org; Sangannavar, Mallikarjunappa
>>>> <mallikarjunappa.sangannavar@intel.com>; D, Lakshmi Sowjanya
>>>> <lakshmi.sowjanya.d@intel.com>; T R, Thejesh Reddy
>>>> <thejesh.reddy.t.r@intel.com>; Hall, Christopher S
>>>> <christopher.s.hall@intel.com>
>>>> 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