diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-09-05 11:34:38 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-09-05 11:34:38 -0400 |
commit | 2386a3b0fbb0c2dcf29694c7df9a72cb268458f0 (patch) | |
tree | a22a9d46f2abd9ac4b2cbbb9e30a11abe0b7e31d /drivers/char | |
parent | ae7a835cc546fc67df90edaaa0c48ae2b22a29fe (diff) | |
parent | fdbeb7de700a3c8fc85d7e595a6bf1c92a27c0a1 (diff) |
Merge branch 'ipmi'
Merge IPMI fixes from:
"A few things for 3.12 from various people"
* emailed patches from Corey Minyard <minyard@acm.org>:
BMC support for PARISC machines
Add MODULE_ALIAS for autoloading ipmi driver on ACPI systems
ipmi: Initialize locals to avoid warning
ipmi: info leak in compat_ipmi_ioctl()
Diffstat (limited to 'drivers/char')
-rw-r--r-- | drivers/char/ipmi/ipmi_devintf.c | 1 | ||||
-rw-r--r-- | drivers/char/ipmi/ipmi_msghandler.c | 2 | ||||
-rw-r--r-- | drivers/char/ipmi/ipmi_si_intf.c | 77 |
3 files changed, 79 insertions, 1 deletions
diff --git a/drivers/char/ipmi/ipmi_devintf.c b/drivers/char/ipmi/ipmi_devintf.c index d5a5f020810a..ec318bf434a6 100644 --- a/drivers/char/ipmi/ipmi_devintf.c +++ b/drivers/char/ipmi/ipmi_devintf.c | |||
@@ -810,6 +810,7 @@ static long compat_ipmi_ioctl(struct file *filep, unsigned int cmd, | |||
810 | struct ipmi_recv __user *precv64; | 810 | struct ipmi_recv __user *precv64; |
811 | struct ipmi_recv recv64; | 811 | struct ipmi_recv recv64; |
812 | 812 | ||
813 | memset(&recv64, 0, sizeof(recv64)); | ||
813 | if (get_compat_ipmi_recv(&recv64, compat_ptr(arg))) | 814 | if (get_compat_ipmi_recv(&recv64, compat_ptr(arg))) |
814 | return -EFAULT; | 815 | return -EFAULT; |
815 | 816 | ||
diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c index 4445fa164a2d..ec4e10fcf1a5 100644 --- a/drivers/char/ipmi/ipmi_msghandler.c +++ b/drivers/char/ipmi/ipmi_msghandler.c | |||
@@ -1848,7 +1848,7 @@ int ipmi_request_settime(ipmi_user_t user, | |||
1848 | int retries, | 1848 | int retries, |
1849 | unsigned int retry_time_ms) | 1849 | unsigned int retry_time_ms) |
1850 | { | 1850 | { |
1851 | unsigned char saddr, lun; | 1851 | unsigned char saddr = 0, lun = 0; |
1852 | int rv; | 1852 | int rv; |
1853 | 1853 | ||
1854 | if (!user) | 1854 | if (!user) |
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c index af4b23ffc5a6..15e4a6031934 100644 --- a/drivers/char/ipmi/ipmi_si_intf.c +++ b/drivers/char/ipmi/ipmi_si_intf.c | |||
@@ -71,6 +71,11 @@ | |||
71 | #include <linux/of_address.h> | 71 | #include <linux/of_address.h> |
72 | #include <linux/of_irq.h> | 72 | #include <linux/of_irq.h> |
73 | 73 | ||
74 | #ifdef CONFIG_PARISC | ||
75 | #include <asm/hardware.h> /* for register_parisc_driver() stuff */ | ||
76 | #include <asm/parisc-device.h> | ||
77 | #endif | ||
78 | |||
74 | #define PFX "ipmi_si: " | 79 | #define PFX "ipmi_si: " |
75 | 80 | ||
76 | /* Measure times between events in the driver. */ | 81 | /* Measure times between events in the driver. */ |
@@ -298,6 +303,9 @@ static int pci_registered; | |||
298 | #ifdef CONFIG_ACPI | 303 | #ifdef CONFIG_ACPI |
299 | static int pnp_registered; | 304 | static int pnp_registered; |
300 | #endif | 305 | #endif |
306 | #ifdef CONFIG_PARISC | ||
307 | static int parisc_registered; | ||
308 | #endif | ||
301 | 309 | ||
302 | static unsigned int kipmid_max_busy_us[SI_MAX_PARMS]; | 310 | static unsigned int kipmid_max_busy_us[SI_MAX_PARMS]; |
303 | static int num_max_busy_us; | 311 | static int num_max_busy_us; |
@@ -2279,6 +2287,8 @@ static struct pnp_driver ipmi_pnp_driver = { | |||
2279 | .remove = ipmi_pnp_remove, | 2287 | .remove = ipmi_pnp_remove, |
2280 | .id_table = pnp_dev_table, | 2288 | .id_table = pnp_dev_table, |
2281 | }; | 2289 | }; |
2290 | |||
2291 | MODULE_DEVICE_TABLE(pnp, pnp_dev_table); | ||
2282 | #endif | 2292 | #endif |
2283 | 2293 | ||
2284 | #ifdef CONFIG_DMI | 2294 | #ifdef CONFIG_DMI |
@@ -2697,6 +2707,62 @@ static struct platform_driver ipmi_driver = { | |||
2697 | .remove = ipmi_remove, | 2707 | .remove = ipmi_remove, |
2698 | }; | 2708 | }; |
2699 | 2709 | ||
2710 | #ifdef CONFIG_PARISC | ||
2711 | static int ipmi_parisc_probe(struct parisc_device *dev) | ||
2712 | { | ||
2713 | struct smi_info *info; | ||
2714 | |||
2715 | info = smi_info_alloc(); | ||
2716 | |||
2717 | if (!info) { | ||
2718 | dev_err(&dev->dev, | ||
2719 | "could not allocate memory for PARISC probe\n"); | ||
2720 | return -ENOMEM; | ||
2721 | } | ||
2722 | |||
2723 | info->si_type = SI_KCS; | ||
2724 | info->addr_source = SI_DEVICETREE; | ||
2725 | info->io_setup = mem_setup; | ||
2726 | info->io.addr_type = IPMI_MEM_ADDR_SPACE; | ||
2727 | info->io.addr_data = dev->hpa.start; | ||
2728 | info->io.regsize = 1; | ||
2729 | info->io.regspacing = 1; | ||
2730 | info->io.regshift = 0; | ||
2731 | info->irq = 0; /* no interrupt */ | ||
2732 | info->irq_setup = NULL; | ||
2733 | info->dev = &dev->dev; | ||
2734 | |||
2735 | dev_dbg(&dev->dev, "addr 0x%lx\n", info->io.addr_data); | ||
2736 | |||
2737 | dev_set_drvdata(&dev->dev, info); | ||
2738 | |||
2739 | if (add_smi(info)) { | ||
2740 | kfree(info); | ||
2741 | return -EBUSY; | ||
2742 | } | ||
2743 | |||
2744 | return 0; | ||
2745 | } | ||
2746 | |||
2747 | static int ipmi_parisc_remove(struct parisc_device *dev) | ||
2748 | { | ||
2749 | cleanup_one_si(dev_get_drvdata(&dev->dev)); | ||
2750 | return 0; | ||
2751 | } | ||
2752 | |||
2753 | static struct parisc_device_id ipmi_parisc_tbl[] = { | ||
2754 | { HPHW_MC, HVERSION_REV_ANY_ID, 0x004, 0xC0 }, | ||
2755 | { 0, } | ||
2756 | }; | ||
2757 | |||
2758 | static struct parisc_driver ipmi_parisc_driver = { | ||
2759 | .name = "ipmi", | ||
2760 | .id_table = ipmi_parisc_tbl, | ||
2761 | .probe = ipmi_parisc_probe, | ||
2762 | .remove = ipmi_parisc_remove, | ||
2763 | }; | ||
2764 | #endif /* CONFIG_PARISC */ | ||
2765 | |||
2700 | static int wait_for_msg_done(struct smi_info *smi_info) | 2766 | static int wait_for_msg_done(struct smi_info *smi_info) |
2701 | { | 2767 | { |
2702 | enum si_sm_result smi_result; | 2768 | enum si_sm_result smi_result; |
@@ -3462,6 +3528,13 @@ static int init_ipmi_si(void) | |||
3462 | spmi_find_bmc(); | 3528 | spmi_find_bmc(); |
3463 | #endif | 3529 | #endif |
3464 | 3530 | ||
3531 | #ifdef CONFIG_PARISC | ||
3532 | register_parisc_driver(&ipmi_parisc_driver); | ||
3533 | parisc_registered = 1; | ||
3534 | /* poking PC IO addresses will crash machine, don't do it */ | ||
3535 | si_trydefaults = 0; | ||
3536 | #endif | ||
3537 | |||
3465 | /* We prefer devices with interrupts, but in the case of a machine | 3538 | /* We prefer devices with interrupts, but in the case of a machine |
3466 | with multiple BMCs we assume that there will be several instances | 3539 | with multiple BMCs we assume that there will be several instances |
3467 | of a given type so if we succeed in registering a type then also | 3540 | of a given type so if we succeed in registering a type then also |
@@ -3608,6 +3681,10 @@ static void cleanup_ipmi_si(void) | |||
3608 | if (pnp_registered) | 3681 | if (pnp_registered) |
3609 | pnp_unregister_driver(&ipmi_pnp_driver); | 3682 | pnp_unregister_driver(&ipmi_pnp_driver); |
3610 | #endif | 3683 | #endif |
3684 | #ifdef CONFIG_PARISC | ||
3685 | if (parisc_registered) | ||
3686 | unregister_parisc_driver(&ipmi_parisc_driver); | ||
3687 | #endif | ||
3611 | 3688 | ||
3612 | platform_driver_unregister(&ipmi_driver); | 3689 | platform_driver_unregister(&ipmi_driver); |
3613 | 3690 | ||