Re: [PATCH] ipr: ipr_remove() marked __devexit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Any comments on this patch? 

It was acked by Brian King, do I need to do anything else to get it
accepted?


Thanks,
Kleber

On Mon, 2009-03-23 at 17:12 -0300, Kleber Sacilotto de Souza wrote:
> Marking the ipr clean up function ipr_remove() as __devexit and using
> __devexit_p() macro in its address reference.
> 
> 
> Signed-off-by: Kleber Sacilotto de Souza <kleber@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Breno Leitao <leitao@xxxxxxxxxxxxxxxxxx>
> Acked-by: Brian King <brking@xxxxxxxxxxxxxxxxxx>
> ---
>  drivers/scsi/ipr.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c
> index 0782900..bc39ac6 100644
> --- a/drivers/scsi/ipr.c
> +++ b/drivers/scsi/ipr.c
> @@ -7674,7 +7674,7 @@ static void __ipr_remove(struct pci_dev *pdev)
>   * Return value:
>   *     none
>   **/
> -static void ipr_remove(struct pci_dev *pdev)
> +static void __devexit ipr_remove(struct pci_dev *pdev)
>  {
>         struct ipr_ioa_cfg *ioa_cfg = pci_get_drvdata(pdev);
> 
> @@ -7850,7 +7850,7 @@ static struct pci_driver ipr_driver = {
>         .name = IPR_NAME,
>         .id_table = ipr_pci_table,
>         .probe = ipr_probe,
> -       .remove = ipr_remove,
> +       .remove = __devexit_p(ipr_remove),
>         .shutdown = ipr_shutdown,
>         .err_handler = &ipr_err_handler,
>  };
> --
> 
-- 
Kleber S. Souza
IBM Linux Technology Center

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux