Re: [PATCH RFC] net: dsa: remove unnecessary phy.h include

2017-01-18 Thread David Miller
From: David Miller Date: Wed, 18 Jan 2017 16:37:14 -0500 (EST) > From: Russell King - ARM Linux > Date: Wed, 18 Jan 2017 00:14:03 + > >> Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an >> unnecessary dependency for quite a large amount of the kernel. There's >> very l

Re: [PATCH RFC] net: dsa: remove unnecessary phy.h include

2017-01-18 Thread Russell King - ARM Linux
On Wed, Jan 18, 2017 at 04:37:14PM -0500, David Miller wrote: > From: Russell King - ARM Linux > Date: Wed, 18 Jan 2017 00:14:03 + > > > Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an > > unnecessary dependency for quite a large amount of the kernel. There's > > very l

Re: [PATCH RFC] net: dsa: remove unnecessary phy.h include

2017-01-18 Thread David Miller
From: Russell King - ARM Linux Date: Wed, 18 Jan 2017 00:14:03 + > Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an > unnecessary dependency for quite a large amount of the kernel. There's > very little which actually requires definitions from phy.h in net/dsa.h > - the

Re: [PATCH RFC] net: dsa: remove unnecessary phy.h include

2017-01-18 Thread David Miller
From: Russell King - ARM Linux Date: Wed, 18 Jan 2017 21:46:28 + > On Wed, Jan 18, 2017 at 04:37:14PM -0500, David Miller wrote: >> From: Russell King - ARM Linux >> Date: Wed, 18 Jan 2017 00:14:03 + >> >> > Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an >> > unne

Re: [PATCH RFC] net: dsa: remove unnecessary phy.h include

2017-01-17 Thread Florian Fainelli
On 01/17/2017 04:14 PM, Russell King - ARM Linux wrote: > Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an > unnecessary dependency for quite a large amount of the kernel. There's > very little which actually requires definitions from phy.h in net/dsa.h > - the include itself

Re: [PATCH RFC] net: dsa: remove unnecessary phy.h include

2017-01-17 Thread Vivien Didelot
Hi Russell, Russell King - ARM Linux writes: > Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an > unnecessary dependency for quite a large amount of the kernel. There's > very little which actually requires definitions from phy.h in net/dsa.h > - the include itself only wan

[PATCH RFC] net: dsa: remove unnecessary phy.h include

2017-01-17 Thread Russell King - ARM Linux
Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an unnecessary dependency for quite a large amount of the kernel. There's very little which actually requires definitions from phy.h in net/dsa.h - the include itself only wants the declaration of a couple of structures and IFNAMSI