All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: PATCH: Enable udev-based autodetection of calypso modem on Freerunner
Date: Tue, 21 Dec 2010 12:16:18 -0600	[thread overview]
Message-ID: <4D10EEF2.1050003@gmail.com> (raw)
In-Reply-To: <1292955183.2658.60.camel@aeonflux>

[-- Attachment #1: Type: text/plain, Size: 960 bytes --]

Hi Marcel,

On 12/21/2010 12:13 PM, Marcel Holtmann wrote:
> Hi Denis,
> 
>>>> I think the better way would be to drop a udev rule directly into
>>>> /etc/udev.d or similar.  The rule should be as simple as:
>>>> KERNEL=="ttySAC0", ENV{OFONO_DRIVER}="calypso"
>>>
>>> Do you mean in another file than 97-ofono.rules?
>>
>> Yep.
> 
> I prefer not to have this inside ofono.git at all. We also leave the IFX
> specific ones out. It is really up to the package maintainers to get
> this sorted.
> 

That is actually what I had in mind.  Maybe I didn't communicate this
properly.

> We could document this as Freerunner specific notes to point people into
> the right direction. And we should maybe to the same for IFX since that
> is currently only documented in the GIT commit logs.

Documentation inside the individual modem driver is the way to go.  And
IFX isn't even documented in git commit logs.  I looked ;)

Regards,
-Denis

  reply	other threads:[~2010-12-21 18:16 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-15 22:12 PATCH: Enable udev-based autodetection of calypso modem on Freerunner Neil Jerram
2010-12-20 23:39 ` Denis Kenzior
2010-12-21 12:01   ` Neil Jerram
2010-12-21 18:01     ` Denis Kenzior
2010-12-21 18:13       ` Marcel Holtmann
2010-12-21 18:16         ` Denis Kenzior [this message]
2010-12-21 19:05           ` Marcel Holtmann
2010-12-21 19:18             ` Denis Kenzior
2010-12-21 19:22               ` Marcel Holtmann
2010-12-21 19:33                 ` Denis Kenzior
2010-12-21 21:34                   ` Neil Jerram
2010-12-22 13:37                     ` Marcel Holtmann
2010-12-23 12:02                       ` Neil Jerram

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=4D10EEF2.1050003@gmail.com \
    --to=denkenz@gmail.com \
    --cc=ofono@ofono.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.