[PATCH 0/9] powerpc/pci: clean up direct access to sysdata

[Michael Ellerman]

On Thu, 2009-04-30 at 08:09 -0500, Kumar Gala wrote:
> We shouldn't be accessing sysdata directly.  In the future we might have
> sysdata be a device_node on ppc32 to match ppc64.  Direct access would make that
> a bit difficult.
> 
> If someone can look at the iseries code I would appreciate that:
> 
> arch/powerpc/platforms/iseries/pci.c:           iomm_table[current_iomm_table_entry] = dev->sysdata;
> arch/powerpc/platforms/iseries/pci.c:                   iseries_ds_addr(dev->sysdata) | (bar_num << 24);
> arch/powerpc/platforms/iseries/pci.c:   pdev->sysdata = node;

It looks like it's basically doing the same logic as
pci_device_to_OF_node(), and fetch_dev_dn(), but there's probably some
reason why it's not using those.

Given how much new iseries hardware there is, I think we'd probably
rather leave the code as is. It's a device node anyway, so it doesn't
block your plans RE ppc32.

cheers
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20090430/b4257d8a/attachment.pgp>