On 27 December 2011 14:33, Andreas Färber <afaer...@suse.de> wrote:
> Further down, this also looks odd:
>
> if (s->conf & (PL080_CONF_M1 | PL080_CONF_M1)) {
> hw_error("pl080_write: Big-endian DMA not implemented\n");
> }
>
> Maybe (..._M1 | ..._M2)?Yes, obvious typo. I've noticed that one before but hadn't got round to submitting a patch to fix it, since it's only guarding a "this is broken" assert. -- PMM
