[PATCH v8 02/12] ARM/ARM64: KVM: Add common header for PSCI related defines
anup at brainfault.org
Mon Apr 14 20:43:59 PDT 2014
On Mon, Apr 14, 2014 at 11:58 PM, Ashwin Chaugule
<ashwin.chaugule at linaro.org> wrote:
> Hi Anup,
> On 3 April 2014 05:05, Anup Patel <anup.patel at linaro.org> wrote:
>> +/* PSCI version decoding (independent of PSCI version) */
>> +#define PSCI_VERSION_MAJOR_MASK 0xffff0000
>> +#define PSCI_VERSION_MINOR_MASK 0x0000ffff
>> +#define PSCI_VERSION_MAJOR_SHIFT 16
>> +#define PSCI_VERSION_MAJOR(ver) \
>> + (((ver) & PSCI_VER_MAJOR_MASK) >> PSCI_VER_MAJOR_SHIFT)
>> +#define PSCI_VERSION_MINOR(ver) ((ver) & PSCI_VER_MINOR_MASK)
> I just noticed this as I was preparing to rebase my patches on top of
> yours. Looks like you modified my original version of these macros,
> but forgot to change them completely. You still have the old "_VER_"
> in there which are now undefined.
> Can you please resend with this fix? If these are already picked up
> for upstreaming, then I can send a follow up fix along with my v0.2
> patchset which is (hopefully) the first user of these macros.
Thanks for spotting the issue. I will send fixed patches soon.
> kvmarm mailing list
> kvmarm at lists.cs.columbia.edu
More information about the linux-arm-kernel