linux-kernel - Re: linux-next: comment in commit in the slave-dma tree (dmaengine: dw

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: <20130108095752.9e1529a2d32d0635490cd16e@canb.auug.org.au>
Date:	Tue, 8 Jan 2013 09:57:52 +1100
From:	Stephen Rothwell <sfr@...b.auug.org.au>
To:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:	Vinod Koul <vinod.koul@...el.com>, linux-next@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Viresh Kumar <viresh.kumar@...aro.org>
Subject: Re: linux-next: comment in commit in the slave-dma tree (dmaengine:
 dw_dmac: Enhance device tree support)

Hi Greg,

On Mon, 7 Jan 2013 14:43:08 -0800 Greg Kroah-Hartman <gregkh@...uxfoundation.org> wrote:
>
> That brings up something I wanted to ask you.
> 
> I'd like to send a patch to Linus to just delete __devinit and friends
> entirely from init.h, but that will break the linux-next tree from
> building in places (like this one, and in a few others.)
> 
> Is that ok?  That should finally get everyone's attention and keep them
> from the cargo-cult programming mode of using these attributes :)

I guess I can apply fixup patches that just remove any instances that
turn up ... and you and the tree owner will get copies of those.

So, if Linus takes it, I will cope.

This is what "git grep -E '__dev(init|exit)'" returns for next-20130107:

arch/ia64/kernel/iosapic.c: *				__init/__devinit attributes.
drivers/dma/dw_dmac.c:__devinit dw_dma_parse_dt(struct platform_device *pdev)
drivers/media/platform/sh_veu.c:static int __devinit sh_veu_probe(struct platform_device *pdev)
drivers/media/platform/sh_veu.c:static int __devexit sh_veu_remove(struct platform_device *pdev)
drivers/media/platform/sh_veu.c:	.remove		= __devexit_p(sh_veu_remove),
drivers/net/ethernet/qlogic/qlcnic/qlcnic_main.c:static int __devinit
drivers/net/ethernet/qlogic/qlcnic/qlcnic_main.c:static void __devexit qlcnic_remove(struct pci_dev *pdev)
drivers/power/reset/restart-poweroff.c:static int __devinit restart_poweroff_probe(struct platform_device *pdev)
drivers/power/reset/restart-poweroff.c:static int __devexit restart_poweroff_remove(struct platform_device *pdev)
drivers/power/reset/restart-poweroff.c:	.remove = __devexit_p(restart_poweroff_remove),
drivers/regulator/lp8755.c:	.remove = __devexit_p(lp8755_remove),
drivers/scsi/ipr.c:static int __devinit ipr_enable_msix(struct ipr_ioa_cfg *ioa_cfg)
drivers/scsi/ipr.c:static int __devinit ipr_enable_msi(struct ipr_ioa_cfg *ioa_cfg)
drivers/scsi/ipr.c:static void __devinit name_msi_vectors(struct ipr_ioa_cfg *ioa_cfg)
drivers/scsi/ipr.c:static int __devinit ipr_request_other_msi_irqs(struct ipr_ioa_cfg *ioa_cfg)
drivers/scsi/ips.c:/* 5.30.00  - use __devexit_p()                                              */
scripts/tags.sh:	-I __initdata,__exitdata,__initconst,__devinitdata	\
scripts/tags.sh:	-I __devinitconst,__cpuinitdata,__initdata_memblock	\
sound/oss/swarm_cs4297a.c:* 11/10/00 bkz - added __devinit to cs4297a_hw_init()

so there are just a few left.

BTW, I am *not* reporting merge fixes I am doing due to your tree wide
removal of __dev*, but they are still turning up :-(

-- 
Cheers,
Stephen Rothwell                    sfr@...b.auug.org.au

Content of type "application/pgp-signature" skipped