From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Welling <mwelling-EkmVulN54Sk@public.gmane.org>
Subject: Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support
Date: Wed, 10 Feb 2016 10:36:22 -0600
Message-ID: <20160210163622.GA1390@deathstar>
References: <20160205172457.GA16778@deathstar>
 <CAEnQRZAjgn3Y0618MgO284DMc_9cE49b9jeUr0j2HwUK6YK+0A@mail.gmail.com>
 <20160206003245.GA1329@deathstar>
 <20160208102459.GC2220@tetsubishi>
 <20160208163632.GA3630@deathstar>
 <01fd01d162a4$8e28c450$aa7a4cf0$@codeaurora.org>
 <20160208191658.GA23145@deathstar>
 <001b01d163fd$bfb15280$3f13f780$@codeaurora.org>
 <CAEnQRZCzkq5wFsS7Ydby+bsFJTbGVq=AX9TLkkhfMKOG8gAZHA@mail.gmail.com>
 <003201d16415$0eb1a8f0$2c14fad0$@codeaurora.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: QUOTED-PRINTABLE
Return-path: <linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Content-Disposition: inline
In-Reply-To: <003201d16415$0eb1a8f0$2c14fad0$@codeaurora.org>
Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
To: Sricharan <sricharan-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
Cc: 'Daniel Baluta' <daniel.baluta-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, 'Wolfram Sang' <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>, 'Jonathan Cameron' <jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, 'Hartmut Knaack' <knaack.h-Mmb7MZpHnFY@public.gmane.org>, 'Lars-Peter Clausen' <lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org>, 'Peter Meerwald-Stadler' <pmeerw-jW+XmwGofnusTnJN9+BGXg@public.gmane.org>, 'Linux Kernel Mailing List' <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, 'Lucas De Marchi' <lucas.demarchi-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, 'Andy Gross' <agross-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>, 'Pramod Gurav' <pramod.gurav-edOiRQu9Xnj5XLMNweQjbQ@public.gmane.org>, 'Bjorn Andersson' <bjorn.andersson-/MT0OVThwyLZJqsBc5GL+g@public.gmane.org>, 'Guenter Roeck' <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>, eibach-dJ+jgKLZIg4@public.gmane.org, linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
List-Id: linux-arm-msm@vger.kernel.org

On Wed, Feb 10, 2016 at 08:39:04PM +0530, Sricharan wrote:
> > Hi Sricharan,
> >=20
> > Are you looking at pca9685_pwm_probe in drivers/pwm/pwm-pca9685.c
> > right?
> >
>  Yes.
> =20
> > I'm asking this because this driver doesn't seem to support runtime=
 pm and
> > there is no check for regmap_write/regmap_write return code in the =
probe
> > function.
>    Hmm to be clear, so it=E2=80=99s the pm_runtime_getsync from i2c-q=
up which fails right ?
>    I was tracking that when there are i2c_xfers from pwm. I did not s=
ee any failures there.
>    So wanted to know the correct sequence to reproduce.
>

The problem was discovered using the patch that this thread is on. The =
PWM driver does
not have the problem.

When the driver in this patch called pm_runtime_get_sync you got -EINVA=
L back.

> Regards,
>   Sricharan
>  =20
>=20
>=20