[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sat, 10 Oct 2015 08:20:50 +0100
From: Wolfram Sang <wsa@...-dreams.de>
To: Krzysztof Kozlowski <k.kozlowski@...sung.com>
Cc: linux-sh@...r.kernel.org, k.kozlowski.k@...il.com,
linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org,
Kukjin Kim <kgene@...nel.org>, linux-i2c@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v2 3/4] i2c: s3c2410: enable RuntimePM before registering
to the core
On Sat, Oct 10, 2015 at 03:08:52PM +0900, Krzysztof Kozlowski wrote:
> W dniu 09.10.2015 o 18:39, Wolfram Sang pisze:
> > From: Wolfram Sang <wsa+renesas@...g-engineering.com>
> >
> > The core may register clients attached to this master which may use
> > funtionality from the master. So, RuntimePM must be enabled before, otherwise
> > this will fail. While here, move drvdata, too.
> >
> > Signed-off-by: Wolfram Sang <wsa+renesas@...g-engineering.com>
> > ---
> > drivers/i2c/busses/i2c-s3c2410.c | 12 +++++++-----
> > 1 file changed, 7 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c
> > index 50bfd8cef5f224..336e34e1ca597d 100644
> > --- a/drivers/i2c/busses/i2c-s3c2410.c
> > +++ b/drivers/i2c/busses/i2c-s3c2410.c
> > @@ -1243,19 +1243,21 @@ static int s3c24xx_i2c_probe(struct platform_device *pdev)
> > i2c->adap.nr = i2c->pdata->bus_num;
> > i2c->adap.dev.of_node = pdev->dev.of_node;
> >
> > + platform_set_drvdata(pdev, i2c);
> > +
> > + pm_runtime_enable(&pdev->dev);
> > + pm_runtime_enable(&i2c->adap.dev);
> > +
> > ret = i2c_add_numbered_adapter(&i2c->adap);
> > if (ret < 0) {
> > dev_err(&pdev->dev, "failed to add bus to i2c core\n");
> > + pm_runtime_disable(&i2c->adap.dev);
> > + pm_runtime_disable(&pdev->dev);
> > s3c24xx_i2c_deregister_cpufreq(i2c);
> > clk_unprepare(i2c->clk);
> > return ret;
> > }
> >
> > - platform_set_drvdata(pdev, i2c);
> > -
> > - pm_runtime_enable(&pdev->dev);
> > - pm_runtime_enable(&i2c->adap.dev);
> > -
> > dev_info(&pdev->dev, "%s: S3C I2C adapter\n", dev_name(&i2c->adap.dev));
> > return 0;
> > }
> >
>
> Hi,
>
> Unfortunately on Odroid XU3, Exynos5420, samsung,s3c2440-i2c,
> next-20151009-1-ga93b1a6f5055:
Thanks for testing! Will send v3 in a minute.
Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)
Powered by blists - more mailing lists