Hi Gavin,

On Fri, 24 Feb 2012 17:38:11 +0800 Gavin Shan <sha...@linux.vnet.ibm.com> wrote:
>
> +#define EEH_DEV_TO_OF_NODE(edev)     (edev->dn)
> +#define EEH_DEV_TO_PCI_DEV(edev)     (edev->pdev)
> +#define OF_NODE_TO_EEH_DEV(dn)               ((struct eeh_dev *)(dn->edev))
> +#define PCI_DEV_TO_EEH_DEV(pdev)     ((struct eeh_dev 
> *)(pdev->dev.archdata.edev))

You need to put parentheses around the use of macro arguments ... or,
even better, make these into "static inline" accessor functions (with
lower case names).

-- 
Cheers,
Stephen Rothwell                    s...@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

Attachment: pgppYc3VZ261t.pgp
Description: PGP signature

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to