linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ohad Ben-Cohen <ohad@wizery.com>
To: Michal Simek <monstr@monstr.eu>
Cc: LKML <linux-kernel@vger.kernel.org>,
	John Williams <john.williams@petalogix.com>,
	linux-arm <linux-arm-kernel@lists.infradead.org>,
	"Grosen, Mark" <mgrosen@ti.com>, Arnd Bergmann <arnd@arndb.de>,
	Grant Likely <grant.likely@secretlab.ca>,
	devicetree-discuss@lists.ozlabs.org
Subject: Re: remoteproc: Load coprocessor code to the specific main memory location
Date: Fri, 17 Feb 2012 02:55:33 +0100	[thread overview]
Message-ID: <CAK=WgbZBJJbFcoeZ+hY=BgyW_4yvOJPVNMZ18_9o+zY7tEc_pQ@mail.gmail.com> (raw)
In-Reply-To: <CAHTX3dJ4PruKO-wJLJU8W0boHFL0f6eoF+QNaoNjm5hu26YEow@mail.gmail.com>

On Thu, Feb 16, 2012 at 9:21 PM, Michal Simek <monstr@monstr.eu> wrote:
> IRC in init part Linux kick rtos which send which services are available.in
> any format which is unknown to me.

Check out struct rpmsg_ns_msg - that's the exact format of the name
service notification messages (it's also mentioned by
Documentation/rpmsg.txt).

Of course, these messages, like any other message that is sent across,
need to begin with the usual rpmsg header. Its format is defined by
struct rpmsg_hdr.

To send the message from the rtos to Linux, you then need to grab a
buffer from the available ring (of the vring which is used by Linux to
receive messages), fill it with the message, put it in the used ring,
and kick the Linux side.

> Please also correct me if I read the code wrongly. that mailbox in omap
> driver is used just to say which buffer contains that message. Is it
> correct?

We're using the mailbox payload in order to indicate which vring has
pending buffers.

> I can create 15 sw signals to tell rtos which buffer contains message.

You can also just use a single interrupt and then, when it is
triggered, check both vrings for pending buffers. It's just another if
statement and shouldn't be too much of an overhead.

> Will be help if I clear my code and send it to you for review? Maybe you
> will see where the problem is.

I'm not sure I'll have the time to review it, but let's try to
progress this way and see how it goes.

Sorry I couldn't answer faster this week - I'm in ELC and was a bit
away from emails. I'll soon be flying back and that will keep me away
from emails again, but b/o next week I'll be online again.

Thanks,
Ohad.

  parent reply	other threads:[~2012-02-17  1:55 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-24 17:20 remoteproc: Load coprocessor code to the specific main memory location Michal Simek
2012-01-25 11:42 ` Ohad Ben-Cohen
2012-01-25 12:41   ` Michal Simek
2012-01-25 13:45     ` Ohad Ben-Cohen
2012-01-25 14:19       ` Michal Simek
2012-01-25 16:24         ` Ohad Ben-Cohen
2012-01-26 10:44           ` Michal Simek
2012-01-27 10:06             ` Michal Simek
2012-01-27 11:38             ` Ohad Ben-Cohen
2012-02-13 14:20               ` Michal Simek
2012-02-14 16:28                 ` Ohad Ben-Cohen
2012-02-15 12:47                   ` Michal Simek
2012-02-15 14:36                     ` Michal Simek
2012-02-16  9:12                       ` Michal Simek
2012-02-16 11:28                         ` Ohad Ben-Cohen
2012-02-16 13:07                           ` Michal Simek
2012-02-16 13:22                             ` Michal Simek
2012-02-16 17:34                               ` Ohad Ben-Cohen
     [not found]                                 ` <CAHTX3dJ4PruKO-wJLJU8W0boHFL0f6eoF+QNaoNjm5hu26YEow@mail.gmail.com>
2012-02-17  1:55                                   ` Ohad Ben-Cohen [this message]
2012-02-20 14:22                                     ` Michal Simek
2012-02-20 20:11                                       ` Ohad Ben-Cohen
2012-02-22 12:10                                         ` Michal Simek
2012-02-22 12:59                                           ` Ohad Ben-Cohen
2012-03-06 15:22                                             ` Michal Simek
2012-03-06 16:26                                               ` Ohad Ben-Cohen
2012-02-16 17:27                             ` Ohad Ben-Cohen

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='CAK=WgbZBJJbFcoeZ+hY=BgyW_4yvOJPVNMZ18_9o+zY7tEc_pQ@mail.gmail.com' \
    --to=ohad@wizery.com \
    --cc=arnd@arndb.de \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@secretlab.ca \
    --cc=john.williams@petalogix.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mgrosen@ti.com \
    --cc=monstr@monstr.eu \
    /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;
as well as URLs for NNTP newsgroup(s).