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

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

 



On 13/08/2024 16:51, Shen Jianping (ME-SE/EAD2) wrote:
> 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

I asked the question on purpose, so you will double check and do not
waste my time with whatever you wan to respond just to avoid work.
Replying something just to silence review is not welcomed.

If you claim you have two modules then please give me the names of the
modules (filenames).

Best regards,
Krzysztof





[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