diff options
author | Kevin P. Fleming <kpfleming@digium.com> | 2008-10-28 21:49:02 +0000 |
---|---|---|
committer | Kevin P. Fleming <kpfleming@digium.com> | 2008-10-28 21:49:02 +0000 |
commit | cb2fd65e3da7a68ef11d458e9ca99463f90b32a0 (patch) | |
tree | 8acd0b016433ced01fd64876a031381c822c30aa /drivers/dahdi/wcte11xp.c | |
parent | 8b0e06e5f588d24af0666b1d605084a83c7c8c25 (diff) |
fix a large number of warnings found by sparse, the kernel code sanity checking tool. some of these fixes are non-optimal (casting 'unsigned long' to '__user void *'), but are unavoidable in many cases. started from tzafrir's patch, did most of the work myself.
(closes issue #13763)
Reported by: tzafrir
Patches:
sparse_fixes_1.diff uploaded by tzafrir (license 46)
git-svn-id: http://svn.asterisk.org/svn/dahdi/linux/trunk@5162 a0bf4364-ded3-4de4-8d8a-66a801d63aff
Diffstat (limited to 'drivers/dahdi/wcte11xp.c')
-rw-r--r-- | drivers/dahdi/wcte11xp.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/dahdi/wcte11xp.c b/drivers/dahdi/wcte11xp.c index c314167..7c2f76e 100644 --- a/drivers/dahdi/wcte11xp.c +++ b/drivers/dahdi/wcte11xp.c @@ -435,7 +435,7 @@ static int t1xxp_ioctl(struct dahdi_chan *chan, unsigned int cmd, unsigned long for (x=0;x<NUM_REGS;x++) regs.regs[x] = t1_framer_in(wc, x); - if (copy_to_user((struct t4_regs *)data, ®s, sizeof(regs))) + if (copy_to_user((__user void *)data, ®s, sizeof(regs))) return -EFAULT; break; default: @@ -1592,12 +1592,12 @@ static struct pci_device_id t1xxp_pci_tbl[] = { MODULE_DEVICE_TABLE(pci,t1xxp_pci_tbl); static struct pci_driver t1xxp_driver = { - name: "wcte11xp", - probe: t1xxp_init_one, - remove: __devexit_p(t1xxp_remove_one), - suspend: NULL, - resume: NULL, - id_table: t1xxp_pci_tbl, + .name = "wcte11xp", + .probe = t1xxp_init_one, + .remove = __devexit_p(t1xxp_remove_one), + .suspend = NULL, + .resume = NULL, + .id_table = t1xxp_pci_tbl, }; static int __init t1xxp_init(void) |