[PATCH v2 4/5] mmc: mxcmmc: enable DMA support on mpc512x
Anatolij Gustschin
agust at denx.de
Tue Apr 2 18:29:44 EST 2013
On Tue, 2 Apr 2013 08:56:54 +0200
Sascha Hauer <s.hauer at pengutronix.de> wrote:
...
> > @@ -336,6 +359,8 @@ static int mxcmci_setup_data(struct mxcmci_host *host, struct mmc_data *data)
> > } else {
> > host->dma_dir = DMA_TO_DEVICE;
> > slave_dirn = DMA_MEM_TO_DEV;
> > +
> > + mxcmci_fixup_buffers(data);
>
> For the register accesses it's fine the way you did in this version, but
> here I think a
>
> if (is_mpc512x_mmc())
> swap_buffer()
>
> would look better.
the swapping function uses optimised st_le32() which is not available
on ARM, that's the reason why this is build time selected. Maybe the
"fixup" in the function name is not pretty, I can change it to "swap"
then.
Thanks,
Anatolij
More information about the devicetree-discuss
mailing list