public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: Haavard Skinnemoen <hskinnemoen@atmel.com>
Cc: Hans-Christian Egtvedt <hcegtvedt@atmel.com>,
	Andrew Victor <andrew@sanpeople.com>,
	kernel-avr32 <kernel@avr32linux.org>,
	Patrice Vilchez <patrice.vilchez@rfo.atmel.com>,
	Nicolas FERRE <nicolas.ferre@rfo.atmel.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/1] Driver for the Atmel on-chip SSC on AT32AP and AT91.
Date: Thu, 19 Jul 2007 07:45:42 -0700	[thread overview]
Message-ID: <200707190745.43151.david-b@pacbell.net> (raw)
In-Reply-To: <20070719113136.4d0a951f@dhcp-255-175.norway.atmel.com>

On Thursday 19 July 2007, Haavard Skinnemoen wrote:

> > So the protocol drivers need some kind of resource management in order
> > to not step on each others' toes, and that's the reason for the export
> > of ssc_request() and ssc_free().
> > 
> > I'm not sure if anyone would have been very mad at me for sneaking this
> > in through the avr32 tree, but I'd really like to know what others, in
> > particular the AT91 people and David, think of this kind of thing
> > before it ends up in mainline.

Does this work with both AT91 and AVR32 instances of SSC-to-at73c213
audio output hardware?  Clearly it "should".

This "kind of thing" seems pretty routine.  I see it all the time with
timer modules and DMA channels ... as well as flexible serial links like
this "SSC" module.  (Other serial-link examples include McBSP on OMAP,
SSP on PXA, and lots of non-AC97 audio data stream links.)

In this case a simple request/free interface is enough, since only one
module will hook up to the external hardware.  (Here, a Codec/AMP chip.)
In other cases the request may imply a more complex choice algorithm
than just matching IDs:  the particular DMA channel or timer returned
may not matter.  So for now I think hardware-specific APIs are OK.

Thing is, these don't fit very nicely into the driver model since not
all instances of that silicon module will use the same driver.  One
instance might manage a bidirectional audio stream through a codec;
another might implement the control interface to that code using SPI;
while a third might manage a custom data acquisition interface.

- Dave

      reply	other threads:[~2007-07-19 14:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-16  6:52 [PATCH 1/1] Driver for the Atmel on-chip SSC on AT32AP and AT91 Hans-Christian Egtvedt
2007-07-19  0:01 ` Andrew Morton
     [not found]   ` <469F0A78.3040808@atmel.com>
     [not found]     ` <20070719110122.262c2778@dhcp-255-175.norway.atmel.com>
2007-07-19  9:31       ` Haavard Skinnemoen
2007-07-19 14:45         ` David Brownell [this message]

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=200707190745.43151.david-b@pacbell.net \
    --to=david-b@pacbell.net \
    --cc=akpm@linux-foundation.org \
    --cc=andrew@sanpeople.com \
    --cc=hcegtvedt@atmel.com \
    --cc=hskinnemoen@atmel.com \
    --cc=kernel@avr32linux.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nicolas.ferre@rfo.atmel.com \
    --cc=patrice.vilchez@rfo.atmel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox