diff options
| author | Grant Likely <grant.likely@secretlab.ca> | 2013-02-17 20:03:27 +0000 |
|---|---|---|
| committer | Grant Likely <grant.likely@secretlab.ca> | 2013-02-17 20:03:27 +0000 |
| commit | 02bbde7849e68e193cefaa1885fe0df0f03c9fcd (patch) | |
| tree | 788a1d2226a1aa3a8bbcf58dfe9739f869ea7be0 /drivers/of | |
| parent | e570d7c15a73f88942885881a84257a65fa44c80 (diff) | |
| download | linux-02bbde7849e68e193cefaa1885fe0df0f03c9fcd.tar.gz linux-02bbde7849e68e193cefaa1885fe0df0f03c9fcd.tar.bz2 linux-02bbde7849e68e193cefaa1885fe0df0f03c9fcd.zip | |
Revert "of: use platform_device_add"
This reverts commit aac73f34542bc7ae4317928d2eabfeb21d247323. That
commit causes two kinds of breakage; it breaks registration of AMBA
devices when one of the parent nodes already contains overlapping
resource regions, and it breaks calls to request_region() by device
drivers in certain conditions where there are overlapping memory
regions. Both of these problems can probably be fixed, but it is better
to back out the commit and get a proper fix designed before trying again.
Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
Diffstat (limited to 'drivers/of')
| -rw-r--r-- | drivers/of/platform.c | 13 |
1 files changed, 2 insertions, 11 deletions
diff --git a/drivers/of/platform.c b/drivers/of/platform.c index 3c3e3cad3e08..b80891b43816 100644 --- a/drivers/of/platform.c +++ b/drivers/of/platform.c @@ -203,7 +203,6 @@ struct platform_device *of_platform_device_create_pdata( struct device *parent) { struct platform_device *dev; - int rc; if (!of_device_is_available(np)) return NULL; @@ -215,24 +214,16 @@ struct platform_device *of_platform_device_create_pdata( #if defined(CONFIG_MICROBLAZE) dev->archdata.dma_mask = 0xffffffffUL; #endif - dev->name = dev_name(&dev->dev); dev->dev.coherent_dma_mask = DMA_BIT_MASK(32); + dev->dev.bus = &platform_bus_type; dev->dev.platform_data = platform_data; - dev->dev.id = PLATFORM_DEVID_NONE; - /* device_add will assume that this device is on the same node as - * the parent. If there is no parent defined, set the node - * explicitly */ - if (!parent) - set_dev_node(&dev->dev, of_node_to_nid(np)); /* We do not fill the DMA ops for platform devices by default. * This is currently the responsibility of the platform code * to do such, possibly using a device notifier */ - rc = platform_device_add(dev); - if (rc) { - dev_err(&dev->dev, "device registration failed\n"); + if (of_device_add(dev) != 0) { platform_device_put(dev); return NULL; } |