From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v3 51/62] arm/acpi: Prepare EFI system table for Dom0 Date: Wed, 18 Nov 2015 12:06:00 +0000 Message-ID: <564C69A8.9030506@citrix.com> References: <1447753261-7552-1-git-send-email-shannon.zhao@linaro.org> <1447753261-7552-52-git-send-email-shannon.zhao@linaro.org> <564B1757.5060008@citrix.com> <564BF1E2.5050003@huawei.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <564BF1E2.5050003@huawei.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: Shannon Zhao , shannon.zhao@linaro.org, xen-devel@lists.xen.org Cc: hangaohuai@huawei.com, stefano.stabellini@citrix.com, ian.campbell@citrix.com, peter.huangpeng@huawei.com List-Id: xen-devel@lists.xenproject.org On 18/11/15 03:34, Shannon Zhao wrote: > On 2015/11/17 20:02, Julien Grall wrote: >> I'm nearly sure I already said it, this code already exists in the tree. >> Why do you need to implement a new version? > Ok, it needs to move the functions to the public place, right? So they > can be called from other places. I don't think you need to move the current function code. AFAICT, it's already exported by common/xz/crc32.c Regards, -- Julien Grall