[PATCH 3/4 v4] mailbox: Enable BCM2835 mailbox support
eric at anholt.net
Fri Mar 20 10:24:20 PDT 2015
Stephen Warren <swarren at wwwdotorg.org> writes:
> On 03/18/2015 05:28 PM, Eric Anholt wrote:
>> Stephen Warren <swarren at wwwdotorg.org> writes:
>>> On 03/12/2015 08:32 PM, Eric Anholt wrote:
>>>> diff --git a/drivers/mailbox/bcm2835-mailbox.c
>>>> +#define MBOX_MSG(chan, data28) (((data28) & ~0xf) | ((chan) &
>>>> 0xf)) +#define MBOX_CHAN(msg) ((msg) & 0xf) +#define
>>>> MBOX_DATA28(msg) ((msg) & ~0xf)
>>> Even the concept of storing channel IDs in the LSBs feels like it
>>> might be RPi-firmware-specific rather than HW-specific?
>> I guess? If we found another firmware protocol, we could have
>> that device's dt just specify a different compatible string. But
>> in the absence of another firmware to talk to, I'm not sure what
>> you want here.
> I would expect the mailbox driver to expose a single channel that just
> transports 32-bit values, since the HW doesn't impose any kind of
> structure on the values it transports AFAIK. Clients of the mailbox
> driver would formulate the messages they send through the mailox using
> the macros above.
> I'm not sure whether the mailbox core allows multiple clients for the
> same mailbox channel though? This HW appears to require it.
Yeah, that's the problem.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 818 bytes
Desc: not available
More information about the linux-rpi-kernel