Re: [linux-sunxi] Re: [PATCH] ARM: dts: sun4i: Add dts file for the pov protab2-ips9 tablet

[Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

Hi,

On 07-09-15 22:56, Maxime Ripard wrote:
> On Mon, Sep 07, 2015 at 09:05:29AM +0200, Hans de Goede wrote:
>>>> +&reg_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.

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