2015-07-25 19:10:43

by Hauke Mehrtens

[permalink] [raw]
Subject: [PATCH] bcma: fix build error when build as module

Currently of_default_bus_match_table is not exported so we can only use
this feature when bcma is build into the kernel. This patch removes
support for child buses when bcma is build as a module as a temporary
fix for a build problem introduces in this commit:

commit cae761b5a6bdc597ba476a040fdcd5b4bc559b85
Author: Rafał Miłecki <[email protected]>
Date: Sun Jun 28 17:17:13 2015 +0200

bcma: populate bus DT subnodes as platform_device-s

Signed-off-by: Hauke Mehrtens <[email protected]>
---
drivers/bcma/main.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/bcma/main.c b/drivers/bcma/main.c
index 5912847..8d973c4 100644
--- a/drivers/bcma/main.c
+++ b/drivers/bcma/main.c
@@ -410,7 +410,11 @@ int bcma_bus_register(struct bcma_bus *bus)
bcma_core_pci_early_init(&bus->drv_pci[0]);
}

- if (bus->host_pdev) {
+ /* TODO: remove check for IS_BUILTIN(CONFIG_BCMA) check when
+ * of_default_bus_match_table is exported or in some other way
+ * accessible. This is just a temporary workaround.
+ */
+ if (IS_BUILTIN(CONFIG_BCMA) && bus->host_pdev) {
struct device *dev = &bus->host_pdev->dev;

of_platform_populate(dev->of_node, of_default_bus_match_table,
--
2.1.4



2015-07-26 11:56:39

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] bcma: fix build error when build as module

Hauke Mehrtens <[email protected]> writes:

> Currently of_default_bus_match_table is not exported so we can only use
> this feature when bcma is build into the kernel. This patch removes
> support for child buses when bcma is build as a module as a temporary
> fix for a build problem introduces in this commit:
>
> commit cae761b5a6bdc597ba476a040fdcd5b4bc559b85
> Author: Rafał Miłecki <[email protected]>
> Date: Sun Jun 28 17:17:13 2015 +0200
>
> bcma: populate bus DT subnodes as platform_device-s
>
> Signed-off-by: Hauke Mehrtens <[email protected]>

Thanks, manually applied to wireless-drivers-next.git. (Still
patchwork.kernel.org has buggy utf-8 support, sigh.) I just added
Reported-by and Fixes lines to the commit log.

I wrote a similar patch but I'll drop that as your has better comments :)

--
Kalle Valo