From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH 0/4] Add AXP209 GPIO driver Date: Wed, 9 Mar 2016 13:17:50 +0100 Message-ID: <56E0146E.6050102@redhat.com> References: <1457520614-32239-1-git-send-email-maxime.ripard@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1457520614-32239-1-git-send-email-maxime.ripard@free-electrons.com> Sender: linux-gpio-owner@vger.kernel.org To: Maxime Ripard , Linus Walleij , Alexandre Courbot , Lee Jones , Chen-Yu Tsai Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com List-Id: devicetree@vger.kernel.org Hi, On 09-03-16 11:50, Maxime Ripard wrote: > Hi, > > The axp209 PMIC used in combination to some Allwinner SoCs has a bunch > of GPIOs accessible. Some boards use these to control their backlight > or a few LEDs. Thanks for working on this, but IMHO this cannot go upstream like this, the gpio pins on the axp pmics need a pinctrl driver, not a gpio driver. I.E. on the axp209 gpio0 and gpio1 can also be used to output an additional low-noise ldo (so as a regulator), or as an adc input. I've been working on gsl1680 touchscreen support lately and on at least a few a23 tablets, the low-noise ldo is used as AVCC for the touchscreen controller. Now these use an axp223 pmic, but nothing is stopping someone from doing something similar with an axp209 and I think it would be best to support this from day one, rather then hope we can retro-fit this later without breaking dts. > There's supposed to be 4 of them, but the fourth one has a different > configuration register scheme, and I couldn't find any board that was > using this GPIO. It will be probably be supported eventually, but > until then, we support only the first 3 GPIOs. He he, this matches what I've done in u-boot, the 4th gpio indeed is somewhat funky (and the docs are not quite clear on how it works exactly IIRC), leaving it out for now indeed seems for the best. Regards, Hans