diff mbox

spi: clps711x: Add MODULE_ALIAS to support module auto-loading

Message ID 1389690114.22958.0.camel@phoenix (mailing list archive)
State Accepted
Commit 350a9b33046ef0d9909f68b87f411f4cf4e286f7
Headers show

Commit Message

Axel Lin Jan. 14, 2014, 9:01 a.m. UTC
Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
 drivers/spi/spi-clps711x.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Mark Brown Jan. 14, 2014, 2:21 p.m. UTC | #1
On Tue, Jan 14, 2014 at 05:01:54PM +0800, Axel Lin wrote:
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

Applied, thanks.
Alexander Shiyan Jan. 14, 2014, 5:22 p.m. UTC | #2
PiBPbiBUdWUsIEphbiAxNCwgMjAxNCBhdCAwNTowMTo1NFBNICswODAwLCBBeGVsIExpbiB3cm90
ZToKPiA+IFNpZ25lZC1vZmYtYnk6IEF4ZWwgTGluIDxheGVsLmxpbkBpbmdpY3MuY29tPgo+IAo+
IEFwcGxpZWQsIHRoYW5rcy4KCk5vIGRlcGVuZGVuY2llcyBvbiB0aGUgZHJpdmVyIGZvciB0aGUg
b3RoZXIgbW9kdWxlcywgd2hpY2ggZ2l2ZXMgdGhpcyBjaGFuZ2U/CgotLS0K
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Mark Brown Jan. 14, 2014, 5:41 p.m. UTC | #3
On Tue, Jan 14, 2014 at 09:22:09PM +0400, Alexander Shiyan wrote:
> > On Tue, Jan 14, 2014 at 05:01:54PM +0800, Axel Lin wrote:
> > > Signed-off-by: Axel Lin <axel.lin@ingics.com>

> > Applied, thanks.

> No dependencies on the driver for the other modules, which gives this change?

I can't parse the above sorry?  If you're asking how inter module
dependencies get resolved the module loader does this automatically
based on the symbols referenced in modules.
Alexander Shiyan Jan. 14, 2014, 5:58 p.m. UTC | #4
0JLRgtC+0YDQvdC40LosIDE0INGP0L3QstCw0YDRjyAyMDE0LCAxNzo0MSBVVEMg0L7RgiBNYXJr
IEJyb3duIDxicm9vbmllQGtlcm5lbC5vcmc+Ogo+IE9uIFR1ZSwgSmFuIDE0LCAyMDE0IGF0IDA5
OjIyOjA5UE0gKzA0MDAsIEFsZXhhbmRlciBTaGl5YW4gd3JvdGU6Cj4gPiA+IE9uIFR1ZSwgSmFu
IDE0LCAyMDE0IGF0IDA1OjAxOjU0UE0gKzA4MDAsIEF4ZWwgTGluIHdyb3RlOgo+ID4gPiA+IFNp
Z25lZC1vZmYtYnk6IEF4ZWwgTGluIDxheGVsLmxpbkBpbmdpY3MuY29tPgo+IAo+ID4gPiBBcHBs
aWVkLCB0aGFua3MuCj4gCj4gPiBObyBkZXBlbmRlbmNpZXMgb24gdGhlIGRyaXZlciBmb3IgdGhl
IG90aGVyIG1vZHVsZXMsIHdoaWNoIGdpdmVzIHRoaXMgY2hhbmdlPwo+IAo+IEkgY2FuJ3QgcGFy
c2UgdGhlIGFib3ZlIHNvcnJ5PyAgSWYgeW91J3JlIGFza2luZyBob3cgaW50ZXIgbW9kdWxlCj4g
ZGVwZW5kZW5jaWVzIGdldCByZXNvbHZlZCB0aGUgbW9kdWxlIGxvYWRlciBkb2VzIHRoaXMgYXV0
b21hdGljYWxseQo+IGJhc2VkIG9uIHRoZSBzeW1ib2xzIHJlZmVyZW5jZWQgaW4gbW9kdWxlcy4K
ClllcywgYnV0IHRoZXJlIGFyZSBubyBtb2R1bGVzIHRoYXQgaGF2ZSBjbHBzNzExeC1zcGkgZGVw
ZW5kZW5jeS4KSSBhbSBub3QgZ3VydSBpbiBtb2R1bGUgaGFuZGxpbmcsIGp1c3Qgd2FudCB0byB1
bmRlcnN0YW5kIHRoZSBuZWVkCnRvIG1ha2UgdGhpcyBjaGFuZ2UuCgotLS0K
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Mark Brown Jan. 14, 2014, 6 p.m. UTC | #5
On Tue, Jan 14, 2014 at 09:58:31PM +0400, Alexander Shiyan wrote:
> ???????, 14 ?????? 2014, 17:41 UTC ?? Mark Brown <broonie@kernel.org>:

> > I can't parse the above sorry?  If you're asking how inter module
> > dependencies get resolved the module loader does this automatically
> > based on the symbols referenced in modules.

> Yes, but there are no modules that have clps711x-spi dependency.
> I am not guru in module handling, just want to understand the need
> to make this change.

It's so that if the driver is built as a module if the device is present
in the system userspace can figure out which module to load to support
it.
Alexander Shiyan Jan. 14, 2014, 6:04 p.m. UTC | #6
0JLRgtC+0YDQvdC40LosIDE0INGP0L3QstCw0YDRjyAyMDE0LCAxODowMCBVVEMg0L7RgiBNYXJr
IEJyb3duIDxicm9vbmllQGtlcm5lbC5vcmc+Ogo+IE9uIFR1ZSwgSmFuIDE0LCAyMDE0IGF0IDA5
OjU4OjMxUE0gKzA0MDAsIEFsZXhhbmRlciBTaGl5YW4gd3JvdGU6Cj4gPiDQktGC0L7RgNC90LjQ
uiwgMTQg0Y/QvdCy0LDRgNGPIDIwMTQsIDE3OjQxIFVUQyDQvtGCIE1hcmsgQnJvd24gPGJyb29u
aWVAa2VybmVsLm9yZz46Cj4gCj4gPiA+IEkgY2FuJ3QgcGFyc2UgdGhlIGFib3ZlIHNvcnJ5PyAg
SWYgeW91J3JlIGFza2luZyBob3cgaW50ZXIgbW9kdWxlCj4gPiA+IGRlcGVuZGVuY2llcyBnZXQg
cmVzb2x2ZWQgdGhlIG1vZHVsZSBsb2FkZXIgZG9lcyB0aGlzIGF1dG9tYXRpY2FsbHkKPiA+ID4g
YmFzZWQgb24gdGhlIHN5bWJvbHMgcmVmZXJlbmNlZCBpbiBtb2R1bGVzLgo+IAo+ID4gWWVzLCBi
dXQgdGhlcmUgYXJlIG5vIG1vZHVsZXMgdGhhdCBoYXZlIGNscHM3MTF4LXNwaSBkZXBlbmRlbmN5
Lgo+ID4gSSBhbSBub3QgZ3VydSBpbiBtb2R1bGUgaGFuZGxpbmcsIGp1c3Qgd2FudCB0byB1bmRl
cnN0YW5kIHRoZSBuZWVkCj4gPiB0byBtYWtlIHRoaXMgY2hhbmdlLgo+IAo+IEl0J3Mgc28gdGhh
dCBpZiB0aGUgZHJpdmVyIGlzIGJ1aWx0IGFzIGEgbW9kdWxlIGlmIHRoZSBkZXZpY2UgaXMgcHJl
c2VudAo+IGluIHRoZSBzeXN0ZW0gdXNlcnNwYWNlIGNhbiBmaWd1cmUgb3V0IHdoaWNoIG1vZHVs
ZSB0byBsb2FkIHRvIHN1cHBvcnQKPiBpdC4KCk9LLCBpZiBzbyB0aGlzIG1ha2VzIHNlbnNlLiBU
aGFuayB5b3UuCgotLS0K
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/spi/spi-clps711x.c b/drivers/spi/spi-clps711x.c
index dafb243..374ba4a 100644
--- a/drivers/spi/spi-clps711x.c
+++ b/drivers/spi/spi-clps711x.c
@@ -257,3 +257,4 @@  module_platform_driver(clps711x_spi_driver);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Alexander Shiyan <shc_work@mail.ru>");
 MODULE_DESCRIPTION("CLPS711X SPI bus driver");
+MODULE_ALIAS("platform:" DRIVER_NAME);