On Mon, 2016-01-04 at 18:36 +0100, Andrew Lunn wrote:
> Add a phydev_name() macro, to help with moving some structure members
> from phy_device.
[]
> diff --git a/include/linux/phy.h b/include/linux/phy.h
[]
> @@ -783,6 +783,8 @@static inline int phy_read_status(struct phy_device
> *phydev)
> #define phydev_dbg(_phydev, format, args...) \
> dev_dbg(&_phydev->dev, format, ##args)
>
> +#define phydev_name(_phydev) dev_name(&_phydev->dev)
> +
This should use parentheses around phydev
#define phydev_name(phydev) dev_name(&(phydev)->dev)
or likely even better be a static inline
static inline const char * phydev_name(const struct phy_device *phydev)
{
return dev_name(&phydev->dev);
}
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
