Re: [PATCH v4 0/4] i2c: designware: improve performance for transfers

2016-08-25 Thread Christian Ruppert
ion, Christian had added: > > Tested-by: Christian Ruppert > on TB101 with Linux-4.7 Tested-by: Christian Ruppert Re-tested this iteration in the same configuration (TB101, Linux-4.7). Still works perfectly. > And Jarkko added his Acked-by: Jarkko Nikula > on patches 1 and 3 (n

Re: [PATCH v3 0/3] i2c: designware: improve performance for transfers

2016-08-12 Thread Christian Ruppert
n BayTrail and CherryTrail, both of them returning true > for > "dynamically update TAR" Tested-by: Christian Ruppert on TB101 with Linux-4.7 Seems to work perfectly now, congratulations and thanks for your patience with our platform... This was a hard one! > José Roberto de Sou

Re: [PATCH] i2c: designware: do not disable adapter after transfer

2016-05-09 Thread Christian Ruppert
Dear Lucas, On 04.05.2016 16:38, Lucas De Marchi wrote: > Hi Christian, > > On Mon, May 2, 2016 at 7:11 AM, Christian Ruppert > wrote: >> Dear Lucas, >> >> On 22.04.2016 17:19, Lucas De Marchi wrote: >>> CC'ing Christian. >>> >>>

Re: [PATCH] i2c: designware: do not disable adapter after transfer

2016-05-02 Thread Christian Ruppert
Dear Lucas, On 22.04.2016 17:19, Lucas De Marchi wrote: > CC'ing Christian. > > On Fri, Apr 22, 2016 at 12:08 PM, Lucas De Marchi > wrote: >> Disabling the adapter after each transfer is pretty bad for sensors and >> other devices doing small transfers at a high rate. It slows down the >> transf

Re: [PATCH] i2c: designware: do not disable adapter after transfer

2016-04-08 Thread Christian Ruppert
On 2016-04-07 19:28, De Marchi, Lucas wrote: > Hi Christian, > > On Thu, 2016-04-07 at 15:37 +0200, Christian Ruppert wrote: >> Dear Lucas, >> >> Sorry for the late reply but I had to put our test environment back >> together to check this patch. I'll kee

Re: [PATCH] i2c: designware: do not disable adapter after transfer

2016-04-07 Thread Christian Ruppert
Dear Lucas, Sorry for the late reply but I had to put our test environment back together to check this patch. I'll keep it around for a while in case you have further iterations to test. On 2016-04-01 04:47, Lucas De Marchi wrote: > From: Lucas De Marchi > > Disabling the adapter after each tra

Re: [PATCH v2 7/9] ARC: use fixed frequencies in arc_set_early_base_baud()

2016-03-08 Thread christian . ruppert
arc_{get|set}_core_freq() and switch to generic clock > framework later on. > > Cc: Christian Ruppert > Signed-off-by: Alexey Brodkin > Signed-off-by: Vineet Gupta Acked-by: Christian Ruppert > --- > Changes v1 -> v2 > - Base baud TB10x needs to be 166.66 not 166.6

Re: [PATCH 7/9] ARC: use fixed frequencies in arc_set_early_base_baud()

2016-02-02 Thread christian . ruppert
Alexey Brodkin wrote on 02.02.2016 13:53:26: > > Adding Christian for Abilis TB10x clocks review. > > On Tue, 2016-02-02 at 16:28 +0530, Vineet Gupta wrote: > > From: Alexey Brodkin > > > > UARTs usually have fixed clock so we're switching to use of > > constant values instead of something der

Re: [PATCH] i2c: designware: use enable on resume instead initialization

2015-06-24 Thread christian . ruppert
> Lucas De Marchi wrote on 23.06.2015 19:02:03: > > > > On Tue, Jun 23, 2015 at 1:45 PM, wrote: > > > > > Hello, > > > > > > > > > > Christian Ruppert/ALi_GVA/ALi wrote on 10.06.2015 17:05:16: > > > > [...] > > > >

Re: [PATCH] i2c: designware: use enable on resume instead initialization

2015-06-24 Thread christian . ruppert
Dear Lucas, Lucas De Marchi wrote on 23.06.2015 19:02:03: > On Tue, Jun 23, 2015 at 1:45 PM, wrote: > > Hello, > > > > Christian Ruppert/ALi_GVA/ALi wrote on 10.06.2015 17:05:16: > [...] > > The result is not very encouraging: Out of five (identical) designware

Re: [PATCH] i2c: designware: use enable on resume instead initialization

2015-06-23 Thread christian . ruppert
Hello, Christian Ruppert/ALi_GVA/ALi wrote on 10.06.2015 17:05:16: > Mika Westerberg wrote on 10.06. > 2015 09:07:22: > > On Tue, Jun 09, 2015 at 03:29:01PM -0300, Lucas De Marchi wrote: > > > Hi Mika, > > > > > > On Tue, Jun 9, 2015 at 5:51 AM, Mika Weste

Re: [PATCH v2] i2c: designware: Suppress error message if platform_get_irq() < 0

2015-03-09 Thread christian . ruppert
Alexey Brodkin wrote on 03/09/2015 10:03:12 AM: > From: Alexey Brodkin > To: linux-...@vger.kernel.org, > Cc: linux-kernel@vger.kernel.org, Alexey Brodkin > , Vineet Gupta > , Christian Ruppert > , Mika Westerberg > , Wolfram Sang , > Andy Shevchenko > Date:

Re: [PATCH] i2c: designware: Suppress error message if platform_get_irq() returns -EPROBE_DEFER

2015-03-03 Thread Christian Ruppert
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2015-03-03 18:21, Wolfram Sang wrote: > >> which omit this type of messages completely. Andy's proposal of >> centralising this looks like a very good solution here (and on >> top of that removes many useless strings from the kernel >> binary). >

Re: [PATCH] i2c: designware: Suppress error message if platform_get_irq() returns -EPROBE_DEFER

2015-03-03 Thread christian . ruppert
Hello Alexey, Alexey Brodkin wrote on 03/03/2015 05:37:31 PM: > From: > > Alexey Brodkin > > To: > > "christian.rupp...@alitech.com" , > > Cc: > > "mika.westerb...@linux.intel.com" , > "linux-kernel@vger.kernel.org" , > "vineet.gup...@synopsys.com" , "wsa@the- > dreams.de" , "andriy.she

Re: [PATCH] i2c: designware: Suppress error message if platform_get_irq() returns -EPROBE_DEFER

2015-03-03 Thread Christian Ruppert
DEFER. > > Signed-off-by: Alexey Brodkin > Cc: Vineet Gupta > Cc: Christian Ruppert > Cc: Mika Westerberg > Cc: Wolfram Sang > --- > drivers/i2c/busses/i2c-designware-platdrv.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/driver

[PATCH] pinctrl/TB10x: Fix signedness bug

2014-04-11 Thread Christian Ruppert
In the TB10x pin database, a port index of -1 is used to indicate unmuxed GPIO pin groups. This bug fixes a 'cast to unsigned' bug of this value. Thanks to Dan Carpenter for highlighting this. CC: Dan Carpenter Signed-off-by: Christian Ruppert --- drivers/pinctrl/pinctrl-tb10x.c |

Re: [PATCH 2/2 v2] x86: Add support for uncompressed kernel images

2013-12-16 Thread Christian Ruppert
On Mon, Nov 18, 2013 at 02:48:30PM +0100, Christian Ruppert wrote: > There seems to be some interest to enable uncompressed kernels also for x86 > in addition to (embedded) ARC platforms. > > Add the code to support uncompressed kernels to lib/, select the respective > Kconfig opti

[PATCH 2/2 v2] x86: Add support for uncompressed kernel images

2013-11-18 Thread Christian Ruppert
-off-by: Christian Ruppert --- arch/x86/Kconfig |1 + arch/x86/boot/compressed/Makefile | 14 ++-- arch/x86/boot/compressed/misc.c |4 +++ lib/decompress_copy.c | 44 + 4 files changed, 56 insertions(+), 7

[PATCH 2/2] x86: Add support for uncompressed kernel images

2013-11-18 Thread Christian Ruppert
Signed-off-by: Christian Ruppert --- arch/x86/Kconfig |1 + arch/x86/boot/compressed/Makefile | 14 ++-- arch/x86/boot/compressed/misc.c |4 +++ lib/decompress_copy.c | 44 + 4 files changed, 56 insertions

[PATCH 1/2] init/Kconfig: add option to disable kernel compression

2013-11-18 Thread Christian Ruppert
to permit disabling of kernel compression. Signed-off-by: Christian Ruppert --- arch/arc/Kconfig |2 ++ init/Kconfig | 13 - 2 files changed, 14 insertions(+), 1 deletions(-) diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig index 2ee0c9b..15f4c3d 100644 --- a/arch/arc/Kconfig +++

Re: [PATCH] [FIX] init/Kconfig: fix option to disable kernel compression

2013-11-18 Thread Christian Ruppert
On Sat, Nov 16, 2013 at 03:12:56PM +0530, Vineet Gupta wrote: > On 11/15/2013 10:21 PM, Christian Ruppert wrote: > > Some architectures with self-decompressing kernel images did not compile > > with commit 69f0554ec261fd686ac7fa1c598cc9eb27b83a80 because they don't > > p

Re: [PATCH 1/2] init/Kconfig: add option to disable kernel compression

2013-11-15 Thread Christian Ruppert
On Thu, Nov 14, 2013 at 03:51:22PM +0530, Vineet Gupta wrote: > +CC Sam for Kconfig wisdom > > On 11/14/2013 02:08 PM, Christian Ruppert wrote: > > Some ARC users say they can boot faster with without kernel compression. > > This probably depends on things like the FL

[PATCH] [FIX] init/Kconfig: fix option to disable kernel compression

2013-11-15 Thread Christian Ruppert
which explicitly state they support them. Signed-off-by: Christian Ruppert --- arch/arc/Kconfig | 2 ++ init/Kconfig | 5 + 2 files changed, 7 insertions(+) diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig index 2ee0c9b..15f4c3d 100644 --- a/arch/arc/Kconfig +++ b/arch/arc/Kconfig @@ -21,6

Re: [PATCH 2/2] x86: Add support for uncompressed kernel images

2013-11-15 Thread Christian Ruppert
On Thu, Nov 14, 2013 at 09:45:12AM -0800, H. Peter Anvin wrote: > On 11/14/2013 12:38 AM, Christian Ruppert wrote: > > diff --git a/lib/decompress_copy.c b/lib/decompress_copy.c > > new file mode 100644 > > index 000..8a41090 > > --- /dev/null > > +++ b/lib/de

Re: [PATCH 2/2] x86: Add support for uncompressed kernel images

2013-11-15 Thread Christian Ruppert
On Thu, Nov 14, 2013 at 09:31:44AM -0800, H. Peter Anvin wrote: > So it sounds like we're starting from the beginning - I presume that > means the patch currently in the kernel should be reverted first? Ack -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a

[PATCH 1/2] init/Kconfig: add option to disable kernel compression

2013-11-14 Thread Christian Ruppert
to permit disabling of kernel compression. Signed-off-by: Christian Ruppert --- arch/arc/Kconfig | 2 ++ init/Kconfig | 12 +++- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig index 91dbb27..3991f03 100644 --- a/arch/arc/Kconfig +++

[PATCH 2/2] x86: Add support for uncompressed kernel images

2013-11-14 Thread Christian Ruppert
Signed-off-by: Christian Ruppert --- arch/x86/Kconfig | 1 + arch/x86/boot/compressed/Makefile | 14 ++-- arch/x86/boot/compressed/misc.c | 4 lib/decompress_copy.c | 47 +++ 4 files changed, 59 insertions(+), 7

Re: Uncompressed kernel doesn't build on x86_64

2013-11-14 Thread Christian Ruppert
On Wed, Nov 13, 2013 at 11:34:18AM -0500, Pavel Roskin wrote: > Hi Christian, > > Your recent patch enables uncompressed kernels, but selecting that > option (CONFIG_KERNEL_UNCOMPRESSED) fails on x86_86: > [...] > arch/x86/boot/compressed/Makefile has no provision for uncompressed > files. There

Re: [PATCH 03/03] GPIO: Add TB10x GPIO driver

2013-10-24 Thread Christian Ruppert
On Wed, Oct 16, 2013 at 02:58:55PM +0200, Christian Ruppert wrote: > On Wed, Oct 16, 2013 at 01:29:44PM +0200, Linus Walleij wrote: > > On Tue, Oct 15, 2013 at 3:45 PM, Christian Ruppert > > wrote: > > > On Wed, Oct 09, 2013 at 02:19:17PM +0200, Linus Walleij wrote: >

Re: [PATCH 03/03] GPIO: Add TB10x GPIO driver

2013-10-16 Thread Christian Ruppert
On Wed, Oct 16, 2013 at 01:29:44PM +0200, Linus Walleij wrote: > On Tue, Oct 15, 2013 at 3:45 PM, Christian Ruppert > wrote: > > On Wed, Oct 09, 2013 at 02:19:17PM +0200, Linus Walleij wrote: > > >> It's not like I'm 100% certain on where to use one or the othe

[PATCH] Add a short note on pinctrl_get_group_pins to pinctrl.txt

2013-10-16 Thread Christian Ruppert
This patch adds a short description of how to use the newly added pinctrl_get_group_pins function to the pinctrl documentation. Signed-off-by: Christian Ruppert --- Documentation/pinctrl.txt |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/Documentation/pinctrl.txt

Re: [PATCH 03/03] GPIO: Add TB10x GPIO driver

2013-10-15 Thread Christian Ruppert
On Wed, Oct 09, 2013 at 02:19:17PM +0200, Linus Walleij wrote: > On Tue, Oct 8, 2013 at 2:25 PM, Christian Ruppert > wrote: > > Overall this driver is looking very nice, we just need to figure out this > group range concept in the other patch. > > > +Example: >

[PATCH V2] pinmux: Add TB10x pinmux driver

2013-10-15 Thread Christian Ruppert
The pinmux driver of the Abilis Systems TB10x platform based on ARC700 CPUs. Used to control the pinmux and is a prerequisite for the GPIO driver. Signed-off-by: Christian Ruppert Signed-off-by: Pierrick Hascoet --- .../bindings/pinctrl/abilis,tb10x-iomux.txt| 80 ++ drivers/pinctrl

[PATCH V2] Make non-linear GPIO ranges accesible from gpiolib

2013-10-15 Thread Christian Ruppert
This patch adds the infrastructure required to register non-linear gpio ranges through gpiolib and the standard GPIO device tree bindings. Signed-off-by: Christian Ruppert --- Documentation/devicetree/bindings/gpio/gpio.txt | 40 ++- drivers/gpio/gpiolib-of.c

Re: [PATCH 01/03] Make non-linear GPIO ranges accesible from gpiolib

2013-10-15 Thread Christian Ruppert
On Wed, Oct 09, 2013 at 04:01:05PM +0200, Linus Walleij wrote: > On Wed, Oct 9, 2013 at 3:28 PM, Christian Ruppert > wrote: > > On Wed, Oct 09, 2013 at 01:58:35PM +0200, Linus Walleij wrote: > >> On Tue, Oct 8, 2013 at 2:25 PM, Christian Ruppert > >> wrote: > &

Re: [PATCH 01/03] Make non-linear GPIO ranges accesible from gpiolib

2013-10-09 Thread Christian Ruppert
On Wed, Oct 09, 2013 at 01:58:35PM +0200, Linus Walleij wrote: > On Tue, Oct 8, 2013 at 2:25 PM, Christian Ruppert > wrote: > > > This patch adds the infrastructure required to register non-linear gpio > > ranges through gpiolib and the standard GPIO device tree bindings.

Re: [PATCH REBASE] Add option to disable kernel compression

2013-10-09 Thread Christian Ruppert
On Tue, Oct 08, 2013 at 12:29:40PM -0700, Andrew Morton wrote: > On Tue, 8 Oct 2013 14:33:32 +0200 Christian Ruppert > wrote: > > > Until now, kernel compression can only be disabled by deselecting > > HAVE_ in the architecture Kconfig. > > Scratching my hea

[PATCH 01/03] Make non-linear GPIO ranges accesible from gpiolib

2013-10-08 Thread Christian Ruppert
This patch adds the infrastructure required to register non-linear gpio ranges through gpiolib and the standard GPIO device tree bindings. Signed-off-by: Christian Ruppert --- Documentation/devicetree/bindings/gpio/gpio.txt | 40 +++ drivers/gpio/gpiolib-of.c

[PATCH 02/03] pinmux: Add TB10x pinmux driver

2013-10-08 Thread Christian Ruppert
The pinmux driver of the Abilis Systems TB10x platform based on ARC700 CPUs. Used to control the pinmux and is a prerequisite for the GPIO driver. Signed-off-by: Christian Ruppert Signed-off-by: Pierrick Hascoet --- .../bindings/pinctrl/abilis,tb10x-iomux.txt| 79 ++ drivers/pinctrl

[PATCH 03/03] GPIO: Add TB10x GPIO driver

2013-10-08 Thread Christian Ruppert
The GPIO driver for the Abilis Systems TB10x series of SOCs based on ARC700 CPUs. It supports GPIO control and GPIO interrupt generation. This driver works in conjunction with the TB10x pinctrl driver. Signed-off-by: Sascha Leuenberger Signed-off-by: Christian Ruppert --- .../devicetree

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-10-08 Thread Christian Ruppert
On Wed, Aug 28, 2013 at 04:47:36PM +0200, Christian Ruppert wrote: > On Thu, Aug 22, 2013 at 02:10:55PM -0600, Stephen Warren wrote: > [...] > > Pin groups are supposed to be something that represents some property of > > the pinctrl HW itself. So, if you have register "

Re: [PATCH] Export symbols for splice in modules

2013-10-08 Thread Christian Ruppert
On Tue, Oct 08, 2013 at 05:52:29AM -0700, Christoph Hellwig wrote: > On Tue, Oct 08, 2013 at 02:38:18PM +0200, Christian Ruppert wrote: > > The symbols splice_to_pipe, splice_grow_spd and splice_shrink_spd are not > > currently exported from the kernel. This prevents the im

[PATCH] Export symbols for splice in modules

2013-10-08 Thread Christian Ruppert
kernel modules. Signed-off-by: Christian Ruppert --- fs/splice.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/fs/splice.c b/fs/splice.c index 3b7ee65..28e19b7 100644 --- a/fs/splice.c +++ b/fs/splice.c @@ -265,6 +265,7 @@ ssize_t splice_to_pipe(struct pipe_inode_info

[PATCH REBASE] Add option to disable kernel compression

2013-10-08 Thread Christian Ruppert
Until now, kernel compression can only be disabled by deselecting HAVE_ in the architecture Kconfig. However, some users/platforms within the same architecture might want to use compression while others might want to disable it. This patch is a solution to the dilemma. Signed-off-by: Christian

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-08-30 Thread Christian Ruppert
On Thu, Aug 29, 2013 at 10:24:30AM +0200, Linus Walleij wrote: > On Thu, Aug 29, 2013 at 9:35 AM, Christian Ruppert > wrote: > > On Wed, Aug 28, 2013 at 08:49:36PM +0200, Linus Walleij wrote: > >> On Wed, Aug 21, 2013 at 5:57 PM, Christian Ruppert > >> wrote: > &

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-08-29 Thread Christian Ruppert
On Wed, Aug 28, 2013 at 08:49:36PM +0200, Linus Walleij wrote: > On Wed, Aug 21, 2013 at 5:57 PM, Christian Ruppert > wrote: > > On Wed, Aug 14, 2013 at 06:53:56PM +0200, Linus Walleij wrote: > >> > >> OK, that can also be called a "bank" or "register&

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-08-28 Thread Christian Ruppert
On Sat, Aug 24, 2013 at 01:58:47PM +0900, Shinya Kuribayashi wrote: > On 8/21/13 11:39 PM, Christian Ruppert wrote: > >On Fri, Aug 16, 2013 at 11:15:12AM +0900, Shinya Kuribayashi wrote: > >>On 8/5/13 6:31 PM, Christian Ruppert wrote: > >>>On Wed, Jul 24, 20

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-08-28 Thread Christian Ruppert
On Thu, Aug 22, 2013 at 02:10:55PM -0600, Stephen Warren wrote: > On 08/21/2013 09:57 AM, Christian Ruppert wrote: > > On Wed, Aug 14, 2013 at 06:53:56PM +0200, Linus Walleij wrote: > >> On Mon, Aug 5, 2013 at 1:51 PM, Christian Ruppert > >> wrote: > >>>

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-08-21 Thread Christian Ruppert
On Wed, Aug 14, 2013 at 06:53:56PM +0200, Linus Walleij wrote: > On Mon, Aug 5, 2013 at 1:51 PM, Christian Ruppert > wrote: > > [Me] > >> I don't see any of the port concept creeping into the device tree > >> in this version and that is how I think it should b

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-08-21 Thread Christian Ruppert
On Fri, Aug 16, 2013 at 11:15:12AM +0900, Shinya Kuribayashi wrote: > Hi, > > On 8/5/13 6:31 PM, Christian Ruppert wrote:> On Wed, Jul 24, 2013 at > 11:31:44PM +0900, Shinya Kuribayashi wrote: > >>As said before, all t_SCL things should go away. Please forget > >&

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-08-12 Thread Christian Ruppert
On Mon, Aug 05, 2013 at 12:02:26PM +0200, Wolfram Sang wrote: > > > > >Would it make sense to add generic I2C device tree properties for those > > > >parameters? These parameters are independent of the actual bus driver, > > > >rather a PCB property... And as such the correct place would be device

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-08-05 Thread Christian Ruppert
On Tue, Jul 30, 2013 at 12:35:03AM +0200, Linus Walleij wrote: > Sorry for taking eternities to look into this. > > On Tue, Jun 18, 2013 at 11:29 AM, Christian Ruppert > wrote: > > > The pinmux driver of the Abilis Systems TB10x platform based on ARC700 CPUs. > > Use

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-08-05 Thread Christian Ruppert
On Wed, Jul 24, 2013 at 11:31:44PM +0900, Shinya Kuribayashi wrote: > On 7/22/13 10:17 PM, Christian Ruppert wrote:> On Wed, Jul 17, 2013 at > 11:39:58PM +0900, Shinya Kuribayashi wrote: > >>On 7/16/13 8:16 PM, Christian Ruppert wrote:> On Sat, Jul 13, 2013 at > &g

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-07-26 Thread Christian Ruppert
On Thu, Jul 18, 2013 at 01:54:18PM -0600, Stephen Warren wrote: > On 07/18/2013 10:07 AM, Christian Ruppert wrote: > ... > > Well, perhaps my definition of "inside"/"outside" pins was not quite > > clear: The pin groups define the set of (kernel internal) p

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-07-22 Thread Christian Ruppert
On Wed, Jul 17, 2013 at 11:39:58PM +0900, Shinya Kuribayashi wrote: > On 7/16/13 8:16 PM, Christian Ruppert wrote:> On Sat, Jul 13, 2013 at > 02:36:43PM +0900, Shinya Kuribayashi wrote: > >>Basically, DW I2C core provides a good enough (and quite direct) way > >>to co

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-07-18 Thread Christian Ruppert
On Tue, Jul 16, 2013 at 10:04:03AM -0600, Stephen Warren wrote: > On 07/16/2013 02:47 AM, Christian Ruppert wrote: > > On Wed, Jul 10, 2013 at 01:27:52PM -0600, Stephen Warren wrote: > >> On 07/08/2013 07:02 AM, Christian Ruppert wrote: > >> ... > >>> OK,

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-07-16 Thread Christian Ruppert
rolled by SCL_HCNT whereas (2) is controlled by SDA_HOLD_TIME. _ scl \__ ... ___ __ sda \_/ ... |-|---| (1) (2) > On Wed, Jul 10, 2013 at 06:56:35PM +0200, Christian Ruppert wrote: > >>I

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-07-16 Thread Christian Ruppert
On Wed, Jul 10, 2013 at 01:27:52PM -0600, Stephen Warren wrote: > On 07/08/2013 07:02 AM, Christian Ruppert wrote: > ... > > OK, a small drawing of our hardware should make this clear, let's take > > an imaginary example of one port with 10 pins, one i2c interface, one &g

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-07-10 Thread Christian Ruppert
On Wed, Jul 10, 2013 at 01:52:15PM +0300, Mika Westerberg wrote: > On Tue, Jul 09, 2013 at 06:19:28PM +0200, Christian Ruppert wrote: > > What I meant is the following: The clock cycle time Tc is composed of > > the four components > > > > Tc = Th + Tf + Tl + Tr &g

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-07-09 Thread Christian Ruppert
On Tue, Jul 09, 2013 at 11:44:02AM +0300, Mika Westerberg wrote: > On Mon, Jul 08, 2013 at 03:42:17PM +0200, Christian Ruppert wrote: > > On Mon, Jul 08, 2013 at 02:45:26PM +0300, Mika Westerberg wrote: > > > The DesignWare I2C controller has high count (HCNT) and low count (LCN

Re: [PATCH 1/2] i2c-designware: make *CNT values configurable

2013-07-08 Thread Christian Ruppert
*/ > struct dw_i2c_dev { > struct device *dev; > @@ -91,6 +99,10 @@ struct dw_i2c_dev { > unsigned intrx_fifo_depth; > int rx_outstanding; > u32 sda_hold_time; > + u16 ss_hcnt; &g

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-07-08 Thread Christian Ruppert
On Fri, Jul 05, 2013 at 12:40:55PM -0600, Stephen Warren wrote: > On 07/05/2013 03:49 AM, Christian Ruppert wrote: > > On Wed, Jun 26, 2013 at 11:40:42AM -0600, Stephen Warren wrote: > >> On 06/26/2013 05:50 AM, Christian Ruppert wrote: > >>> On Wed, Jun 19, 201

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-07-05 Thread Christian Ruppert
On Wed, Jun 26, 2013 at 11:40:42AM -0600, Stephen Warren wrote: > On 06/26/2013 05:50 AM, Christian Ruppert wrote: > > On Wed, Jun 19, 2013 at 04:35:14PM -0600, Stephen Warren wrote: > >> On 06/18/2013 03:29 AM, Christian Ruppert wrote: > [...] > >>> +Example &g

Re: [PATCH v10] i2c-designware: make SDA hold time configurable

2013-07-03 Thread Christian Ruppert
On Wed, Jul 03, 2013 at 04:44:09PM +0200, Arnd Bergmann wrote: > On Wednesday 03 July 2013, Christian Ruppert wrote: > > On Wed, Jul 03, 2013 at 04:20:03PM +0200, Arnd Bergmann wrote: > > > On Wednesday 03 July 2013, Christian Ruppert wrote: > > > > On Wed, Jul 03,

Re: [PATCH v10] i2c-designware: make SDA hold time configurable

2013-07-03 Thread Christian Ruppert
On Wed, Jul 03, 2013 at 04:20:03PM +0200, Arnd Bergmann wrote: > On Wednesday 03 July 2013, Christian Ruppert wrote: > > On Wed, Jul 03, 2013 at 01:43:11PM +0200, Arnd Bergmann wrote: > > > On Wednesday 26 June 2013, Wolfram Sang wrote: > > > > On Wed, Jun 26, 2013

Re: [PATCH v10] i2c-designware: make SDA hold time configurable

2013-07-03 Thread Christian Ruppert
On Wed, Jul 03, 2013 at 01:43:11PM +0200, Arnd Bergmann wrote: > On Wednesday 26 June 2013, Wolfram Sang wrote: > > On Wed, Jun 26, 2013 at 10:55:06AM +0200, Christian Ruppert wrote: > > > This patch makes the SDA hold time configurable through device tree. > > > >

Re: [PATCH v2] pinctrl: elaborate a bit on arrangements in doc

2013-07-03 Thread Christian Ruppert
ere > e.g. GPIO direction is determined by a register in the pin controller HW > module rather than the GPIO HW module. > - I agree, this is really worth mentioning in some place, maybe even a more prominent one than here. -- Christian Ruppert ,

Re: [PATCH linux-next] i2c-designware: use div_u64 to fix link

2013-07-03 Thread Christian Ruppert
good. I also tested it on our ARC based platform, no issues with compilation or functionality. Thanks for pointing this out. Reviewed-by: Christian Ruppert > Signed-off-by: Vincent Stehlé > Cc: Wolfram Sang > Cc: Christian Ruppert > Cc: Pierrick Hascoet > --- > > >

Re: [PATCH v2] pinctrl: elaborate a bit on arrangements in doc

2013-06-27 Thread Christian Ruppert
On Thu, Jun 27, 2013 at 11:54:47AM +0200, Linus Walleij wrote: > From: Linus Walleij > > This elaborates a bit on the pin control and pin muxing > logic vs GPIO arangements in the hardware. > > Inspired by some drawings in a mail from Christian Ruppert. > Both arrangem

[PATCH] ARC: [TB10x] Updates for irqchip driver

2013-06-26 Thread Christian Ruppert
Device tree and Kconfig updates for irqchip driver. Signed-off-by: Christian Ruppert --- arch/arc/boot/dts/abilis_tb100.dtsi | 32 ++-- arch/arc/boot/dts/abilis_tb101.dtsi | 32 ++-- arch/arc/boot/dts/abilis_tb10x.dtsi | 32

Re: [PATCH] pinctrl: elaborate a bit on arrangements in doc

2013-06-26 Thread Christian Ruppert
On Tue, Jun 25, 2013 at 04:19:12PM +0200, Linus Walleij wrote: > From: Linus Walleij > > This elaborates a bit on the pinctrl vs GPIO arangements > in the hardware. > > Inspired by some drawings in a mail from Christian > Ruppert. > > Cc: Rob Landley > Cc: Chri

Re: [PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-06-26 Thread Christian Ruppert
On Wed, Jun 19, 2013 at 04:35:14PM -0600, Stephen Warren wrote: > On 06/18/2013 03:29 AM, Christian Ruppert wrote: > > The pinmux driver of the Abilis Systems TB10x platform based on ARC700 CPUs. > > Used to control the pinmux and is a prerequisite for the GPIO driver. > >

Re: [PATCH 1/4] Make non-linear GPIO ranges accesible from gpiolib

2013-06-26 Thread Christian Ruppert
On Wed, Jun 19, 2013 at 04:27:44PM -0600, Stephen Warren wrote: > On 06/18/2013 03:29 AM, Christian Ruppert wrote: > > This patch adds the infrastructure required to register non-linear gpio > > ranges through gpiolib and the standard GPIO device tree bindings. > > I review t

Re: [PATCH 1/4] Make non-linear GPIO ranges accesible from gpiolib

2013-06-26 Thread Christian Ruppert
On Wed, Jun 19, 2013 at 12:15:13PM -0600, Stephen Warren wrote: > On 06/19/2013 06:03 AM, Linus Walleij wrote: > > On Tue, Jun 18, 2013 at 11:29 AM, Christian Ruppert > > wrote: > > > >> This patch adds the infrastructure required to register non-linear gpio >

[PATCH v10] i2c-designware: make SDA hold time configurable

2013-06-26 Thread Christian Ruppert
This patch makes the SDA hold time configurable through device tree. Signed-off-by: Christian Ruppert Signed-off-by: Pierrick Hascoet --- .../devicetree/bindings/i2c/i2c-designware.txt | 15 +++ arch/arc/boot/dts/abilis_tb100_dvk.dts | 10 +- arch/arc

[tip:irq/core] irqchip: Add TB10x interrupt controller driver

2013-06-25 Thread tip-bot for Christian Ruppert
Commit-ID: b06eb0173ef1d9366d14b4ca3e8e38dc72b03e8b Gitweb: http://git.kernel.org/tip/b06eb0173ef1d9366d14b4ca3e8e38dc72b03e8b Author: Christian Ruppert AuthorDate: Tue, 25 Jun 2013 18:29:57 +0200 Committer: Thomas Gleixner CommitDate: Tue, 25 Jun 2013 18:54:21 +0200 irqchip: Add

[PATCH V4] irqchip: Add TB10x interrupt controller driver

2013-06-25 Thread Christian Ruppert
The SOC interrupt controller driver for the Abilis Systems TB10x series of SOCs based on ARC700 CPUs. Signed-off-by: Christian Ruppert Signed-off-by: Pierrick Hascoet --- This patch depends on commits eb76bdd407d8a90e59a06cb0158886df390e5d1c and 712bc93df9e7f14b8a163148d2aa7c778e151627 from

Re: [PATCH V3] irqchip: Add TB10x interrupt controller driver

2013-06-25 Thread Christian Ruppert
On Tue, Jun 25, 2013 at 03:58:43PM +0200, Thomas Gleixner wrote: > On Fri, 31 May 2013, Christian Ruppert wrote: > > The SOC interrupt controller driver for the Abilis Systems TB10x series of > > SOCs based on ARC700 CPUs. > > > > Th

Re: [PATCH V3] irqchip: Add TB10x interrupt controller driver

2013-06-25 Thread Christian Ruppert
interrupts and the rest of the system. Do you see a better way to drive this kind of hardware? Do you have any other comments on the driver? Without this driver, arch/arc/plat-tb10x and related drivers will not work and it would thus be good to

Re: [PATCH 2/2] Make non-linear GPIO ranges accesible from gpiolib

2013-06-25 Thread Christian Ruppert
On Fri, Jun 21, 2013 at 03:17:33PM -0600, Stephen Warren wrote: > On 06/20/2013 05:57 AM, Christian Ruppert wrote: > > Hello Stephen, > > > > On Wed, Jun 19, 2013 at 12:27:56PM -0600, Stephen Warren wrote: > >> On 06/19/2013 12:10 PM, Stephen Warren wrote: > >

[PATCH v9] i2c-designware: make SDA hold time configurable

2013-06-21 Thread Christian Ruppert
This patch makes the SDA hold time configurable through device tree. Signed-off-by: Christian Ruppert Signed-off-by: Pierrick Hascoet --- .../devicetree/bindings/i2c/i2c-designware.txt | 14 ++ arch/arc/boot/dts/abilis_tb100_dvk.dts | 10 +- arch/arc

Re: [PATCH 2/2] Make non-linear GPIO ranges accesible from gpiolib

2013-06-20 Thread Christian Ruppert
Hello Stephen, On Wed, Jun 19, 2013 at 12:27:56PM -0600, Stephen Warren wrote: > On 06/19/2013 12:10 PM, Stephen Warren wrote: > > On 06/14/2013 03:12 AM, Christian Ruppert wrote: > >> On Thu, Jun 13, 2013 at 03:38:09PM -0600, Stephen Warren wrote: > >>> On 06/13/20

Re: [PATCH REBASE] i2c-designware: make SDA hold time configurable

2013-06-20 Thread Christian Ruppert
tion of three factors: . Drive strength . Bus load (Fan out) . Internal timings of all devices on the bus (setup/hold times etc.) This applies to ASIC internal timing. My PCB experience is quite limited and I'm not sure if this can be applied as such. That said, the I2C specification does

Re: [PATCH REBASE] i2c-designware: make SDA hold time configurable

2013-06-19 Thread Christian Ruppert
On Wed, Jun 19, 2013 at 11:45:40AM +0200, Wolfram Sang wrote: > Hi, > > On Wed, Jun 12, 2013 at 04:47:45PM +0200, Christian Ruppert wrote: > > On Mon, Jun 10, 2013 at 05:29:55PM +0200, Wolfram Sang wrote: > > > On Tue, May 14, 2013 at 03:04:02PM +0200, Christian Ruppert w

[PATCH 1/4] Make non-linear GPIO ranges accesible from gpiolib

2013-06-18 Thread Christian Ruppert
This patch adds the infrastructure required to register non-linear gpio ranges through gpiolib and the standard GPIO device tree bindings. Signed-off-by: Christian Ruppert --- Documentation/devicetree/bindings/gpio/gpio.txt | 44 + drivers/gpio/gpiolib-of.c

[PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-06-18 Thread Christian Ruppert
The pinmux driver of the Abilis Systems TB10x platform based on ARC700 CPUs. Used to control the pinmux and is a prerequisite for the GPIO driver. Signed-off-by: Christian Ruppert Signed-off-by: Pierrick Hascoet --- .../bindings/pinctrl/abilis,tb10x-iomux.txt| 81 ++ drivers/pinctrl

[PATCH 3/4] GPIO: Add TB10x GPIO driver

2013-06-18 Thread Christian Ruppert
The GPIO driver for the Abilis Systems TB10x series of SOCs based on ARC700 CPUs. It supports GPIO control and GPIO interrupt generation. This driver works in conjunction with the TB10x pinctrl driver. Signed-off-by: Sascha Leuenberger Signed-off-by: Christian Ruppert --- .../devicetree

[PATCH 4/4] Add Abilis Systems Sarl to device tree vendor prefixes

2013-06-18 Thread Christian Ruppert
Signed-off-by: Christian Ruppert --- .../devicetree/bindings/vendor-prefixes.txt|1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt index 6931c43..e2a8fff

Re: [PATCH 1/4] Make non-linear GPIO ranges accesible from gpiolib

2013-06-18 Thread Christian Ruppert
On Tue, Jun 18, 2013 at 10:09:47AM +0200, Linus Walleij wrote: > On Mon, Jun 17, 2013 at 6:04 PM, Christian Ruppert > wrote: > > > This patch adds the infrastructure required to register non-linear gpio > > ranges through gpiolib and the standard GPIO device tree bindings.

[PATCH v8] i2c-designware: make SDA hold time configurable

2013-06-18 Thread Christian Ruppert
This patch makes the SDA hold time configurable through device tree. Signed-off-by: Christian Ruppert Signed-off-by: Pierrick Hascoet --- .../devicetree/bindings/i2c/i2c-designware.txt | 14 ++ arch/arc/boot/dts/abilis_tb100_dvk.dts | 10 +- arch/arc

[PATCH 1/4] Make non-linear GPIO ranges accesible from gpiolib

2013-06-17 Thread Christian Ruppert
This patch adds the infrastructure required to register non-linear gpio ranges through gpiolib and the standard GPIO device tree bindings. Signed-off-by: Christian Ruppert --- Documentation/devicetree/bindings/gpio/gpio.txt | 44 ++ arch/arc/boot/dts/abilis_tb100.dtsi

[PATCH 2/4] pinmux: Add TB10x pinmux driver

2013-06-17 Thread Christian Ruppert
The pinmux driver of the Abilis Systems TB10x platform based on ARC700 CPUs. Signed-off-by: Christian Ruppert Signed-off-by: Pierrick Hascoet --- .../bindings/pinctrl/abilis,tb10x-iomux.txt| 81 ++ drivers/pinctrl/Kconfig|4 + drivers/pinctrl/Makefile

[PATCH 3/4] GPIO: Add TB10x GPIO driver

2013-06-17 Thread Christian Ruppert
The GPIO driver for the Abilis Systems TB10x series of SOCs based on ARC700 CPUs. It supports GPIO control and GPIO interrupt generation. Signed-off-by: Sascha Leuenberger Signed-off-by: Christian Ruppert --- .../devicetree/bindings/gpio/abilis,tb10x-gpio.txt | 36 ++ arch/arc/boot/dts

[PATCH 4/4] Add Abilis Systems Sarl to device tree vendor prefixes

2013-06-17 Thread Christian Ruppert
Signed-off-by: Christian Ruppert --- .../devicetree/bindings/vendor-prefixes.txt|1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt index 6931c43..e2a8fff

Re: [PATCH 2/2] Make non-linear GPIO ranges accesible from gpiolib

2013-06-17 Thread Christian Ruppert
On Thu, Jun 13, 2013 at 03:38:09PM -0600, Stephen Warren wrote: > On 06/13/2013 06:55 AM, Christian Ruppert wrote: > [...] > > diff --git a/Documentation/devicetree/bindings/gpio/gpio.txt > > b/Documentation/devicetree/bindings/gpio/gpio.txt > > > +In addition, named g

Re: [PATCH V2] i2c: designware: fix race between subsequent xfers

2013-06-17 Thread Christian Ruppert
On Mon, Jun 17, 2013 at 10:33:36AM +0200, Jean Delvare wrote: > On Mon, 17 Jun 2013 10:19:33 +0200, Christian Ruppert wrote: > > On Fri, Jun 14, 2013 at 04:37:41PM +0200, Wolfram Sang wrote: > > > BTW since I am currently here: i2c-designware-core should be in the > >

Re: [PATCH V2] i2c: designware: fix race between subsequent xfers

2013-06-17 Thread Christian Ruppert
On Fri, Jun 14, 2013 at 04:37:41PM +0200, Wolfram Sang wrote: > On Fri, Jun 07, 2013 at 10:51:23AM +0200, Christian Ruppert wrote: > [...] > > + /* > > +* We must disable the adapter before unlocking the &dev->lock mutex > > +* below. Otherwise the har

Re: [PATCH] pinctrl: update GPIO range doc

2013-06-17 Thread Christian Ruppert
. Thanks for stepping in. Reviewed-by: Christian Ruppert > Cc: Christian Ruppert > Signed-off-by: Linus Walleij > --- > Documentation/pinctrl.txt | 24 +--- > 1 file changed, 21 insertions(+), 3 deletions(-) > > diff --git a/Documentation/pinctrl.txt b/

Re: [PATCH 2/2] Make non-linear GPIO ranges accesible from gpiolib

2013-06-14 Thread Christian Ruppert
On Thu, Jun 13, 2013 at 03:38:09PM -0600, Stephen Warren wrote: > On 06/13/2013 06:55 AM, Christian Ruppert wrote: > > This patch adds the infrastructure required to register non-linear gpio > > ranges through gpiolib and the standard GPIO device tree bindings. > > That&#x

[PATCH] Fix comment on pinctrl_gpio_range.pin_base

2013-06-14 Thread Christian Ruppert
The comment introduced with the recently added pinctrl_gpio_range.pins element was wrong. This corrects it. Thanks to Patrice Chotard for pointing this out. Signed-off-by: Christian Ruppert --- include/linux/pinctrl/pinctrl.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff

  1   2   >