The ID table already has respective entry and MODULE_DEVICE_TABLE and
creates proper alias for SPI driver. Having another MODULE_ALIAS causes
the alias to be duplicated.
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/misc/eeprom/at25.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/misc/eeprom/at25.c b/drivers/misc/eeprom/at25.c
index 65d49a6de1a7..595ceb9a7126 100644
--- a/drivers/misc/eeprom/at25.c
+++ b/drivers/misc/eeprom/at25.c
@@ -529,4 +529,3 @@ module_spi_driver(at25_driver);
MODULE_DESCRIPTION("Driver for most SPI EEPROMs");
MODULE_AUTHOR("David Brownell");
MODULE_LICENSE("GPL");
-MODULE_ALIAS("spi:at25");
--
2.34.1
The ID table already has respective entry and MODULE_DEVICE_TABLE and
creates proper alias for SPI driver. Having another MODULE_ALIAS causes
the alias to be duplicated.
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/mtd/devices/mchp23k256.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/mtd/devices/mchp23k256.c b/drivers/mtd/devices/mchp23k256.c
index d533475fda15..cef5f9677d39 100644
--- a/drivers/mtd/devices/mchp23k256.c
+++ b/drivers/mtd/devices/mchp23k256.c
@@ -257,4 +257,3 @@ module_spi_driver(mchp23k256_driver);
MODULE_DESCRIPTION("MTD SPI driver for MCHP23K256 RAM chips");
MODULE_AUTHOR("Andrew Lunn <[email protected]>");
MODULE_LICENSE("GPL v2");
-MODULE_ALIAS("spi:mchp23k256");
--
2.34.1
The ID table already has respective entry and MODULE_DEVICE_TABLE and
creates proper alias for SPI driver. Having another MODULE_ALIAS causes
the alias to be duplicated.
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/misc/eeprom/eeprom_93xx46.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/misc/eeprom/eeprom_93xx46.c b/drivers/misc/eeprom/eeprom_93xx46.c
index e78a76d74ff4..45c8ae0db8f9 100644
--- a/drivers/misc/eeprom/eeprom_93xx46.c
+++ b/drivers/misc/eeprom/eeprom_93xx46.c
@@ -578,5 +578,3 @@ MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("Driver for 93xx46 EEPROMs");
MODULE_AUTHOR("Anatolij Gustschin <[email protected]>");
MODULE_ALIAS("spi:93xx46");
-MODULE_ALIAS("spi:eeprom-93xx46");
-MODULE_ALIAS("spi:93lc46b");
--
2.34.1
On Sun, Apr 14, 2024 at 05:49:57PM +0200, Krzysztof Kozlowski wrote:
> The ID table already has respective entry and MODULE_DEVICE_TABLE and
> creates proper alias for SPI driver. Having another MODULE_ALIAS causes
> the alias to be duplicated.
..
> MODULE_ALIAS("spi:93xx46");
I was stumbled over this (leftover?).
Commit message doesn't elaborate this bit.
Any comments?
> -MODULE_ALIAS("spi:eeprom-93xx46");
> -MODULE_ALIAS("spi:93lc46b");
--
With Best Regards,
Andy Shevchenko
On 08/05/2024 19:53, Andy Shevchenko wrote:
> On Sun, Apr 14, 2024 at 05:49:57PM +0200, Krzysztof Kozlowski wrote:
>> The ID table already has respective entry and MODULE_DEVICE_TABLE and
>> creates proper alias for SPI driver. Having another MODULE_ALIAS causes
>> the alias to be duplicated.
>
> ...
>
>> MODULE_ALIAS("spi:93xx46");
>
> I was stumbled over this (leftover?).
> Commit message doesn't elaborate this bit.
> Any comments?
It is not present in ID table and commit msg removes only duplicated
aliases. That alias has meaning - someone might be actually relying on it.
Best regards,
Krzysztof
On Wed, May 08, 2024 at 08:15:00PM +0200, Krzysztof Kozlowski wrote:
> On 08/05/2024 19:53, Andy Shevchenko wrote:
> > On Sun, Apr 14, 2024 at 05:49:57PM +0200, Krzysztof Kozlowski wrote:
> >> The ID table already has respective entry and MODULE_DEVICE_TABLE and
> >> creates proper alias for SPI driver. Having another MODULE_ALIAS causes
> >> the alias to be duplicated.
..
> >> MODULE_ALIAS("spi:93xx46");
> >
> > I was stumbled over this (leftover?).
> > Commit message doesn't elaborate this bit.
> > Any comments?
>
> It is not present in ID table and commit msg removes only duplicated
> aliases. That alias has meaning - someone might be actually relying on it.
It seems no users for it. The only user of platform data of this EEPROM uses
board files which AFAIU bypasses modalias matching.
--
With Best Regards,
Andy Shevchenko
On 08/05/2024 20:28, Andy Shevchenko wrote:
> On Wed, May 08, 2024 at 08:15:00PM +0200, Krzysztof Kozlowski wrote:
>> On 08/05/2024 19:53, Andy Shevchenko wrote:
>>> On Sun, Apr 14, 2024 at 05:49:57PM +0200, Krzysztof Kozlowski wrote:
>>>> The ID table already has respective entry and MODULE_DEVICE_TABLE and
>>>> creates proper alias for SPI driver. Having another MODULE_ALIAS causes
>>>> the alias to be duplicated.
>
> ...
>
>>>> MODULE_ALIAS("spi:93xx46");
>>>
>>> I was stumbled over this (leftover?).
>>> Commit message doesn't elaborate this bit.
>>> Any comments?
>>
>> It is not present in ID table and commit msg removes only duplicated
>> aliases. That alias has meaning - someone might be actually relying on it.
>
> It seems no users for it. The only user of platform data of this EEPROM uses
> board files which AFAIU bypasses modalias matching.
I don't think that's correct. The modalias of SPI board is there on
purpose. Why do you think it is not used? IOW, what changed in kernel
since the issue was encountered and fixed by adding that alias (see
explanation and users in kernel of modalias from that commit)?
Best regards,
Krzysztof
On Wed, May 8, 2024, at 20:44, Krzysztof Kozlowski wrote:
> On 08/05/2024 20:28, Andy Shevchenko wrote:
>> On Wed, May 08, 2024 at 08:15:00PM +0200, Krzysztof Kozlowski wrote:
>>> On 08/05/2024 19:53, Andy Shevchenko wrote:
>>>> On Sun, Apr 14, 2024 at 05:49:57PM +0200, Krzysztof Kozlowski wrote:
>>>>> The ID table already has respective entry and MODULE_DEVICE_TABLE and
>>>>> creates proper alias for SPI driver. Having another MODULE_ALIAS causes
>>>>> the alias to be duplicated.
>>
>> ...
>>
>>>>> MODULE_ALIAS("spi:93xx46");
>>>>
>>>> I was stumbled over this (leftover?).
>>>> Commit message doesn't elaborate this bit.
>>>> Any comments?
>>>
>>> It is not present in ID table and commit msg removes only duplicated
>>> aliases. That alias has meaning - someone might be actually relying on it.
>>
>> It seems no users for it. The only user of platform data of this EEPROM uses
>> board files which AFAIU bypasses modalias matching.
>
> I don't think that's correct. The modalias of SPI board is there on
> purpose.
Right, but I think a better workaround would have been to change
the board_info to pick a modalias that is part of the ID table:
--- a/drivers/misc/eeprom/digsy_mtc_eeprom.c
+++ b/drivers/misc/eeprom/digsy_mtc_eeprom.c
@@ -76,7 +76,7 @@ static struct gpiod_lookup_table eeprom_spi_gpiod_table = {
static struct spi_board_info digsy_mtc_eeprom_info[] __initdata = {
{
- .modalias = "93xx46",
+ .modalias = "eeprom-93xx46",
.max_speed_hz = 1000000,
.bus_num = EE_SPI_BUS_NUM,
.chip_select = 0,
Arnd
On 08/05/2024 21:11, Arnd Bergmann wrote:
> On Wed, May 8, 2024, at 20:44, Krzysztof Kozlowski wrote:
>> On 08/05/2024 20:28, Andy Shevchenko wrote:
>>> On Wed, May 08, 2024 at 08:15:00PM +0200, Krzysztof Kozlowski wrote:
>>>> On 08/05/2024 19:53, Andy Shevchenko wrote:
>>>>> On Sun, Apr 14, 2024 at 05:49:57PM +0200, Krzysztof Kozlowski wrote:
>>>>>> The ID table already has respective entry and MODULE_DEVICE_TABLE and
>>>>>> creates proper alias for SPI driver. Having another MODULE_ALIAS causes
>>>>>> the alias to be duplicated.
>>>
>>> ...
>>>
>>>>>> MODULE_ALIAS("spi:93xx46");
>>>>>
>>>>> I was stumbled over this (leftover?).
>>>>> Commit message doesn't elaborate this bit.
>>>>> Any comments?
>>>>
>>>> It is not present in ID table and commit msg removes only duplicated
>>>> aliases. That alias has meaning - someone might be actually relying on it.
>>>
>>> It seems no users for it. The only user of platform data of this EEPROM uses
>>> board files which AFAIU bypasses modalias matching.
>>
>> I don't think that's correct. The modalias of SPI board is there on
>> purpose.
>
> Right, but I think a better workaround would have been to change
> the board_info to pick a modalias that is part of the ID table:
>
> --- a/drivers/misc/eeprom/digsy_mtc_eeprom.c
> +++ b/drivers/misc/eeprom/digsy_mtc_eeprom.c
> @@ -76,7 +76,7 @@ static struct gpiod_lookup_table eeprom_spi_gpiod_table = {
>
> static struct spi_board_info digsy_mtc_eeprom_info[] __initdata = {
> {
> - .modalias = "93xx46",
> + .modalias = "eeprom-93xx46",
> .max_speed_hz = 1000000,
> .bus_num = EE_SPI_BUS_NUM,
> .chip_select = 0,
>
That's kind of independent change. This commit *only* removed duplicated
aliases, thus should have not functional impact except dropping
redundant code. What you propose is to change spi board matching, which
is reasonable, just a different thing.
Best regards,
Krzysztof
On Wed, May 08, 2024 at 09:17:59PM +0200, Krzysztof Kozlowski wrote:
> On 08/05/2024 21:11, Arnd Bergmann wrote:
> > On Wed, May 8, 2024, at 20:44, Krzysztof Kozlowski wrote:
> >> On 08/05/2024 20:28, Andy Shevchenko wrote:
> >>> On Wed, May 08, 2024 at 08:15:00PM +0200, Krzysztof Kozlowski wrote:
> >>>> On 08/05/2024 19:53, Andy Shevchenko wrote:
> >>>>> On Sun, Apr 14, 2024 at 05:49:57PM +0200, Krzysztof Kozlowski wrote:
> >>>>>> The ID table already has respective entry and MODULE_DEVICE_TABLE and
> >>>>>> creates proper alias for SPI driver. Having another MODULE_ALIAS causes
> >>>>>> the alias to be duplicated.
> >>>
> >>> ...
> >>>
> >>>>>> MODULE_ALIAS("spi:93xx46");
> >>>>>
> >>>>> I was stumbled over this (leftover?).
> >>>>> Commit message doesn't elaborate this bit.
> >>>>> Any comments?
> >>>>
> >>>> It is not present in ID table and commit msg removes only duplicated
> >>>> aliases. That alias has meaning - someone might be actually relying on it.
> >>>
> >>> It seems no users for it. The only user of platform data of this EEPROM uses
> >>> board files which AFAIU bypasses modalias matching.
> >>
> >> I don't think that's correct. The modalias of SPI board is there on
> >> purpose.
> >
> > Right, but I think a better workaround would have been to change
> > the board_info to pick a modalias that is part of the ID table:
> >
> > --- a/drivers/misc/eeprom/digsy_mtc_eeprom.c
> > +++ b/drivers/misc/eeprom/digsy_mtc_eeprom.c
> > @@ -76,7 +76,7 @@ static struct gpiod_lookup_table eeprom_spi_gpiod_table = {
> >
> > static struct spi_board_info digsy_mtc_eeprom_info[] __initdata = {
> > {
> > - .modalias = "93xx46",
> > + .modalias = "eeprom-93xx46",
> > .max_speed_hz = 1000000,
> > .bus_num = EE_SPI_BUS_NUM,
> > .chip_select = 0,
> >
>
> That's kind of independent change.
I have done similar change in my patch series, after which I am 100% sure the
modalias can be removed. Can you look at that?
https://lore.kernel.org/r/[email protected]
> This commit *only* removed duplicated
> aliases, thus should have not functional impact except dropping
> redundant code. What you propose is to change spi board matching, which
> is reasonable, just a different thing.
--
With Best Regards,
Andy Shevchenko