From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753489AbdEEO57 (ORCPT ); Fri, 5 May 2017 10:57:59 -0400 Received: from foss.arm.com ([217.140.101.70]:52504 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752677AbdEEO55 (ORCPT ); Fri, 5 May 2017 10:57:57 -0400 Date: Fri, 5 May 2017 15:58:00 +0100 From: Will Deacon To: David Daney Cc: Hanjun Guo , Geetha sowjanya , robin.murphy@arm.com, lorenzo.pieralisi@arm.com, sudeep.holla@arm.com, iommu@lists.linux-foundation.org, Geetha Sowjanya , jcm@redhat.com, linu.cherian@cavium.com, linux-kernel@vger.kernel.org, geethasowjanya.akula@gmail.com, linux-acpi@vger.kernel.org, robert.richter@cavium.com, catalin.marinas@arm.com, sgoutham@cavium.com, linux-arm-kernel@lists.infradead.org, Charles.Garcia-Tobin@arm.com Subject: Re: [PATCH v3 3/7] ACPICA: IORT: Add Cavium ThunderX2 SMMUv3 model definition. Message-ID: <20170505145800.GG14111@arm.com> References: <1493986091-30521-1-git-send-email-gakula@caviumnetworks.com> <1493986091-30521-4-git-send-email-gakula@caviumnetworks.com> <330d0438-f9ed-ef65-8ffd-d556d149b4e1@linaro.org> <6b04ab17-449f-74e0-72f4-b5a9bc0639d8@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6b04ab17-449f-74e0-72f4-b5a9bc0639d8@gmail.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 05, 2017 at 07:56:17AM -0700, David Daney wrote: > On 05/05/2017 06:53 AM, Hanjun Guo wrote: > >On 2017/5/5 20:08, Geetha sowjanya wrote: > >>From: Linu Cherian > >> > >>Add SMMUv3 model definition for ThunderX2. > >> > >>Signed-off-by: Linu Cherian > >>Signed-off-by: Geetha Sowjanya > >>--- > >> include/acpi/actbl2.h | 2 ++ > >> 1 file changed, 2 insertions(+) > >> > >>diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h > >>index faa9f2c..76a6f5d 100644 > >>--- a/include/acpi/actbl2.h > >>+++ b/include/acpi/actbl2.h > >>@@ -779,6 +779,8 @@ struct acpi_iort_smmu { > >> #define ACPI_IORT_SMMU_CORELINK_MMU400 0x00000002 /* ARM Corelink > >>MMU-400 */ > >> #define ACPI_IORT_SMMU_CORELINK_MMU500 0x00000003 /* ARM Corelink > >>MMU-500 */ > >> > >>+#define ACPI_IORT_SMMU_V3_CAVIUM_CN99XX 0x00000002 /* Cavium ThunderX2 > >>SMMUv3 */ > > > >There are some other model numbers in the unreleased spec, > >I think we need to wait for the updated IORT spec to > >be released. > > > > ... or if we are fairly confident that the identifier will not need to > change, we can merge this as is and establish a de facto specification that > the Real IORT specification will then be forced to follow. > > Is there anything other than bureaucratic inertia holding up the real > specification? My understanding is that IORT is going to be published imminently (i.e. before the next kernel release), so it makes sense to wait rather than fork the spec. Will