From mboxrd@z Thu Jan 1 00:00:00 1970 From: Manish Jaggi Subject: Re: [PATCH v1 1/3] xen/arm: smmu: Rename arm_smmu_xen_device with, device_iommu_info Date: Mon, 6 Apr 2015 16:15:21 +0530 Message-ID: <552263C1.7010204@caviumnetworks.com> References: <551504D8.1030102@caviumnetworks.com>, <5515543F.5090001@linaro.org> <1427462515927.43521@caviumnetworks.com>, <55155CA0.5010304@linaro.org> <1427479254399.31116@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1427479254399.31116@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: "Jaggi, Manish" , Julien Grall , Xen Devel , Stefano Stabellini , Ian Campbell , "Prasun.kapoor@cavium.com" , "Kumar, Vijaya" Ian Campbell List-Id: xen-devel@lists.xenproject.org On Friday 27 March 2015 11:30 PM, Jaggi, Manish wrote: > ________________________________ > From: Julien Grall > Sent: Friday, March 27, 2015 7:05 PM > To: Jaggi, Manish; Xen Devel; Stefano Stabellini; Ian Campbell; Prasun.kapoor@cavium.com; Kumar, Vijaya > Subject: Re: [PATCH v1 1/3] xen/arm: smmu: Rename arm_smmu_xen_device with, device_iommu_info > > On 27/03/15 13:21, Jaggi, Manish wrote: >> >> Regards, >> Manish Jaggi > Could you please try to configure you email client correctly? It's > rather confusing the "regards, Manish Jaggi" at the beginning of the mail. > > [manish] Fixed. Thanks for pointing out > >> ________________________________________ >> From: Julien Grall >> Sent: Friday, March 27, 2015 6:29 PM >> To: Jaggi, Manish; Xen Devel; Stefano Stabellini; Ian Campbell; Prasun.kapoor@cavium.com; Kumar, Vijaya >> Subject: Re: [PATCH v1 1/3] xen/arm: smmu: Rename arm_smmu_xen_device with, device_iommu_info >> >> Hi Manish, >> >> On 27/03/15 07:20, Manish Jaggi wrote: >>> arm_smmu_xen_device is not an intuitive name for a datastructure which >>> represents >>> device->archdata.iommu. Rename arm_smmu_xen_device with device_iommu_info >> device_iommu_info is not more intuitive... At least arm_smmu_xen_device >> shows that it's a specific Xen structure and not coming from the Linux >> drivers. >> >> [manish] But that is not a valid reason for a non intuitive naming. It is really hard to keep us readability of the code with arm_smmu_xen_device. It is not clear that it is referring to a device attached to smmu or smmu itself. There is another data structure arm_smmu_device as well. > Did you read the comment explaining the structure arm_smmu_xen_device? > It's just above the definition. > > "arm_smmu" is the prefix for any structure within this file. > "xen" means it's a structure added for Xen. > "device" means it's data stored for a device. > >> Please choose another name I can take it but arm_smmu_xen_device is really confusing > I won't choose a name myself for a name that I think valid... > > If you really want to change the name, you have to put at least > arm_smmu_xen_ in the name. > > [manish] what about device_archdata_priv, this is denoting what it is. > >> Regards, As per Ians mail in other thread, %s/arm_smmu_xen_device/arch_smm_xen_device/g is ok with you ? > Regards, > > -- > Julien Grall