From: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>
Cc: Maxime Ripard
<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>,
linux-arm-kernel
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
linux-sunxi <linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org>
Subject: Re: [linux-sunxi] Re: [PATCH] ARM: dts: sun4i: Add dts file for the pov protab2-ips9 tablet
Date: Tue, 8 Sep 2015 14:49:33 +0200 [thread overview]
Message-ID: <55EED95D.20004@redhat.com> (raw)
In-Reply-To: <CAGb2v64BGPP9Fs0gM0ghJZ0tO8QdGZD9GfpquGCGKiZCeTkyww-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
Hi,
On 09/08/2015 10:40 AM, Chen-Yu Tsai wrote:
> On Tue, Sep 8, 2015 at 3:45 PM, Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:
>> Hi,
>>
>> On 07-09-15 22:56, Maxime Ripard wrote:
>>>
>>> On Mon, Sep 07, 2015 at 09:05:29AM +0200, Hans de Goede wrote:
>>>>>>
>>>>>> +®_ldo3 {
>>>>>> + /*
>>>>>> + * We need to always power the camera sensor, otherwhise all
>>>>>> access
>>>>>> + * to i2c1 is blocked.
>>>>>> + */
>>>>>> + regulator-always-on;
>>>>>> + regulator-min-microvolt = <2800000>;
>>>>>> + regulator-max-microvolt = <2800000>;
>>>>>> + regulator-name = "vdd-csi";
>>>>>> +};
>>>>>
>>>>>
>>>>> What is connected on i2c1 ? Just the camera sensor? or it has some
>>>>> other devices there?
>>>>
>>>>
>>>> The bma250 accelerometer sits there, and the kernel already has a driver
>>>> for it. That driver needs to have devicetree binding support added, and
>>>> then we should be able to use the accelerometer.
>>>
>>>
>>> Ok, so if this regulator is disable, you can't access the other
>>> devices as well, right?
>>
>>
>> Right, the controller reports the bus as being stuck.
>>
>>> Do you know why? Is it the regulator providing
>>> the pull-up voltage?
>>
>>
>> I've tried enabling the pull ups on the SoC i2c pins, so I do not think
>> that it is that, it seems that somehow when not powered the camera sensor is
>> actively keeping the lines low. Either it has multiple power planes, or
>> it is using normally-on fet-s between ground and its i2c lines.
>
> FYI the reference designs use one regulator to power the pull-ups, VCC-PX
> (X for X pin group), and VDDIO (IO power)on the camera sensors. AVDD, DVDD
> (actual power) for the sensors are another (or more) regulators.
But that is a A23 or some such generation thing, right? This is an A10 based
tablet!
Regards,
Hans
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-09-08 12:49 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-05 8:21 [PATCH] ARM: dts: sun4i: Add dts file for the pov protab2-ips9 tablet Hans de Goede
[not found] ` <1441441319-10658-1-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-06 16:30 ` Maxime Ripard
2015-09-07 7:05 ` Hans de Goede
[not found] ` <55ED3739.609-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-07 7:49 ` Priit Laes
[not found] ` <1441612142.14422.34.camel-q/aMd4JkU83YtjvyW6yDsg@public.gmane.org>
2015-09-07 8:49 ` [linux-sunxi] " Hans de Goede
[not found] ` <55ED4F8F.6080502-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-07 9:14 ` Priit Laes
[not found] ` <1441617262.21530.4.camel-q/aMd4JkU83YtjvyW6yDsg@public.gmane.org>
2015-09-07 9:30 ` Hans de Goede
[not found] ` <55ED591B.7050705-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-07 20:52 ` Maxime Ripard
2015-09-08 7:48 ` Hans de Goede
[not found] ` <55EE92E0.1010805-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-10 13:56 ` Maxime Ripard
2015-09-07 20:56 ` Maxime Ripard
2015-09-08 7:45 ` [linux-sunxi] " Hans de Goede
[not found] ` <55EE9230.70209-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-08 8:40 ` Chen-Yu Tsai
[not found] ` <CAGb2v64BGPP9Fs0gM0ghJZ0tO8QdGZD9GfpquGCGKiZCeTkyww-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-09-08 12:49 ` Hans de Goede [this message]
[not found] ` <55EED95D.20004-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-08 13:14 ` Chen-Yu Tsai
2015-09-13 15:22 ` Maxime Ripard
2015-09-13 17:33 ` Hans de Goede
[not found] ` <55F5B370.7040203-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-22 15:02 ` Maxime Ripard
2015-09-22 15:24 ` [linux-sunxi] " Hans de Goede
[not found] ` <560172B5.8090709-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-10-10 12:32 ` Hans de Goede
2015-10-19 19:43 ` Maxime Ripard
2015-10-20 21:59 ` Hans de Goede
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55EED95D.20004@redhat.com \
--to=hdegoede-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
--cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=wens-jdAy2FN1RRM@public.gmane.org \
--cc=wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).