Led驱动Platform设备注册后无法调用probe函数

fsjack

做了个简单的led驱动，网上和书上都说device和driver注册后，如果name匹配的话就会调用probe进行加载，可是当我注册了device和driver后没有调用probe，不知道为什么，请各位大大帮忙看看，感激不尽！：）

led.h

1. #include <linux/device.h>
   
2. #include <linux/module.h>
   
3. #include <linux/kernel.h>
   
4. #include <linux/miscdevice.h>
   
5. #include <linux/fs.h>
   
6. #include <linux/slab.h>
   
7. #include <linux/err.h>
   
8. #include <asm/arch/regs-gpio.h>
   
9. #include <asm/arch/hardware.h>
   
10. #include <asm/hardware.h>
    
11. #include <asm/delay.h>
    
12. 
    
13. #define LED_MINOR 0
    
14. #define LED_DRIVER "Led Driver For Test"
    
15. #define DEV_NAME "LED"
    
16. 
    
17. static int led_open (struct inode *node, struct file *filp);
    
18. static int led_ioctl (struct inode *node, struct file *filp, unsigned int cmd, unsigned long arg);
    
19. static ssize_t led_write (struct file *filp, const char __user *buf, size_t count, loff_t *f_ops);
    
20. static int led_probe(struct device * dev);
    
21. static int led_remove(struct device * dev);
    
22. static int led_release (struct inode *node, struct file *filp);
    
23. 
    
24. static struct file_operations led_fops = {
    
25.     .owner  =   THIS_MODULE,
    
26.     .open   =   led_open,
    
27.     .write  =   led_write,
    
28.     .ioctl  =   led_ioctl,
    
29.     .release    =   led_release,
    
30. };
    
31. 
    
32. static struct miscdevice led_miscdev = {
    
33.     MISC_DYNAMIC_MINOR,
    
34.     DEV_NAME,
    
35.     &led_fops,
    
36. };
    
37. 
    
38. struct platform_device *led_dev;
    
39. static struct device_driver led_driver = {
    
40.     .name   =   DEV_NAME,
    
41.     .bus    =   &platform_bus_type,
    
42.     .probe  =   led_probe,
    
43.     .remove =   led_remove,
    
44. };
    
45. 
    
46. static unsigned long led_table[] = {
    
47.     S3C2410_GPF4,
    
48.     S3C2410_GPF5,
    
49.     S3C2410_GPF6,
    
50.     S3C2410_GPF7,
    
51. };
    
52. 
    
53. static unsigned long led_cfg_table[] = {
    
54.     S3C2410_GPF4_OUTP,
    
55.     S3C2410_GPF5_OUTP,
    
56.     S3C2410_GPF6_OUTP,
    
57.     S3C2410_GPF7_OUTP,
    
58. };
    
59. 
    
60. 
    

复制代码

led.c

1. #include "led.h"
   
2. 
   
3. static void led_on_off(int led_num,int led_on_off){
   
4.     //1 for on,2 for off
   
5.     if (led_num>ARRAY_SIZE(led_table)-1)
   
6.         return;
   
7. 
   
8.     s3c2410_gpio_setpin(led_table[led_num],led_on_off );
   
9. }
   
10. 
    
11. static void all_led_on(void){
    
12.     int i;
    
13.     for(i=0;i<ARRAY_SIZE(led_table);i++){
    
14.         s3c2410_gpio_setpin(led_table[i],0);
    
15.     }
    
16. }
    
17. 
    
18. static void all_led_out(void){
    
19.     int i;
    
20.     for(i=0;i<ARRAY_SIZE(led_table);i++){
    
21.         s3c2410_gpio_setpin(led_table[i],1);
    
22.     }
    
23. }
    
24. 
    
25. static void leds_init(void){
    
26.     int i;
    
27.     for(i=0;i<ARRAY_SIZE(led_table);i++){
    
28.         s3c2410_gpio_cfgpin(led_table[i],led_cfg_table[i]);
    
29.     }
    
30. }
    
31.    
    
32. 
    
33. static ssize_t led_write (struct file *filp, const char __user *buf, size_t count, loff_t *f_ops){
    
34.     return 0;
    
35. }
    
36. 
    
37. static int led_ioctl (struct inode *node, struct file *filp, unsigned int cmd, unsigned long arg){
    
38.     return 0;
    
39. }
    
40. 
    
41. 
    
42. static int led_open (struct inode *node, struct file *filp){
    
43.     printk("led driver is opened.\n");
    
44.     return 0;
    
45. }
    
46. 
    
47. static int led_release (struct inode *node, struct file *filp){
    
48.     printk("led driver is closed.\n");
    
49.     return 0;
    
50. }
    
51. 
    
52. static int led_probe(struct device * dev){
    
53.     misc_register(&led_miscdev);
    
54.     printk("%s is installed.\n",LED_DRIVER);
    
55. 
    
56.     all_led_on();
    
57.    
    
58.     printk("%s probe!\n",DEV_NAME);
    
59.     return 0;
    
60. }
    
61. 
    
62. static int led_remove(struct device * dev){
    
63.     all_led_out();
    
64.    
    
65.     if(misc_deregister(&led_miscdev))
    
66.       printk("misc device is deregister!\n");
    
67. 
    
68.     printk("%s remove!\n",DEV_NAME);
    
69.     return 0;
    
70. }
    
71. 
    
72. static int __init led_init(void){
    
73.     int ret;
    
74. 
    
75.     led_dev = platform_device_register_simple(DEV_NAME,-1,NULL,0);
    
76.     if(IS_ERR(led_dev)){
    
77.         printk("failed to register device %s",DEV_NAME);
    
78.         return -EINVAL;
    
79.     }
    
80.     printk("Device is sucessfully registered.\n");
    
81. 
    
82.     ret = driver_register(&led_driver);
    
83.     if(ret){
    
84.         platform_device_unregister(led_dev);
    
85.         printk("failed to register driver %s",LED_DRIVER);
    
86.         return -EINVAL;
    
87.     }
    
88.     printk("Driver is sucessfully registered.\n");
    
89. 
    
90. 
    
91.     leds_init();
    
92.     return 0;
    
93. }
    
94. 
    
95. static void __exit led_exit(void){
    
96.     driver_unregister(&led_driver);
    
97.     platform_device_unregister(led_dev);
    
98. 
    
99.     printk("led modules exit!\n");
    
100. }
     
101. 
     
102. 
     
103. module_init(led_init);
     
104. module_exit(led_exit);
     
105. 
     
106. MODULE_AUTHOR("Jackie Cheung");
     
107. MODULE_LICENSE("GPL");
     

复制代码

fsjack

顶上去~！

amarant

对着例子模仿下吧，要不多加点printk看看跑了哪些，多调一下就明白了~

（不愿仔细看代码,)

jn200002

你从内核拷贝个led.c的驱动，然后把所有的都删掉。


然后留下注册，和probe然后一步一步加你想要的功能.


不愿意这样做，你可以在你所有的函数里加printk，像这样贴一篇代码上来，大多数人只看个驱动名称，是led，愿意再看的就几乎没有了。

lemin9538

我觉得你应该用platform_driver_register()注册相对应的平台驱动结构体，使他们属于platform总线下。而不是driver_register()，使用driver_register()后他们不是属于同一总线下，是不会匹配的吧？

Trigger_Huang

首先，你应该定义的是struct platform_device 结构体对象，而不是指针，并把把它的name赋值
其次你应该定义的是   struct platform_driver 结构体对象，而不是device_driver的对象，且要对其中的.driver.name赋值一样的名字，或者为id_table赋值。
第三，应该是在init函数中用platform_device_register以及platform_driver_register进行注册。device和device_driver只是虚基类，一般不会直接引用的，像platform, PCI USB等都是继承的该类。

Trigger_Huang

而且最好将device和driver分开编写成两个模块，然后先insmod device 然后再insmod driver。写在一起的这种做法貌似不是很推荐。

whjzbp

如果是arm核的话，那要在/arch/arm/mach-xxxx/devs.c中添加struct platform_device  my_device_led,, 然后在/arch/arm/mach-xxxx/mach-xxxx.c中的static struct platform_device *smdkxxx_devices[] __initdata = {中添加&my_device_led

请参考可以否

jounehou

看来楼主的麻烦还是比较多的 。。最好多加点打印信息哦。。看走到哪里去了。还有 我不明白的是 楼主是把这个驱动编译成模块的 还是直接编进内核的呢？

fsjack

非常感谢各位！
原来Linux设备模型只适用于编译进内核么？就是说platform_device和platform_driver的形式来写驱动不适用于编译成模块再insmod加载？

另外我用的是FS2410开发板，预装Linux2.6.8，所以我懒得移植新版内核就用这个了，而这个内核是没有Platform_driver的，所以就用device_driver了。