Hello, On 24 March 2014 09:26, Rob Herring <rob.herring@xxxxxxxxxx> wrote: >>> >>> My suggestion would be to have: >>> arch/arm/include/uapi/asm/psci.h >>> arch/arm64/include/uapi/asm/psci.h >> >> I think the suggestion was to move all this to include/uapi/linux/psci.h >> or something equivalent so also arm and arm64 can share these defines, >> but maybe I read too much into it. > > Yes. It is perfectly fine for a somewhat architecture specific header > to live in include/linux (or include/uapi/linux). > > While I'm aware that we wouldn't want to push seemingly irrelevant stuff into this header, I'm thinking of ways to unify the ARM and ARM64 PSCI code. A lot of it is reusable between the two. So, can this header also include all the common PSCI functions between ARM and ARM64? And can KVM possibly use these functions as well? Alternately, is drivers/psci.c a more suitable place? In this case, at least one function will be guarded by CONFIG_ARM64. Cheers, Ashwin _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
