All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Arun MURTHY <arun.murthy@stericsson.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
	Sjur BRENDELAND <sjur.brandeland@stericsson.com>
Subject: Re: [PATCHv2 1/4] modem_shm: Add Modem Access Framework
Date: Wed, 8 Aug 2012 06:37:48 -0700	[thread overview]
Message-ID: <20120808133748.GA14459@kroah.com> (raw)
In-Reply-To: <F45880696056844FA6A73F415B568C695B0E796A03@EXDCVYMBSTM006.EQ1STM.local>

On Wed, Aug 08, 2012 at 05:36:05AM +0200, Arun MURTHY wrote:
> > On Tue, Aug 07, 2012 at 12:24:28PM +0530, Arun Murthy wrote:
> > > Adds Modem Access Framework, which allows for registering platform
> > specific
> > > modem access mechanisms. The framework also exposes APIs for client
> > drivers
> > > for getting and releasing access to modem, regardless of the
> > underlying
> > > platform specific access mechanism.
> > 
> > The term "modems" here has a lot of legacy connotations.  First of
> > which
> > is, userspace handles this today as tty devices, why aren't you doing
> > the same here?  Why does this have to be something "special"?
> > 
> 
> The main focus over there the modem IPC.

Over where?

What "modem IPC"?

You need to really explain what you are doing here, as I have no idea.
And I have no idea why you still can't just use a tty device, why can't
you?

> In doing so, there are some functionality like waking the modem, or
> releasing the modem etc. These will be used by the modem IPC drivers
> and also few others like sim driver and security drivers.

What's a "sim driver", and what type of "security drivers" are you
referring to?

> Since this is a shared call and hence has to be synchronized. Hence so a
> small framework like is being done to monitor the modem access related only
> operations.

Again, why can't the tty layer do this for you?

greg k-h

  reply	other threads:[~2012-08-08 13:37 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-07  6:54 [PATCHv2 0/4] modem_shm: U8500 SHaRed Memory driver(SHRM) Arun Murthy
2012-08-07  6:54 ` [PATCHv2 1/4] modem_shm: Add Modem Access Framework Arun Murthy
2012-08-07 18:38   ` Greg KH
2012-08-08  3:36     ` Arun MURTHY
2012-08-08 13:37       ` Greg KH [this message]
2012-08-09  9:53         ` Arun MURTHY
2012-08-09 11:38           ` Alan Cox
2012-08-13 13:33             ` Linus Walleij
2012-08-09 10:14         ` Arun MURTHY
2012-08-07  6:54 ` [PATCHv2 2/4] modem_shm: Register u8500 client for MAF Arun Murthy
2012-08-07  6:54 ` [PATCHv2 3/4] modem_shm: u8500-shm: U8500 Shared Memory Driver Arun Murthy
2012-08-07 10:01   ` Alan Cox
2012-08-08  3:03     ` Arun MURTHY
2012-08-08 10:46       ` Alan Cox
2012-08-09 10:11         ` Arun MURTHY
2012-08-07 18:39   ` Greg KH
2012-08-08  3:36     ` Arun MURTHY
2012-08-07  6:54 ` [PATCHv2 4/4] Doc: Add u8500_shrm document Arun Murthy

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=20120808133748.GA14459@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=arun.murthy@stericsson.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=sjur.brandeland@stericsson.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 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.