RE: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-06-13 Thread Philip, Avinash
On Thu, Jun 13, 2013 at 13:59:53, Nori, Sekhar wrote: > On 6/13/2013 1:02 PM, Philip, Avinash wrote: > > > With tnetv107x_defconfig build is failing > > > > arch/arm/mach-davinci/board-tnetv107x-evm.c:282:15: error: > > 'davinci_timer_init' undeclared here (not in a function) > > arch/arm/mach-d

Re: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-06-13 Thread Sekhar Nori
On 6/13/2013 1:02 PM, Philip, Avinash wrote: > With tnetv107x_defconfig build is failing > > arch/arm/mach-davinci/board-tnetv107x-evm.c:282:15: error: > 'davinci_timer_init' undeclared here (not in a function) > arch/arm/mach-davinci/board-tnetv107x-evm.c:284:15: error: > 'davinci_init_late' u

RE: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-06-13 Thread Philip, Avinash
On Thu, Jun 13, 2013 at 11:47:52, Nori, Sekhar wrote: > On 6/12/2013 5:40 PM, Philip, Avinash wrote: > > On Wed, Jun 12, 2013 at 13:13:59, Nori, Sekhar wrote: > >> On 6/11/2013 6:25 PM, Philip, Avinash wrote: > >> > >>> On Tue, Jun 11, 2013 at 17:26:06, Nori, Sekhar wrote: > >> > On 5/22/2013

Re: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-06-12 Thread Sekhar Nori
On 6/12/2013 5:40 PM, Philip, Avinash wrote: > On Wed, Jun 12, 2013 at 13:13:59, Nori, Sekhar wrote: >> On 6/11/2013 6:25 PM, Philip, Avinash wrote: >> >>> On Tue, Jun 11, 2013 at 17:26:06, Nori, Sekhar wrote: >> On 5/22/2013 12:40 PM, Philip Avinash wrote: >> > @@ -179,13 +204,10 @@ stati

RE: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-06-12 Thread Philip, Avinash
On Wed, Jun 12, 2013 at 13:13:59, Nori, Sekhar wrote: > On 6/11/2013 6:25 PM, Philip, Avinash wrote: > > > On Tue, Jun 11, 2013 at 17:26:06, Nori, Sekhar wrote: > > >> On 5/22/2013 12:40 PM, Philip Avinash wrote: > > >>> @@ -179,13 +204,10 @@ static int __init davinci_gpio_setup(void) > >>>

Re: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-06-12 Thread Sekhar Nori
On 6/11/2013 6:25 PM, Philip, Avinash wrote: > On Tue, Jun 11, 2013 at 17:26:06, Nori, Sekhar wrote: >> On 5/22/2013 12:40 PM, Philip Avinash wrote: >>> @@ -179,13 +204,10 @@ static int __init davinci_gpio_setup(void) >>> gpiochip_add(&ctlrs[i].chip); >>> } >>> >>> - soc_info

RE: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-06-11 Thread Philip, Avinash
, base += 32) { > > ctlrs[i].chip.label = "DaVinci"; > > @@ -179,13 +204,10 @@ static int __init davinci_gpio_setup(void) > > gpiochip_add(&ctlrs[i].chip); > > } > > > > - soc_info->gpio_ctlrs = ctlrs; >

Re: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-06-11 Thread Sekhar Nori
On 5/22/2013 12:40 PM, Philip Avinash wrote: > From: KV Sujith > > Modify GPIO Davinci driver to be compliant to standard platform drivers. > The driver did not have platform driver structure or a probe. Instead, > had a davinci_gpio_setup() function which is called in the pure_init > sequence.

Re: [PATCH 03/11] gpio: davinci: Modify to platform driver

2013-05-30 Thread Linus Walleij
On Wed, May 22, 2013 at 9:10 AM, Philip Avinash wrote: Why not squash patch 1 into this patch? Anyway, this bring us closer to the model of other drivers so: Acked-by: Linus Walleij Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a

[PATCH 03/11] gpio: davinci: Modify to platform driver

2013-05-22 Thread Philip Avinash
From: KV Sujith Modify GPIO Davinci driver to be compliant to standard platform drivers. The driver did not have platform driver structure or a probe. Instead, had a davinci_gpio_setup() function which is called in the pure_init sequence. The function also had dependency on davinci_soc_info struc