lkml.org 
[lkml]   [2024]   [Apr]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 1/4] i2c: designware: Create shared header hosting driver name
From


On 4/23/2024 4:59 PM, Andy Shevchenko wrote:
> Tue, Apr 23, 2024 at 04:36:19PM -0700, Florian Fainelli kirjoitti:
>> We have a number of drivers that reference the string "i2c_designware"
>> yet this is copied all over the places with opportunities for this
>> string being mis-used. Create a shared header that defines this as a
>> constant that other drivers can reference.
>
> ...
>
>> #include <linux/i2c.h>
>> +#include <linux/i2c-designware.h>
>
> Can it be hidden in the subfolder?

That would require the MFD and ethernet drivers to include relative to
where they are in the source tree, do we really want that?

>
> ...
>
>> -#define DRIVER_NAME "i2c-designware-pci"
>> +#define DRIVER_NAME I2C_DESIGNWARE_NAME "-pci"
>
> Oh, this makes all the things hard to read.

OK, besides there is a change for '_' when it was a '-' before, so maybe
I should drop that hunk.

>
>> /* Work with hotplug and coldplug */
>> -MODULE_ALIAS("i2c_designware-pci");
>> +MODULE_ALIAS(DRIVER_NAME);
>
> I believe we shouldn't use MODULE_ALIAS() without real justification.

Pre-existing change.
--
Florian
[unhandled content-type:application/pkcs7-signature]
\
 
 \ /
  Last update: 2024-04-24 03:22    [W:0.119 / U:0.164 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site