On Thu, Jul 07, 2022 at 01:37:42PM -0700, Florian Fainelli wrote: > The implementation is exactly the same, so avoid open-coding it in two > different locations. > > Signed-off-by: Florian Fainelli <f.fainelli@xxxxxxxxx> Seems reasonable. Thanks. Reviewed-by: Serge Semin <fancer.lancer@xxxxxxxxx> *Note it now looks exactly as the generic phys_to_virt(). -Sergey > --- > arch/mips/include/asm/io.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/mips/include/asm/io.h b/arch/mips/include/asm/io.h > index 6f5c86d2bab4..880048a54362 100644 > --- a/arch/mips/include/asm/io.h > +++ b/arch/mips/include/asm/io.h > @@ -131,7 +131,7 @@ static inline phys_addr_t virt_to_phys(const volatile void *x) > */ > static inline void * phys_to_virt(unsigned long address) > { > - return (void *)(address + PAGE_OFFSET - PHYS_OFFSET); > + return __va(address); > } > > /* > -- > 2.25.1 >