From: Marcel Holtmann <marcel@holtmann.org>
To: ofono@ofono.org
Subject: Re: [n900 PATCH 2/5] isigen: generic modem driver plugin for isimodems
Date: Fri, 17 Sep 2010 10:18:11 +0900 [thread overview]
Message-ID: <1284686291.2405.176.camel@localhost.localdomain> (raw)
In-Reply-To: <53bb2abc324cd8002bbeaab6db1164e1@chewa.net>
[-- Attachment #1: Type: text/plain, Size: 1032 bytes --]
Hi Remi,
> >> >> That is the goal if and when we have up-to-date kernel, but
> >> >> we are not there yet.
> >> >
> >> > I don't see the point of adding temporary cruft to the main oFono
> >> > repository then.
> >>
> >> Well, I see value in having oFono upstream work as-is with the current
> >> MeeGo kernel. The plugin could of course reside somewhere else, but
> >> that is just adding extra work for packagers. And it's not like we
> >> can't fix the plugin once MeeGo has a proper kernel in place.
> >
> > what is the current state of Phonet support (including N900) and the
> > current MeeGo kernel? Are we missing patches?
>
> I did not fully grasp what the extra Netlink code does. There should be no
> need to CONFIGURE the Phonet stack via Netlink. If there is, then it's a
> kernel bug.
you are the expert here and have to give us some guideline on how we
should use Phonet the right way. If you say that we should remove the
netlink code from oFono, I am all for it.
Regards
Marcel
prev parent reply other threads:[~2010-09-17 1:18 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-15 16:59 [n900 PATCH 0/5] n900 modem drivers Pekka.Pessi
2010-09-15 16:59 ` [n900 PATCH 1/5] Include isimodem sources always in dist Pekka.Pessi
2010-09-15 16:59 ` [n900 PATCH 2/5] isigen: generic modem driver plugin for isimodems Pekka.Pessi
2010-09-15 16:59 ` [n900 PATCH 3/5] isimodem: remove the modem driver parts Pekka.Pessi
2010-09-15 16:59 ` [n900 PATCH 4/5] n900: modem driver plugin for Nokia N900 Pekka.Pessi
2010-09-15 16:59 ` [n900 PATCH 5/5] n900maemo: modem driver plugin for N900 with Maemo Pekka.Pessi
2010-09-15 18:07 ` [n900 PATCH 2/5] isigen: generic modem driver plugin for isimodems =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2010-09-15 21:27 ` Pekka Pessi
2010-09-15 22:00 ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2010-09-16 6:02 ` Aki Niemi
2010-09-16 6:26 ` Marcel Holtmann
2010-09-16 23:45 ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2010-09-17 1:18 ` Marcel Holtmann [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=1284686291.2405.176.camel@localhost.localdomain \
--to=marcel@holtmann.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox