Re: linux-next: manual merge of the devicetree tree with thespi-current tree

From: Grant Likely
Date: Tue Feb 15 2011 - 23:42:09 EST


On Tue, Feb 15, 2011 at 9:09 PM, Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
> Hi Grant,
>
> Today's linux-next merge of the devicetree tree got a conflict in
> drivers/spi/pxa2xx_spi_pci.c between commit
> 0f3e1d27a7e3f98d996d707d649128e229b65deb ("spi/pxa2xx pci: fix the
> release - remove race") from the spi-current tree and commit
> c9e358dfc4a8cb2227172ef77908c2e0ee17bcb9 ("driver-core: remove
> conditionals around devicetree pointers") from the devicetree tree.
>
> I fixed it up (see below) and can carry the fix as necessary.

Heh, seeing as those are *both* my trees, I'll take care of merging
appropriately to sort this out. :-)

g.

> --
> Cheers,
> Stephen Rothwell                    sfr@xxxxxxxxxxxxxxxx
>
> diff --cc drivers/spi/pxa2xx_spi_pci.c
> index 19752b0,b6589bb..0000000
> --- a/drivers/spi/pxa2xx_spi_pci.c
> +++ b/drivers/spi/pxa2xx_spi_pci.c
> @@@ -75,24 -84,25 +75,22 @@@ static int __devinit ce4100_spi_probe(s
>                return ret;
>        }
>
>  +      pdev = platform_device_alloc("pxa2xx-spi", dev->devfn);
>        spi_info = kzalloc(sizeof(*spi_info), GFP_KERNEL);
>  -      if (!spi_info) {
>  +      if (!pdev || !spi_info ) {
>                ret = -ENOMEM;
>  -              goto err_kz;
>  +              goto err_nomem;
>        }
>  -      ssp = &spi_info->ssp;
>  -      pdev = &spi_info->spi_pdev;
>  -      spi_pdata =  &spi_info->spi_pdata;
>  +      memset(&spi_pdata, 0, sizeof(spi_pdata));
>  +      spi_pdata.num_chipselect = dev->devfn;
>
>  -      pdev->name = "pxa2xx-spi";
>  -      pdev->id = dev->devfn;
>  -      pdev->dev.parent = &dev->dev;
>  -      pdev->dev.platform_data = &spi_info->spi_pdata;
>  +      ret = platform_device_add_data(pdev, &spi_pdata, sizeof(spi_pdata));
>  +      if (ret)
>  +              goto err_nomem;
>
>  +      pdev->dev.parent = &dev->dev;
> - #ifdef CONFIG_OF
>        pdev->dev.of_node = dev->dev.of_node;
> - #endif
>  -      pdev->dev.release = plat_dev_release;
>  -
>  -      spi_pdata->num_chipselect = dev->devfn;
>  -
>  +      ssp = &spi_info->ssp;
>        ssp->phys_base = pci_resource_start(dev, 0);
>        ssp->mmio_base = ioremap(phys_beg, phys_len);
>        if (!ssp->mmio_base) {
>



--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/