All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Richardson <jonathar@broadcom.com>
To: Richard Cochran <richardcochran@gmail.com>
Cc: Arnd Bergmann <arnd@arndb.de>,
	Darren Edamura <dedamura@broadcom.com>,
	One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>,
	Scott Branden <sbranden@broadcom.com>,
	Pawel Moll <pawel.moll@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Ray Jui <rjui@broadcom.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	<linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	<bcm-kernel-feedback-list@broadcom.com>,
	Kumar Gala <galak@codeaurora.org>,
	Mark Rutland <mark.rutland@arm.com>,
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 2/2] misc: Add initial Digital Timing Engine (DTE) driver for cygnus
Date: Thu, 21 May 2015 10:48:32 -0700	[thread overview]
Message-ID: <555E1A70.90305@broadcom.com> (raw)
In-Reply-To: <20150521063303.GA1558@localhost.localdomain>

On 15-05-20 11:33 PM, Richard Cochran wrote:
> On Wed, May 20, 2015 at 04:38:02PM -0700, Jonathan Richardson wrote:
>>
>> Richard, this design isn't going to work. We need to have both kernel
>> and user space consumers.
> 
> But you did not implement even a single kernel consumer.
> 
>> We don't want all GPIO's in a common timestamp
>> buffer either, as it presents problems I mentioned previously. Currently
>> the network input is a gpio.
> 
> That won't work.  Consider what happens when the MAC drops a packet.
> 
>> After some discussion here I think we'll
>> have to keep this driver out of the kernel for now.
> 
> Fine with me.

We'll keep this discussion and try to figure out the best interface
keeping in mind what you've said as we move further along with it. Not a
wasted discussion. Requirements of this need to be better defined and
thought out also.

Thanks,
Jon

> 
> Thanks,
> Richard
> 


WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Richardson <jonathar@broadcom.com>
To: Richard Cochran <richardcochran@gmail.com>
Cc: Arnd Bergmann <arnd@arndb.de>,
	Darren Edamura <dedamura@broadcom.com>,
	One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>,
	Scott Branden <sbranden@broadcom.com>,
	Pawel Moll <pawel.moll@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Ray Jui <rjui@broadcom.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	Rob Herring <robh+dt@kernel.org>,
	bcm-kernel-feedback-list@broadcom.com,
	Kumar Gala <galak@codeaurora.org>,
	Mark Rutland <mark.rutland@arm.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 2/2] misc: Add initial Digital Timing Engine (DTE) driver for cygnus
Date: Thu, 21 May 2015 10:48:32 -0700	[thread overview]
Message-ID: <555E1A70.90305@broadcom.com> (raw)
In-Reply-To: <20150521063303.GA1558@localhost.localdomain>

On 15-05-20 11:33 PM, Richard Cochran wrote:
> On Wed, May 20, 2015 at 04:38:02PM -0700, Jonathan Richardson wrote:
>>
>> Richard, this design isn't going to work. We need to have both kernel
>> and user space consumers.
> 
> But you did not implement even a single kernel consumer.
> 
>> We don't want all GPIO's in a common timestamp
>> buffer either, as it presents problems I mentioned previously. Currently
>> the network input is a gpio.
> 
> That won't work.  Consider what happens when the MAC drops a packet.
> 
>> After some discussion here I think we'll
>> have to keep this driver out of the kernel for now.
> 
> Fine with me.

We'll keep this discussion and try to figure out the best interface
keeping in mind what you've said as we move further along with it. Not a
wasted discussion. Requirements of this need to be better defined and
thought out also.

Thanks,
Jon

> 
> Thanks,
> Richard
> 

WARNING: multiple messages have this Message-ID (diff)
From: jonathar@broadcom.com (Jonathan Richardson)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] misc: Add initial Digital Timing Engine (DTE) driver for cygnus
Date: Thu, 21 May 2015 10:48:32 -0700	[thread overview]
Message-ID: <555E1A70.90305@broadcom.com> (raw)
In-Reply-To: <20150521063303.GA1558@localhost.localdomain>

On 15-05-20 11:33 PM, Richard Cochran wrote:
> On Wed, May 20, 2015 at 04:38:02PM -0700, Jonathan Richardson wrote:
>>
>> Richard, this design isn't going to work. We need to have both kernel
>> and user space consumers.
> 
> But you did not implement even a single kernel consumer.
> 
>> We don't want all GPIO's in a common timestamp
>> buffer either, as it presents problems I mentioned previously. Currently
>> the network input is a gpio.
> 
> That won't work.  Consider what happens when the MAC drops a packet.
> 
>> After some discussion here I think we'll
>> have to keep this driver out of the kernel for now.
> 
> Fine with me.

We'll keep this discussion and try to figure out the best interface
keeping in mind what you've said as we move further along with it. Not a
wasted discussion. Requirements of this need to be better defined and
thought out also.

Thanks,
Jon

> 
> Thanks,
> Richard
> 

  reply	other threads:[~2015-05-21 17:49 UTC|newest]

Thread overview: 67+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-22 23:22 [PATCH 0/2] Add DTE driver for Cygnus Jonathan Richardson
2015-04-22 23:22 ` Jonathan Richardson
2015-04-22 23:22 ` Jonathan Richardson
2015-04-22 23:22 ` [PATCH 1/2] misc: Add DT binding for cygnus Digital Timing Engine (DTE) driver Jonathan Richardson
2015-04-22 23:22   ` Jonathan Richardson
2015-04-22 23:22   ` Jonathan Richardson
2015-04-22 23:22 ` [PATCH 2/2] misc: Add initial Digital Timing Engine (DTE) driver for cygnus Jonathan Richardson
2015-04-22 23:22   ` Jonathan Richardson
2015-04-22 23:22   ` Jonathan Richardson
2015-04-23  8:04   ` Arnd Bergmann
2015-04-23  8:04     ` Arnd Bergmann
2015-04-23  8:04     ` Arnd Bergmann
2015-04-23 18:07     ` Jonathan Richardson
2015-04-23 18:07       ` Jonathan Richardson
2015-04-23 18:07       ` Jonathan Richardson
2015-05-01 19:01     ` Jonathan Richardson
2015-05-01 19:01       ` Jonathan Richardson
2015-05-01 19:01       ` Jonathan Richardson
2015-05-01 19:30       ` One Thousand Gnomes
2015-05-01 19:30         ` One Thousand Gnomes
2015-05-01 19:30         ` One Thousand Gnomes
2015-05-01 19:40         ` Arnd Bergmann
2015-05-01 19:40           ` Arnd Bergmann
2015-05-08 20:02           ` Jonathan Richardson
2015-05-08 20:02             ` Jonathan Richardson
2015-05-08 20:02             ` Jonathan Richardson
2015-05-13  1:03             ` Jonathan Richardson
2015-05-13  1:03               ` Jonathan Richardson
2015-05-13  1:03               ` Jonathan Richardson
2015-05-13 12:19             ` Arnd Bergmann
2015-05-13 12:19               ` Arnd Bergmann
2015-05-13 14:37               ` Richard Cochran
2015-05-13 14:37                 ` Richard Cochran
2015-05-13 14:51                 ` Richard Cochran
2015-05-13 14:51                   ` Richard Cochran
2015-05-13 15:35             ` Richard Cochran
2015-05-13 15:35               ` Richard Cochran
2015-05-13 19:50               ` Jonathan Richardson
2015-05-13 19:50                 ` Jonathan Richardson
2015-05-13 19:50                 ` Jonathan Richardson
2015-05-13 20:27                 ` Richard Cochran
2015-05-13 20:27                   ` Richard Cochran
2015-05-13 23:25                   ` Jonathan Richardson
2015-05-13 23:25                     ` Jonathan Richardson
2015-05-13 23:25                     ` Jonathan Richardson
2015-05-14 11:30                     ` Richard Cochran
2015-05-14 11:30                       ` Richard Cochran
2015-05-14 11:30                       ` Richard Cochran
2015-05-20 23:38                       ` Jonathan Richardson
2015-05-20 23:38                         ` Jonathan Richardson
2015-05-20 23:38                         ` Jonathan Richardson
2015-05-21  6:33                         ` Richard Cochran
2015-05-21  6:33                           ` Richard Cochran
2015-05-21 17:48                           ` Jonathan Richardson [this message]
2015-05-21 17:48                             ` Jonathan Richardson
2015-05-21 17:48                             ` Jonathan Richardson
2015-05-13 15:21       ` Richard Cochran
2015-05-13 15:21         ` Richard Cochran
2015-05-13 15:21         ` Richard Cochran
2015-05-13 19:38         ` Jonathan Richardson
2015-05-13 19:38           ` Jonathan Richardson
2015-05-13 19:38           ` Jonathan Richardson
2015-05-13 19:42           ` Richard Cochran
2015-05-13 19:42             ` Richard Cochran
2015-05-13 19:39         ` Jonathan Richardson
2015-05-13 19:39           ` Jonathan Richardson
2015-05-13 19:39           ` Jonathan Richardson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=555E1A70.90305@broadcom.com \
    --to=jonathar@broadcom.com \
    --cc=arnd@arndb.de \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=dedamura@broadcom.com \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=gnomes@lxorguk.ukuu.org.uk \
    --cc=gregkh@linuxfoundation.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=richardcochran@gmail.com \
    --cc=rjui@broadcom.com \
    --cc=robh+dt@kernel.org \
    --cc=sbranden@broadcom.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.