Re: [PATCH V2 3/3] ARM: davinci: da850: add NAND driver entries

2013-01-17 Thread Linus Walleij
On Wed, Jan 9, 2013 at 1:47 PM, Sekhar Nori wrote: > On 1/8/2013 1:50 PM, Kumar, Anil wrote: >> +&pmx_core{ >> + pinctrl-names = "default"; >> + pinctrl-0 = < >> + &nand_cs3_pins >> + >; > > This means that the NAND pins are configured even if NAND is not > probed. Right?

Re: [PATCH V2 3/3] ARM: davinci: da850: add NAND driver entries

2013-01-15 Thread Sekhar Nori
On 1/15/2013 4:06 PM, Kumar, Anil wrote: > On Thu, Jan 10, 2013 at 17:49:13, Nori, Sekhar wrote: >> On 1/10/2013 1:07 PM, Kumar, Anil wrote: >>> On Wed, Jan 09, 2013 at 18:17:46, Nori, Sekhar wrote: >> >>> I do not think that it is good idea to move NAND pin mux information >>> into da850.dtsi beca

RE: [PATCH V2 3/3] ARM: davinci: da850: add NAND driver entries

2013-01-15 Thread Kumar, Anil
On Thu, Jan 10, 2013 at 17:49:13, Nori, Sekhar wrote: > On 1/10/2013 1:07 PM, Kumar, Anil wrote: > > On Wed, Jan 09, 2013 at 18:17:46, Nori, Sekhar wrote: > > > I do not think that it is good idea to move NAND pin mux information > > into da850.dtsi because this information is evm specific. > > if

Re: [PATCH V2 3/3] ARM: davinci: da850: add NAND driver entries

2013-01-10 Thread Sekhar Nori
On 1/10/2013 1:07 PM, Kumar, Anil wrote: > On Wed, Jan 09, 2013 at 18:17:46, Nori, Sekhar wrote: > I do not think that it is good idea to move NAND pin mux information > into da850.dtsi because this information is evm specific. > if we will use this approach then we must use the same approach for

RE: [PATCH V2 3/3] ARM: davinci: da850: add NAND driver entries

2013-01-09 Thread Kumar, Anil
On Wed, Jan 09, 2013 at 18:17:46, Nori, Sekhar wrote: > On 1/8/2013 1:50 PM, Kumar, Anil wrote: > > Add NAND driver entries to export NAND functionality on da850 EVM and > > NAND pinctrl node to do pin mux according to pinctrl-single driver. > > Subject line should have DT and da850 evm somewhere.

Re: [PATCH V2 3/3] ARM: davinci: da850: add NAND driver entries

2013-01-09 Thread Sekhar Nori
On 1/8/2013 1:50 PM, Kumar, Anil wrote: > Add NAND driver entries to export NAND functionality on da850 EVM and > NAND pinctrl node to do pin mux according to pinctrl-single driver. Subject line should have DT and da850 evm somewhere. Otherwise NAND has been supported on DA850 for long. > > Sign