public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: marc@plastictigers.com (Marc Butler)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH] slimbus message encoding/decoding
Date: Fri, 9 Mar 2012 17:39:52 -0500	[thread overview]
Message-ID: <20120309223952.GA5669@plastictigers.com> (raw)
In-Reply-To: <4F5A5F00.70508@codeaurora.org>

On Fri, Mar 09, 2012 at 12:50:24PM -0700, Sagar Dharia wrote:
> On 03/08/2012 05:21 PM, Marc Butler wrote:
> 
> Thanks for providing this patch. My understanding is that this patch
> provides helper functions for h/w controllers which rely on s/w to
> provide the exact message to be sent out on the bus.

Yes. The ti omap4430 expects the messages, to be complete except for
the PI, MR, and MI fields which are encoded by the h/w.

> Just like CRC, some other fields may be populated by HW. e.g.
> Qualcomm chip also populates Arbitration priority, Arbitration
> Extension fields. So it expects SW to only program a few fields,
> a.k.a logical/elemental address, MT,MC, DT etc.(that too, SW
> programs this based on software interface manual). The hardware will
> convert this information to actual slimbus-compliant message and
> send it on the bus.

Understood. The functions are not incompatible with construction
partial messages, but as it will not likely work when inserting the
header field, as it goes about trying to compute the remaining length,
based on the current (icomplete) message including the arbitration
header.

> Being said that, I am not aware of how majority of h/w controllers
> work and will be open to have this patch if other hardwares rely on
> s/w to provide the exact message as well.

The ti omap4430 requires this support.

Regards,
-m

      reply	other threads:[~2012-03-09 22:39 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-09  0:21 [RFC PATCH] slimbus message encoding/decoding Marc Butler
2012-03-09 19:50 ` Sagar Dharia
2012-03-09 22:39   ` Marc Butler [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=20120309223952.GA5669@plastictigers.com \
    --to=marc@plastictigers.com \
    --cc=linux-arm-kernel@lists.infradead.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