On Tue, Jul 14, 2015 at 9:42 AM, Sergei Shtylyov
<[email protected]> wrote:
> Hello.
>
> On 07/14/2015 06:32 PM, [email protected] wrote:
>
>> From: Anuradha Karuppiah <[email protected]>
>
>
>> protodown can be set by user space applications like MLAG on detecting
>> errors on a switch port. This patch provides sample switch driver changes
>> for handling protodown. Rocker PHYS disables the port in response to
>> protodown.
>
>
>> Signed-off-by: Anuradha Karuppiah <[email protected]>
>> Signed-off-by: Andy Gospodarek <[email protected]>
>> Signed-off-by: Roopa Prabhu <[email protected]>
>> Signed-off-by: Wilson Kok <[email protected]>
>> ---
>> drivers/net/ethernet/rocker/rocker.c | 19 ++++++++++++++++++-
>> 1 file changed, 18 insertions(+), 1 deletion(-)
>
>
>> diff --git a/drivers/net/ethernet/rocker/rocker.c
>> b/drivers/net/ethernet/rocker/rocker.c
>> index 2d8578cade..ec950d9 100644
>> --- a/drivers/net/ethernet/rocker/rocker.c
>> +++ b/drivers/net/ethernet/rocker/rocker.c
>> @@ -3983,7 +3983,8 @@ static int rocker_port_open(struct net_device *dev)
>>
>> napi_enable(&rocker_port->napi_tx);
>> napi_enable(&rocker_port->napi_rx);
>> - rocker_port_set_enable(rocker_port, true);
>> + if (!(dev->proto_down))
>
>
> Inner parens not needed.
Ack.
>
>> + rocker_port_set_enable(rocker_port, true);
>> netif_start_queue(dev);
>> return 0;
>>
>> @@ -4167,6 +4168,21 @@ static int rocker_port_get_phys_port_name(struct
>> net_device *dev,
>> return err ? -EOPNOTSUPP : 0;
>> }
>>
>> +static int rocker_port_change_proto_down(struct net_device *dev,
>> + bool proto_down)
>> +{
>> + struct rocker_port *rocker_port = netdev_priv(dev);
>> +
>> + if (rocker_port->dev->flags & IFF_UP) {
>> + if (proto_down)
>> + rocker_port_set_enable(rocker_port, false);
>> + else
>> + rocker_port_set_enable(rocker_port, true);
>
>
> Why not:
>
> rocker_port_set_enable(rocker_port, !proto_down);
yes, that would be better. thanks for the review.
>
> [...]
>
> MBR, Sergei
>
--
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
