From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jochen Friedrich Subject: Re: [PATCH] rtc-pcf8563: Add device ids table Date: Wed, 23 Apr 2008 15:26:34 +0200 Message-ID: <480F390A.90707@scram.de> References: <200804171708.32847.laurentp@cse-semaphore.com> <200804221612.06745.laurentp@cse-semaphore.com> <20080423131657.71d6312f@hyperion.delvare> <200804231412.40038.laurentp@cse-semaphore.com> <20080423144756.3fc9b1bf@hyperion.delvare> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20080423144756.3fc9b1bf@hyperion.delvare> 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: Jean Delvare Cc: Alessandro Zummo , rtc-linux@googlegroups.com, linuxppc-dev list , i2c@lm-sensors.org, Andrew Morton List-Id: linux-i2c@vger.kernel.org Hi Jean, > Jochen, I'm a bit confused by the dependencies that exist - or not - > between these 7 patches you sent at once. I thought they had to be > applied in sequence but it seems not? And some of them should > apparently go through me i2c tree but others (e.g. [7/7]) not? [1/7] and [2/7] are forward ports of patches from you. I'm currently just running a make allmodconfig compile to check if it really caught all affected i2c drivers. These are the patches we are talking about. [3/7] is the OF helper stuff which translates between OF names and i2c types. It does NOT translate OF names to module names, but relies on [1/7] to do so. Without [1/7], [3/7] still applies, but module auto loading won't work. This has been applied to 2.6.26. > I would appreciate if you could summarize quickly which patches depend > on others in which way. If we can make smaller subsets of patches, that > will be easier for me to review and push upstream on my limited time. [4/7] is the patch from Jon Smirl to convert i2c-mpc to OF. This relies on [3/7] instead of Jons initial OF-autoloading patches you didn't like. As there were no comments at all, I would however postpone this to 2.6.27. [5/7] is the cleanup patch (originating from you) which completely removes old driver matching scheme. This should probably wait until 2.6.27, as well. This relies on [1/7] and [2/7]. [6/7] and [7/7] are a new driver and depend on [3/7]. This can wait for 2.6.27, as well (i would rather like to see those in -mm to get more testing, though). Thanks, Jochen