From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [213.79.90.228] (helo=buildserver.ru.mvista.com) by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux)) id 1MYUla-00056v-Mm for linux-mtd@lists.infradead.org; Wed, 05 Aug 2009 00:54:10 +0000 Date: Wed, 5 Aug 2009 04:54:04 +0400 From: Anton Vorontsov To: David Brownell Subject: Re: [PATCH 0/7] Device table matching for SPI subsystem Message-ID: <20090805005404.GA14451@oksana.dev.rtsoft.ru> References: <20090729170345.GA26787@oksana.dev.rtsoft.ru> <200908031916.02880.david-b@pacbell.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <200908031916.02880.david-b@pacbell.net> Cc: David Brownell , linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org, Grant Likely , linuxppc-dev@ozlabs.org, linux-mtd@lists.infradead.org, Jean Delvare , Andrew Morton , David Woodhouse Reply-To: avorontsov@ru.mvista.com List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, Aug 03, 2009 at 07:16:02PM -0700, David Brownell wrote: > On Wednesday 29 July 2009, Anton Vorontsov wrote: > > platform_data is overkill for m25p80 chips, the > > driver only needs to know exact chip model, and that's what device > > tables are for. > > To be fair, the platform_data also supports partitioning > and labeling e.g. for cmdlinepart ... With OF we do partitioning and labling via drivers/mtd/ofpart.c. Though, so far there is no support for OF partitions in the m25p80 driver. I think we'd better put the OF partitions probing into parse_mtd_partitions(), and avoid calling of_mtd_parse_partitons() directly... Thanks, -- Anton Vorontsov email: cbouatmailru@gmail.com irc://irc.freenode.net/bd2 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anton Vorontsov Date: Wed, 05 Aug 2009 00:54:04 +0000 Subject: Re: [lm-sensors] [PATCH 0/7] Device table matching for SPI subsystem Message-Id: <20090805005404.GA14451@oksana.dev.rtsoft.ru> List-Id: References: <20090729170345.GA26787@oksana.dev.rtsoft.ru> <200908031916.02880.david-b@pacbell.net> In-Reply-To: <200908031916.02880.david-b@pacbell.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: David Brownell Cc: Andrew Morton , David Brownell , David Woodhouse , Grant Likely , Jean Delvare , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org, linuxppc-dev@ozlabs.org On Mon, Aug 03, 2009 at 07:16:02PM -0700, David Brownell wrote: > On Wednesday 29 July 2009, Anton Vorontsov wrote: > > platform_data is overkill for m25p80 chips, the > > driver only needs to know exact chip model, and that's what device > > tables are for. > > To be fair, the platform_data also supports partitioning > and labeling e.g. for cmdlinepart ... With OF we do partitioning and labling via drivers/mtd/ofpart.c. Though, so far there is no support for OF partitions in the m25p80 driver. I think we'd better put the OF partitions probing into parse_mtd_partitions(), and avoid calling of_mtd_parse_partitons() directly... Thanks, -- Anton Vorontsov email: cbouatmailru@gmail.com irc://irc.freenode.net/bd2 _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [203.10.76.45]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "mx.ozlabs.org", Issuer "CA Cert Signing Authority" (verified OK)) by bilbo.ozlabs.org (Postfix) with ESMTPS id E015BB7079 for ; Wed, 5 Aug 2009 10:54:08 +1000 (EST) Received: from buildserver.ru.mvista.com (unknown [213.79.90.228]) by ozlabs.org (Postfix) with ESMTP id 67C0ADDD04 for ; Wed, 5 Aug 2009 10:54:07 +1000 (EST) Date: Wed, 5 Aug 2009 04:54:04 +0400 From: Anton Vorontsov To: David Brownell Subject: Re: [PATCH 0/7] Device table matching for SPI subsystem Message-ID: <20090805005404.GA14451@oksana.dev.rtsoft.ru> References: <20090729170345.GA26787@oksana.dev.rtsoft.ru> <200908031916.02880.david-b@pacbell.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: <200908031916.02880.david-b@pacbell.net> Cc: David Brownell , linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org, linuxppc-dev@ozlabs.org, linux-mtd@lists.infradead.org, Jean Delvare , Andrew Morton , David Woodhouse Reply-To: avorontsov@ru.mvista.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, Aug 03, 2009 at 07:16:02PM -0700, David Brownell wrote: > On Wednesday 29 July 2009, Anton Vorontsov wrote: > > platform_data is overkill for m25p80 chips, the > > driver only needs to know exact chip model, and that's what device > > tables are for. > > To be fair, the platform_data also supports partitioning > and labeling e.g. for cmdlinepart ... With OF we do partitioning and labling via drivers/mtd/ofpart.c. Though, so far there is no support for OF partitions in the m25p80 driver. I think we'd better put the OF partitions probing into parse_mtd_partitions(), and avoid calling of_mtd_parse_partitons() directly... Thanks, -- Anton Vorontsov email: cbouatmailru@gmail.com irc://irc.freenode.net/bd2 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932765AbZHEAyH (ORCPT ); Tue, 4 Aug 2009 20:54:07 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754520AbZHEAyF (ORCPT ); Tue, 4 Aug 2009 20:54:05 -0400 Received: from ru.mvista.com ([213.79.90.228]:47575 "EHLO buildserver.ru.mvista.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1753553AbZHEAyE (ORCPT ); Tue, 4 Aug 2009 20:54:04 -0400 Date: Wed, 5 Aug 2009 04:54:04 +0400 From: Anton Vorontsov To: David Brownell Cc: Andrew Morton , David Brownell , David Woodhouse , Grant Likely , Jean Delvare , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org, linuxppc-dev@ozlabs.org Subject: Re: [PATCH 0/7] Device table matching for SPI subsystem Message-ID: <20090805005404.GA14451@oksana.dev.rtsoft.ru> Reply-To: avorontsov@ru.mvista.com References: <20090729170345.GA26787@oksana.dev.rtsoft.ru> <200908031916.02880.david-b@pacbell.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <200908031916.02880.david-b@pacbell.net> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 03, 2009 at 07:16:02PM -0700, David Brownell wrote: > On Wednesday 29 July 2009, Anton Vorontsov wrote: > > platform_data is overkill for m25p80 chips, the > > driver only needs to know exact chip model, and that's what device > > tables are for. > > To be fair, the platform_data also supports partitioning > and labeling e.g. for cmdlinepart ... With OF we do partitioning and labling via drivers/mtd/ofpart.c. Though, so far there is no support for OF partitions in the m25p80 driver. I think we'd better put the OF partitions probing into parse_mtd_partitions(), and avoid calling of_mtd_parse_partitons() directly... Thanks, -- Anton Vorontsov email: cbouatmailru@gmail.com irc://irc.freenode.net/bd2