免费注册 查看新帖 |

Chinaunix

  平台 论坛 博客 文库
最近访问板块 发新帖
查看: 7651 | 回复: 4
打印 上一主题 下一主题

watchdog驱动insmod: No such device 跪求解决方法 [复制链接]

论坛徽章:
0
跳转到指定楼层
1 [收藏(0)] [报告]
发表于 2011-10-11 10:03 |只看该作者 |倒序浏览
ATMEL ARM at91sam9g45 的处理器
watchdog是AT91SAM9G45自带的
以模块形式在rsS中加载的
看门狗驱动编译正常,insmod时提示No such device

输出结果是:
AT91SAM9 Watchdog:before platform_driver_probe
insmod: can't insert 'at91sam9_wdt.ko': No such device

完整代码如下:

#include <linux/errno.h>
#include <linux/fs.h>
#include <linux/init.h>
#include <linux/io.h>
#include <linux/kernel.h>
#include <linux/miscdevice.h>
#include <linux/module.h>
#include <linux/moduleparam.h>
#include <linux/platform_device.h>
#include <linux/types.h>
#include <linux/watchdog.h>
#include <linux/jiffies.h>
#include <linux/timer.h>
#include <linux/bitops.h>
#include <linux/uaccess.h>

#include <mach/at91_wdt.h>

#define DRV_NAME "AT91SAM9 Watchdog"

/* AT91SAM9 watchdog runs a 12bit counter @ 256Hz,
* use this to convert a watchdog
* value from/to milliseconds.
*/
#define ms_to_ticks(t) (((t << / 1000) - 1)
#define ticks_to_ms(t) (((t + 1) * 1000) >>

/* Hardware timeout in seconds */
#define WDT_HW_TIMEOUT 2

/* Timer heartbeat (500ms) */
#define WDT_TIMEOUT (HZ/2)

/* User land timeout */
#define WDT_HEARTBEAT 15
static int heartbeat = WDT_HEARTBEAT;
module_param(heartbeat, int, 0);
MODULE_PARM_DESC(heartbeat, "Watchdog heartbeats in seconds. "
"(default = " __MODULE_STRING(WDT_HEARTBEAT) "";

static int nowayout = WATCHDOG_NOWAYOUT;
module_param(nowayout, int, 0);
MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started "
"(default=" __MODULE_STRING(WATCHDOG_NOWAYOUT) "";

static void at91_ping(unsigned long data);

static struct {
unsigned long next_heartbeat; /* the next_heartbeat for the timer */
unsigned long open;
char expect_close;
struct timer_list timer; /* The timer that pings the watchdog */
} at91wdt_private;

/* ......................................................................... */


/*
* Reload the watchdog timer. (ie, pat the watchdog)
*/
static inline void at91_wdt_reset(void)
{
at91_sys_write(AT91_WDT_CR, AT91_WDT_KEY | AT91_WDT_WDRSTT);
}

/*
* Timer tick
*/
static void at91_ping(unsigned long data)
{
if (time_before(jiffies, at91wdt_private.next_heartbeat) ||
(!nowayout && !at91wdt_private.open)) {
at91_wdt_reset();
mod_timer(&at91wdt_private.timer, jiffies + WDT_TIMEOUT);
} else
printk(KERN_CRIT DRV_NAME": I will reset your machine !\n";
}

/*
* Watchdog device is opened, and watchdog starts running.
*/
static int at91_wdt_open(struct inode *inode, struct file *file)
{
if (test_and_set_bit(0, &at91wdt_private.open))
return -EBUSY;

at91wdt_private.next_heartbeat = jiffies + heartbeat * HZ;
mod_timer(&at91wdt_private.timer, jiffies + WDT_TIMEOUT);

return nonseekable_open(inode, file);
}

/*
* Close the watchdog device.
*/
static int at91_wdt_close(struct inode *inode, struct file *file)
{
clear_bit(0, &at91wdt_private.open);

/* stop internal ping */
if (!at91wdt_private.expect_close)
del_timer(&at91wdt_private.timer);

at91wdt_private.expect_close = 0;
return 0;
}

/*
* Set the watchdog time interval in 1/256Hz (write-once)
* Counter is 12 bit.
*/
static int at91_wdt_settimeout(unsigned int timeout)
{
unsigned int reg;
unsigned int mr;

/* Check if disabled */
mr = at91_sys_read(AT91_WDT_MR);
if (mr & AT91_WDT_WDDIS) {
printk(KERN_ERR DRV_NAME": sorry, watchdog is disabled\n";
return -EIO;
}

/*
* All counting occurs at SLOW_CLOCK / 128 = 256 Hz
*
* Since WDV is a 12-bit counter, the maximum period is
* 4096 / 256 = 16 seconds.
*/
reg = AT91_WDT_WDRSTEN /* causes watchdog reset */
/* | AT91_WDT_WDRPROC causes processor reset only */
| AT91_WDT_WDDBGHLT /* disabled in debug mode */
| AT91_WDT_WDD /* restart at any time */
| (timeout & AT91_WDT_WDV); /* timer value */
at91_sys_write(AT91_WDT_MR, reg);

return 0;
}

static const struct watchdog_info at91_wdt_info = {
.identity = DRV_NAME,
.options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING,
};

/*
* Handle commands from user-space.
*/
static long at91_wdt_ioctl(struct file *file,
unsigned int cmd, unsigned long arg)
{
void __user *argp = (void __user *)arg;
int __user *p = argp;
int new_value;

switch (cmd) {
case WDIOC_GETSUPPORT:
return copy_to_user(argp, &at91_wdt_info,
sizeof(at91_wdt_info)) ? -EFAULT : 0;

case WDIOC_GETSTATUS:
case WDIOC_GETBOOTSTATUS:
return put_user(0, p);

case WDIOC_KEEPALIVE:
at91wdt_private.next_heartbeat = jiffies + heartbeat * HZ;
return 0;

case WDIOC_SETTIMEOUT:
if (get_user(new_value, p))
return -EFAULT;

heartbeat = new_value;
at91wdt_private.next_heartbeat = jiffies + heartbeat * HZ;

return put_user(new_value, p); /* return current value */

case WDIOC_GETTIMEOUT:
return put_user(heartbeat, p);
}
return -ENOTTY;
}

/*
* Pat the watchdog whenever device is written to.
*/
static ssize_t at91_wdt_write(struct file *file, const char *data, size_t len,
loff_t *ppos)
{
if (!len)
return 0;

/* Scan for magic character */
if (!nowayout) {
size_t i;

at91wdt_private.expect_close = 0;

for (i = 0; i < len; i++) {
char c;
if (get_user(c, data + i))
return -EFAULT;
if (c == 'V') {
at91wdt_private.expect_close = 42;
break;
}
}
}

at91wdt_private.next_heartbeat = jiffies + heartbeat * HZ;

return len;
}

/* ......................................................................... */

static const struct file_operations at91wdt_fops = {
.owner = THIS_MODULE,
.llseek = no_llseek,
.unlocked_ioctl = at91_wdt_ioctl,
.open = at91_wdt_open,
.release = at91_wdt_close,
.write = at91_wdt_write,
};

static struct miscdevice at91wdt_miscdev = {
.minor = WATCHDOG_MINOR,
.name = DRV_NAME, //li change "watchdog",
.fops = &at91wdt_fops,
};

static int __init at91wdt_probe(struct platform_device *pdev)
{
int res;

printk(KERN_INFO DRV_NAME "before 0";
if (at91wdt_miscdev.parent)
return -EBUSY;
at91wdt_miscdev.parent = &pdev->dev;
printk(KERN_INFO DRV_NAME "before settimeout";
/* Set watchdog */
res = at91_wdt_settimeout(ms_to_ticks(WDT_HW_TIMEOUT * 1000));
if (res)
{
printk(KERN_ERR DRV_NAME": sorry, watchdog at91_wdt_settimeout is return\n";
return res;
}

printk(KERN_INFO DRV_NAME "before misc_register";
res = misc_register(&at91wdt_miscdev);
if (res)
{
printk(KERN_ERR DRV_NAME": sorry, watchdog misc_register is return\n");
return res;
}

printk(KERN_INFO DRV_NAME "After misc_register");
at91wdt_private.next_heartbeat = jiffies + heartbeat * HZ;
setup_timer(&at91wdt_private.timer, at91_ping, 0);
mod_timer(&at91wdt_private.timer, jiffies + WDT_TIMEOUT);

printk(KERN_INFO DRV_NAME " enabled (heartbeat=%d sec, nowayout=%d)\n",
heartbeat, nowayout);

return 0;
}

static int __exit at91wdt_remove(struct platform_device *pdev)
{
int res;

res = misc_deregister(&at91wdt_miscdev);
if (!res)
at91wdt_miscdev.parent = NULL;

return res;
}

#ifdef CONFIG_PM

static int at91wdt_suspend(struct platform_device *pdev, pm_message_t message)
{
return 0;
}

static int at91wdt_resume(struct platform_device *pdev)
{
return 0;
}

#else
#define at91wdt_suspend NULL
#define at91wdt_resume NULL
#endif

static struct platform_driver at91wdt_driver = {
.probe = at91wdt_probe,
.remove = __exit_p(at91wdt_remove),
.suspend = at91wdt_suspend,
.resume = at91wdt_resume,
.driver = {
.name = DRV_NAME, // "at91_wdt"
.owner = THIS_MODULE,
},
};


static int __init at91sam_wdt_init(void)
{
printk(KERN_INFO DRV_NAME ":before platform_driver_probe\n");
return platform_driver_probe(&at91wdt_driver, at91wdt_probe);
//return at91wdt_probe(&at91wdt_driver);
}

static void __exit at91sam_wdt_exit(void)
{
platform_driver_unregister(&at91wdt_driver);
}

module_init(at91sam_wdt_init);
module_exit(at91sam_wdt_exit);

MODULE_AUTHOR("Renaud CERRATO <r.cerrato@til-technologies.fr>");
MODULE_DESCRIPTION("Watchdog driver for Atmel AT91SAM9x processors");
MODULE_LICENSE("GPL");
MODULE_ALIAS_MISCDEV(WATCHDOG_MINOR);

论坛徽章:
0
2 [报告]
发表于 2011-10-11 10:11 |只看该作者
分析了下就是:
int __init_or_module platform_driver_probe(struct platform_driver *drv,
                int (*probe)(struct platform_device *))
{
        int retval, code;

        /* temporary section violation during probe() */
        drv->probe = probe;
        retval = code = platform_driver_register(drv);

        /* Fixup that section violation, being paranoid about code scanning
         * the list of drivers in order to probe new devices.  Check to see
         * if the probe was successful, and make sure any forced probes of
         * new devices fail.
         */
        spin_lock(&platform_bus_type.p->klist_drivers.k_lock);
        drv->probe = NULL;
        if (code == 0 && list_empty(&drv->driver.p->klist_devices.k_list))
                retval = -ENODEV;
        drv->driver.probe = platform_drv_probe_fail;
        spin_unlock(&platform_bus_type.p->klist_drivers.k_lock);

        if (code != retval)
                platform_driver_unregister(drv);
        return retval;
}
返回了 -ENODEV

论坛徽章:
17
水瓶座
日期:2013-08-29 12:09:27白羊座
日期:2014-08-07 12:36:42丑牛
日期:2014-07-24 12:44:41寅虎
日期:2014-04-16 16:15:33寅虎
日期:2014-03-12 09:28:43摩羯座
日期:2014-03-06 13:22:04技术图书徽章
日期:2014-03-06 11:34:50天蝎座
日期:2014-01-09 11:31:44寅虎
日期:2013-12-27 17:01:44双子座
日期:2013-12-27 12:32:29双子座
日期:2013-12-25 09:03:33丑牛
日期:2013-12-24 16:18:44
3 [报告]
发表于 2011-10-12 09:21 |只看该作者
回复 1# xiaoli96123

我估计问题应该是出在platform_driver_register中

platform_driver_register()->driver_register()->bus_add_driver()->driver_attach()

在调用driver_attach()时应该在总线上没有找到匹配的设备,所以
  1. if (code == 0 && list_empty(&drv->driver.p->klist_devices.k_list))
  2.                 retval = -ENODEV;
复制代码
这里返回了-ENODEV

LZ应该检查自己注册过去的probe函数是否正确,driver的名称和platform_device(即看门狗)的名称是否一致,还有就是platform_device是否register

论坛徽章:
0
4 [报告]
发表于 2011-10-12 14:53 |只看该作者
谢谢3楼,是我的platform_device没egister。

论坛徽章:
0
5 [报告]
发表于 2016-03-24 22:12 |只看该作者

你有ATMEL ARM at91sam9g45 开发版的资料么,谢谢
您需要登录后才可以回帖 登录 | 注册

本版积分规则 发表回复

  

北京盛拓优讯信息技术有限公司. 版权所有 京ICP备16024965号-6 北京市公安局海淀分局网监中心备案编号:11010802020122 niuxiaotong@pcpop.com 17352615567
未成年举报专区
中国互联网协会会员  联系我们:huangweiwei@itpub.net
感谢所有关心和支持过ChinaUnix的朋友们 转载本站内容请注明原作者名及出处

清除 Cookies - ChinaUnix - Archiver - WAP - TOP