From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v3 2/2] arm: export platform_op XENPF_settime64 Date: Wed, 11 Nov 2015 18:23:39 +0000 Message-ID: <564387AB.5070005@citrix.com> References: <1447262371-1555-2-git-send-email-stefano.stabellini@eu.citrix.com> <56437C57.4000900@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Stefano Stabellini Cc: dgdegra@tycho.nsa.gov, xen-devel@lists.xensource.com, Ian.Campbell@citrix.com List-Id: xen-devel@lists.xenproject.org On 11/11/15 18:14, Stefano Stabellini wrote: > On Wed, 11 Nov 2015, Julien Grall wrote: >>> diff --git a/xen/arch/arm/vtimer.c b/xen/arch/arm/vtimer.c >>> index 1418092..aa0fde6 100644 >>> --- a/xen/arch/arm/vtimer.c >>> +++ b/xen/arch/arm/vtimer.c >>> @@ -22,13 +22,13 @@ >>> #include >>> #include >>> #include >>> +#include >>> #include >>> #include >>> #include >>> #include >>> #include >>> >>> -extern s_time_t ticks_to_ns(uint64_t ticks); >> >> If it's a valid change you should mention it in the commit message >> because it's not related to this patch. >> >> Otherwise it looks spurious... > > Sorry, that's not a valid change anymore, I meant to remove it. Thanks > for checking. If it wasn't clear, I think this change is valid. Defining the prototype in the C file is a call to have the prototype not matching the declaration. Any prototypes should be declared in the header and not that way. I'm fine with this, but it should be mentioned in the commit message. Regards, -- Julien Grall