Re: [U-Boot] [PATCH v2 0/7] net: add support for phy speed limitation

[Alexey Brodkin]

Hi Joe,

On Tue, 2016-01-19 at 07:02 +, Alexey Brodkin wrote:
> Hi Joe,
> 
> On Wed, 2016-01-13 at 16:59 +0300, Alexey Brodkin wrote:
> > In some cases Ethernet PHY and network itself might support speeds
> > which are not supported by Ethernet controller (MAC).
> > 
> > To support that kind of situation it is required to force limit
> > PHY so it is not configured for unsupported mode or even if PHY 
> > autonegotiated too high speed it will be reconfigured for slower mode.
> > 
> > Note this series is v2 because v1 patches were sent separately not being
> > aranged in series. And now I'm putting all patches in one series
> > and marking it v2 even though no changes were done in patches itself.
> > 
> > Alexey Brodkin (4):
> >   drivers/net/phy: introduce phy_set_supported()
> >   include/net.h: add max_speed member in struct eth_pdata
> >   net/designware: do explicit port selection for 1Gb mode
> >   net/designware: add support of max-speed device tree property
> > 
> > Florian Fainelli (2):
> >   net: phy: ensure Gigabit features are masked off if requested
> >   net: phy: breakdown PHY_*_FEATURES defines
> > 
> > Sascha Hauer (1):
> >   net: phy: genphy: Allow overwriting features
> > 
> >  drivers/net/designware.c | 16 +-
> >  drivers/net/designware.h |  1 +
> >  drivers/net/phy/phy.c| 77 
> > ++--
> >  include/net.h|  2 ++
> >  include/phy.h| 23 ++-
> >  5 files changed, 89 insertions(+), 30 deletions(-)
> 
> Please treat this as a polite reminder to review
> that series sometime soon.

Thank you for reviewing that series.
I'm wondering who's going to pick it up?

-Alexey
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 0/7] net: add support for phy speed limitation

[Joe Hershberger]

On Wed, Jan 27, 2016 at 10:23 AM, Alexey Brodkin
 wrote:
> Hi Joe,
>
> On Tue, 2016-01-19 at 07:02 +, Alexey Brodkin wrote:
>> Hi Joe,
>>
>> On Wed, 2016-01-13 at 16:59 +0300, Alexey Brodkin wrote:
>> > In some cases Ethernet PHY and network itself might support speeds
>> > which are not supported by Ethernet controller (MAC).
>> >
>> > To support that kind of situation it is required to force limit
>> > PHY so it is not configured for unsupported mode or even if PHY
>> > autonegotiated too high speed it will be reconfigured for slower mode.
>> >
>> > Note this series is v2 because v1 patches were sent separately not being
>> > aranged in series. And now I'm putting all patches in one series
>> > and marking it v2 even though no changes were done in patches itself.
>> >
>> > Alexey Brodkin (4):
>> >   drivers/net/phy: introduce phy_set_supported()
>> >   include/net.h: add max_speed member in struct eth_pdata
>> >   net/designware: do explicit port selection for 1Gb mode
>> >   net/designware: add support of max-speed device tree property
>> >
>> > Florian Fainelli (2):
>> >   net: phy: ensure Gigabit features are masked off if requested
>> >   net: phy: breakdown PHY_*_FEATURES defines
>> >
>> > Sascha Hauer (1):
>> >   net: phy: genphy: Allow overwriting features
>> >
>> >  drivers/net/designware.c | 16 +-
>> >  drivers/net/designware.h |  1 +
>> >  drivers/net/phy/phy.c| 77 
>> > ++--
>> >  include/net.h|  2 ++
>> >  include/phy.h| 23 ++-
>> >  5 files changed, 89 insertions(+), 30 deletions(-)
>>
>> Please treat this as a polite reminder to review
>> that series sometime soon.
>
> Thank you for reviewing that series.
> I'm wondering who's going to pick it up?

I'm picking it up now and build testing it.

Thanks,
-Joe
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 0/7] net: add support for phy speed limitation

[Alexey Brodkin]

Hi Joe,

On Wed, 2016-01-27 at 12:48 -0600, Joe Hershberger wrote:
> On Wed, Jan 27, 2016 at 10:23 AM, Alexey Brodkin
>  wrote:
> > Hi Joe,
> > 
> > On Tue, 2016-01-19 at 07:02 +, Alexey Brodkin wrote:
> > > Hi Joe,
> > > 
> > > On Wed, 2016-01-13 at 16:59 +0300, Alexey Brodkin wrote:
> > > > In some cases Ethernet PHY and network itself might support speeds
> > > > which are not supported by Ethernet controller (MAC).
> > > > 
> > > > To support that kind of situation it is required to force limit
> > > > PHY so it is not configured for unsupported mode or even if PHY
> > > > autonegotiated too high speed it will be reconfigured for slower mode.
> > > > 
> > > > Note this series is v2 because v1 patches were sent separately not being
> > > > aranged in series. And now I'm putting all patches in one series
> > > > and marking it v2 even though no changes were done in patches itself.
> > > > 
> > > > Alexey Brodkin (4):
> > > >   drivers/net/phy: introduce phy_set_supported()
> > > >   include/net.h: add max_speed member in struct eth_pdata
> > > >   net/designware: do explicit port selection for 1Gb mode
> > > >   net/designware: add support of max-speed device tree property
> > > > 
> > > > Florian Fainelli (2):
> > > >   net: phy: ensure Gigabit features are masked off if requested
> > > >   net: phy: breakdown PHY_*_FEATURES defines
> > > > 
> > > > Sascha Hauer (1):
> > > >   net: phy: genphy: Allow overwriting features
> > > > 
> > > >  drivers/net/designware.c | 16 +-
> > > >  drivers/net/designware.h |  1 +
> > > >  drivers/net/phy/phy.c| 77 
> > > > ++--
> > > >  include/net.h|  2 ++
> > > >  include/phy.h| 23 ++-
> > > >  5 files changed, 89 insertions(+), 30 deletions(-)
> > > 
> > > Please treat this as a polite reminder to review
> > > that series sometime soon.
> > 
> > Thank you for reviewing that series.
> > I'm wondering who's going to pick it up?
> 
> I'm picking it up now and build testing it.

Thanks for doing that!

-Alexey
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 0/7] net: add support for phy speed limitation

[Alexey Brodkin]

Hi Joe,

On Wed, 2016-01-13 at 16:59 +0300, Alexey Brodkin wrote:
> In some cases Ethernet PHY and network itself might support speeds
> which are not supported by Ethernet controller (MAC).
> 
> To support that kind of situation it is required to force limit
> PHY so it is not configured for unsupported mode or even if PHY 
> autonegotiated too high speed it will be reconfigured for slower mode.
> 
> Note this series is v2 because v1 patches were sent separately not being
> aranged in series. And now I'm putting all patches in one series
> and marking it v2 even though no changes were done in patches itself.
> 
> Alexey Brodkin (4):
>   drivers/net/phy: introduce phy_set_supported()
>   include/net.h: add max_speed member in struct eth_pdata
>   net/designware: do explicit port selection for 1Gb mode
>   net/designware: add support of max-speed device tree property
> 
> Florian Fainelli (2):
>   net: phy: ensure Gigabit features are masked off if requested
>   net: phy: breakdown PHY_*_FEATURES defines
> 
> Sascha Hauer (1):
>   net: phy: genphy: Allow overwriting features
> 
>  drivers/net/designware.c | 16 +-
>  drivers/net/designware.h |  1 +
>  drivers/net/phy/phy.c| 77 
> ++--
>  include/net.h|  2 ++
>  include/phy.h| 23 ++-
>  5 files changed, 89 insertions(+), 30 deletions(-)

Please treat this as a polite reminder to review
that series sometime soon.

-Alexey
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v2 0/7] net: add support for phy speed limitation

[Alexey Brodkin]

In some cases Ethernet PHY and network itself might support speeds
which are not supported by Ethernet controller (MAC).

To support that kind of situation it is required to force limit
PHY so it is not configured for unsupported mode or even if PHY 
autonegotiated too high speed it will be reconfigured for slower mode.

Note this series is v2 because v1 patches were sent separately not being
aranged in series. And now I'm putting all patches in one series
and marking it v2 even though no changes were done in patches itself.

Alexey Brodkin (4):
  drivers/net/phy: introduce phy_set_supported()
  include/net.h: add max_speed member in struct eth_pdata
  net/designware: do explicit port selection for 1Gb mode
  net/designware: add support of max-speed device tree property

Florian Fainelli (2):
  net: phy: ensure Gigabit features are masked off if requested
  net: phy: breakdown PHY_*_FEATURES defines

Sascha Hauer (1):
  net: phy: genphy: Allow overwriting features

 drivers/net/designware.c | 16 +-
 drivers/net/designware.h |  1 +
 drivers/net/phy/phy.c| 77 ++--
 include/net.h|  2 ++
 include/phy.h| 23 ++-
 5 files changed, 89 insertions(+), 30 deletions(-)

-- 
2.4.3

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot