On Wed, Aug 5, 2009 at 1:41 PM, Magnus Damm<magnus.damm@xxxxxxxxx> wrote: >> On Tue, Aug 4, 2009 at 6:20 PM, Magnus Damm<magnus.damm@xxxxxxxxx> wrote: >>> From: Magnus Damm <damm@xxxxxxxxxx> >>> >>> This patch removes the ARM dependency from the generic "onenand" >>> platform device driver. This change makes the driver useful for >>> other architectures as well. Needed for the SuperH kfr2r09 board. > > On Wed, Aug 5, 2009 at 1:24 PM, Kyungmin Park<kmpark@xxxxxxxxxxxxx> wrote: >> Good idea add the onenand_platform_data, but dont' agree the renaming >> the onenand-flash. >> Other boards are use it even though it's not released it > > I suspected so. But this is the only reason why changed the name. =) > > If we don't change the name then the platform device will be attached > to the platform driver as usual, but the old platform data structure > will use a different binary format compated to what the driver > expects. > > Changing the driver name makes sure that the old device disappears and > that people can move over to the new name and at the same time update > the platform data to the new format. > >> Others are good. > > Thanks. > > I can post an updated version where I keep the driver name unchanged > if you prefer that. But I'm pretty sure out-of-tree drivers will break > with NULL pointer accesses or similar if we update the platform data > format without changing the name. I don't think you want that. =) Any update on this? I posted a platform device patch for the kfr2r09 board to the linux-sh list yesterday, so with that platform device patch and this driver patch all in-tree users are converted. Thanks in advance, / magnus -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
