Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-28 Thread Andrew Lunn
Hi Stephen > There already is LED control via ethtool. > It is more important that the existing API (ethtool) still work. I'm having trouble finding this. The man page for ethtool only mentions -p --identify with respect to LEDs. I also don't see anything in include/uapi/linux/ethtool.h Please

Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-28 Thread Stephen Hemminger
On Wed, 23 Mar 2016 18:51:37 +0100 Vishal Thanki wrote: > Hi all, > > Based on suggestions from Andrew and Florian, I have made some changes > to expose the ethernet PHY LEDs using kernel LED subsystem. The following > ALPHA patchset introduces two new LED triggers: > > 1) -eth-phy-link: > To m

Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-24 Thread Andrew Lunn
> Hi Andrew, > > I still have some questions. Will the phylib call this > led_trigger_register_to_led() function for registering the trigger > instead of calling led_trigger_register()? Each phy driver needs to call led_classdev_register() for each LED it has. It then also calls led_trigger_regis

Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-24 Thread Vishal Thanki
>> >> The eth-phy-activity trigger uses the blink_set which I think uses the >> >> hardware acceleration if available. I am not sure how to handles LEDs >> >> which does not have hardware acceleration for this (eth-phy-activity) >> >> trigger. >> > >> > We want the LED to blink on activity, real pa

Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-24 Thread Andrew Lunn
On Wed, Mar 23, 2016 at 10:24:45PM +0100, Vishal Thanki wrote: > On Wed, Mar 23, 2016 at 10:10 PM, Andrew Lunn wrote: > > On Wed, Mar 23, 2016 at 09:24:00PM +0100, Vishal Thanki wrote: > >> Hi, > >> > >> > My suggestion was that the hardware needs to control the LEDs. You > >> > have software doin

Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-23 Thread Vishal Thanki
On Wed, Mar 23, 2016 at 10:10 PM, Andrew Lunn wrote: > On Wed, Mar 23, 2016 at 09:24:00PM +0100, Vishal Thanki wrote: >> Hi, >> >> > My suggestion was that the hardware needs to control the LEDs. You >> > have software doing it. You might be able to do this with the PHY >> > state machine for link

Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-23 Thread Andrew Lunn
On Wed, Mar 23, 2016 at 09:24:00PM +0100, Vishal Thanki wrote: > Hi, > > > My suggestion was that the hardware needs to control the LEDs. You > > have software doing it. You might be able to do this with the PHY > > state machine for link. But activity is never going to be accepted if > > software

Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-23 Thread Vishal Thanki
Hi, > My suggestion was that the hardware needs to control the LEDs. You > have software doing it. You might be able to do this with the PHY > state machine for link. But activity is never going to be accepted if > software control. > > The LED trigger attached to an LED should be used to configur

Re: [PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-23 Thread Andrew Lunn
On Wed, Mar 23, 2016 at 06:51:37PM +0100, Vishal Thanki wrote: > Hi all, > > Based on suggestions from Andrew and Florian, I have made some changes > to expose the ethernet PHY LEDs using kernel LED subsystem. The following > ALPHA patchset introduces two new LED triggers: > > 1) -eth-phy-link: >

[PATCH 0/3] Control ethernet PHY LEDs via LED subsystem

2016-03-23 Thread Vishal Thanki
Hi all, Based on suggestions from Andrew and Florian, I have made some changes to expose the ethernet PHY LEDs using kernel LED subsystem. The following ALPHA patchset introduces two new LED triggers: 1) -eth-phy-link: To monitor the PHY Link status 2) -eth-phy-activity: To monitor the PHY activ