Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-21 Thread Sekhar Nori
On Tuesday 22 August 2017 06:47 AM, Franklin S Cooper Jr wrote: > > > On 08/21/2017 04:05 AM, Sekhar Nori wrote: >> On Thursday 17 August 2017 03:47 AM, Franklin S Cooper Jr wrote: >> >>> @@ -802,7 +821,6 @@ static int davinci_i2c_probe(struct platform_device >>> *pdev) >>> dev->clk =

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-21 Thread Sekhar Nori
On Tuesday 22 August 2017 06:47 AM, Franklin S Cooper Jr wrote: > > > On 08/21/2017 04:05 AM, Sekhar Nori wrote: >> On Thursday 17 August 2017 03:47 AM, Franklin S Cooper Jr wrote: >> >>> @@ -802,7 +821,6 @@ static int davinci_i2c_probe(struct platform_device >>> *pdev) >>> dev->clk =

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-21 Thread Franklin S Cooper Jr
On 08/21/2017 04:05 AM, Sekhar Nori wrote: > On Thursday 17 August 2017 03:47 AM, Franklin S Cooper Jr wrote: > >> @@ -802,7 +821,6 @@ static int davinci_i2c_probe(struct platform_device >> *pdev) >> dev->clk = devm_clk_get(>dev, NULL); >> if (IS_ERR(dev->clk)) >> return

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-21 Thread Franklin S Cooper Jr
On 08/21/2017 04:05 AM, Sekhar Nori wrote: > On Thursday 17 August 2017 03:47 AM, Franklin S Cooper Jr wrote: > >> @@ -802,7 +821,6 @@ static int davinci_i2c_probe(struct platform_device >> *pdev) >> dev->clk = devm_clk_get(>dev, NULL); >> if (IS_ERR(dev->clk)) >> return

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-21 Thread Sekhar Nori
Hi Wolfram, On Thursday 17 August 2017 09:54 PM, Wolfram Sang wrote: > On Wed, Aug 16, 2017 at 05:17:14PM -0500, Franklin S Cooper Jr wrote: >> 66AK2G has I2C instances that are not apart of the ALWAYS_ON power domain >> like other Keystone 2 SoCs and OMAPL138. Therefore, pm_runtime >> is

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-21 Thread Sekhar Nori
Hi Wolfram, On Thursday 17 August 2017 09:54 PM, Wolfram Sang wrote: > On Wed, Aug 16, 2017 at 05:17:14PM -0500, Franklin S Cooper Jr wrote: >> 66AK2G has I2C instances that are not apart of the ALWAYS_ON power domain >> like other Keystone 2 SoCs and OMAPL138. Therefore, pm_runtime >> is

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-21 Thread Sekhar Nori
On Thursday 17 August 2017 03:47 AM, Franklin S Cooper Jr wrote: > @@ -802,7 +821,6 @@ static int davinci_i2c_probe(struct platform_device *pdev) > dev->clk = devm_clk_get(>dev, NULL); > if (IS_ERR(dev->clk)) > return PTR_ERR(dev->clk); > -

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-21 Thread Sekhar Nori
On Thursday 17 August 2017 03:47 AM, Franklin S Cooper Jr wrote: > @@ -802,7 +821,6 @@ static int davinci_i2c_probe(struct platform_device *pdev) > dev->clk = devm_clk_get(>dev, NULL); > if (IS_ERR(dev->clk)) > return PTR_ERR(dev->clk); > -

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-17 Thread Wolfram Sang
On Wed, Aug 16, 2017 at 05:17:14PM -0500, Franklin S Cooper Jr wrote: > 66AK2G has I2C instances that are not apart of the ALWAYS_ON power domain > like other Keystone 2 SoCs and OMAPL138. Therefore, pm_runtime > is required to insure the power domain used by the specific I2C instance is >

Re: [PATCH v2 2/3] i2c: davinci: Add PM Runtime Support

2017-08-17 Thread Wolfram Sang
On Wed, Aug 16, 2017 at 05:17:14PM -0500, Franklin S Cooper Jr wrote: > 66AK2G has I2C instances that are not apart of the ALWAYS_ON power domain > like other Keystone 2 SoCs and OMAPL138. Therefore, pm_runtime > is required to insure the power domain used by the specific I2C instance is >