On Fri, Aug 12, 2016 at 11:14 PM, Sebastian Reichel [off-list ref] wrote:

If the option is a bool, then you don't need module.h or any MODULE
macros, nor do you need any __exit or unregister related code.

Alternatively, if there is a use case for it to be modular, then maybe you
want to change the above to a tristate.

Paul.
--

+       depends on BT_HCIUART
+       help
+         Nokia H4+ is serial protocol for communication between Bluetooth
+         device and host. This protocol is required for Bluetooth devices
+         with UART interface in Nokia devices.
+
+         Say Y here to compile support for Nokia's H4+ protocol.
+
 config BT_HCIUART_BCSP
        bool "BCSP protocol support"
        depends on BT_HCIUART

[...]

+       { .compatible = "nokia,ti,wl1271-bluetooth", },
+       {},
+};
+MODULE_DEVICE_TABLE(of, nokia_bluetooth_of_match);
+
+static struct platform_driver platform_nokia_driver = {
+       .driver = {
+               .name = "nokia-bluetooth",
+               .of_match_table = nokia_bluetooth_of_match,
+       },
+       .probe = nokia_bluetooth_probe,
+};
+
+int __init nokia_init(void)
+{
+       platform_driver_register(&platform_nokia_driver);
+       return hci_uart_register_proto(&nokia_proto);
+}
+
+int __exit nokia_deinit(void)
+{
+       platform_driver_unregister(&platform_nokia_driver);
+       return hci_uart_unregister_proto(&nokia_proto);
+}