Re: GTE - The hardware timestamping engine

2021-03-23 Thread Dipen Patel
On 3/23/21 3:06 AM, Thierry Reding wrote: > On Tue, Mar 23, 2021 at 10:08:00AM +0100, Linus Walleij wrote: >> On Mon, Mar 22, 2021 at 9:17 PM Dipen Patel wrote: >> >>> My follow-up concerns on both Linus's and Kent's feedback: >>> >>> 1. Please correct me if I am wrong, lineevent in the gpioli

Re: GTE - The hardware timestamping engine

2021-03-23 Thread Marc Zyngier
On Tue, 23 Mar 2021 10:06:39 +, Thierry Reding wrote: [...] > Obviously if we don't integrate this with IRQs directly, it becomes a > bit more difficult to relate the captured timestamps to the events > across subsystem boundaries. I'm not sure how this would be solved > properly. If the eve

Re: GTE - The hardware timestamping engine

2021-03-23 Thread Dipen Patel
On 3/23/21 3:06 AM, Thierry Reding wrote: > On Tue, Mar 23, 2021 at 10:08:00AM +0100, Linus Walleij wrote: >> On Mon, Mar 22, 2021 at 9:17 PM Dipen Patel wrote: >> >>> My follow-up concerns on both Linus's and Kent's feedback: >>> >>> 1. Please correct me if I am wrong, lineevent in the gpioli

Re: GTE - The hardware timestamping engine

2021-03-23 Thread Dipen Patel
On 3/23/21 2:08 AM, Linus Walleij wrote: > On Mon, Mar 22, 2021 at 9:17 PM Dipen Patel wrote: > >> My follow-up concerns on both Linus's and Kent's feedback: >> >> 1. Please correct me if I am wrong, lineevent in the gpiolib* is only >> serves the userspace clients. >> 1.a What about kern

Re: GTE - The hardware timestamping engine

2021-03-23 Thread Richard Cochran
On Tue, Mar 23, 2021 at 10:03:18AM +0100, Thierry Reding wrote: > I agree. My understanding is the the TSC is basically an SoC-wide clock > that can be (and is) used by several hardware blocks. There's an > interface for software to read out the value, but it's part of a block > called TKE (time-ke

Re: GTE - The hardware timestamping engine

2021-03-23 Thread Thierry Reding
On Tue, Mar 23, 2021 at 10:08:00AM +0100, Linus Walleij wrote: > On Mon, Mar 22, 2021 at 9:17 PM Dipen Patel wrote: > > > My follow-up concerns on both Linus's and Kent's feedback: > > > > 1. Please correct me if I am wrong, lineevent in the gpiolib* is only > > serves the userspace clients.

Re: GTE - The hardware timestamping engine

2021-03-23 Thread Linus Walleij
On Mon, Mar 22, 2021 at 9:17 PM Dipen Patel wrote: > My follow-up concerns on both Linus's and Kent's feedback: > > 1. Please correct me if I am wrong, lineevent in the gpiolib* is only > serves the userspace clients. > 1.a What about kernel drivers wanting to use this feature for monitoring

Re: GTE - The hardware timestamping engine

2021-03-23 Thread Thierry Reding
On Mon, Mar 22, 2021 at 01:33:38PM -0700, Dipen Patel wrote: > Hi Richard, > > Thanks for your input and time. Please see below follow up. > > On 3/20/21 8:38 AM, Richard Cochran wrote: > > On Sat, Mar 20, 2021 at 01:44:20PM +0100, Arnd Bergmann wrote: > >> Adding Richard Cochran as well, for dri

Re: GTE - The hardware timestamping engine

2021-03-22 Thread Kent Gibson
On Mon, Mar 22, 2021 at 09:09:50PM -0700, Dipen Patel wrote: > > > On 3/22/21 7:59 PM, Kent Gibson wrote: > > On Mon, Mar 22, 2021 at 06:53:10PM -0700, Dipen Patel wrote: > >> > >> > >> On 3/22/21 5:32 PM, Kent Gibson wrote: > >>> On Mon, Mar 22, 2021 at 01:21:46PM -0700, Dipen Patel wrote: > >>>

Re: GTE - The hardware timestamping engine

2021-03-22 Thread Dipen Patel
On 3/22/21 7:59 PM, Kent Gibson wrote: > On Mon, Mar 22, 2021 at 06:53:10PM -0700, Dipen Patel wrote: >> >> >> On 3/22/21 5:32 PM, Kent Gibson wrote: >>> On Mon, Mar 22, 2021 at 01:21:46PM -0700, Dipen Patel wrote: Hi Linus and Kent, > > [snip] > >>> In response to all your comments

Re: GTE - The hardware timestamping engine

2021-03-22 Thread Kent Gibson
On Mon, Mar 22, 2021 at 06:53:10PM -0700, Dipen Patel wrote: > > > On 3/22/21 5:32 PM, Kent Gibson wrote: > > On Mon, Mar 22, 2021 at 01:21:46PM -0700, Dipen Patel wrote: > >> Hi Linus and Kent, > >> [snip] > > In response to all your comments above... > > > > Firstly, I'm not suggesting that

Re: GTE - The hardware timestamping engine

2021-03-22 Thread Dipen Patel
On 3/22/21 5:32 PM, Kent Gibson wrote: > On Mon, Mar 22, 2021 at 01:21:46PM -0700, Dipen Patel wrote: >> Hi Linus and Kent, >> >> Thanks you so much for your valuable inputs and time, Please see below, my >> follow ups. >> >> On 3/21/21 11:00 PM, Kent Gibson wrote: >>> On Sat, Mar 20, 2021 at 1

Re: GTE - The hardware timestamping engine

2021-03-22 Thread Kent Gibson
On Mon, Mar 22, 2021 at 01:21:46PM -0700, Dipen Patel wrote: > Hi Linus and Kent, > > Thanks you so much for your valuable inputs and time, Please see below, my > follow ups. > > On 3/21/21 11:00 PM, Kent Gibson wrote: > > On Sat, Mar 20, 2021 at 12:56:36PM +0100, Linus Walleij wrote: > >> Hi Di

Re: GTE - The hardware timestamping engine

2021-03-22 Thread Dipen Patel
Hi Richard, Thanks for your input and time. Please see below follow up. On 3/20/21 8:38 AM, Richard Cochran wrote: > On Sat, Mar 20, 2021 at 01:44:20PM +0100, Arnd Bergmann wrote: >> Adding Richard Cochran as well, for drivers/ptp/, he may be able to >> identify whether this should be integrated

Re: GTE - The hardware timestamping engine

2021-03-22 Thread Dipen Patel
Hi Linus and Kent, Thanks you so much for your valuable inputs and time, Please see below, my follow ups. On 3/21/21 11:00 PM, Kent Gibson wrote: > On Sat, Mar 20, 2021 at 12:56:36PM +0100, Linus Walleij wrote: >> Hi Dipen, >> >> thanks for your mail! >> >> I involved some other kernel people to

Re: GTE - The hardware timestamping engine

2021-03-21 Thread Kent Gibson
On Sat, Mar 20, 2021 at 12:56:36PM +0100, Linus Walleij wrote: > Hi Dipen, > > thanks for your mail! > > I involved some other kernel people to get some discussion. > I think Kent Gibson can be of great help because he is using > GPIOs with high precision. > Actually I just extended the cdev uA

Re: GTE - The hardware timestamping engine

2021-03-20 Thread Richard Cochran
On Sat, Mar 20, 2021 at 01:44:20PM +0100, Arnd Bergmann wrote: > Adding Richard Cochran as well, for drivers/ptp/, he may be able to > identify whether this should be integrated into that framework in some > form. I'm not familiar with the GTE, but it sounds like it is a (free running?) clock with

Re: GTE - The hardware timestamping engine

2021-03-20 Thread Arnd Bergmann
On Sat, Mar 20, 2021 at 12:56 PM Linus Walleij wrote: > > Hi Dipen, > > thanks for your mail! > > I involved some other kernel people to get some discussion. > I think Kent Gibson can be of great help because he is using > GPIOs with high precision. > > We actually discussed this a bit when adding

Re: GTE - The hardware timestamping engine

2021-03-20 Thread Linus Walleij
Hi Dipen, thanks for your mail! I involved some other kernel people to get some discussion. I think Kent Gibson can be of great help because he is using GPIOs with high precision. We actually discussed this a bit when adding support for realtime timestamps. On Wed, Mar 17, 2021 at 11:29 PM Dipe

GTE - The hardware timestamping engine

2021-03-17 Thread Dipen Patel
Hi All, I wanted to discuss few implementation details regarding the GTE module and wanted to know your feedback on certain aspects as below. == GTE introductions: == Nvidia Tegra SoCs have generic timestamping engine (GTE) hardware module which can monitor SoC sig