> +static int bcm2835_needs_bounce(struct device *dev, dma_addr_t dma_addr, > size_t size)
Too long line..
> +void __init bcm2835_init_early(void)
> +{
> + if(of_machine_is_compatible("brcm,bcm2711"))
Odd formatting.
Otherwise this looks good to me.

