RE: [PATCH v2 2/2] iio: imu: smi240: imu driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi


...
>>>>> +
>>>>> +MODULE_AUTHOR("Markus Lochmann
><markus.lochmann@xxxxxxxxxxxx>");
>>>>> +MODULE_AUTHOR("Stefan Gutmann <stefan.gutmann@xxxxxxxxxxxx>");
>>>>> +MODULE_DESCRIPTION("Bosch SMI240 driver"); MODULE_LICENSE("Dual
>>>>> +BSD/GPL");
>>>>
>>>> Hm? How many modules do you have here? What are their names?
>>>>
>>>> We have one module, named  "Bosch SMI240 driver". Any problem here?
>>>
>>> Yes, you put MODULE_* to how many files? Two? Three? Why is it needed
>>> everywhere?
>>
>> We put MODULE_* in all the *.c , just like the other IMU drivers already in source
>tree. They do the same.
>
>That's not true. First, look at adis_buffer.c. no MODULE_XXX. Second, maybe they
>have multiple modules, so the macros are expected. That's why I asked you and
>you said you do not have more than one module.
>
 
Now I get the point , what you mean for "module".  Yes we have two modules. 
One module named "Bosch SMI240 driver" therefore we put MODULE_* in smi240_core.c 
And we have the second module named " Bosch SMI240 SPI driver " and we put therefore MODULE_* in smi240_spi.c




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux