Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs

2018-12-12 Thread Florian Fainelli
On 12/12/18 4:33 AM, Joakim Tjernlund wrote: > This allows to control carrier from /sys/class/net/ethX/carrier for > Fixed PHYs. > > Signed-off-by: Joakim Tjernlund > --- > v2 - Only allow carrier changes for Fixed PHYs > > Florian: I have reimpl. this as I think you meant by registering >

Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs

2018-12-12 Thread Joakim Tjernlund
On Wed, 2018-12-12 at 15:51 +0100, Andrew Lunn wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > > fast check, would you be happy with this in fixed PHY: > > --

Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs

2018-12-12 Thread Andrew Lunn
> fast check, would you be happy with this in fixed PHY: > --- a/drivers/net/phy/fixed_phy.c > +++ b/drivers/net/phy/fixed_phy.c > @@ -25,6 +25,7 @@ > #include > #include > #include > +#include > > #include "swphy.h" > > @@ -38,6 +39,7 @@ struct fixed_phy { > struct phy_device *

Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs

2018-12-12 Thread Joakim Tjernlund
> > From: Andrew Lunn > > Sent: Wednesday, December 12, 2018 2:43 PM > > To: jo...@infinera.com > > Cc: netdev @ vger . kernel . org ; Claudiu Manoil > > ; Florian Fainelli > > Subject: Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs > > >

Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs

2018-12-12 Thread Joakim Tjernlund
> > From: Andrew Lunn > > Sent: Wednesday, December 12, 2018 2:43 PM > > To: jo...@infinera.com > > Cc: netdev @ vger . kernel . org ; Claudiu Manoil > > ; Florian Fainelli > > Subject: Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs > > >

RE: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs

2018-12-12 Thread Claudiu Manoil
>-Original Message- >From: Andrew Lunn >Sent: Wednesday, December 12, 2018 2:43 PM >To: jo...@infinera.com >Cc: netdev @ vger . kernel . org ; Claudiu Manoil >; Florian Fainelli >Subject: Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs > >On

Re: [PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs

2018-12-12 Thread Andrew Lunn
On Wed, Dec 12, 2018 at 01:33:08PM +0100, Joakim Tjernlund wrote: > This allows to control carrier from /sys/class/net/ethX/carrier for > Fixed PHYs. > > Signed-off-by: Joakim Tjernlund > --- > v2 - Only allow carrier changes for Fixed PHYs > > Florian: I have reimpl. this as I think you meant

[PATCHv2] gianfar: Add gfar_change_carrier() for Fixed PHYs

2018-12-12 Thread Joakim Tjernlund
This allows to control carrier from /sys/class/net/ethX/carrier for Fixed PHYs. Signed-off-by: Joakim Tjernlund --- v2 - Only allow carrier changes for Fixed PHYs Florian: I have reimpl. this as I think you meant by registering a Fixed PHY callback. Andrew: Are happy with this as we