[PATCH] Add Amstrad Delta NAND support.
Jörn Engel
joern at wohnheim.fh-wedel.de
Thu May 18 12:57:28 EDT 2006
On Thu, 18 May 2006 17:09:41 +0100, Jonathan McDowell wrote:
>
> +static struct mtd_info *ams_delta_mtd = NULL;
> + switch(cmd){
^ ^
Add spaces
> + omap_writew(0, (OMAP_MPUIO_BASE + OMAP_MPUIO_IO_CNTL));
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Could that be done in a macro?
> + udelay(0.04);
Floating point in the kernel?
> + ams_delta_mtd = kmalloc (sizeof(struct mtd_info) +
^
> + sizeof (struct nand_chip), GFP_KERNEL);
Remove space
And please create a structure containing both struct mtd_info and
struct nand_chip. Then use sizeof(that structure)...
> + /* Get pointer to private data */
> + this = (struct nand_chip *) (&ams_delta_mtd[1]);
...and remove this cast.
> + /* Initialize structures */
> + memset((char *) ams_delta_mtd, 0, sizeof(struct mtd_info));
> + memset((char *) this, 0, sizeof(struct nand_chip));
And those as well, while you're at it.
> + if (nand_scan (ams_delta_mtd, 1)) {
^
> + kfree (ams_delta_mtd);
^
> +static void __exit ams_delta_cleanup (void)
^
> + nand_release (ams_delta_mtd);
^
> + kfree (ams_delta_mtd);
^
Jörn
--
Happiness isn't having what you want, it's wanting what you have.
-- unknown
More information about the linux-mtd
mailing list