Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-21 Thread Alexandre Belloni
Hi Mark, On 20/02/2014 at 10:12:32 +0100, Nicolas Ferre wrote : > NAK. Either this is a atmel,at91rm9200-pinctrl node or a simple-bus. Not > both; that doesn't make any sense. > > >>> Simply a copy paste, I'll fix that here and also the 6 other atmel > >>> dtsi includes. > >>> >

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-21 Thread Alexandre Belloni
Hi Mark, On 20/02/2014 at 10:12:32 +0100, Nicolas Ferre wrote : NAK. Either this is a atmel,at91rm9200-pinctrl node or a simple-bus. Not both; that doesn't make any sense. Simply a copy paste, I'll fix that here and also the 6 other atmel dtsi includes. What is your preference for

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-20 Thread Nicolas Ferre
On 20/02/2014 09:59, Boris BREZILLON : > Hi Mark, > > On 19/02/2014 18:54, Mark Rutland wrote: >> On Wed, Feb 19, 2014 at 05:31:42PM +, Alexandre Belloni wrote: >>> On 19/02/2014 at 17:00:20 +, Mark Rutland wrote : On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote:

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-20 Thread Boris BREZILLON
Hi Mark, On 19/02/2014 18:54, Mark Rutland wrote: On Wed, Feb 19, 2014 at 05:31:42PM +, Alexandre Belloni wrote: On 19/02/2014 at 17:00:20 +, Mark Rutland wrote : On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl.

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-20 Thread Nicolas Ferre
On 19/02/2014 18:10, Boris BREZILLON : > On 19/02/2014 17:19, Jean-Jacques Hiblot wrote: >> Hi boris, >> >> I don't know if splitting the patch is needed. > >> For the 9261 I was >> asked to keep the patch number low > Okay, but then you mix DT modifications with source code modification. > >

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-20 Thread Maxime Ripard
Hi Mark, On Wed, Feb 19, 2014 at 05:00:20PM +, Mark Rutland wrote: > > + adc0: adc@fffd { > > + compatible = "atmel,at91sam9260-adc"; > > + reg = <0xfffd 0x100>; > > +

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-20 Thread Maxime Ripard
Hi Mark, On Wed, Feb 19, 2014 at 05:00:20PM +, Mark Rutland wrote: + adc0: adc@fffd { + compatible = atmel,at91sam9260-adc; + reg = 0xfffd 0x100; + interrupts = 20

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-20 Thread Nicolas Ferre
On 19/02/2014 18:10, Boris BREZILLON : On 19/02/2014 17:19, Jean-Jacques Hiblot wrote: Hi boris, I don't know if splitting the patch is needed. For the 9261 I was asked to keep the patch number low Okay, but then you mix DT modifications with source code modification. IMHO, we should

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-20 Thread Boris BREZILLON
Hi Mark, On 19/02/2014 18:54, Mark Rutland wrote: On Wed, Feb 19, 2014 at 05:31:42PM +, Alexandre Belloni wrote: On 19/02/2014 at 17:00:20 +, Mark Rutland wrote : On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl.

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-20 Thread Nicolas Ferre
On 20/02/2014 09:59, Boris BREZILLON : Hi Mark, On 19/02/2014 18:54, Mark Rutland wrote: On Wed, Feb 19, 2014 at 05:31:42PM +, Alexandre Belloni wrote: On 19/02/2014 at 17:00:20 +, Mark Rutland wrote : On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: This adds

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Mark Rutland
On Wed, Feb 19, 2014 at 05:31:42PM +, Alexandre Belloni wrote: > On 19/02/2014 at 17:00:20 +, Mark Rutland wrote : > > On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: > > > This adds preliminary DT support for the at91sam9rl. > > > > > > Signed-off-by: Alexandre Belloni

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Boris BREZILLON
Hi Mark, On 19/02/2014 18:00, Mark Rutland wrote: On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/at91sam9rl.dtsi | 628 ++

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Alexandre Belloni
On 19/02/2014 at 17:00:20 +, Mark Rutland wrote : > On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: > > This adds preliminary DT support for the at91sam9rl. > > > > Signed-off-by: Alexandre Belloni > > --- > > arch/arm/boot/dts/at91sam9rl.dtsi | 628 > >

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Boris BREZILLON
On 19/02/2014 17:19, Jean-Jacques Hiblot wrote: Hi boris, I don't know if splitting the patch is needed. For the 9261 I was asked to keep the patch number low Okay, but then you mix DT modifications with source code modification. IMHO, we should keep these modifications in different

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Mark Rutland
On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: > This adds preliminary DT support for the at91sam9rl. > > Signed-off-by: Alexandre Belloni > --- > arch/arm/boot/dts/at91sam9rl.dtsi | 628 > ++ > arch/arm/mach-at91/at91sam9rl.c | 16 + >

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Boris BREZILLON
Hi Alexandre, On 19/02/2014 16:32, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/at91sam9rl.dtsi | 628 ++ arch/arm/mach-at91/at91sam9rl.c | 16 + 2 files changed,

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Jean-Jacques Hiblot
Hi boris, I don't know if splitting the patch is needed. For the 9261 I was asked to keep the patch number low and the board won't boot up if the one of the 2 patches is missing. Jean-Jacques 2014-02-19 17:01 GMT+01:00 Boris BREZILLON : > Hi Alexandre, > > > On 19/02/2014 16:32, Alexandre

[PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Alexandre Belloni
This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/at91sam9rl.dtsi | 628 ++ arch/arm/mach-at91/at91sam9rl.c | 16 + 2 files changed, 644 insertions(+) create mode 100644

[PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Alexandre Belloni
This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni alexandre.bell...@free-electrons.com --- arch/arm/boot/dts/at91sam9rl.dtsi | 628 ++ arch/arm/mach-at91/at91sam9rl.c | 16 + 2 files changed, 644 insertions(+) create

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Jean-Jacques Hiblot
Hi boris, I don't know if splitting the patch is needed. For the 9261 I was asked to keep the patch number low and the board won't boot up if the one of the 2 patches is missing. Jean-Jacques 2014-02-19 17:01 GMT+01:00 Boris BREZILLON b.brezil...@overkiz.com: Hi Alexandre, On 19/02/2014

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Boris BREZILLON
Hi Alexandre, On 19/02/2014 16:32, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni alexandre.bell...@free-electrons.com --- arch/arm/boot/dts/at91sam9rl.dtsi | 628 ++

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Mark Rutland
On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni alexandre.bell...@free-electrons.com --- arch/arm/boot/dts/at91sam9rl.dtsi | 628 ++

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Boris BREZILLON
On 19/02/2014 17:19, Jean-Jacques Hiblot wrote: Hi boris, I don't know if splitting the patch is needed. For the 9261 I was asked to keep the patch number low Okay, but then you mix DT modifications with source code modification. IMHO, we should keep these modifications in different

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Alexandre Belloni
On 19/02/2014 at 17:00:20 +, Mark Rutland wrote : On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni alexandre.bell...@free-electrons.com --- arch/arm/boot/dts/at91sam9rl.dtsi |

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Boris BREZILLON
Hi Mark, On 19/02/2014 18:00, Mark Rutland wrote: On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni alexandre.bell...@free-electrons.com --- arch/arm/boot/dts/at91sam9rl.dtsi | 628

Re: [PATCHv2 1/8] ARM: at91: Add at91sam9rl DT SoC support

2014-02-19 Thread Mark Rutland
On Wed, Feb 19, 2014 at 05:31:42PM +, Alexandre Belloni wrote: On 19/02/2014 at 17:00:20 +, Mark Rutland wrote : On Wed, Feb 19, 2014 at 03:32:24PM +, Alexandre Belloni wrote: This adds preliminary DT support for the at91sam9rl. Signed-off-by: Alexandre Belloni