All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <guenter.roeck@ericsson.com>
To: Sid Boyce <sboyce@blueyonder.co.uk>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] misc: Driver for Silicon Labs Si570 and compatibles
Date: Mon, 9 May 2011 11:31:38 -0700	[thread overview]
Message-ID: <20110509183138.GB4964@ericsson.com> (raw)
In-Reply-To: <4DB17A3B.3060609@blueyonder.co.uk>

Hi Sid,

On Fri, Apr 22, 2011 at 08:53:15AM -0400, Sid Boyce wrote:
> Guenter Roeck wrote:
> Depends. In our case, turns out the devices consuming the clock
> have user mode drivers. Lots of history there, but the chip vendors
> provide those user mode drivers, and the teams responsible for
> integrating the drivers decided to not rewrite it to kernel mode drivers.
> Also, for special purposes such as margining, it is necessary to control
> the clock from userspace. So, for our use case, I need the user-visible
> interface. I _don't_ need the kernel interface, at least not right now,
> which is why I did not add it.
> 
> Browsing through the web, it seems the chip is somewhat popular with
> Amateur Radio. No idea if it would ever be controlled for such a purpose
> from Linux, but if so, it would also require a user configurable frequency.
> 
> If there is a better place for such a driver than misc, please let me know.
> 
> Thanks,
> Guenter
> =============================================================================
> Support for these devices included in several Amateur radio programs 
> tend to differ greatly which means reinventing the wheel in most of them.
> Programs like Linrad, sdr-shell, quisk, lysdr, ghpsdr3, etc.
> It's a small addition to the kernel that would significantly help in the 
> development of software for the myriad of SDR (Software Defined Radio) 
> designs based on this chip family.
> I have 5 Amateur Radio transceivers using this chip and 2 more soon to 
> be added.
> They are all open hardware heavily reliant on Linux.
> The addition of UAC2 (USB Audio Class 2) kernel support has helped our 
> SDR hardware design efforts significantly.
> I and the rest of the Linux SDR community would be happy to see it included.
> Regards
> Sid.

Thanks for the feedback. Somehow I missed your reply at the time.

As you probably have noticed from the discussion, I seems to be unlikely that
the driver will be accepted into the kernel, at least not unless there is
significant demand for it from others.

It may help if you can provide a list of requirements from your side (do you need 
a kernel API ? Do you need an ABI to set the frequency from applications ?).

I would be happy to provide and maintain the driver as separate patch until it 
is accepted (if that ever happens), and work with application maintainers to 
integrate it. Let me know if there is interest, and feel free to forward this
e-mail to interested parties.

Thanks,
Guenter

  reply	other threads:[~2011-05-09 18:31 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-22 12:53 [PATCH v2] misc: Driver for Silicon Labs Si570 and compatibles Sid Boyce
2011-05-09 18:31 ` Guenter Roeck [this message]
2011-05-09 20:18   ` Sid Boyce
  -- strict thread matches above, loose matches on Subject: below --
2011-04-19 21:36 Guenter Roeck
2011-04-19 21:36 ` Guenter Roeck
2011-04-20  9:23 ` Jonathan Cameron
     [not found]   ` <4DAEA618.4040801-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org>
2011-04-20 16:34     ` Guenter Roeck
2011-04-20 16:34       ` Guenter Roeck
2011-04-20 16:44 ` Arnd Bergmann
2011-04-20 18:16   ` Guenter Roeck
2011-04-21 11:11     ` Arnd Bergmann
2011-04-21 15:56       ` Guenter Roeck
     [not found]         ` <20110421155658.GA28245-IzeFyvvaP7pWk0Htik3J/w@public.gmane.org>
2011-04-21 16:23           ` Arnd Bergmann
2011-04-21 16:23             ` Arnd Bergmann
2011-04-21 18:47             ` Guenter Roeck
2011-04-21 19:00               ` Arnd Bergmann
2011-04-21 21:58                 ` Guenter Roeck
2011-04-21 23:34       ` Hans J. Koch
2011-04-22 19:40         ` Guenter Roeck
2011-04-22 19:40           ` Guenter Roeck
2011-04-26 14:00           ` Arnd Bergmann
2011-04-26 14:00             ` Arnd Bergmann
2011-04-26 15:33             ` Guenter Roeck

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=20110509183138.GB4964@ericsson.com \
    --to=guenter.roeck@ericsson.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sboyce@blueyonder.co.uk \
    /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.