diff mbox

ASoC: zx: fix platform_no_drv_owner.cocci warnings

Message ID 20150604171126.GA226227@ivytown2 (mailing list archive)
State Accepted
Commit 67f72776b6674ca2b4602996bd3e235a4f38c4b4
Headers show

Commit Message

Fengguang Wu June 4, 2015, 5:11 p.m. UTC
sound/soc/zte/zx296702-i2s.c:428:3-8: No need to set .owner here. The core will do it.

 Remove .owner field if calls are used which set it automatically

Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci

CC: Jun Nie <jun.nie@linaro.org>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

 zx296702-i2s.c |    1 -
 1 file changed, 1 deletion(-)

Comments

Mark Brown June 4, 2015, 5:16 p.m. UTC | #1
On Fri, Jun 05, 2015 at 01:11:27AM +0800, kbuild test robot wrote:
> sound/soc/zte/zx296702-i2s.c:428:3-8: No need to set .owner here. The core will do it.
> 
>  Remove .owner field if calls are used which set it automatically

Applied, thanks.
Mark Brown June 4, 2015, 5:53 p.m. UTC | #2
On Thu, Jun 04, 2015 at 06:16:59PM +0100, Mark Brown wrote:
> On Fri, Jun 05, 2015 at 01:11:27AM +0800, kbuild test robot wrote:
> > sound/soc/zte/zx296702-i2s.c:428:3-8: No need to set .owner here. The core will do it.
> > 
> >  Remove .owner field if calls are used which set it automatically
> 
> Applied, thanks.

These mails are CCed to kbuild-all@01.org which is generating
backscatter since it's subscribers only (not even moderated for
non-subscribers) - you can you please disable the mails to
non-subscribers or open the list up?
Fengguang Wu June 5, 2015, 1:09 a.m. UTC | #3
On Thu, Jun 04, 2015 at 06:53:00PM +0100, Mark Brown wrote:
> On Thu, Jun 04, 2015 at 06:16:59PM +0100, Mark Brown wrote:
> > On Fri, Jun 05, 2015 at 01:11:27AM +0800, kbuild test robot wrote:
> > > sound/soc/zte/zx296702-i2s.c:428:3-8: No need to set .owner here. The core will do it.
> > > 
> > >  Remove .owner field if calls are used which set it automatically
> > 
> > Applied, thanks.
> 
> These mails are CCed to kbuild-all@01.org which is generating
> backscatter since it's subscribers only (not even moderated for
> non-subscribers) - you can you please disable the mails to
> non-subscribers or open the list up?

Sure, I'll open up the list to all email addresses that have 10+
commits in the recent 1 year's git log.

Thanks,
Fengguang
Joe Perches June 5, 2015, 1:21 a.m. UTC | #4
On Fri, 2015-06-05 at 09:09 +0800, Fengguang Wu wrote:
> On Thu, Jun 04, 2015 at 06:53:00PM +0100, Mark Brown wrote:
[]
> > These mails are CCed to kbuild-all@01.org which is generating
> > backscatter since it's subscribers only (not even moderated for
> > non-subscribers) - you can you please disable the mails to
> > non-subscribers or open the list up?
> 
> Sure, I'll open up the list to all email addresses that have 10+
> commits in the recent 1 year's git log.

10 is a pretty high number, though it's ~1100 addresses

If you are adding filters, perhaps you could add all the
listed MAINTAINERS too.
Fengguang Wu June 5, 2015, 2:27 a.m. UTC | #5
On Thu, Jun 04, 2015 at 06:21:05PM -0700, Joe Perches wrote:
> On Fri, 2015-06-05 at 09:09 +0800, Fengguang Wu wrote:
> > On Thu, Jun 04, 2015 at 06:53:00PM +0100, Mark Brown wrote:
> []
> > > These mails are CCed to kbuild-all@01.org which is generating
> > > backscatter since it's subscribers only (not even moderated for
> > > non-subscribers) - you can you please disable the mails to
> > > non-subscribers or open the list up?
> > 
> > Sure, I'll open up the list to all email addresses that have 10+
> > commits in the recent 1 year's git log.
> 
> 10 is a pretty high number, though it's ~1100 addresses

I'll also whitelist the whole domain if there are 2+ contributors
without counting their number of commits. This should reduce the
whitelist size as well as improve coverage.

> If you are adding filters, perhaps you could add all the
> listed MAINTAINERS too.

I'd expect all MAINTAINERS to be an active committer (he will at least
have some merge commits ;), so git log should be enough.

This is the script I used to generate the white list.

cd /c/repo/linux || exit

WHITELIST=/c/kernel-tests/kernel-developer-email-pattern

kernel_developer_email_list()
{
	git shortlog -e --since '1 year ago' next/master |
	grep "$1):$" |
	awk -F'[<>]' '{ print $2 }' |
	grep -v -f $WHITELIST
}

:> $WHITELIST

# show domain with 2+ contributors
kernel_developer_email_list '[0-9]' |
cut -f2 -d@ |
sed -e 's/^/^.*@/' |
sort |
uniq -c |
sort -nr |
awk '{ if ($1 >= 2) print $2 }' > $WHITELIST-new
cp $WHITELIST-new $WHITELIST

# show individual with 10+ commits
kernel_developer_email_list '[0-9][0-9]' |
sed -e 's/^/^/' >> $WHITELIST-new
mv $WHITELIST-new $WHITELIST

Here are the output:

^.*@gmail.com
^.*@intel.com
^.*@redhat.com
^.*@samsung.com
^.*@freescale.com
^.*@linaro.org
^.*@linux.vnet.ibm.com
^.*@linux.intel.com
^.*@huawei.com
^.*@ti.com
^.*@google.com
^.*@codeaurora.org
^.*@chromium.org
^.*@oracle.com
^.*@mellanox.com
^.*@qlogic.com
^.*@arm.com
^.*@amd.com
^.*@marvell.com
^.*@broadcom.com
^.*@nvidia.com
^.*@canonical.com
^.*@suse.de
^.*@kernel.org
^.*@rock-chips.com
^.*@imgtec.com
^.*@gmx.de
^.*@st.com
^.*@emulex.com
^.*@xilinx.com
^.*@renesas.com
^.*@csr.com
^.*@cn.fujitsu.com
^.*@suse.cz
^.*@avagotech.com
^.*@mentor.com
^.*@hp.com
^.*@qca.qualcomm.com
^.*@googlemail.com
^.*@citrix.com
^.*@atmel.com
^.*@iki.fi
^.*@de.ibm.com
^.*@pengutronix.de
^.*@suse.com
^.*@free-electrons.com
^.*@fb.com
^.*@windriver.com
^.*@tieto.com
^.*@microsoft.com
^.*@mediatek.com
^.*@apm.com
^.*@vmware.com
^.*@synopsys.com
^.*@realtek.com
^.*@primarydata.com
^.*@lge.com
^.*@cisco.com
^.*@web.de
^.*@sgi.com
^.*@openwrt.org
^.*@linux.com
^.*@gmx.net
^.*@diasemi.com
^.*@chelsio.com
^.*@akamai.com
^.*@sonymobile.com
^.*@ni.com
^.*@nicira.com
^.*@infradead.org
^.*@free.fr
^.*@ens.fr
^.*@codethink.co.uk
^.*@unisys.com
^.*@solarflare.com
^.*@qti.qualcomm.com
^.*@parallels.com
^.*@mvista.com
^.*@lip6.fr
^.*@hotmail.com
^.*@denx.de
^.*@cumulusnetworks.com
^.*@collabora.co.uk
^.*@au1.ibm.com
^.*@analog.com
^.*@6wind.com
^.*@xyratex.com
^.*@wizery.com
^.*@us.ibm.com
^.*@opensource.altera.com
^.*@linbit.com
^.*@jp.fujitsu.com
^.*@cogentembedded.com
^.*@caviumnetworks.com
^.*@brocade.com
^.*@axis.com
^.*@altera.com
^.*@163.com
^.*@yandex-team.ru
^.*@yahoo.fr
^.*@yahoo.com
^.*@visteon.com
^.*@vanguardiasur.com.ar
^.*@tuxdriver.com
^.*@stratus.com
^.*@pmcs.com
^.*@outlook.com
^.*@nsn.com
^.*@nokia.com
^.*@micron.com
^.*@hisilicon.com
^.*@gmx.com
^.*@gentoo.org
^.*@fastmail.fm
^.*@ericsson.com
^.*@Emulex.Com
^.*@emc.com.tw
^.*@dev.mellanox.co.il
^.*@cavium.com
^.*@amazon.de
^.*@acm.org
^.*@zoho.com
^.*@xsens.com
^.*@xes-inc.com
^.*@virtualopensystems.com
^.*@users.sourceforge.net
^.*@smartplayin.com
^.*@semihalf.com
^.*@samba.org
^.*@realsil.com.cn
^.*@profitbricks.com
^.*@posteo.de
^.*@plumgrid.com
^.*@partner.samsung.com
^.*@openvz.org
^.*@opensource.wolfsonmicro.com
^.*@netapp.com
^.*@mm-sol.com
^.*@mail.ru
^.*@linutronix.de
^.*@lemote.com
^.*@lab.ntt.co.jp
^.*@keymile.com
^.*@inktank.com
^.*@hitachi.com
^.*@globallogic.com
^.*@fau.de
^.*@ezchip.com
^.*@debian.org
^.*@ddn.com
^.*@daterainc.com
^.*@c-s.fr
^.*@cray.com
^.*@collabora.com
^.*@cloudius-systems.com
^.*@ce.jp.nec.com
^.*@baylibre.com
^.*@arista.com
^.*@android.com
^.*@alliedtelesis.co.nz
^.*@alibaba-inc.com
^.*@xsilon.com
^.*@xs4all.nl
^.*@wp.pl
^.*@vodalys.com
^.*@visionengravers.com
^.*@verge.net.au
^.*@umich.edu
^.*@uli-eckhardt.de
^.*@uclouvain.be
^.*@twitter.com
^.*@tuxera.com
^.*@tu-dresden.de
^.*@tronnes.org
^.*@toradex.com
^.*@theobroma-systems.com
^.*@tencent.com
^.*@sunrus.com.cn
^.*@stressinduktion.org
^.*@spreadtrum.com
^.*@sntech.de
^.*@skynet.be
^.*@skitlab.ru
^.*@silabs.com
^.*@secunet.com
^.*@savoirfairelinux.com
^.*@sang-engineering.com
^.*@sandisk.com
^.*@ru.ibm.com
^.*@qq.com
^.*@purestorage.com
^.*@protonic.nl
^.*@overkiz.com
^.*@osg.samsung.com
^.*@open-mesh.com
^.*@open.eurogiciel.org
^.*@online.de
^.*@noironetworks.com
^.*@newterm.pl
^.*@networkplumber.org
^.*@netfilter.org
^.*@neratec.com
^.*@motorola.com
^.*@men.de
^.*@memblaze.com
^.*@mellanox.co.il
^.*@lst.de
^.*@linux-vserver.org
^.*@linux-mips.org
^.*@linux-m68k.org
^.*@linuxfoundation.org
^.*@linux-foundation.org
^.*@laposte.net
^.*@konsulko.com
^.*@kernkonzept.com
^.*@kernellabs.com
^.*@kernel.crashing.org
^.*@jolla.com
^.*@ispras.ru
^.*@igel.co.jp
^.*@ieee.org
^.*@ideasonboard.com
^.*@hauppauge.com
^.*@green-communications.fr
^.*@googlers.com
^.*@goodmis.org
^.*@goldelico.com
^.*@ge.com
^.*@gatech.edu
^.*@gaisler.com
^.*@fr.zoreil.com
^.*@Freescale.com
^.*@freebox.fr
^.*@foxmail.com
^.*@flawful.org
^.*@ffwll.ch
^.*@fedoraproject.org
^.*@eukrea.com
^.*@endlessm.com
^.*@elproma.com.pl
^.*@ellerman.id.au
^.*@elisp.net
^.*@de.bosch.com
^.*@davemloft.net
^.*@cs.jhu.edu
^.*@comcast.net
^.*@codesourcery.com
^.*@clara.co.uk
^.*@cirrus.com
^.*@cea.fr
^.*@bjorling.me
^.*@arm.linux.org.uk
^.*@armadeus.com
^.*@amacapital.net
^.*@altlinux.org
^.*@alphalink.fr
^.*@1wt.eu
^ahmed.darwish@valeo.com
^viro@ZenIV.linux.org.uk
^viro@zeniv.linux.org.uk
^stern@rowland.harvard.edu
^alex@alex-smith.me.uk
^alexander.stein@systec-electronic.com
^aik@ozlabs.ru
^alistair@popple.id.au
^rupran@einserver.de
^aduggan@synaptics.com
^andrew@lunn.ch
^ao2@ao2.it
^arno@natisbad.org
^arnd@arndb.de
^arvid.brodin@alten.se
^asaf.vertz@tandemg.com
^axel.lin@ingics.com
^baruch@tkos.co.il
^hadess@hadess.net
^behanw@converseincode.com
^greearb@candelatech.com
^ben@decadent.org.uk
^benjamin@southpole.se
^bjorn@mork.no
^me@bobcopeland.com
^carlo@caione.org
^casey@schaufler-ca.com
^c@24.io
^wens@csie.org
^ching2048@areca.com.tw
^chris@rorvick.com
^chris@chris-wilson.co.uk
^chris@zankel.net
^email@christophjaeger.info
^clemens@ladisch.de
^cwang@twopensource.com
^clg@fr.ibm.com
^dja@axtens.net
^daniel@iogearbox.net
^daniel@zonque.org
^david@fromorbit.com
^david@hardeman.nu
^dave@stgolabs.net
^kda@linux-powerpc.org
^lifshitz@compulab.co.il
^dgilbert@interlog.com
^dudl@cypress.com
^eliot@blennerhassett.gen.nz
^emilio@elopez.com.ar
^eballetbo@iseebcn.com
^sandeen@sandeen.net
^ebiederm@xmission.com
^fthain@telegraphics.com.au
^fw@strlen.de
^frank.praznik@oh.rr.com
^geert+renesas@glider.be
^linux@horizon.com
^gerg@uclinux.org
^linux@roeck-us.net
^gclement@baobob.org
^hans@hanshq.net
^hauke@hauke-m.de
^herbert@gondor.apana.org.au
^abbotti@mev.co.uk
^ipm@chirality.org.uk
^JBottomley@Parallels.com
^jan.kiszka@siemens.com
^jason@lakedaemon.net
^jlayton@poochiereds.net
^jbarnes@virtuousgeek.org
^jiri@resnulli.us
^joe@perches.com
^joel@jms.id.au
^joern@logfs.org
^hannes@cmpxchg.org
^John.C.Harrison@Intel.com
^corbet@lwn.net
^josh@joshtriplett.org
^kadlec@blackhole.kfki.hu
^ja@ssi.bg
^julien.chauveau@neo-technologies.fr
^karl.beldan@rivierawaves.com
^Larry.Finger@lwfinger.net
^lars@metafoo.de
^LW@KARO-electronics.de
^luca.abeni@unitn.it
^luca@lucaceresoli.net
^luca.ellero@brickedbrain.com
^m.v.b@runbox.com
^damm+renesas@opensource.se
^manfred@colorfullife.com
^marcel@holtmann.org
^mgreer@animalcreek.com
^stockhausen@collogia.de
^kernel@martin.sperl.org
^yamada.m@jp.panasonic.com
^yamada.masahiro@socionext.com
^mail@beyermatthias.de
^mkrufky@linuxtv.org
^mikey@neuling.org
^madcatxster@devoid-pointer.net
^mina86@mina86.com
^michaelc@cs.wisc.edu
^mike.looijmans@topic.nl
^namit@cs.technion.ac.il
^n-horiguchi@ah.jp.nec.com
^nab@linux-iscsi.org
^der.herr@hofr.at
^hofrat@osadl.org
^nick.dyer@itdev.co.uk
^nicolas.iooss_linux@m4x.org
^green@linuxhacker.ru
^linux@rempel-privat.de
^olivier@sobrie.be
^olof@lixom.net
^osandov@osandov.com
^linux@rainbow-software.org
^kaber@trash.net
^pebolle@tiscali.nl
^paul@pwsan.com
^pavel@ucw.cz
^pefoley2@pefoley.com
^peter@hurleysoftware.com
^pmeerw@pmeerw.net
^peda@axentia.se
^plaes@plaes.org
^rabin@rab.in
^linux@rasmusvillemoes.dk
^richard@nod.at
^rickard_strandqvist@spectrumdigital.se
^imr@rtschenk.de
^rspliet@eclipso.eu
^rusty@rustcorp.com.au
^sd@queasysnail.net
^lkml14@scotdoyle.com
^popovich_sergei@mail.ua
^simon.horman@netronome.com
^stefan@agner.ch
^stefan.bruens@rwth-aachen.de
^stefan.sorensen@spectralink.com
^stefan.wahren@i2se.com
^stefano.stabellini@eu.citrix.com
^smueller@chronox.de
^s.grosjean@peak-system.com
^sfr@canb.auug.org.au
^sds@tycho.nsa.gov
^swise@opengridcomputing.com
^stratosk@semaphore.gr
^sylvain.rochet@finsecur.com
^o-takashi@sakamocchi.jp
^penguin-kernel@I-love.SAKURA.ne.jp
^tytso@mit.edu
^tgraf@suug.ch
^thomas@net.t-labs.tu-berlin.de
^niederp@physik.uni-kl.de
^tilman@imap.cc
^tharvey@gateworks.com
^tjakobi@math.uni-bielefeld.de
^tklauser@distanz.ch
^tom@herbertland.com
^tonyb@cybernetics.com
^tony@atomide.com
^varkab@cdac.in
^vincent.weaver@maine.edu
^QCA_vkondrat@QCA.qualcomm.com
^yongjun_wei@trendmicro.com.cn
^wsa@the-dreams.de
^lixiubo@cmss.chinamobile.com
^ydroneaud@opteya.com
^yguerrini@tomshardware.fr
^ysato@users.sourceforge.jp

Thanks,
Fengguang
diff mbox

Patch

--- a/sound/soc/zte/zx296702-i2s.c
+++ b/sound/soc/zte/zx296702-i2s.c
@@ -425,7 +425,6 @@  static struct platform_driver i2s_driver
 	.probe = zx_i2s_probe,
 	.driver = {
 		.name = "zx-i2s",
-		.owner = THIS_MODULE,
 		.of_match_table = zx_i2s_dt_ids,
 	},
 };