From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andreas Herrmann Subject: Re: [PATCH v2 08/11] of: Increase MAX_PHANDLE_ARGS Date: Wed, 29 Jan 2014 18:16:11 +0100 Message-ID: <20140129171611.GB13543@alberich> References: <1389876263-25759-1-git-send-email-andreas.herrmann@calxeda.com> <1389876263-25759-9-git-send-email-andreas.herrmann@calxeda.com> <20140117110830.GW3471@alberich> <52E92842.3000001@amd.com> <52E93360.1000904@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <52E93360.1000904-5C7GfCeVMHo@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Suravee Suthikulanit Cc: "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Will Deacon , "iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org" , Rob Herring , Rob Herring , Grant Likely , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: devicetree@vger.kernel.org On Wed, Jan 29, 2014 at 11:59:12AM -0500, Suravee Suthikulanit wrote: > On 1/29/2014 10:57 AM, Rob Herring wrote: > >>> diff --git a/include/linux/of.h b/include/linux/of.h > >>> >>index 276c546..24e1b28 100644 > >>> >>--- a/include/linux/of.h > >>> >>+++ b/include/linux/of.h > >>> >>@@ -67,7 +67,7 @@ struct device_node { > >>> >> #endif > >>> >> }; > >>> >> > >>> >>-#define MAX_PHANDLE_ARGS 8 > >>> >>+#define MAX_PHANDLE_ARGS 16 > >> > > >> > > >> >Since the MMU-500 specify "Number of SMRs" upto 128 registers, shouldn't > >> >this be changed to be able to support 128 StreamIDs as well? Although I am > >> >not sure if this would be too big to have on the stack per Rob's comment in > >> >the previous patch set. > > Do you actually need 128 now? If not, then we can deal with that when > > we get there. There are lots of things in spec's that are not actually > > implemented or supported. > > Actually, we are using 32 on the AMD system. So, do you think we can set > this to 32 instead? I think that's ok. But are we really talking about number of SMRs or number of StreamIDs per master device here? Ie. are you just having 32 SMRs for an SMMU on your AMD system or do you have master devices which have 32 StreamIDs? If it's just number of SMRs we don't need to modify this macro. Andreas From mboxrd@z Thu Jan 1 00:00:00 1970 From: andreas.herrmann@calxeda.com (Andreas Herrmann) Date: Wed, 29 Jan 2014 18:16:11 +0100 Subject: [PATCH v2 08/11] of: Increase MAX_PHANDLE_ARGS In-Reply-To: <52E93360.1000904@amd.com> References: <1389876263-25759-1-git-send-email-andreas.herrmann@calxeda.com> <1389876263-25759-9-git-send-email-andreas.herrmann@calxeda.com> <20140117110830.GW3471@alberich> <52E92842.3000001@amd.com> <52E93360.1000904@amd.com> Message-ID: <20140129171611.GB13543@alberich> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Jan 29, 2014 at 11:59:12AM -0500, Suravee Suthikulanit wrote: > On 1/29/2014 10:57 AM, Rob Herring wrote: > >>> diff --git a/include/linux/of.h b/include/linux/of.h > >>> >>index 276c546..24e1b28 100644 > >>> >>--- a/include/linux/of.h > >>> >>+++ b/include/linux/of.h > >>> >>@@ -67,7 +67,7 @@ struct device_node { > >>> >> #endif > >>> >> }; > >>> >> > >>> >>-#define MAX_PHANDLE_ARGS 8 > >>> >>+#define MAX_PHANDLE_ARGS 16 > >> > > >> > > >> >Since the MMU-500 specify "Number of SMRs" upto 128 registers, shouldn't > >> >this be changed to be able to support 128 StreamIDs as well? Although I am > >> >not sure if this would be too big to have on the stack per Rob's comment in > >> >the previous patch set. > > Do you actually need 128 now? If not, then we can deal with that when > > we get there. There are lots of things in spec's that are not actually > > implemented or supported. > > Actually, we are using 32 on the AMD system. So, do you think we can set > this to 32 instead? I think that's ok. But are we really talking about number of SMRs or number of StreamIDs per master device here? Ie. are you just having 32 SMRs for an SMMU on your AMD system or do you have master devices which have 32 StreamIDs? If it's just number of SMRs we don't need to modify this macro. Andreas