2013/6/4 Arnd Bergmann [off-list ref]:

On Tuesday 04 June 2013 16:36:50 Florian Fainelli wrote:

quoted

It seems to me that what David proposes is to have say an
arch/arm/mach-foo/phy-fixups.c file which is only enabled when
CONFIG_PHYLIB is set (obj-$(CONFIG_PHYLIB) += phy-fixup.o), such that
it does not need to have any conditionnals when calling
phy_register_fixup. This sounds a little unusual, but why not.

I don't think it would actually help us, because then we still need
to declare a local function that gets called from the board init
code. Instead of doing

        if (IS_ENABLED(CONFIG_PHYLIB))
                 phy_register_fixup_for_uid(phy_id, foo_phy_fixup);

we would then do

        if (IS_ENABLED(CONFIG_PHYLIB))
                foo_phy_fixup_register();

which is not much different at all.

You would just need to define a stub for your arch_foo_phy_fixup()
which has a different definition depending on whether CONFIG_PHYLIB is
defined or not.

This would be just one function, instead of the whole bunch of stubs
needed for phylib. Right now its probably 1 vs 3, so it does not make
that much of a difference but who knows, if we had more phylib stubs
and forget to update the stubs? (which tends to happen pretty often).

The size savings are exactly the same in both approaches anyway.
--
Florian