From mboxrd@z Thu Jan 1 00:00:00 1970 From: Linus Walleij Subject: Re: [PATCH 27/32] gpio: Add support for the FXL6408 GPIO expander. Date: Wed, 8 Jun 2016 11:10:58 +0200 Message-ID: References: <1464817421-8519-1-git-send-email-kraxel@redhat.com> <1464817421-8519-28-git-send-email-kraxel@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <1464817421-8519-28-git-send-email-kraxel@redhat.com> Sender: linux-gpio-owner@vger.kernel.org To: Gerd Hoffmann Cc: linux-rpi-kernel , Eric Anholt , Florian Fainelli , Ray Jui , Scott Branden , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Catalin Marinas , Will Deacon , Alexandre Courbot , "open list:BROADCOM BCM281XX/BCM11XXX/BCM216XX ARM ARCHITE..." , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM64 PORT (AARCH64 ARCHITECTURE)" , open list , "open list:GPIO SUBSYSTEM" List-Id: devicetree@vger.kernel.org On Wed, Jun 1, 2016 at 11:43 PM, Gerd Hoffmann wrote: > +#define FXL6408_OUTPUT 0x05 > +/* Bits here make the output High-Z, instead of the OUTPUT value. */ > +#define FXL6408_OUTPUT_HIGH_Z 0x07 (...) > + /* Disable High-Z of outputs, so that our OUTPUT updates > + * actually take effect. > + */ > + i2c_smbus_write_byte_data(client, FXL6408_OUTPUT_HIGH_Z, 0); High-Z is probably the same as open drain right? And if not open drain, it is push-pull. So you want to implement .set_single_ended() for configuring the lines in "high-z", i.e. open drain output mode. Yours, Linus Walleij