kernel: backport bcma init patch from 4.5
authorRafał Miłecki <zajec5@gmail.com>
Fri, 26 Feb 2016 07:09:57 +0000 (07:09 +0000)
committerRafał Miłecki <zajec5@gmail.com>
Fri, 26 Feb 2016 07:09:57 +0000 (07:09 +0000)
This simply replaces init fix with a final version and puts it in a
generic dir. This will allow backporting some trivial changes from 4.6.

Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
SVN-Revision: 48788

target/linux/bcm53xx/patches-3.18/800-bcma-use-two-different-initcalls-if-built-in.patch [deleted file]
target/linux/generic/patches-3.18/031-bcma-from-4.5.patch [new file with mode: 0644]

diff --git a/target/linux/bcm53xx/patches-3.18/800-bcma-use-two-different-initcalls-if-built-in.patch b/target/linux/bcm53xx/patches-3.18/800-bcma-use-two-different-initcalls-if-built-in.patch
deleted file mode 100644 (file)
index 9f2cd39..0000000
+++ /dev/null
@@ -1,65 +0,0 @@
-From 666bdfc027cde41a171862dc698987a378c8b66a Mon Sep 17 00:00:00 2001
-From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
-Date: Mon, 9 Feb 2015 18:00:42 +0100
-Subject: [PATCH RFC] bcma: use two different initcalls if built-in
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-This is needed as we can't initialize bus during fs_initcall.
-Initialization requires SPROM which depends on NVRAM which depends on
-mtd. Since mtd, spi, nand, spi-nor use standard module_init, we have to
-do the same in bcma.
-Without this we'll try to initialize SPROM without having a ready SPROM
-proviver registered using bcma_arch_register_fallback_sprom.
-
-Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
----
-While this patch seems to work and I can compile bcma as built-in and
-module, I'm not too proud of it. I don't really like these #if(n)def
-tricks and I'm afraid bcma_modinit may be called even if
-bcma_modinit_early failed.
-
-Do you see any better idea of solving this?
----
- drivers/bcma/main.c | 16 ++++++++++++++--
- 1 file changed, 14 insertions(+), 2 deletions(-)
-
---- a/drivers/bcma/main.c
-+++ b/drivers/bcma/main.c
-@@ -637,13 +637,25 @@ static int bcma_device_uevent(struct dev
-                             core->id.rev, core->id.class);
- }
-+/* Bus has to be registered early, before any bcma driver */
-+static int __init bcma_modinit_early(void)
-+{
-+      return bus_register(&bcma_bus_type);
-+}
-+#ifndef MODULE
-+fs_initcall(bcma_modinit_early);
-+#endif
-+
-+/* Initialization has to be done later with SPI/mtd/NAND/SPROM available */
- static int __init bcma_modinit(void)
- {
-       int err;
--      err = bus_register(&bcma_bus_type);
-+#ifdef MODULE
-+      err = bcma_modinit_early();
-       if (err)
-               return err;
-+#endif
-       err = bcma_host_soc_register_driver();
-       if (err) {
-@@ -660,7 +672,7 @@ static int __init bcma_modinit(void)
-       return err;
- }
--fs_initcall(bcma_modinit);
-+module_init(bcma_modinit);
- static void __exit bcma_modexit(void)
- {
diff --git a/target/linux/generic/patches-3.18/031-bcma-from-4.5.patch b/target/linux/generic/patches-3.18/031-bcma-from-4.5.patch
new file mode 100644 (file)
index 0000000..171395d
--- /dev/null
@@ -0,0 +1,49 @@
+--- a/drivers/bcma/main.c
++++ b/drivers/bcma/main.c
+@@ -637,11 +637,36 @@ static int bcma_device_uevent(struct dev
+                             core->id.rev, core->id.class);
+ }
+-static int __init bcma_modinit(void)
++static unsigned int bcma_bus_registered;
++
++/*
++ * If built-in, bus has to be registered early, before any driver calls
++ * bcma_driver_register.
++ * Otherwise registering driver would trigger BUG in driver_register.
++ */
++static int __init bcma_init_bus_register(void)
+ {
+       int err;
++      if (bcma_bus_registered)
++              return 0;
++
+       err = bus_register(&bcma_bus_type);
++      if (!err)
++              bcma_bus_registered = 1;
++
++      return err;
++}
++#ifndef MODULE
++fs_initcall(bcma_init_bus_register);
++#endif
++
++/* Main initialization has to be done with SPI/mtd/NAND/SPROM available */
++static int __init bcma_modinit(void)
++{
++      int err;
++
++      err = bcma_init_bus_register();
+       if (err)
+               return err;
+@@ -660,7 +685,7 @@ static int __init bcma_modinit(void)
+       return err;
+ }
+-fs_initcall(bcma_modinit);
++module_init(bcma_modinit);
+ static void __exit bcma_modexit(void)
+ {