DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
  s=s1024; d=pacbell.net;
  h=Message-ID:X-YMail-OSG:Received:X-Mailer:Date:From:Subject:To:Cc:In-Reply-To:MIME-Version:Content-Type;
  b=6UZy/9E34L24Amei6BiAoUNqXRxNyB+sHMGMQGvl+sH/vpLBLEEsblrvC1yw/ekX/UO627ji6mS3Z8iF6oTQOS/UwAHOX1VfszSXS2GJt/8xkuy62KVVAIrQD7HtoF35ldwA3002OQAyyRbR9ybrgo3jY2CpQXs6tKCA49nrHhE=;
Message-ID: <54038.80733.qm@web180305.mail.gq1.yahoo.com>
Date: Wed, 7 Jul 2010 04:08:54 -0700 (PDT)
From: David Brownell <david-b@pacbell.net>
Subject: RE: [PATCH 1/2] mtdpart: memory accessor interface for MTD layer
To: "'David Woodhouse'" <dwmw2@infradead.org>,
       "'Kevin Hilman'" <khilman@deeprootsystems.com>,
       Sudhakar Rajashekhara <sudhakar.raj@ti.com>
Cc: "'Bernd Schmidt'" <bernd.schmidt@analog.com>,
       "'Nicolas Pitre'" <nico@fluxnic.net>, linux-kernel@vger.kernel.org,
       "'David Howells'" <dhowells@redhat.com>,
       "'David Brownell'" <dbrownell@users.sourceforge.net>,
       linux-mtd@lists.infradead.org,
       "'Andrew Morton'" <akpm@linux-foundation.org>
In-Reply-To: <012f01cb1dc3$09584880$1c08d980$@raj@ti.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 920
Lines: 29


> > > This patch adds two new members to the
> mtd_partition structure,
> > > a function handler which will be
> > > called during setup of the
> > > partition and an argument to be passed to this
> setup function.
> > 
> > Ick.
> > 
> > I don't mind providing the mtd_macc_{read,write}
> helper functions,
> > although they should work on generic MTD
> > >devices not just on partitions.
> > But do we really have to do the callout to
> > arbitrary functions from
> > _within_ the core MTD code... why can't we
> > do that in the board driver?

I think the short answer is that the callout is
what provides the board drivers enough information
to make the correct calls.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/