Re: [PATCH 05/10] mailbox: apple: Add DockChannel FIFO controller

From: Michael Reeves

Date: Tue Jul 07 2026 - 09:47:41 EST


On Sun, Jul 5, 2026 at 6:14 AM Yureka Lilian <yureka@xxxxxxxxxxxxxx> wrote:
[...]
> > + while (block) {
> > + writeb_relaxed(*p++, dc->data_base + DATA_TX8);
>
> This must be writel_relaxed, as writeb_relaxed in this mmio region will
> cause an SError. This is why we have separate addresses putting 8-bits
> and 32-bits into the FIFO I guess.
Ah, thanks for the pickup. Sorry for the silly typo from me, must have missed
it because all the HID writes are 4-byte aligned. Will correct in v2
which I should
be able to send out by the weekend unless I get busy.

Thanks again,
Michael