> -void __iomem *arch_ioremap(phys_addr_t phys_addr, size_t size, unsigned long prot); > +void __iomem * > +arch_ioremap(phys_addr_t *paddr, size_t size, unsigned long *prot_val); It seems a bit odd to do this in two steps vs just doing the entire change in the first patch. Any good reason for that?