On Tuesday 21 April 2015 13:37:13 Eric Anholt wrote: > Arnd Bergmann <arnd@xxxxxxxx> writes: > > > On Tuesday 21 April 2015 11:09:52 Eric Anholt wrote: > >> +static struct map_desc bcm2836_io_map __initdata = { > >> + .virtual = BCM2835_PERIPH_VIRT, > >> + .pfn = __phys_to_pfn(BCM2836_PERIPH_PHYS), > >> + .length = BCM2835_PERIPH_SIZE, > >> + .type = MT_DEVICE > >> +}; > >> + > >> static void __init bcm2835_map_io(void) > >> { > >> - iotable_init(&io_map, 1); > >> + iotable_init(&bcm2835_io_map, 1); > >> +} > >> + > >> +static void __init bcm2836_map_io(void) > >> +{ > >> + iotable_init(&bcm2836_io_map, 1); > >> } > >> > > > > Can you explain what this is needed for? Most platform ports don't > > do this any more. > > Nope, I can't! I'm not sure what the bcm2835 side of it does, and I was > just replicating that for 2836. > > Should it be removed from 2835, too? > Hard to know. Does anything reference BCM2835_PERIPH_VIRT? Does it work if you remove it? Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html