linux-kernel - Re: [PATCH] power: Remove unnecessary MODULE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150831083546.GD14484@pali>
Date:	Mon, 31 Aug 2015 10:35:46 +0200
From:	Pali Rohár <pali.rohar@...il.com>
To:	Javier Martinez Canillas <javier@....samsung.com>
Cc:	linux-kernel@...r.kernel.org,
	"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	Sebastian Reichel <sre@...nel.org>,
	Dmitry Eremin-Solenikov <dbaryshkov@...il.com>,
	David Woodhouse <dwmw2@...radead.org>
Subject: Re: [PATCH] power: Remove unnecessary MODULE_ALIAS() for I2C drivers

On Monday 31 August 2015 10:17:11 Javier Martinez Canillas wrote:
> [adding cc list again]
> 
> Hello Pali,
> 
> On 08/31/2015 09:17 AM, Pali Rohár wrote:
> > On Monday 31 August 2015 01:18:35 Javier Martinez Canillas wrote:
> >> These drivers already have an I2C device id table that is used to create
> >> module aliases and the used MODULE_ALIAS() was either already in the I2C
> >> table so it was redundant or wasn't a valid I2C id so it was never used.
> >>
> >> Signed-off-by: Javier Martinez Canillas <javier@....samsung.com>
> >>
> >> ---
> >>
> >> diff --git a/drivers/power/bq27x00_battery.c b/drivers/power/bq27x00_battery.c
> >> index 8287261fd978..d0b2f3b47b8f 100644
> >> --- a/drivers/power/bq27x00_battery.c
> >> +++ b/drivers/power/bq27x00_battery.c
> >> @@ -1120,10 +1120,6 @@ module_exit(bq27x00_battery_exit);
> >>  MODULE_ALIAS("platform:bq27000-battery");
> >>  #endif
> >>  
> >> -#ifdef CONFIG_BATTERY_BQ27X00_I2C
> >> -MODULE_ALIAS("i2c:bq27000-battery");
> >> -#endif
> >> -
> >>  MODULE_AUTHOR("Rodolfo Giometti <giometti@...ux.it>");
> >>  MODULE_DESCRIPTION("BQ27x00 battery monitor driver");
> >>  MODULE_LICENSE("GPL");
> > 
> > Hi! This part of patch is already in queue list. So then your patch will
> > conflict with other pending patches...
> >
> 
> Sorry, with which patch it will conflict? It applies cleanly on top of
> the power supply git tree next branch:
> 
> https://git.kernel.org/cgit/linux/kernel/git/sre/linux-power-supply.git/log/?h=next
> 
> Best regards,

There are bq27x00 patches waiting on mailing list...

v1: https://lkml.org/lkml/2015/7/22/705
v2: https://lkml.org/lkml/2015/7/28/929
v3: https://lkml.org/lkml/2015/8/5/597

Your patch will conflict with this (v2) patch:
https://lkml.org/lkml/2015/7/28/923

Some of v1 patches are already merged into tree, some v2 are Acked by
me, but not merged yet... Probably Sebastian (maintainer) did not have
time to review all patches yet.

-- 
Pali Rohár
pali.rohar@...il.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/