Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add firmware driver #1037

merged 11 commits into from Jul 13, 2015

mailbox/bcm2835: Fix mailbox full detection.

With the VC reader blocked and the ARM writing, MAIL0_STA reads empty
permanently while MAIL1_STA goes from empty (0x40000000) to non-empty
(0x00000001-0x00000007) to full (0x80000008).

This bug ended up having no effect on us, because all of our
transactions in the client driver were synchronous and under a mutex.

Suggested-by: Phil Elwell <>
Signed-off-by: Eric Anholt <>
Acked-by: Stephen Warren <>
Signed-off-by: Jassi Brar <>
  • Loading branch information...
anholt authored and notro committed May 13, 2015
commit 4e811cd34d8c9b38d329e9bdbb1851efe0f9c201
@@ -49,6 +49,7 @@
#define MAIL0_STA (ARM_0_MAIL0 + 0x18)
#define MAIL0_CNF (ARM_0_MAIL0 + 0x1C)
#define MAIL1_WRT (ARM_0_MAIL1 + 0x00)
#define MAIL1_STA (ARM_0_MAIL1 + 0x18)

/* Status register: FIFO state. */
#define ARM_MS_FULL BIT(31)
@@ -117,7 +118,7 @@ static bool bcm2835_last_tx_done(struct mbox_chan *link)
bool ret;

ret = !(readl(mbox->regs + MAIL0_STA) & ARM_MS_FULL);
ret = !(readl(mbox->regs + MAIL1_STA) & ARM_MS_FULL);
return ret;
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.