summaryrefslogtreecommitdiffstats
path: root/drivers/of
diff options
context:
space:
mode:
authorGrant Likely <grant.likely@secretlab.ca>2013-02-17 20:03:27 +0000
committerGrant Likely <grant.likely@secretlab.ca>2013-02-17 20:03:27 +0000
commit02bbde7849e68e193cefaa1885fe0df0f03c9fcd (patch)
tree788a1d2226a1aa3a8bbcf58dfe9739f869ea7be0 /drivers/of
parente570d7c15a73f88942885881a84257a65fa44c80 (diff)
downloadop-kernel-dev-02bbde7849e68e193cefaa1885fe0df0f03c9fcd.zip
op-kernel-dev-02bbde7849e68e193cefaa1885fe0df0f03c9fcd.tar.gz
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.c13
1 files changed, 2 insertions, 11 deletions
diff --git a/drivers/of/platform.c b/drivers/of/platform.c
index 3c3e3ca..b80891b 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;
}
OpenPOWER on IntegriCloud