On Tue, 2019-12-24 at 09:35 +0800, Yingjie Bai wrote: > Hi Michael, > Thanks for pointing out the issue. My mistake... > This patch should indeed make sense only when > CONFIG_PHYS_64BIT=y > > I could not find corenet32_smp_defconfig, but I guess in your config, > CONFIG_PHYS_64BIT=n ? > I will update the patch later today
corenet32_smp_defconfig is a makefile rule that pulls in multiple config fragments. It has CONFIG_PHYS_64BIT=y, but __pa() returns an unsigned long regardless (which obviously needs to be fixed if DDR starting beyond 4G is to be supported). What 32-bit config are you using where this actually builds? -Scott