From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v1 3/3] xen/arm: smmu: Renaming struct iommu_domain *domain to, struct iommu_domain *iommu_domain Date: Fri, 27 Mar 2015 13:04:23 +0000 Message-ID: <55155557.7020103@linaro.org> References: <551505B3.9020505@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <551505B3.9020505@caviumnetworks.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Manish Jaggi , Xen Devel , "Prasun.kapoor@cavium.com" , "Kumar, Vijaya" , Ian Campbell , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org Hi manish, On 27/03/15 07:24, Manish Jaggi wrote: > It is good for code readability as there are many structures ending with > the name domain. > Also a code like this one is now easy to understand with the rename > old: dev_iommu_domain(dev) = domain; > new: dev_iommu_domain(dev) = iommu_domain; > > Also in current code struct smmu_domain pointer variable name is always > smmu_domain. > The change is on the same lines You are modifying the code from Linux just for your own comprehension. And we are trying to not diverge from a specific Linux commit in order to easily backport patch. NAcked-by: Julien Grall Regards, -- Julien Grall