mirror of
https://github.com/torvalds/linux.git
synced 2024-12-26 21:02:19 +00:00
4538d0ca71
When no interrupt is specified the pata_of_platform fills the irq_res resource with -1, which is wrong to do for two reasons: 1. By definition, 'no irq' should be IRQ 0, not some negative integer; 2. pata_platform checks for irq_res.start > 0, but since irq_res.start is unsigned type, the check will be true for `-1'. Reported-by: Steven A. Falco <sfalco@harris.com> Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
115 lines
2.9 KiB
C
115 lines
2.9 KiB
C
/*
|
|
* OF-platform PATA driver
|
|
*
|
|
* Copyright (c) 2007 MontaVista Software, Inc.
|
|
* Anton Vorontsov <avorontsov@ru.mvista.com>
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License (Version 2) as
|
|
* published by the Free Software Foundation.
|
|
*/
|
|
|
|
#include <linux/kernel.h>
|
|
#include <linux/module.h>
|
|
#include <linux/of_platform.h>
|
|
#include <linux/ata_platform.h>
|
|
|
|
static int __devinit pata_of_platform_probe(struct of_device *ofdev,
|
|
const struct of_device_id *match)
|
|
{
|
|
int ret;
|
|
struct device_node *dn = ofdev->node;
|
|
struct resource io_res;
|
|
struct resource ctl_res;
|
|
struct resource irq_res;
|
|
unsigned int reg_shift = 0;
|
|
int pio_mode = 0;
|
|
int pio_mask;
|
|
const u32 *prop;
|
|
|
|
ret = of_address_to_resource(dn, 0, &io_res);
|
|
if (ret) {
|
|
dev_err(&ofdev->dev, "can't get IO address from "
|
|
"device tree\n");
|
|
return -EINVAL;
|
|
}
|
|
|
|
if (of_device_is_compatible(dn, "electra-ide")) {
|
|
/* Altstatus is really at offset 0x3f6 from the primary window
|
|
* on electra-ide. Adjust ctl_res and io_res accordingly.
|
|
*/
|
|
ctl_res = io_res;
|
|
ctl_res.start = ctl_res.start+0x3f6;
|
|
io_res.end = ctl_res.start-1;
|
|
} else {
|
|
ret = of_address_to_resource(dn, 1, &ctl_res);
|
|
if (ret) {
|
|
dev_err(&ofdev->dev, "can't get CTL address from "
|
|
"device tree\n");
|
|
return -EINVAL;
|
|
}
|
|
}
|
|
|
|
ret = of_irq_to_resource(dn, 0, &irq_res);
|
|
if (ret == NO_IRQ)
|
|
irq_res.start = irq_res.end = 0;
|
|
else
|
|
irq_res.flags = 0;
|
|
|
|
prop = of_get_property(dn, "reg-shift", NULL);
|
|
if (prop)
|
|
reg_shift = *prop;
|
|
|
|
prop = of_get_property(dn, "pio-mode", NULL);
|
|
if (prop) {
|
|
pio_mode = *prop;
|
|
if (pio_mode > 6) {
|
|
dev_err(&ofdev->dev, "invalid pio-mode\n");
|
|
return -EINVAL;
|
|
}
|
|
} else {
|
|
dev_info(&ofdev->dev, "pio-mode unspecified, assuming PIO0\n");
|
|
}
|
|
|
|
pio_mask = 1 << pio_mode;
|
|
pio_mask |= (1 << pio_mode) - 1;
|
|
|
|
return __pata_platform_probe(&ofdev->dev, &io_res, &ctl_res, &irq_res,
|
|
reg_shift, pio_mask);
|
|
}
|
|
|
|
static int __devexit pata_of_platform_remove(struct of_device *ofdev)
|
|
{
|
|
return __pata_platform_remove(&ofdev->dev);
|
|
}
|
|
|
|
static struct of_device_id pata_of_platform_match[] = {
|
|
{ .compatible = "ata-generic", },
|
|
{ .compatible = "electra-ide", },
|
|
{},
|
|
};
|
|
MODULE_DEVICE_TABLE(of, pata_of_platform_match);
|
|
|
|
static struct of_platform_driver pata_of_platform_driver = {
|
|
.name = "pata_of_platform",
|
|
.match_table = pata_of_platform_match,
|
|
.probe = pata_of_platform_probe,
|
|
.remove = __devexit_p(pata_of_platform_remove),
|
|
};
|
|
|
|
static int __init pata_of_platform_init(void)
|
|
{
|
|
return of_register_platform_driver(&pata_of_platform_driver);
|
|
}
|
|
module_init(pata_of_platform_init);
|
|
|
|
static void __exit pata_of_platform_exit(void)
|
|
{
|
|
of_unregister_platform_driver(&pata_of_platform_driver);
|
|
}
|
|
module_exit(pata_of_platform_exit);
|
|
|
|
MODULE_DESCRIPTION("OF-platform PATA driver");
|
|
MODULE_AUTHOR("Anton Vorontsov <avorontsov@ru.mvista.com>");
|
|
MODULE_LICENSE("GPL");
|