From mboxrd@z Thu Jan  1 00:00:00 1970
From: Julien Grall <julien.grall@linaro.org>
Subject: Re: [PATCH v2 07/16] xen/arm: IRQ: Move IRQ management
 from gic.c to irq.c
Date: Mon, 07 Apr 2014 14:34:58 +0100
Message-ID: <5342A982.40406@linaro.org>
References: <1396557727-19102-1-git-send-email-julien.grall@linaro.org>
	<1396557727-19102-8-git-send-email-julien.grall@linaro.org>
	<1396876077.22845.76.camel@kazak.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta3.messagelabs.com ([195.245.230.39])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <julien.grall@linaro.org>) id 1WX9hV-0003tt-Nv
	for xen-devel@lists.xenproject.org; Mon, 07 Apr 2014 13:35:01 +0000
Received: by mail-ee0-f48.google.com with SMTP id b57so643172eek.21
	for <xen-devel@lists.xenproject.org>;
	Mon, 07 Apr 2014 06:34:59 -0700 (PDT)
In-Reply-To: <1396876077.22845.76.camel@kazak.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: xen-devel@lists.xenproject.org, tim@xen.org, stefano.stabellini@citrix.com
List-Id: xen-devel@lists.xenproject.org

On 04/07/2014 02:07 PM, Ian Campbell wrote:
> On Thu, 2014-04-03 at 21:41 +0100, Julien Grall wrote:
>> The file gic.c contains functions and variables which is not related to the GIC:
>>     - release_irq
>>     - setup_irq
>>     - gic_route_irq_to_guest
>>     - {,local_}irq_desc
>>
>> Move all theses functions/variables in irq.c and split gic_route_irq_to_guest
>> in 2 parts:
> 
> Please can you separate pure code motion from refactoring.
> 
> e.g. You could do the split first leaving route_dt_irq_to_guest in gic.c
> and then move it along with everything else.

Sure, as the refactoring was simple I didn't create a new patch.

>>     - route_dt_irq_to_guest: setup the desc
>>     - gic_route_irq_to_guest: setup correctly the GIC and the desc
>>     handler
>>  
>> +void release_irq(unsigned int irq)
>> +{
>> +    struct irq_desc *desc;
>> +    unsigned long flags;
>> +   struct irqaction *action;
> 
> Indentation.

I've blindly copied for the previous location. I think, I fix the
indention on a previous patch. I will fix it here.

Regards

-- 
Julien Grall