Thanks Andrew, will try to change as suggested.
> On Wed, Sep 06, 2017 at 10:51:44AM +0000, nisar.sa...@microchip.com > wrote: > > From: Nisar Sayed <nisar.sa...@microchip.com> > > > > Add fixed_phy device support for LAN7801 device > > > > When LAN7801 device connected to PHY Device which does not have > > MDIO/MDC access, fixex_phy device will be added. > > Please try to find a way to do this without all the #ifdefs. They can be > acceptable in header files, but should be avoided in .c files. > > Andrew