On Sep 7, 2009, at 8:02 AM, Premi, Sanjeev wrote:


I think it is a great step in the right direction. My only concern is
that the usage conventions are confusing:

OMAP3_HAS_FEATURE(neon, NEON)

[sp] This is only used to declare a function that would translate to:
    unsigned int omap3_has_neon(void)
    {
             return (omap_features & OMAP_HAS_NEON);
    }
    EXPORT_SYMBOL(omap3_has_neon);

    A user would always do something like:
    if (omap_has_neon())
        do_neon_specific_stuff();


Ok, all is well then. :)


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to