Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node

From	Carlo Caione <carlo@caione.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node
Date	2016-03-01 14:00 +0100
Message-ID	<r7S4b-Pg-25@gated-at.bofh.it> (permalink)
References	<r7FJD-1gw-7@gated-at.bofh.it> <r7FJE-1gw-25@gated-at.bofh.it> <r7QlI-8lK-1@gated-at.bofh.it> <r7Qvo-8px-11@gated-at.bofh.it> <r7RUt-LY-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tue, Mar 1, 2016 at 1:43 PM, Andre Przywara <andre.przywara@arm.com> wrote:
> Hi,
>
> On 01/03/16 11:18, Andreas Färber wrote:
>> Hi Andre,
>>
>> Am 01.03.2016 um 12:01 schrieb Andre Przywara:
>>> On 29/02/16 23:44, Andreas Färber wrote:
>>>> Add GICH and GICV resources for HYP mode - guess based on other vendors.
>>>
>>> Do you know if the firmware allows the kernel to be entered in EL2
>>> (which is the arm64 name for HYP)?
>>> So can we run kvm?
>>> If you have a booted kernel, can you grep for "EL2" and "kvm" in the dmesg?
>>
>> I do not have a rootfs yet (MMC v5 patches by Carlo are still waiting
>> for review), but with this change the KVM driver initializes okay - the
>> purpose of this patch!
>>
>>> Also you should merge this patch into 3/8, same for 8/8.
>>
>> If people confirm this is generally or specifically for this SoC correct
>> then sure. So far 3/8 seems a safe subset for lack of public documentation.
>
> The GIC is an integral part of the SoC, so this clearly belongs in there.
>
>>>> Signed-off-by: Andreas Färber <afaerber@suse.de>
>>>> ---
>>>>  arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 4 +++-
>
> In general I was wondering if this naming is correct?
> Shouldn't it be something with "s905" in it? Because this the SoC that
> is driving all those hardware and the peripherals that you describe in
> there are clearly within the SoC.
> So something like meson-s905.dtsi or the like?

When I first submitted support for the meson8 and meson8b I picked up
the names according to the Amlogic SDK.
In the latest Amlogic drop this SoC is identified as meson-gxbb so
probably we should stick to this name.

-- 
Carlo Caione

Thread

[PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Andreas Färber <afaerber@suse.de> - 2016-03-01 00:50 +0100
  Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Andre Przywara <andre.przywara@arm.com> - 2016-03-01 12:10 +0100
    Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Andreas Färber <afaerber@suse.de> - 2016-03-01 12:20 +0100
      Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Matthias Brugger <mbrugger@suse.com> - 2016-03-01 12:50 +0100
      Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Andre Przywara <andre.przywara@arm.com> - 2016-03-01 13:50 +0100
        Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Carlo Caione <carlo@caione.org> - 2016-03-01 14:00 +0100
          Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Andre Przywara <andre.przywara@arm.com> - 2016-03-01 15:20 +0100
        Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Andreas Färber <afaerber@suse.de> - 2016-03-01 23:50 +0100
          Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node André Przywara <andre.przywara@arm.com> - 2016-03-02 00:00 +0100
        Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Andreas Färber <afaerber@suse.de> - 2016-03-02 00:40 +0100
          Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node Andreas Färber <afaerber@suse.de> - 2016-03-02 01:10 +0100
          Re: [PATCH 7/8] ARM64: dts: amlogic: Extend GXBaby GIC node André Przywara <andre.przywara@arm.com> - 2016-03-02 01:10 +0100

csiph-web