From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Steven A. Falco" Subject: Re: [PATCH 0/7 RFC] Handle I2C GPIO controllers with the OF (was: pca9539 I2C gpio expander) Date: Fri, 17 Oct 2008 12:07:15 -0400 Message-ID: <48F8B833.7080807@harris.com> References: <20081016171222.GA24812@oksana.dev.rtsoft.ru> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20081016171222.GA24812@oksana.dev.rtsoft.ru> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linuxppc-dev-bounces+glppd-linuxppc64-dev=m.gmane.org@ozlabs.org Errors-To: linuxppc-dev-bounces+glppd-linuxppc64-dev=m.gmane.org@ozlabs.org To: avorontsov@ru.mvista.com Cc: David Brownell , linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org, i2c@lm-sensors.org, Jean Delvare , David Miller List-Id: linux-i2c@vger.kernel.org Anton Vorontsov wrote: > Hi all, > > Recently there was a question about I2C GPIO controllers and how should > we handle them with the OpenFirmware and such. > > Here is the attempt to "connect" I2C GPIO controllers to the > "OpenFirmware" device tree, without writing an OF-specific bindings > for each driver. > > The salt is in these two patches: > > [PATCH 3/7] of: fill the archdata for I2C devices > ^ Here we're storing the device tree node into the I2C device. > > [PATCH 5/7] of/gpio: implement of_dev_gpiochip_{add,remove} calls > ^ And here we extracting the the stored node to put the registered > of_gpio_chip into that node. > > > How does it look? I've just tested this with a pca9539 chip attached to my Sequoia board, and it works perfectly for me. I don't have the mcu_mpc8349emitx.c file in my tree, so I was not able to apply part 7 of 7, but the rest looks fine. Thanks for doing this. Acked by: Steve Falco