[PATCH 2/2] mfd: pm8008: drop bogus i2c module alias

Johan Hovold posted 2 patches 12 months ago
[PATCH 2/2] mfd: pm8008: drop bogus i2c module alias
Posted by Johan Hovold 12 months ago
Prior to commit af503716ac14 ("i2c: core: report OF style module alias
for devices registered via OF") drivers using OF matching needed an i2c
module alias such as "i2c:pm8008" for module autoloading to work.

Drop the bogus i2c alias from the pm8008 driver which was merged long
after i2c core was fixed and whose alias did not match the driver name
("pm8008") to begin with.

Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
---
 drivers/mfd/qcom-pm8008.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/mfd/qcom-pm8008.c b/drivers/mfd/qcom-pm8008.c
index 29ec3901564b..c4cef3936ea3 100644
--- a/drivers/mfd/qcom-pm8008.c
+++ b/drivers/mfd/qcom-pm8008.c
@@ -211,4 +211,3 @@ static struct i2c_driver pm8008_mfd_driver = {
 module_i2c_driver(pm8008_mfd_driver);
 
 MODULE_LICENSE("GPL v2");
-MODULE_ALIAS("i2c:qcom-pm8008");
-- 
2.39.3
Re: [PATCH 2/2] mfd: pm8008: drop bogus i2c module alias
Posted by Lee Jones 11 months, 2 weeks ago
On Fri, 26 May 2023, Johan Hovold wrote:

> Prior to commit af503716ac14 ("i2c: core: report OF style module alias
> for devices registered via OF") drivers using OF matching needed an i2c
> module alias such as "i2c:pm8008" for module autoloading to work.
> 
> Drop the bogus i2c alias from the pm8008 driver which was merged long
> after i2c core was fixed and whose alias did not match the driver name
> ("pm8008") to begin with.
> 
> Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
> ---
>  drivers/mfd/qcom-pm8008.c | 1 -
>  1 file changed, 1 deletion(-)

Applied, thanks

-- 
Lee Jones [李琼斯]