All of lore.kernel.org
 help / color / mirror / Atom feed
From: ludovic.desroches@atmel.com (ludovic.desroches)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/4] ARM: at91: at91 based machines specify their own irq handler at run time
Date: Thu, 7 Jun 2012 09:25:03 +0200	[thread overview]
Message-ID: <4FD0574F.6000200@atmel.com> (raw)
In-Reply-To: <20120606094748.GA26504@taskit.de>

Hi Christian,

On 06/06/2012 11:47 AM, Christian Glindkamp wrote:
> On 2012-06-05 16:47, ludovic.desroches at atmel.com wrote:
>> From: Ludovic Desroches<ludovic.desroches@atmel.com>
>>
>> SOC_AT91SAM9 selects MULTI_IRQ_HANDLER in order to let machines specify their
>> own IRQ handler at run time.
>>
>> Signed-off-by: Ludovic Desroches<ludovic.desroches@atmel.com>
>> ---
>>   arch/arm/mach-at91/board-cpu9krea.c           |    1 +
>>   arch/arm/mach-at91/board-sam9g20ek.c          |    1 +
>>   arch/arm/mach-at91/board-sam9261ek.c          |    1 +
>>   arch/arm/mach-at91/board-stamp9g20.c          |    1 +
>>   arch/arm/mach-at91/board-usb-a926x.c          |    1 +
>
> At least these board files contain more than one MACHINE_START, but
> you only update the last one. Same for the third patch.

Thanks for your feedback, I'll correct it in V2.

>
> I have not looked at the rest of the patch.
>
>

Regards

Ludovic

WARNING: multiple messages have this Message-ID (diff)
From: ludovic.desroches <ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
To: Christian Glindkamp
	<christian.glindkamp-7uXufW8GZECzQB+pC5nmwQ@public.gmane.org>
Cc: will.deacon-5wv7dgnIgG8@public.gmane.org,
	devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH 1/4] ARM: at91: at91 based machines specify their own irq handler at run time
Date: Thu, 7 Jun 2012 09:25:03 +0200	[thread overview]
Message-ID: <4FD0574F.6000200@atmel.com> (raw)
In-Reply-To: <20120606094748.GA26504-7uXufW8GZECzQB+pC5nmwQ@public.gmane.org>

Hi Christian,

On 06/06/2012 11:47 AM, Christian Glindkamp wrote:
> On 2012-06-05 16:47, ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org wrote:
>> From: Ludovic Desroches<ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
>>
>> SOC_AT91SAM9 selects MULTI_IRQ_HANDLER in order to let machines specify their
>> own IRQ handler at run time.
>>
>> Signed-off-by: Ludovic Desroches<ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
>> ---
>>   arch/arm/mach-at91/board-cpu9krea.c           |    1 +
>>   arch/arm/mach-at91/board-sam9g20ek.c          |    1 +
>>   arch/arm/mach-at91/board-sam9261ek.c          |    1 +
>>   arch/arm/mach-at91/board-stamp9g20.c          |    1 +
>>   arch/arm/mach-at91/board-usb-a926x.c          |    1 +
>
> At least these board files contain more than one MACHINE_START, but
> you only update the last one. Same for the third patch.

Thanks for your feedback, I'll correct it in V2.

>
> I have not looked at the rest of the patch.
>
>

Regards

Ludovic

  reply	other threads:[~2012-06-07  7:25 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-05 14:47 [PATCH 0/4] ARM: at91: introduce AIC5 support ludovic.desroches at atmel.com
2012-06-05 14:47 ` ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w
2012-06-05 14:47 ` [PATCH 1/4] ARM: at91: at91 based machines specify their own irq handler at run time ludovic.desroches at atmel.com
2012-06-05 14:47   ` ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w
2012-06-06  9:47   ` Christian Glindkamp
2012-06-06  9:47     ` Christian Glindkamp
2012-06-07  7:25     ` ludovic.desroches [this message]
2012-06-07  7:25       ` ludovic.desroches
2012-06-05 14:47 ` [PATCH 2/4] ARM: at91: fix irq_alloc_descs parameters for sparse irq ludovic.desroches at atmel.com
2012-06-05 14:47   ` ludovic.desroches
2012-06-05 14:47 ` [PATCH 3/4] ARM: at91: enable use of " ludovic.desroches at atmel.com
2012-06-05 14:47   ` ludovic.desroches
2012-06-05 14:47 ` [PATCH 4/4] ARM: at91: add AIC5 support ludovic.desroches at atmel.com
2012-06-05 14:47   ` ludovic.desroches

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FD0574F.6000200@atmel.com \
    --to=ludovic.desroches@atmel.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.