From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jochen Friedrich Subject: Re: build breakage from of i2c helper patch Date: Fri, 18 Apr 2008 14:35:29 +0200 Message-ID: <48089591.608@scram.de> References: <48071778.2000405@scram.de> <18440.34234.28845.7398@cargo.ozlabs.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <18440.34234.28845.7398-UYQwCShxghk5kJ7NmlRacFaTQe2KTcn/@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: i2c-bounces-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org Errors-To: i2c-bounces-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org To: Paul Mackerras Cc: Stephen Rothwell , Kumar Gala , Linux I2C , "linuxppc-dev-mnsaURCQ41sdnm+yROfE0A@public.gmane.org list" List-Id: linux-i2c@vger.kernel.org Hi Paul, > Which way around is this dependency? Do you mean that the patches you > listed depend on the "OF helpers for the i2c API" patch, or that the > OF helpers patch depends on these other ones? The OF helpers patch depends on the patches below to support autoloading i2c modules using the alias mechanism. Otherwise, we need to translate between i2c module type and driver_name in the helper patch (like currently done in fsl_soc.c function of_find_i2c_driver()). However, this old way doesn't really scale well. > >> http://patchwork.ozlabs.org/linuxppc/patch?id=17833 >> http://patchwork.ozlabs.org/linuxppc/patch?id=17834 >> >> or the original ones from Jean Delvare: >> >> http://patchwork.ozlabs.org/linuxppc/patch?id=16282 >> http://patchwork.ozlabs.org/linuxppc/patch?id=16283 >> http://patchwork.ozlabs.org/linuxppc/patch?id=16284 > > What's the best way to fix this now? We need to get a reasonable fix > in before I ask Linux to pull the powerpc tree. Thanks, Jochen _______________________________________________ i2c mailing list i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org http://lists.lm-sensors.org/mailman/listinfo/i2c From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.scram.de (mail0.scram.de [78.47.204.202]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "mail.scram.de", Issuer "scram e.V. CA" (not verified)) by ozlabs.org (Postfix) with ESMTPS id F1477DE1EE for ; Fri, 18 Apr 2008 22:35:44 +1000 (EST) Message-ID: <48089591.608@scram.de> Date: Fri, 18 Apr 2008 14:35:29 +0200 From: Jochen Friedrich MIME-Version: 1.0 To: Paul Mackerras Subject: Re: build breakage from of i2c helper patch References: <48071778.2000405@scram.de> <18440.34234.28845.7398@cargo.ozlabs.ibm.com> In-Reply-To: <18440.34234.28845.7398@cargo.ozlabs.ibm.com> Content-Type: text/plain; charset=ISO-8859-1 Cc: Stephen Rothwell , Linux I2C , "linuxppc-dev@ozlabs.org list" List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi Paul, > Which way around is this dependency? Do you mean that the patches you > listed depend on the "OF helpers for the i2c API" patch, or that the > OF helpers patch depends on these other ones? The OF helpers patch depends on the patches below to support autoloading i2c modules using the alias mechanism. Otherwise, we need to translate between i2c module type and driver_name in the helper patch (like currently done in fsl_soc.c function of_find_i2c_driver()). However, this old way doesn't really scale well. > >> http://patchwork.ozlabs.org/linuxppc/patch?id=17833 >> http://patchwork.ozlabs.org/linuxppc/patch?id=17834 >> >> or the original ones from Jean Delvare: >> >> http://patchwork.ozlabs.org/linuxppc/patch?id=16282 >> http://patchwork.ozlabs.org/linuxppc/patch?id=16283 >> http://patchwork.ozlabs.org/linuxppc/patch?id=16284 > > What's the best way to fix this now? We need to get a reasonable fix > in before I ask Linux to pull the powerpc tree. Thanks, Jochen