Re: arc platform code updates (was Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller)

2014-10-10 Thread Christian Ruppert
On Tue, Oct 07, 2014 at 12:35:29PM +, Vineet Gupta wrote: +CC Guenter Roeck [...] However, the kernel-doc comment for init_machine in mach_desc.h is now slightly confusing (still mentioning device tree). A platform of future can still call of_platform_populate() etc to reparse the

arc platform code updates (was Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller)

2014-10-07 Thread Vineet Gupta
+CC Guenter Roeck On Tuesday 07 October 2014 03:00 PM, Christian Ruppert wrote: Hi Vineet, On Fri, Sep 26, 2014 at 09:20:20AM +0530, Vineet Gupta wrote: Hi Chrisitian, On Monday 22 September 2014 10:53 PM, Christian Ruppert wrote: If there is no designware i2c in any of your platforms,

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-29 Thread Viresh Kumar
On Tue, Sep 23, 2014 at 3:05 AM, Mika Westerberg mika.westerb...@linux.intel.com wrote: arch/arm/boot/dts/spear1310.dtsi arch/arm/boot/dts/spear320.dtsi arch/arm/boot/dts/spear3xx.dtsi arch/arm/boot/dts/spear600.dtsi I think we are good to go with the first three but not sure about the

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-29 Thread Wolfram Sang
On Mon, Sep 22, 2014 at 12:12:07PM +0300, Mika Westerberg wrote: On Sat, Sep 20, 2014 at 11:36:34AM +0200, Wolfram Sang wrote: On Thu, Sep 18, 2014 at 12:26:07PM +0300, Mika Westerberg wrote: From: Carl Peng carlpeng...@gmail.com Add support for AMD version of the DW I2C host

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-29 Thread Mika Westerberg
On Mon, Sep 29, 2014 at 11:24:50PM +0200, Wolfram Sang wrote: On Mon, Sep 22, 2014 at 12:12:07PM +0300, Mika Westerberg wrote: On Sat, Sep 20, 2014 at 11:36:34AM +0200, Wolfram Sang wrote: On Thu, Sep 18, 2014 at 12:26:07PM +0300, Mika Westerberg wrote: From: Carl Peng

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-29 Thread Wolfram Sang
1) They do not apply cleanly on top of other i2c-designware changes I applied (check i2c/for-next I just pushed out). The conflicts might not be hard, but they were not trivial enough for me to do them inbetween other things. I'd be very happy about a repost on top of i2c/for-next. OK,

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-25 Thread Vineet Gupta
Hi Chrisitian, On Monday 22 September 2014 10:53 PM, Christian Ruppert wrote: If there is no designware i2c in any of your platforms, simply ignore my message. From my point of view there is no need to move select COMMON_CLK up to the ARC architecture level. I still ended up moving COMMON_CLK

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-23 Thread Mika Westerberg
On Mon, Sep 22, 2014 at 07:22:52PM +0200, Christian Ruppert wrote: On Mon, Sep 22, 2014 at 02:16:25PM +, Vineet Gupta wrote: On Monday 22 September 2014 07:30 PM, Mika Westerberg wrote: COMMON_CLK is not selected by the ARC architecture in general. However, we do select COMMON_CLK

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-22 Thread Mika Westerberg
On Sat, Sep 20, 2014 at 11:36:34AM +0200, Wolfram Sang wrote: On Thu, Sep 18, 2014 at 12:26:07PM +0300, Mika Westerberg wrote: From: Carl Peng carlpeng...@gmail.com Add support for AMD version of the DW I2C host controller. The device is enumerated from ACPI namespace with ACPI ID

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-22 Thread Christian Ruppert
Dear Mika, On Mon, Sep 22, 2014 at 12:12:07PM +0300, Mika Westerberg wrote: On Sat, Sep 20, 2014 at 11:36:34AM +0200, Wolfram Sang wrote: On Thu, Sep 18, 2014 at 12:26:07PM +0300, Mika Westerberg wrote: From: Carl Peng carlpeng...@gmail.com Add support for AMD version of the DW I2C

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-22 Thread Mika Westerberg
On Mon, Sep 22, 2014 at 01:48:57PM +, Vineet Gupta wrote: On Monday 22 September 2014 05:59 PM, Christian Ruppert wrote: Dear Mika, On Mon, Sep 22, 2014 at 12:12:07PM +0300, Mika Westerberg wrote: On Sat, Sep 20, 2014 at 11:36:34AM +0200, Wolfram Sang wrote: On Thu, Sep 18, 2014 at

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-22 Thread Vineet Gupta
On Monday 22 September 2014 07:30 PM, Mika Westerberg wrote: COMMON_CLK is not selected by the ARC architecture in general. However, we do select COMMON_CLK in the TB10x platform which uses the designware I2C driver so this new dependency is no problem for us. Vineet, Do you see

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-22 Thread Mika Westerberg
On Mon, Sep 22, 2014 at 02:16:25PM +, Vineet Gupta wrote: On Monday 22 September 2014 07:30 PM, Mika Westerberg wrote: COMMON_CLK is not selected by the ARC architecture in general. However, we do select COMMON_CLK in the TB10x platform which uses the designware I2C driver so

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-22 Thread Christian Ruppert
On Mon, Sep 22, 2014 at 02:16:25PM +, Vineet Gupta wrote: On Monday 22 September 2014 07:30 PM, Mika Westerberg wrote: COMMON_CLK is not selected by the ARC architecture in general. However, we do select COMMON_CLK in the TB10x platform which uses the designware I2C driver so

Re: [PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-20 Thread Wolfram Sang
On Thu, Sep 18, 2014 at 12:26:07PM +0300, Mika Westerberg wrote: From: Carl Peng carlpeng...@gmail.com Add support for AMD version of the DW I2C host controller. The device is enumerated from ACPI namespace with ACPI ID AMD0010. Because the core driver needs an input source clock, and this

[PATCH 2/2] i2c: designware: Add support for AMD I2C controller

2014-09-18 Thread Mika Westerberg
From: Carl Peng carlpeng...@gmail.com Add support for AMD version of the DW I2C host controller. The device is enumerated from ACPI namespace with ACPI ID AMD0010. Because the core driver needs an input source clock, and this is not an Intel LPSS device where clocks are provided through