Subject: Re: [PATCH v6 08/10] KVM: arm/arm64: arch_timer: Rely on the arch
 timer to parse the firmware tables
To: Julien Grall <julien.grall@arm.com>, kvmarm@lists.cs.columbia.edu
References: <1460388780-914-1-git-send-email-julien.grall@arm.com>
 <1460388780-914-9-git-send-email-julien.grall@arm.com>
Cc: wei@redhat.com, christoffer.dall@linaro.org, al.stone@linaro.org,
        kvm@vger.kernel.org, hanjun.guo@linaro.org, marc.zyngier@arm.com,
        linux-kernel@vger.kernel.org, Gleb Natapov <gleb@kernel.org>,
        fu.wei@linaro.org, Paolo Bonzini <pbonzini@redhat.com>,
        linux-arm-kernel@lists.infradead.org, gg@slimlogic.co.uk
From: Shanker Donthineni <shankerd@codeaurora.org>
Message-ID: <571C3844.7090103@codeaurora.org>
Date: Sat, 23 Apr 2016 22:06:44 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.6.0
MIME-Version: 1.0
In-Reply-To: <1460388780-914-9-git-send-email-julien.grall@arm.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 812
Lines: 22


On 04/11/2016 10:32 AM, Julien Grall wrote:
> The firmware table is currently parsed by the virtual timer code in
> order to retrieve the virtual timer interrupt. However, this is already
> done by the arch timer driver.
>
> To avoid code duplication, use the newly function
> arch_timer_get_kvm_info()
> which return all the information required by the virtual timer code.
>
> Signed-off-by: Julien Grall <julien.grall@arm.com>
> Acked-by: Christoffer Dall <christoffer.dall@linaro.org>
>
Tested-by: Shanker Donthineni <shankerd@codeaurora.org>

Using the Qualcomm Technologies QDF2XXX server platform.

-- 
Shanker Donthineni
Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project