Re: [PATCH 2/2] arm: Support for platforms with split GIC cpu interface registers.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 22 November 2013 12:34, Anup Patel <anup@xxxxxxxxxxxxxx> wrote:
> On Fri, Nov 22, 2013 at 12:24 PM, Anup Patel <anup@xxxxxxxxxxxxxx> wrote:
>> On Fri, Nov 1, 2013 at 3:43 PM, Ian Campbell <ian.campbell@xxxxxxxxxx> wrote:
>>> At least one platform (APM Storm) places the two pages of the GIC cpu interface
>>> (and the vcpu side) at non-contiguous locations. Document two additional
>>> regions to cover this split and update the corresponding dtsi. Note that Linux
>>> (including KVM) does not use any registers in the second page so there is no
>>> associated code change here.  Xen will use these new regions, although I've not
>>> written the corresponding code yet.
>>>
>>> The ordering of these new regs is slightly counter intuitive but is inteded to
>>> be backward compatible. It is also assumed that all such systems will implement
>>> the GIC virtualisation extensions.
>>>
>>> Add comments to all of the reg examples to help clarify what is going on.
>>
>> Adding Catalin, Marc Z, and Will D.
>>
>> I looked up the GIC-400 TRM (more specifically page 33 of
>> DDI0471B_gic400_r0p1_trm.pdf) but could not find any info
>> on additional page for CPU interface.
>>
>> Am I looking at the wrong doc?
>
> Never mind, found the GICC_DIR (offset 0x1000) register.
>
> --
> Anup
>
>>
>> --
>> Anup
>>
>>>
>>> Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
>>> Cc: devicetree@xxxxxxxxxxxxxxx
>>> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
>>> Cc: Kumar Sankaran <ksankaran@xxxxxxx>
>>> Cc: Loc Ho <lho@xxxxxxx>
>>> Cc: Feng Kan <fkan@xxxxxxx>
>>> ---
>>>  Documentation/devicetree/bindings/arm/gic.txt | 57 +++++++++++++++++++++------
>>>  arch/arm64/boot/dts/apm-storm.dtsi            |  6 ++-
>>>  2 files changed, 50 insertions(+), 13 deletions(-)
>>>
>>> diff --git a/Documentation/devicetree/bindings/arm/gic.txt b/Documentation/devicetree/bindings/arm/gic.txt
>>> index ea215e8..00836d9 100644
>>> --- a/Documentation/devicetree/bindings/arm/gic.txt
>>> +++ b/Documentation/devicetree/bindings/arm/gic.txt
>>> @@ -38,7 +38,8 @@ Main node required properties:
>>>
>>>  - reg : Specifies base physical address(s) and size of the GIC registers. The
>>>    first region is the GIC distributor register base and size. The 2nd region is
>>> -  the GIC cpu interface register base and size.
>>> +  the GIC cpu interface register base and size (See also "Split GIC
>>> +  cpu interface", below).
>>>
>>>  Optional
>>>  - interrupts   : Interrupt source of the parent interrupt controller on
>>> @@ -56,8 +57,8 @@ Example:
>>>                 #interrupt-cells = <3>;
>>>                 #address-cells = <1>;
>>>                 interrupt-controller;
>>> -               reg = <0xfff11000 0x1000>,
>>> -                     <0xfff10100 0x100>;
>>> +               reg = <0xfff11000 0x1000>,      /* GIC Dist */
>>> +                     <0xfff10100 0x100>;       /* GIC CPU */
>>>         };
>>>
>>>
>>> @@ -70,9 +71,11 @@ primary interrupt controller).
>>>  Required properties:
>>>
>>>  - reg : Additional regions specifying the base physical address and
>>> -  size of the VGIC registers. The first additional region is the GIC
>>> -  virtual interface control register base and size. The 2nd additional
>>> -  region is the GIC virtual cpu interface register base and size.
>>> +  size of the VGIC registers. The first additional region (i.e. third
>>> +  overall) is the GIC virtual interface control register base and
>>> +  size. The 2nd additional region (i.e. forth overall) is the GIC
>>> +  virtual cpu interface register base and size (See also "Split GIC
>>> +  cpu interface", below).
>>>
>>>  - interrupts : VGIC maintenance interrupt.
>>>
>>> @@ -82,9 +85,41 @@ Example:
>>>                 compatible = "arm,cortex-a15-gic";
>>>                 #interrupt-cells = <3>;
>>>                 interrupt-controller;
>>> -               reg = <0x2c001000 0x1000>,
>>> -                     <0x2c002000 0x2000>,
>>> -                     <0x2c004000 0x2000>,
>>> -                     <0x2c006000 0x2000>;
>>> -               interrupts = <1 9 0xf04>;
>>> +               reg = <0x2c001000 0x1000>,      /* GIC Dist */
>>> +                     <0x2c002000 0x2000>,      /* GIC CPU */
>>> +                     <0x2c004000 0x2000>,      /* GIC VCPU Control */
>>> +                     <0x2c006000 0x2000>;      /* GIC VCPU1 */
>>> +               interrupts = <1 9 0xf04>;       /* GIC Maintenence IRQ */
>>> +       };
>>> +
>>> +* Split GIC cpu interface
>>> +
>>> +  The cpu interfaces (bare-metal in region 2 and virtual in region 4)
>>> +  may be spread over two pages, with the GICC_DIR (Deactivate
>>> +  Interrupt Register) register falling at the start of the second
>>> +  page. If these pages are contiguous then this is described via the
>>> +  size of the second and fourth entries as described above
>>> +  (e.g. 0x2000 rather than 0x1000).
>>> +
>>> +  However if the two pages are not contiguous then two additional
>>> +  regions are present (5th and 6th) describing the location of the the
>>> +  second half of the GIC cpu interface and GIC virtual cpu interface
>>> +  respectively.
>>> +
>>> +  It is assumed that all such systems will implement the GIC
>>> +  virtualisation extensions.
>>> +
>>> +Example:
>>> +
>>> +       gic: interrupt-controller@78010000 {
>>> +               compatible = "arm,cortex-a15-gic";
>>> +               #interrupt-cells = <3>;
>>> +               interrupt-controller;
>>> +               reg = <0x78010000 0x1000>,      /* GIC Dist */
>>> +                     <0x78020000 0x1000>,      /* GIC CPU1 */
>>> +                     <0x78040000 0x2000>,      /* GIC VCPU Control */
>>> +                     <0x78060000 0x1000>,      /* GIC VCPU1 */
>>> +                     <0x78030000 0x1000>,      /* GIC CPU2 */
>>> +                     <0x78070000 0x1000>;      /* GIC VCPU2 */
>>> +               interrupts = <1 9 0xf04>;       /* GIC Maintenence IRQ */
>>>         };
>>> diff --git a/arch/arm64/boot/dts/apm-storm.dtsi b/arch/arm64/boot/dts/apm-storm.dtsi
>>> index bfdc578..b5d141a 100644
>>> --- a/arch/arm64/boot/dts/apm-storm.dtsi
>>> +++ b/arch/arm64/boot/dts/apm-storm.dtsi
>>> @@ -82,9 +82,11 @@
>>>                 #interrupt-cells = <3>;
>>>                 interrupt-controller;
>>>                 reg = <0x0 0x78010000 0x0 0x1000>,      /* GIC Dist */
>>> -                     <0x0 0x78020000 0x0 0x1000>,      /* GIC CPU */
>>> +                     <0x0 0x78020000 0x0 0x1000>,      /* GIC CPU1 */
>>>                       <0x0 0x78040000 0x0 0x2000>,      /* GIC VCPU Control */
>>> -                     <0x0 0x78060000 0x0 0x2000>;      /* GIC VCPU */
>>> +                     <0x0 0x78060000 0x0 0x1000>,      /* GIC VCPU1 */
>>> +                     <0x0 0x78030000 0x0 0x1000>,      /* GIC CPU2 */
>>> +                     <0x0 0x78070000 0x0 0x1000>;      /* GIC VCPU2 */
>>>                 interrupts = <1 9 0xf04>;       /* GIC Maintenence IRQ */
>>>         };
>>>
>>> --
>>> 1.8.4.rc3
>>>

I have looked at the proposed device tree binding change
from APM X-Gene perspective.

This change looks good to me. (Vinayak ??)

Regards,
Anup

>>>
>>> _______________________________________________
>>> linux-arm-kernel mailing list
>>> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
>>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux