Re: checking status semantics with compatible functions

[Adding proper lists on CC]

- k

On Mar 30, 2011, at 3:07 AM, Kumar Gala wrote:

Grant,
=20
We have a few helper functions like:
=20
drivers/of/base.c:EXPORT_SYMBOL(of_device_is_compatible);
drivers/of/base.c:EXPORT_SYMBOL(of_find_compatible_node);
=20
That we use in places like arch/powerpc/platform/85xx/mpc85xx_ds.c:
=20
       for_each_node_by_type(np, "pci") {
               if (of_device_is_compatible(np, "fsl,mpc8540-pci") ||
                   of_device_is_compatible(np, "fsl,mpc8548-pcie") ||
                   of_device_is_compatible(np, "fsl,p2020-pcie")) {
=20
It seems like we should check 'status' in these cases with =

'of_device_is_available'.  I'm thinking that we should have versions of =
the helpers that also do the 'of_device_is_available' checking.  Or =
should we just change the helpers to call of_device_is_available?

=20
- k
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss