All of lore.kernel.org
 help / color / mirror / Atom feed
From: Carsten Behling <carsten.behling@ridgerun.com>
To: linux-usb@vger.kernel.org
Cc: balbi@ti.com, linux-kernel@vger.kernel.org
Subject: Re: usb: musb: Scheduling of interrupt endpoints
Date: Tue, 23 Dec 2014 13:39:14 -0600	[thread overview]
Message-ID: <5499C4E2.8090407@ridgerun.com> (raw)
In-Reply-To: <5499835B.508@ridgerun.com>

The following comment can be found in 'musb_schedule()':

'* REVISIT what we really want here is a regular schedule tree
  * like e.g. OHCI uses.'

So I assume the best practice would be to make an implementation based
on the code in in ohci-q.c. And it would be waste of time to port the old
interrupt endpoint scheduling feature of TI.

Am I right?

On 12/23/2014 08:59 AM, Carsten Behling wrote:
> Would it help if I send a patch as a suggestion and as basis for 
> discussion?
>
> On 12/19/2014 01:38 PM, Carsten Behling wrote:
>> Hi all,
>>
>> Long time ago, TI shipped a kernel named 
>> linux-2.6.32.17-psp03.01.01.39 with an additional kernel option
>> for scheduling of interrupt endpoints.
>>
>> AFAIK, this seems to be the only possibility to attach more that 4 in 
>> endpoints to MUSB (at least on a DM368).
>>
>> This feature reserves one hardware endpoint unit to time schedule 
>> interrupt in endpoints based
>> on its bInterval value triggered by the SOF interrupt.
>>
>> I didn't find any discussion about adding such a feature to the 
>> mainline kernel.
>> IMHO, this feature is absolutely necessary. But there may be reasons, 
>> not to add it (e.g. CPU load).
>>
>> Please let me know your thoughts and ideas.
>>
>> Best regards
>> -Carsten
>>
>


  parent reply	other threads:[~2014-12-23 19:39 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <54947E99.4010908@ridgerun.com>
2014-12-23 14:59 ` usb: musb: Scheduling of interrupt endpoints Carsten Behling
2014-12-23 18:27   ` Felipe Balbi
2014-12-23 20:16     ` Carsten Behling
2014-12-23 23:48       ` Felipe Balbi
2014-12-23 19:39   ` Carsten Behling [this message]
2014-12-23 23:50     ` Felipe Balbi

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=5499C4E2.8090407@ridgerun.com \
    --to=carsten.behling@ridgerun.com \
    --cc=balbi@ti.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    /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.