Re: [RFC 27/29] build: convert HAS_GICV3 use to Kconfig

[George Dunlap <george.dunlap@citrix.com>]

On 06/10/15 11:02, Julien Grall wrote:
> 
> 
> On 06/10/2015 10:56, George Dunlap wrote:
>> On Mon, Oct 5, 2015 at 11:25 PM, Julien Grall
>> <julien.grall@citrix.com> wrote:
>>> Hi,
>>>
>>> On 05/10/2015 23:03, Doug Goldstein wrote:
>>>>
>>>> Use the Kconfig generated CONFIG_HAS_GICV3 defines in the code base.
>>>
>>>
>>> If you are going to rename all HAS_* to CONFIG_HAS_, please drop the HAS
>>> which is now redundant.
>>>
>>>>
>>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
>>>> ---
>>>>    xen/arch/arm/Kconfig         | 4 ++++
>>>>    xen/arch/arm/Makefile        | 2 +-
>>>>    xen/arch/arm/Rules.mk        | 2 --
>>>>    xen/arch/arm/vgic.c          | 2 +-
>>>>    xen/include/asm-arm/domain.h | 3 ++-
>>>>    xen/include/asm-arm/gic.h    | 4 ++--
>>>>    xen/include/asm-arm/vgic.h   | 2 +-
>>>>    7 files changed, 11 insertions(+), 8 deletions(-)
>>>>
>>>> diff --git a/xen/arch/arm/Kconfig b/xen/arch/arm/Kconfig
>>>> index f100f17..01744c7 100644
>>>> --- a/xen/arch/arm/Kconfig
>>>> +++ b/xen/arch/arm/Kconfig
>>>> @@ -28,6 +28,10 @@ config ARCH_DEFCONFIG
>>>>          default "arch/arm/arm32_defconfig" if ARM_32
>>>>          default "arch/arm/arm64_defconfig" if ARM_64
>>>>
>>>> +# Select HAS_GICV3 if Generic Interrupt Connect (GICv3) is supported
>>>
>>>
>>> s/Connect/Controller/ although saying GICv3 is enough. No need to
>>> spell out
>>> the acronym.
>>>
>>> If you really want to spell it it should be Generic Interrupt
>>> Controller v3.
>>
>> Do remember that those comments are for users, not developers.  *You*
>> may know without thinking that "GIC" is an interrupt controller, but
>> many users would appreciate at least a hint so that they know whether
>> they can safely ignore it or go look further into it.
> 
> Even with that in mind, the term "GICvN" is commonly used in the ARM
> world. It's like the term APIC on x86, you don't spell out.

So you don't -- good comparison. :-)

 -George