On Wed, May 11, 2011 at 7:30 AM, Milton Miller <milt...@bga.com> wrote:
> The msi platform device driver was abusing dev.platform_data for its
> platform_driver_data. Use the correct pointer for storage.
>
> Platform_data is supposed to be for platforms to communicate to drivers
> parameters that are not otherwise discoverable. Its lifetime matches
> the platform_device not the platform device driver. It is generally
> not needed for drivers that only support systems with device trees.
>
> Signed-off-by: Milton Miller <milt...@bga.com>
Acked-by: Grant Likely <grant.lik...@secretlab.ca>
> ---
> ---
> arch/powerpc/sysdev/fsl_msi.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/powerpc/sysdev/fsl_msi.c b/arch/powerpc/sysdev/fsl_msi.c
> index d5679dc..077776c 100644
> --- a/arch/powerpc/sysdev/fsl_msi.c
> +++ b/arch/powerpc/sysdev/fsl_msi.c
> @@ -253,7 +253,7 @@ unlock:
>
> static int fsl_of_msi_remove(struct platform_device *ofdev)
> {
> - struct fsl_msi *msi = ofdev->dev.platform_data;
> + struct fsl_msi *msi = platform_get_drvdata(ofdev);
> int virq, i;
> struct fsl_msi_cascade_data *cascade_data;
>
> @@ -327,7 +327,7 @@ static int __devinit fsl_of_msi_probe(struct
> platform_device *dev)
> dev_err(&dev->dev, "No memory for MSI structure\n");
> return -ENOMEM;
> }
> - dev->dev.platform_data = msi;
> + platform_set_drvdata(dev, msi);
>
> msi->irqhost = irq_alloc_host(dev->dev.of_node, IRQ_HOST_MAP_LINEAR,
> NR_MSI_IRQS, &fsl_msi_host_ops, 0);
> --
> 1.7.0.4
>
>
--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
