Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-19 Thread Lee Jones
> > Am not very comfortable with this idea. > > > > As there is no guarantee that the interrupt number/memory map and the > > i2c numbering will be same in future SOCs or other IPs. > > > > You might be already aware that the number of i2cs on each SOC are > > different as example on STiH415 we

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-19 Thread Maxime COQUELIN
On 09/19/2013 03:01 PM, Srinivas KANDAGATLA wrote: > On 19/09/13 08:16, Maxime COQUELIN wrote: >> Hi Srini, >> >> On 09/18/2013 03:17 PM, Srinivas KANDAGATLA wrote: >>> On 18/09/13 13:46, Maxime COQUELIN wrote: On 09/18/2013 02:03 PM, Lee Jones wrote: This patch supplies I2C

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-19 Thread Srinivas KANDAGATLA
On 19/09/13 08:16, Maxime COQUELIN wrote: > Hi Srini, > > On 09/18/2013 03:17 PM, Srinivas KANDAGATLA wrote: >> On 18/09/13 13:46, Maxime COQUELIN wrote: >>> On 09/18/2013 02:03 PM, Lee Jones wrote: >>> This patch supplies I2C configuration to STiH416 SoC. >>> >>> Cc: Srinivas

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-19 Thread Maxime COQUELIN
Hi Srini, On 09/18/2013 03:17 PM, Srinivas KANDAGATLA wrote: > On 18/09/13 13:46, Maxime COQUELIN wrote: >> On 09/18/2013 02:03 PM, Lee Jones wrote: >> This patch supplies I2C configuration to STiH416 SoC. >> >> Cc: Srinivas Kandagatla >> Signed-off-by: Maxime Coquelin >>

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-19 Thread Maxime COQUELIN
Hi Srini, On 09/18/2013 03:17 PM, Srinivas KANDAGATLA wrote: On 18/09/13 13:46, Maxime COQUELIN wrote: On 09/18/2013 02:03 PM, Lee Jones wrote: This patch supplies I2C configuration to STiH416 SoC. Cc: Srinivas Kandagatla srinivas.kandaga...@st.com Signed-off-by: Maxime Coquelin

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-19 Thread Srinivas KANDAGATLA
On 19/09/13 08:16, Maxime COQUELIN wrote: Hi Srini, On 09/18/2013 03:17 PM, Srinivas KANDAGATLA wrote: On 18/09/13 13:46, Maxime COQUELIN wrote: On 09/18/2013 02:03 PM, Lee Jones wrote: This patch supplies I2C configuration to STiH416 SoC. Cc: Srinivas Kandagatla

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-19 Thread Maxime COQUELIN
On 09/19/2013 03:01 PM, Srinivas KANDAGATLA wrote: On 19/09/13 08:16, Maxime COQUELIN wrote: Hi Srini, On 09/18/2013 03:17 PM, Srinivas KANDAGATLA wrote: On 18/09/13 13:46, Maxime COQUELIN wrote: On 09/18/2013 02:03 PM, Lee Jones wrote: This patch supplies I2C configuration to STiH416 SoC.

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-19 Thread Lee Jones
Am not very comfortable with this idea. As there is no guarantee that the interrupt number/memory map and the i2c numbering will be same in future SOCs or other IPs. You might be already aware that the number of i2cs on each SOC are different as example on STiH415 we have 10 SSCs and

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-18 Thread Srinivas KANDAGATLA
On 18/09/13 13:46, Maxime COQUELIN wrote: > On 09/18/2013 02:03 PM, Lee Jones wrote: >>> >> This patch supplies I2C configuration to STiH416 SoC. >>> >> >>> >> Cc: Srinivas Kandagatla >>> >> Signed-off-by: Maxime Coquelin >>> >> --- >>> >> arch/arm/boot/dts/stih416-pinctrl.dtsi | 35

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-18 Thread Maxime COQUELIN
On 09/18/2013 02:03 PM, Lee Jones wrote: >> This patch supplies I2C configuration to STiH416 SoC. >> >> Cc: Srinivas Kandagatla >> Signed-off-by: Maxime Coquelin >> --- >> arch/arm/boot/dts/stih416-pinctrl.dtsi | 35 >> arch/arm/boot/dts/stih416.dtsi | 57 >>

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-18 Thread Lee Jones
> This patch supplies I2C configuration to STiH416 SoC. > > Cc: Srinivas Kandagatla > Signed-off-by: Maxime Coquelin > --- > arch/arm/boot/dts/stih416-pinctrl.dtsi | 35 > arch/arm/boot/dts/stih416.dtsi | 57 > > 2 files

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-18 Thread Lee Jones
This patch supplies I2C configuration to STiH416 SoC. Cc: Srinivas Kandagatla srinivas.kandaga...@st.com Signed-off-by: Maxime Coquelin maxime.coque...@st.com --- arch/arm/boot/dts/stih416-pinctrl.dtsi | 35 arch/arm/boot/dts/stih416.dtsi | 57

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-18 Thread Maxime COQUELIN
On 09/18/2013 02:03 PM, Lee Jones wrote: This patch supplies I2C configuration to STiH416 SoC. Cc: Srinivas Kandagatla srinivas.kandaga...@st.com Signed-off-by: Maxime Coquelin maxime.coque...@st.com --- arch/arm/boot/dts/stih416-pinctrl.dtsi | 35

Re: [PATCH 2/4] ARM: STi: Supply I2C configuration to STiH416 SoC

2013-09-18 Thread Srinivas KANDAGATLA
On 18/09/13 13:46, Maxime COQUELIN wrote: On 09/18/2013 02:03 PM, Lee Jones wrote: This patch supplies I2C configuration to STiH416 SoC. Cc: Srinivas Kandagatla srinivas.kandaga...@st.com Signed-off-by: Maxime Coquelin maxime.coque...@st.com --- arch/arm/boot/dts/stih416-pinctrl.dtsi