Re: [2.6 patch] ata_piix.c:piix_init_one() must be __devinit

[Jeff Garzik <jgarzik@pobox.com>]

Adrian Bunk wrote:
> This patch fixes the following section mismatches:
> 
> <--  snip  -->
> 
> ...
> WARNING: drivers/ata/built-in.o(.text+0x15072): Section mismatch in reference from the function piix_init_one() to the function .devinit.text:piix_init_sata_map()
> WARNING: drivers/ata/built-in.o(.text+0x150dd): Section mismatch in reference from the function piix_init_one() to the function .devinit.text:piix_init_pcs()
> WARNING: drivers/ata/built-in.o(.text+0x150e5): Section mismatch in reference from the function piix_init_one() to the function .devinit.text:piix_init_sidpr()
> WARNING: drivers/ata/built-in.o(.text+0x15107): Section mismatch in reference from the function piix_init_one() to the function .devinit.text:piix_check_450nx_errata()
> ...
> 
> <--  snip  -->
> 
> Signed-off-by: Adrian Bunk <bunk@kernel.org>
> 
> ---
> 222e97f95d38ce7cb41980671cc4383df28b5f7a 
> diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
> index a65c8ae..7a3b9ef 100644
> --- a/drivers/ata/ata_piix.c
> +++ b/drivers/ata/ata_piix.c
> @@ -1595,7 +1595,8 @@ static void piix_iocfg_bit18_quirk(struct pci_dev *pdev)
>   *	Zero on success, or -ERRNO value.
>   */
>  
> -static int piix_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
> +static int __devinit piix_init_one(struct pci_dev *pdev,
> +				   const struct pci_device_id *ent)
>  {
>  	static int printed_version;
>  	struct device *dev = &pdev->dev;
> 

applied