Mercurial > pt1
diff driver/pt1_pci.c @ 36:65c8ac567074
cleaning up:
- eliminate warnings
- remove dangling white spaces
- changed new line code in Makefile from CR+LF to LF
author | Yoshiki Yazawa <yaz@honeyplanet.jp> |
---|---|
date | Fri, 13 Mar 2009 14:32:44 +0900 |
parents | 289794dc265f |
children | c359e7adf700 |
line wrap: on
line diff
--- a/driver/pt1_pci.c Fri Mar 13 14:29:56 2009 +0900 +++ b/driver/pt1_pci.c Fri Mar 13 14:32:44 2009 +0900 @@ -17,11 +17,11 @@ #include <asm/uaccess.h> #include <linux/version.h> #include <linux/mutex.h> -#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,23) +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,23) #include <linux/freezer.h> #else #define set_freezable() -#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,11) +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,11) typedef struct pm_message { int event; } pm_message_t; @@ -69,7 +69,7 @@ #define MAX_READ_BLOCK 4 // 1度に読み出す最大DMAバッファ数 #define MAX_PCI_DEVICE 128 // 最大64枚 #define DMA_SIZE 4096 // DMAバッファサイズ -#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,22) +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,22) #define DMA_RING_SIZE 128 // RINGサイズ #define DMA_RING_MAX 511 // 1RINGにいくつ詰めるか(1023はNGで511まで) #define CHANEL_DMA_SIZE (2*1024*1024) // 地デジ用(16Mbps) @@ -182,7 +182,6 @@ int lp ; int chno ; int lp2 ; - __u32 addr ; __u32 *dataptr ; __u32 *curdataptr ; __u32 val ; @@ -328,7 +327,6 @@ } static int pt1_release(struct inode *inode, struct file *file) { - int minor = iminor(inode); PT1_CHANNEL *channel = file->private_data; mutex_lock(&channel->ptr->lock); @@ -353,7 +351,7 @@ { PT1_CHANNEL *channel = file->private_data; __u32 size ; - + unsigned long dummy; // 4K単位で起こされるのを待つ(CPU負荷対策) if(channel->size < DMA_SIZE){ @@ -367,12 +365,12 @@ if(cnt < channel->size){ // バッファが足りない場合は残りを移動する size = cnt ; - copy_to_user(buf, channel->buf, cnt); + dummy = copy_to_user(buf, channel->buf, cnt); memmove(channel->buf, &channel->buf[cnt], (channel->size - cnt)); channel->size -= cnt ; }else{ size = channel->size ; - copy_to_user(buf, channel->buf, size); + dummy = copy_to_user(buf, channel->buf, size); channel->size = 0 ; } } @@ -433,16 +431,18 @@ } return 0 ; } -static int pt1_ioctl(struct inode *inode, struct file *file, unsigned int cmd, void *arg) +static int pt1_ioctl(struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg0) { PT1_CHANNEL *channel = file->private_data; - int signal ; + int signal ; + unsigned long dummy; + void *arg = (void *)arg0; switch(cmd){ case SET_CHANNEL: { FREQUENCY freq ; - copy_from_user(&freq, arg, sizeof(FREQUENCY)); + dummy = copy_from_user(&freq, arg, sizeof(FREQUENCY)); return SetFreq(channel, &freq); } case START_REC: @@ -462,11 +462,11 @@ break ; case CHANNEL_TYPE_ISDB_T: // calc C/N - signal = isdb_t_read_signal_strength(channel->ptr->regs, + signal = isdb_t_read_signal_strength(channel->ptr->regs, &channel->ptr->lock, channel->address); break ; } - copy_to_user(arg, &signal, sizeof(int)); + dummy = copy_to_user(arg, &signal, sizeof(int)); return 0 ; case LNB_ENABLE: if(lnb){ @@ -578,6 +578,7 @@ u16 cmd ; PT1_DEVICE *dev_conf ; PT1_CHANNEL *channel ; + struct resource *dummy; rc = pci_enable_device(pdev); if (rc) @@ -608,8 +609,8 @@ // PCIアドレスをマップする dev_conf->mmio_start = pci_resource_start(pdev, 0); dev_conf->mmio_len = pci_resource_len(pdev, 0); - rc = request_mem_region(dev_conf->mmio_start, dev_conf->mmio_len, DEV_NAME); - if (!rc) { + dummy = request_mem_region(dev_conf->mmio_start, dev_conf->mmio_len, DEV_NAME); + if (!dummy) { printk(KERN_ERR "PT1: cannot request iomem (0x%llx).\n", (unsigned long long) dev_conf->mmio_start); goto out_err_regbase; } @@ -642,9 +643,9 @@ } // 初期化完了 for(lp = 0 ; lp < MAX_CHANNEL ; lp++){ - set_sleepmode(dev_conf->regs, &dev_conf->lock, + set_sleepmode(dev_conf->regs, &dev_conf->lock, i2c_address[lp], channeltype[lp], TYPE_SLEEP); - + schedule_timeout_interruptible(msecs_to_jiffies(50)); } rc = alloc_chrdev_region(&dev_conf->dev, 0, MAX_CHANNEL, DEV_NAME); @@ -839,6 +840,7 @@ static int __init pt1_pci_init(void) { + printk(version); pt1video_class = class_create(THIS_MODULE, DRIVERNAME); if (IS_ERR(pt1video_class)) return PTR_ERR(pt1video_class);