Commit ec0482e6 authored by Ben Dooks's avatar Ben Dooks

[MTD] [NAND] S3C2410: Move to using platform device table

Commit 57fee4a5 added an
method to specify the platform device compatibility by using
an id-table instead of registering multiple drivers.

Move the S3C24XX NAND driver to using this ID table.
Signed-off-by: default avatarBen Dooks <ben-linux@fluff.org>
CC: Eric Miao <eric.miao@marvell.com>
parent 3218911f
...@@ -593,7 +593,7 @@ static inline void s3c2410_nand_cpufreq_deregister(struct s3c2410_nand_info *inf ...@@ -593,7 +593,7 @@ static inline void s3c2410_nand_cpufreq_deregister(struct s3c2410_nand_info *inf
/* device management functions */ /* device management functions */
static int s3c2410_nand_remove(struct platform_device *pdev) static int s3c24xx_nand_remove(struct platform_device *pdev)
{ {
struct s3c2410_nand_info *info = to_nand_info(pdev); struct s3c2410_nand_info *info = to_nand_info(pdev);
...@@ -788,18 +788,17 @@ static void s3c2410_nand_update_chip(struct s3c2410_nand_info *info, ...@@ -788,18 +788,17 @@ static void s3c2410_nand_update_chip(struct s3c2410_nand_info *info,
} }
} }
/* s3c2410_nand_probe /* s3c24xx_nand_probe
* *
* called by device layer when it finds a device matching * called by device layer when it finds a device matching
* one our driver can handled. This code checks to see if * one our driver can handled. This code checks to see if
* it can allocate all necessary resources then calls the * it can allocate all necessary resources then calls the
* nand layer to look for devices * nand layer to look for devices
*/ */
static int s3c24xx_nand_probe(struct platform_device *pdev)
static int s3c24xx_nand_probe(struct platform_device *pdev,
enum s3c_cpu_type cpu_type)
{ {
struct s3c2410_platform_nand *plat = to_nand_plat(pdev); struct s3c2410_platform_nand *plat = to_nand_plat(pdev);
enum s3c_cpu_type cpu_type;
struct s3c2410_nand_info *info; struct s3c2410_nand_info *info;
struct s3c2410_nand_mtd *nmtd; struct s3c2410_nand_mtd *nmtd;
struct s3c2410_nand_set *sets; struct s3c2410_nand_set *sets;
...@@ -809,6 +808,8 @@ static int s3c24xx_nand_probe(struct platform_device *pdev, ...@@ -809,6 +808,8 @@ static int s3c24xx_nand_probe(struct platform_device *pdev,
int nr_sets; int nr_sets;
int setno; int setno;
cpu_type = platform_get_device_id(pdev)->driver_data;
pr_debug("s3c2410_nand_probe(%p)\n", pdev); pr_debug("s3c2410_nand_probe(%p)\n", pdev);
info = kmalloc(sizeof(*info), GFP_KERNEL); info = kmalloc(sizeof(*info), GFP_KERNEL);
...@@ -922,7 +923,7 @@ static int s3c24xx_nand_probe(struct platform_device *pdev, ...@@ -922,7 +923,7 @@ static int s3c24xx_nand_probe(struct platform_device *pdev,
return 0; return 0;
exit_error: exit_error:
s3c2410_nand_remove(pdev); s3c24xx_nand_remove(pdev);
if (err == 0) if (err == 0)
err = -EINVAL; err = -EINVAL;
...@@ -983,50 +984,30 @@ static int s3c24xx_nand_resume(struct platform_device *dev) ...@@ -983,50 +984,30 @@ static int s3c24xx_nand_resume(struct platform_device *dev)
/* driver device registration */ /* driver device registration */
static int s3c2410_nand_probe(struct platform_device *dev) static struct platform_device_id s3c24xx_driver_ids[] = {
{ {
return s3c24xx_nand_probe(dev, TYPE_S3C2410); .name = "s3c2410-nand",
} .driver_data = TYPE_S3C2410,
}, {
static int s3c2440_nand_probe(struct platform_device *dev) .name = "s3c2440-nand",
{ .driver_data = TYPE_S3C2440,
return s3c24xx_nand_probe(dev, TYPE_S3C2440); }, {
} .name = "s3c2412-nand",
.driver_data = TYPE_S3C2412,
static int s3c2412_nand_probe(struct platform_device *dev)
{
return s3c24xx_nand_probe(dev, TYPE_S3C2412);
}
static struct platform_driver s3c2410_nand_driver = {
.probe = s3c2410_nand_probe,
.remove = s3c2410_nand_remove,
.suspend = s3c24xx_nand_suspend,
.resume = s3c24xx_nand_resume,
.driver = {
.name = "s3c2410-nand",
.owner = THIS_MODULE,
}, },
{ }
}; };
static struct platform_driver s3c2440_nand_driver = { MODULE_DEVICE_TABLE(platform, s3c24xx_driver_ids);
.probe = s3c2440_nand_probe,
.remove = s3c2410_nand_remove,
.suspend = s3c24xx_nand_suspend,
.resume = s3c24xx_nand_resume,
.driver = {
.name = "s3c2440-nand",
.owner = THIS_MODULE,
},
};
static struct platform_driver s3c2412_nand_driver = { static struct platform_driver s3c24xx_nand_driver = {
.probe = s3c2412_nand_probe, .probe = s3c24xx_nand_probe,
.remove = s3c2410_nand_remove, .remove = s3c24xx_nand_remove,
.suspend = s3c24xx_nand_suspend, .suspend = s3c24xx_nand_suspend,
.resume = s3c24xx_nand_resume, .resume = s3c24xx_nand_resume,
.id_table = s3c24xx_driver_ids,
.driver = { .driver = {
.name = "s3c2412-nand", .name = "s3c24xx-nand",
.owner = THIS_MODULE, .owner = THIS_MODULE,
}, },
}; };
...@@ -1035,16 +1016,12 @@ static int __init s3c2410_nand_init(void) ...@@ -1035,16 +1016,12 @@ static int __init s3c2410_nand_init(void)
{ {
printk("S3C24XX NAND Driver, (c) 2004 Simtec Electronics\n"); printk("S3C24XX NAND Driver, (c) 2004 Simtec Electronics\n");
platform_driver_register(&s3c2412_nand_driver); return platform_driver_register(&s3c24xx_nand_driver);
platform_driver_register(&s3c2440_nand_driver);
return platform_driver_register(&s3c2410_nand_driver);
} }
static void __exit s3c2410_nand_exit(void) static void __exit s3c2410_nand_exit(void)
{ {
platform_driver_unregister(&s3c2412_nand_driver); platform_driver_unregister(&s3c24xx_nand_driver);
platform_driver_unregister(&s3c2440_nand_driver);
platform_driver_unregister(&s3c2410_nand_driver);
} }
module_init(s3c2410_nand_init); module_init(s3c2410_nand_init);
...@@ -1053,6 +1030,3 @@ module_exit(s3c2410_nand_exit); ...@@ -1053,6 +1030,3 @@ module_exit(s3c2410_nand_exit);
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
MODULE_AUTHOR("Ben Dooks <ben@simtec.co.uk>"); MODULE_AUTHOR("Ben Dooks <ben@simtec.co.uk>");
MODULE_DESCRIPTION("S3C24XX MTD NAND driver"); MODULE_DESCRIPTION("S3C24XX MTD NAND driver");
MODULE_ALIAS("platform:s3c2410-nand");
MODULE_ALIAS("platform:s3c2412-nand");
MODULE_ALIAS("platform:s3c2440-nand");
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment