Hi Catalin,
On 16/10/17 14:52, Catalin Marinas wrote:
> On Fri, Sep 22, 2017 at 07:26:11PM +0100, James Morse wrote:
>> diff --git a/include/linux/cpuhotplug.h b/include/linux/cpuhotplug.h
>> index f24bfb2b9a2d..466b949474df 100644
>> --- a/include/linux/cpuhotplug.h
>> +++ b/include/linux/cpuhotplug.h
>> @@ -88,6 +88,7 @@ enum cpuhp_state {
>> CPUHP_AP_PERF_XTENSA_STARTING,
>> CPUHP_AP_PERF_METAG_STARTING,
>> CPUHP_AP_MIPS_OP_LOONGSON3_STARTING,
>> + CPUHP_AP_SDEI_STARTING,
>
> Nitpick: how generic is this as to apply to other architectures?
> Probably not, so shall we prefix this with ARM_?
This may get used on 32bit ARM. I blindly copied PSCI, which doesn't have a
prefix. But it makes sense to have one.
> Not actually a strong preferences but similar question for the
> definitions in the sdei.h files (and maybe the filenames themselves).
Thanks,
James
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
