On Wednesday 04 November 2015 03:17:38 Scott Wood wrote: > > I guess this is fixed per architecture, so you could also do this as > > > > static inline void gpio_out32(u32 val, void __iomem *addr) > > { > > if (IS_ENABLED(CONFIG_ARM)) > > iowrite32(val, addr); > > else if (IS_ENABLED(CONFIG_PPC)) > > iowrite32be(val, addr); > > else > > BUG(); > > } > > Unfortunately that guess is wrong. Some of our ARM chips have big-endian I/O > and some have little-endian I/O. Ok, I see. In that case, using run-time detection everywhere is probably best. 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