Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-07 Thread Alexandre Belloni
On 06/07/2013 12:26, Jonathan Cameron wrote: > On 06/24/2013 06:24 PM, Alexandre Belloni wrote: >> From: Maxime Ripard >> >> The ADCs connected to this bus have been experiencing some timeout >> issues when using the iMX28 i2c controller. Switching back to bitbanging >> solves this. >> >>

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-07 Thread Alexandre Belloni
On 06/07/2013 12:26, Jonathan Cameron wrote: On 06/24/2013 06:24 PM, Alexandre Belloni wrote: From: Maxime Ripard maxime.rip...@free-electrons.com The ADCs connected to this bus have been experiencing some timeout issues when using the iMX28 i2c controller. Switching back to bitbanging

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-06 Thread Jonathan Cameron
On 06/24/2013 06:24 PM, Alexandre Belloni wrote: > From: Maxime Ripard > > The ADCs connected to this bus have been experiencing some timeout > issues when using the iMX28 i2c controller. Switching back to bitbanging > solves this. > > Signed-off-by: Maxime Ripard > Signed-off-by: Alexandre

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-06 Thread Jonathan Cameron
On 06/24/2013 06:24 PM, Alexandre Belloni wrote: From: Maxime Ripard maxime.rip...@free-electrons.com The ADCs connected to this bus have been experiencing some timeout issues when using the iMX28 i2c controller. Switching back to bitbanging solves this. Signed-off-by: Maxime Ripard

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Alexandre Belloni
On 02/07/2013 18:33, Marek Vasut wrote: > Dear Alexandre Belloni, > >> On 02/07/2013 13:50, Alexandre Belloni wrote: >>> On 02/07/2013 13:45, Fabio Estevam wrote: Shouldn't this be i2c@1 { reg = <1>; ? >>> >>> No, we have 4 devices on

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Marek Vasut
Dear Alexandre Belloni, > On 02/07/2013 13:50, Alexandre Belloni wrote: > > On 02/07/2013 13:45, Fabio Estevam wrote: > >> Shouldn't this be > >> > >> i2c@1 { > >> > >> reg = <1>; ? > > > > No, we have 4 devices on that mux and 2 pins to select the muxing. >

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Alexandre Belloni
On 02/07/2013 13:50, Alexandre Belloni wrote: > On 02/07/2013 13:45, Fabio Estevam wrote: > >> Shouldn't this be >> >> i2c@1 { >> reg = <1>; ? >> > > No, we have 4 devices on that mux and 2 pins to select the muxing. > OK, got it working. So, the results:

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Alexandre Belloni
On 02/07/2013 13:45, Fabio Estevam wrote: > Shouldn't this be > > i2c@1 { > reg = <1>; ? > No, we have 4 devices on that mux and 2 pins to select the muxing. -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Fabio Estevam
On Tue, Jul 2, 2013 at 8:35 AM, Alexandre Belloni wrote: > mxs-i2c 8005a000.i2c: Failed to get PIO reg. write descriptor. > > That is the one line 243. I'm open to any suggestion. > > BTW, I also tested PIO mode with and without touchscreen support as that > seemed to toggle the issue on your

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Alexandre Belloni
Hi, On 02/07/2013 04:45, Fabio Estevam wrote: > Are you able to use the mxs i2c controller instead of bitbang if you > use this patch? > http://www.spinics.net/lists/stable/msg13202.html > Actually, it gets worse. I'm probably doing something wrong but I get: mxs-i2c 8005a000.i2c: Failed to

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Alexandre Belloni
On 02/07/2013 18:33, Marek Vasut wrote: Dear Alexandre Belloni, On 02/07/2013 13:50, Alexandre Belloni wrote: On 02/07/2013 13:45, Fabio Estevam wrote: Shouldn't this be i2c@1 { reg = 1; ? No, we have 4 devices on that mux and 2 pins to select the

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Alexandre Belloni
Hi, On 02/07/2013 04:45, Fabio Estevam wrote: Are you able to use the mxs i2c controller instead of bitbang if you use this patch? http://www.spinics.net/lists/stable/msg13202.html Actually, it gets worse. I'm probably doing something wrong but I get: mxs-i2c 8005a000.i2c: Failed to get

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Fabio Estevam
On Tue, Jul 2, 2013 at 8:35 AM, Alexandre Belloni alexandre.bell...@free-electrons.com wrote: mxs-i2c 8005a000.i2c: Failed to get PIO reg. write descriptor. That is the one line 243. I'm open to any suggestion. BTW, I also tested PIO mode with and without touchscreen support as that seemed

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Alexandre Belloni
On 02/07/2013 13:45, Fabio Estevam wrote: Shouldn't this be i2c@1 { reg = 1; ? No, we have 4 devices on that mux and 2 pins to select the muxing. -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Alexandre Belloni
On 02/07/2013 13:50, Alexandre Belloni wrote: On 02/07/2013 13:45, Fabio Estevam wrote: Shouldn't this be i2c@1 { reg = 1; ? No, we have 4 devices on that mux and 2 pins to select the muxing. OK, got it working. So, the results: bitbanging: # time cat

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-02 Thread Marek Vasut
Dear Alexandre Belloni, On 02/07/2013 13:50, Alexandre Belloni wrote: On 02/07/2013 13:45, Fabio Estevam wrote: Shouldn't this be i2c@1 { reg = 1; ? No, we have 4 devices on that mux and 2 pins to select the muxing. OK, got it working.

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-01 Thread Fabio Estevam
Hi Alexandre, On Mon, Jun 24, 2013 at 2:24 PM, Alexandre Belloni wrote: > From: Maxime Ripard > > The ADCs connected to this bus have been experiencing some timeout > issues when using the iMX28 i2c controller. Switching back to bitbanging > solves this. Are you able to use the mxs i2c

Re: [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-07-01 Thread Fabio Estevam
Hi Alexandre, On Mon, Jun 24, 2013 at 2:24 PM, Alexandre Belloni alexandre.bell...@free-electrons.com wrote: From: Maxime Ripard maxime.rip...@free-electrons.com The ADCs connected to this bus have been experiencing some timeout issues when using the iMX28 i2c controller. Switching back to

[PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-06-24 Thread Alexandre Belloni
From: Maxime Ripard The ADCs connected to this bus have been experiencing some timeout issues when using the iMX28 i2c controller. Switching back to bitbanging solves this. Signed-off-by: Maxime Ripard Signed-off-by: Alexandre Belloni --- arch/arm/boot/dts/imx28-cfa10049.dts | 108

[PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging

2013-06-24 Thread Alexandre Belloni
From: Maxime Ripard maxime.rip...@free-electrons.com The ADCs connected to this bus have been experiencing some timeout issues when using the iMX28 i2c controller. Switching back to bitbanging solves this. Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com Signed-off-by: Alexandre