Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread srinivas kandagatla
On 02/12/13 13:43, Lee Jones wrote: > On Mon, 02 Dec 2013, srinivas kandagatla wrote: > >> On 02/12/13 13:20, Lee Jones wrote: > + /* FSM */ > + spifsm: spifsm@fe902000{ > + compatible = "st,spi-fsm", "simple-bus"; "simple-bus" is redundant

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread Lee Jones
On Mon, 02 Dec 2013, srinivas kandagatla wrote: > On 02/12/13 13:20, Lee Jones wrote: > >>> + /* FSM */ > >>> + spifsm: spifsm@fe902000{ > >>> + compatible = "st,spi-fsm", "simple-bus"; > >> "simple-bus" is redundant here. > > > > Yes I agree, good spot. >

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread srinivas kandagatla
On 02/12/13 13:20, Lee Jones wrote: >>> + /* FSM */ >>> + spifsm: spifsm@fe902000{ >>> + compatible = "st,spi-fsm", "simple-bus"; >> "simple-bus" is redundant here. > > Yes I agree, good spot. > > I removed it from the Documentation, but missed this

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread Lee Jones
> > + /* FSM */ > > + spifsm: spifsm@fe902000{ > > + compatible = "st,spi-fsm", "simple-bus"; > "simple-bus" is redundant here. Yes I agree, good spot. I removed it from the Documentation, but missed this one. If I remove it, can I add your Ack for

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread srinivas kandagatla
Hi Lee, On 29/11/13 12:19, Lee Jones wrote: > + /* FSM */ > + spifsm: spifsm@fe902000{ > + compatible = "st,spi-fsm", "simple-bus"; "simple-bus" is redundant here. > + reg= <0xfe902000 0x1000>; > +

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread srinivas kandagatla
Hi Lee, On 29/11/13 12:19, Lee Jones wrote: + /* FSM */ + spifsm: spifsm@fe902000{ + compatible = st,spi-fsm, simple-bus; simple-bus is redundant here. + reg= 0xfe902000 0x1000; +

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread Lee Jones
+ /* FSM */ + spifsm: spifsm@fe902000{ + compatible = st,spi-fsm, simple-bus; simple-bus is redundant here. Yes I agree, good spot. I removed it from the Documentation, but missed this one. If I remove it, can I add your Ack for the remainder

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread srinivas kandagatla
On 02/12/13 13:20, Lee Jones wrote: + /* FSM */ + spifsm: spifsm@fe902000{ + compatible = st,spi-fsm, simple-bus; simple-bus is redundant here. Yes I agree, good spot. I removed it from the Documentation, but missed this one. If I remove

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread Lee Jones
On Mon, 02 Dec 2013, srinivas kandagatla wrote: On 02/12/13 13:20, Lee Jones wrote: + /* FSM */ + spifsm: spifsm@fe902000{ + compatible = st,spi-fsm, simple-bus; simple-bus is redundant here. Yes I agree, good spot. I removed it from the

Re: [PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-12-02 Thread srinivas kandagatla
On 02/12/13 13:43, Lee Jones wrote: On Mon, 02 Dec 2013, srinivas kandagatla wrote: On 02/12/13 13:20, Lee Jones wrote: + /* FSM */ + spifsm: spifsm@fe902000{ + compatible = st,spi-fsm, simple-bus; simple-bus is redundant here. Yes I agree, good

[PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-11-29 Thread Lee Jones
Here we add the necessary device nodes required for successful device probing and Pinctrl setup for the FSM. Cc: Srinivas Kandagatla Signed-off-by: Lee Jones --- arch/arm/boot/dts/stih416-b2105.dts| 14 ++ arch/arm/boot/dts/stih416-pinctrl.dtsi | 12 2 files

[PATCH v3 36/36] ARM: STi: Add support for the FSM Serial Flash Controller

2013-11-29 Thread Lee Jones
Here we add the necessary device nodes required for successful device probing and Pinctrl setup for the FSM. Cc: Srinivas Kandagatla srinivas.kandaga...@st.com Signed-off-by: Lee Jones lee.jo...@linaro.org --- arch/arm/boot/dts/stih416-b2105.dts| 14 ++